Messages in this thread Patch in this message |  | | From | Hans de Goede <> | Subject | [RFC 2/5] gpiolib: acpi: Make acpi_gpio_in_ignore_list() more generic | Date | Sun, 31 Oct 2021 17:24:25 +0100 |
| |
Pass the ignore list to parse into acpi_gpio_in_ignore_list() instead of hardcoding it to the ignore_wake module-param.
This is a preparation patch for adding support to fully ignore some _AEI entries.
Signed-off-by: Hans de Goede <hdegoede@redhat.com> --- drivers/gpio/gpiolib-acpi.c | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-)
diff --git a/drivers/gpio/gpiolib-acpi.c b/drivers/gpio/gpiolib-acpi.c index 47712b6903b5..342219a58a32 100644 --- a/drivers/gpio/gpiolib-acpi.c +++ b/drivers/gpio/gpiolib-acpi.c @@ -320,13 +320,15 @@ 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 *ignore_list, + const char *controller_in, + int pin_in) { const char *controller, *pin_str; int len, pin; char *endp; - controller = ignore_wake; + controller = ignore_list; while (controller) { pin_str = strchr(controller, '@'); if (!pin_str) @@ -350,8 +352,8 @@ static bool acpi_gpio_in_ignore_list(const char *controller_in, int pin_in) return false; err: - pr_err_once("Error invalid value for gpiolib_acpi.ignore_wake: %s\n", - ignore_wake); + pr_err_once("Error invalid value for gpiolib_acpi.ignore_xxx: %s\n", + ignore_list); return false; } @@ -363,7 +365,7 @@ static bool acpi_gpio_irq_is_wake(struct device *parent, if (agpio->wake_capable != ACPI_WAKE_CAPABLE) return false; - if (acpi_gpio_in_ignore_list(dev_name(parent), pin)) { + if (acpi_gpio_in_ignore_list(ignore_wake, dev_name(parent), pin)) { dev_info(parent, "Ignoring wakeup on pin %d\n", pin); return false; } -- 2.31.1
|  |