diff mbox series

[v1,2/3] gpiolib: acpi: Use named item for enum gpiod_flags variable

Message ID 20201014133154.30610-2-andriy.shevchenko@linux.intel.com
State Superseded
Headers show
Series [v1,1/3] gpiolib: acpi: Respect bias settings for GpioInt() resource | expand

Commit Message

Andy Shevchenko Oct. 14, 2020, 1:31 p.m. UTC
Use named item instead of plain integer for enum gpiod_flags
to make it clear that even 0 has its own meaning.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
---
 drivers/gpio/gpiolib-acpi.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Mika Westerberg Oct. 21, 2020, 9:59 a.m. UTC | #1
On Wed, Oct 14, 2020 at 04:31:53PM +0300, Andy Shevchenko wrote:
> Use named item instead of plain integer for enum gpiod_flags

> to make it clear that even 0 has its own meaning.

> 

> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>


Reviewed-by: Mika Westerberg <mika.westerberg@linux.intel.com>
diff mbox series

Patch

diff --git a/drivers/gpio/gpiolib-acpi.c b/drivers/gpio/gpiolib-acpi.c
index 52b961673f16..1b372f0dfe63 100644
--- a/drivers/gpio/gpiolib-acpi.c
+++ b/drivers/gpio/gpiolib-acpi.c
@@ -1136,7 +1136,7 @@  acpi_gpiochip_parse_own_gpio(struct acpi_gpio_chip *achip,
 	int ret;
 
 	*lflags = GPIO_LOOKUP_FLAGS_DEFAULT;
-	*dflags = 0;
+	*dflags = GPIOD_ASIS;
 	*name = NULL;
 
 	ret = fwnode_property_read_u32_array(fwnode, "gpios", gpios,