[PATCH v3 4/5] PCI: imx6: Convert to agnostic GPIO API

Linus Walleij linus.walleij at linaro.org
Mon May 6 05:10:24 PDT 2024


On Mon, Apr 29, 2024 at 12:25 PM Andy Shevchenko
<andriy.shevchenko at linux.intel.com> wrote:

> The of_gpio.h is going to be removed. In preparation of that convert
> the driver to the agnostic API.
>
> Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam at linaro.org>
> Reviewed-by: Frank Li <Frank.Li at nxp.com>
> Signed-off-by: Andy Shevchenko <andriy.shevchenko at linux.intel.com>

I think there is a bug here, the code is respecting the OF property
"reset-gpio-active-high"
but the code in drivers/gpio/gpiolib-of.h actually has a quirk for
this so you can just
delete all the active high handling and rely on 1 = asserted and 0 =
deasserted when
using GPIO descriptors.

Just delete this thing:
imx6_pcie->gpio_active_high = of_property_read_bool(node,
                                           "reset-gpio-active-high");

Yours,
Linus Walleij



More information about the linux-amlogic mailing list