From 8a655272d7b440dc3eeda598eb2d05819c0f75c2 Mon Sep 17 00:00:00 2001 From: Simon Glass Date: Sat, 23 Jan 2021 11:08:38 -0700 Subject: [PATCH] acpi: Tidy up documentation for struct acpi_gpio Some comments were provided after this patch was applied. Address them. Signed-off-by: Simon Glass Reviewed-by: Bin Meng --- include/acpi/acpi_device.h | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/include/acpi/acpi_device.h b/include/acpi/acpi_device.h index ed4acd912a..2c8846250b 100644 --- a/include/acpi/acpi_device.h +++ b/include/acpi/acpi_device.h @@ -171,10 +171,11 @@ enum acpi_gpio_polarity { * @io_restrict: I/O restriction setting * @polarity: GPIO polarity * - * Note that GpioIo doesn't have any means of Active Low / High setting, so a - * _DSD must be provided to mitigate this. + * Note that GpioIo() doesn't have any means of Active Low / High setting, so a + * _DSD must be provided to mitigate this. This parameter does not make sense + * for GpioInt() since it has its own means to define it. * - * GpioIo doesn't properly communicate the initial state of the output pin, + * GpioIo() doesn't properly communicate the initial state of the output pin, * thus Linux assumes the simple rule: * * Pull Bias Polarity Requested... @@ -184,7 +185,7 @@ enum acpi_gpio_polarity { * assuming non-active (Polarity = !Pull Bias) * * Down Low as low, assuming active - * Down High as high, assuming non-active + * Down High as low, assuming non-active * Up Low as high, assuming non-active * Up High as high, assuming active *