From: Simon Glass Date: Sat, 23 Jan 2021 18:08:38 +0000 (-0700) Subject: acpi: Tidy up documentation for struct acpi_gpio X-Git-Tag: v2021.10~335^2~11 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=8a655272d7b440dc3eeda598eb2d05819c0f75c2;p=platform%2Fkernel%2Fu-boot.git 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 --- diff --git a/include/acpi/acpi_device.h b/include/acpi/acpi_device.h index ed4acd9..2c88462 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 *