[PATCH v3] PCI: imx6: Replace legacy gpio interface for gpiod interface
Linus Walleij
linus.walleij at linaro.org
Wed Apr 20 16:24:51 PDT 2022
Hi Maira and sorry for being slow on reviews.
On Tue, Nov 2, 2021 at 2:04 AM Maíra Canal <maira.canal at usp.br> wrote:
> - gpio_set_value_cansleep(imx6_pcie->reset_gpio,
> + gpiod_set_raw_value_cansleep(imx6_pcie->reset_gpio,
> !imx6_pcie->gpio_active_high);
Hm I see you got advised to use the raw api. I'm not so sure about
that I like v1 better.
> + imx6_pcie->reset_gpio = devm_gpiod_get_optional(dev, "reset",
> + imx6_pcie->gpio_active_high ? GPIOD_OUT_HIGH : GPIOD_OUT_LOW);
> + if (IS_ERR(imx6_pcie->reset_gpio))
> + return dev_err_probe(dev, PTR_ERR(imx6_pcie->reset_gpio),
> + "unable to get reset gpio\n");
Where is this descriptor coming from? Device trees? Can't we just fix the
DTS file(s) in that case given how wrong they are if they don't set
GPIO_ACTIVE_LOW flag on this IRQ.
Yours,
Linus Walleij
More information about the linux-arm-kernel
mailing list