Commit 0ebb4fbe authored by Bartosz Golaszewski's avatar Bartosz Golaszewski
Browse files

Merge tag 'intel-gpio-v5.18-2' of... 

Merge tag 'intel-gpio-v5.18-2' of gitolite.kernel.org:pub/scm/linux/kernel/git/andy/linux-gpio-intel into gpio/for-current

intel-gpio for v5.18-2

* Couple of fixes related to handling unsigned value of the pin from ACPI

gpiolib:
 -  acpi: Convert type for pin to be unsigned
 -  acpi: use correct format characters
parents 3836c73e 0c2cae09

drivers/gpio/gpiolib-acpi.c

+12 −10
Original line number Diff line number Diff line
@@ -108,7 +108,7 @@ static int acpi_gpiochip_find(struct gpio_chip *gc, void *data) 
 * controller does not have GPIO chip registered at the moment. This is to

 * support probe deferral.

 */

static struct gpio_desc *acpi_get_gpiod(char *path, int pin)

static struct gpio_desc *acpi_get_gpiod(char *path, unsigned int pin)

{

	struct gpio_chip *chip;

	acpi_handle handle;
@@ -136,7 +136,7 @@ static struct gpio_desc *acpi_get_gpiod(char *path, int pin) 
 * as it is intended for use outside of the GPIO layer (in a similar fashion to

 * gpiod_get_index() for example) it also holds a reference to the GPIO device.

 */

struct gpio_desc *acpi_get_and_request_gpiod(char *path, int pin, char *label)

struct gpio_desc *acpi_get_and_request_gpiod(char *path, unsigned int pin, char *label)

{

	struct gpio_desc *gpio;

	int ret;
@@ -317,11 +317,12 @@ static struct gpio_desc *acpi_request_own_gpiod(struct gpio_chip *chip, 
	return desc;

}



static bool acpi_gpio_in_ignore_list(const char *controller_in, int pin_in)

static bool acpi_gpio_in_ignore_list(const char *controller_in, unsigned int pin_in)

{

	const char *controller, *pin_str;

	int len, pin;

	unsigned int pin;

	char *endp;

	int len;



	controller = ignore_wake;

	while (controller) {
@@ -354,13 +355,13 @@ static bool acpi_gpio_in_ignore_list(const char *controller_in, int pin_in) 
static bool acpi_gpio_irq_is_wake(struct device *parent,

				  struct acpi_resource_gpio *agpio)

{

	int pin = agpio->pin_table[0];

	unsigned int pin = agpio->pin_table[0];



	if (agpio->wake_capable != ACPI_WAKE_CAPABLE)

		return false;



	if (acpi_gpio_in_ignore_list(dev_name(parent), pin)) {

		dev_info(parent, "Ignoring wakeup on pin %d\n", pin);

		dev_info(parent, "Ignoring wakeup on pin %u\n", pin);

		return false;

	}
@@ -378,7 +379,8 @@ static acpi_status acpi_gpiochip_alloc_event(struct acpi_resource *ares, 
	struct acpi_gpio_event *event;

	irq_handler_t handler = NULL;

	struct gpio_desc *desc;

	int ret, pin, irq;

	unsigned int pin;

	int ret, irq;



	if (!acpi_gpio_get_irq_resource(ares, &agpio))

		return AE_OK;
@@ -387,8 +389,8 @@ static acpi_status acpi_gpiochip_alloc_event(struct acpi_resource *ares, 
	pin = agpio->pin_table[0];



	if (pin <= 255) {

		char ev_name[5];

		sprintf(ev_name, "_%c%02hhX",

		char ev_name[8];

		sprintf(ev_name, "_%c%02X",

			agpio->triggering == ACPI_EDGE_SENSITIVE ? 'E' : 'L',

			pin);

		if (ACPI_SUCCESS(acpi_get_handle(handle, ev_name, &evt_handle)))
@@ -1098,7 +1100,7 @@ acpi_gpio_adr_space_handler(u32 function, acpi_physical_address address, 


	length = min_t(u16, agpio->pin_table_length, pin_index + bits);

	for (i = pin_index; i < length; ++i) {

		int pin = agpio->pin_table[i];

		unsigned int pin = agpio->pin_table[i];

		struct acpi_gpio_connection *conn;

		struct gpio_desc *desc;

		bool found;

include/linux/gpio/consumer.h

+7 −1
Original line number Diff line number Diff line
@@ -688,7 +688,7 @@ void acpi_dev_remove_driver_gpios(struct acpi_device *adev); 
int devm_acpi_dev_add_driver_gpios(struct device *dev,

				   const struct acpi_gpio_mapping *gpios);



struct gpio_desc *acpi_get_and_request_gpiod(char *path, int pin, char *label);

struct gpio_desc *acpi_get_and_request_gpiod(char *path, unsigned int pin, char *label);



#else  /* CONFIG_GPIOLIB && CONFIG_ACPI */
@@ -705,6 +705,12 @@ static inline int devm_acpi_dev_add_driver_gpios(struct device *dev, 
	return -ENXIO;

}



static inline struct gpio_desc *acpi_get_and_request_gpiod(char *path, unsigned int pin,

							   char *label)

{

	return ERR_PTR(-ENOSYS);

}



#endif /* CONFIG_GPIOLIB && CONFIG_ACPI */