Re: [RFT PATCH 1/4] platform/x86: int3472: provide a helper for getting GPIOs from lookups

From: Andy Shevchenko
Date: Tue Sep 26 2023 - 11:25:59 EST


On Tue, Sep 26, 2023 at 04:59:40PM +0200, Bartosz Golaszewski wrote:
> From: Bartosz Golaszewski <bartosz.golaszewski@xxxxxxxxxx>
>
> gpiod_toggle_active_low() should have never existed in the first place
> and once it was added, it should have never been used outside the MMC
> slot code.
>
> Stop using it in the int3472 driver and use temporary lookup tables
> instead. First: add a helper wrapping the common code in one function.

> #include <linux/clk-provider.h>

...

> +#include <linux/device.h>

No need to have this in the header, use forward declaration instead.

--
With Best Regards,
Andy Shevchenko