On Wed, Sep 04, 2024 at 11:52:06AM GMT, Jani Nikula wrote:
> ARCH=arm build fails with:
>
> CC [M] drivers/gpu/drm/bridge/ti-tdp158.o
> ../drivers/gpu/drm/bridge/ti-tdp158.c: In function ‘tdp158_enable’:
> ../drivers/gpu/drm/bridge/ti-tdp158.c:31:9: error: implicit declaration of
> function ‘gpiod_set_value_cansleep’ [-Werror=implicit-function-declaration]
> 31 | gpiod_set_value_cansleep(tdp158->enable, 1);
> | ^~~~~~~~~~~~~~~~~~~~~~~~
> ../drivers/gpu/drm/bridge/ti-tdp158.c: In function ‘tdp158_probe’:
> ../drivers/gpu/drm/bridge/ti-tdp158.c:80:26: error: implicit declaration of
> function ‘devm_gpiod_get_optional’; did you mean
> ‘devm_regulator_get_optional’? [-Werror=implicit-function-declaration]
> 80 | tdp158->enable = devm_gpiod_get_optional(dev, "enable",
> GPIOD_OUT_LOW);
> | ^~~~~~~~~~~~~~~~~~~~~~~
> | devm_regulator_get_optional
> ../drivers/gpu/drm/bridge/ti-tdp158.c:80:65: error: ‘GPIOD_OUT_LOW’
> undeclared (first use in this function)
> 80 | tdp158->enable = devm_gpiod_get_optional(dev, "enable",
> GPIOD_OUT_LOW);
> |
> ^~~~~~~~~~~~~
> ../drivers/gpu/drm/bridge/ti-tdp158.c:80:65: note: each undeclared identifier
> is reported only once for each function it appears in
>
> Add the proper gpio consumer #include to fix this, and juggle the
> include order to be a bit more pleasant on the eye while at it.
>
> Fixes: a15710027afb ("drm/bridge: add support for TI TDP158")
> Cc: Marc Gonzalez <mgonza...@freebox.fr>
> Cc: Robert Foss <rf...@kernel.org>
> Cc: Andrzej Hajda <andrzej.ha...@intel.com>
> Cc: Neil Armstrong <neil.armstr...@linaro.org>
> Cc: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
> Cc: Jonas Karlman <jo...@kwiboo.se>
> Cc: Jernej Skrabec <jernej.skra...@gmail.com>
> Signed-off-by: Jani Nikula <jani.nik...@intel.com>
> ---
> drivers/gpu/drm/bridge/ti-tdp158.c | 7 +++++--
> 1 file changed, 5 insertions(+), 2 deletions(-)
>
Reviewed-by: Dmitry Baryshkov <dmitry.barysh...@linaro.org>
--
With best wishes
Dmitry
