On 19/06/04 10:26PM, Maxime Jourdan wrote:
> Some boards don't have such a regulator, and don't need one to enable
> HDMI display. Make it optional, fixing hdmi display for those boards.
>
> Also surround the regulator code with a config check on DM_REGULATOR.
>
> Reported-by: Mohammad Rasim <[email protected]>
> Signed-off-by: Maxime Jourdan <[email protected]>
> ---
> drivers/video/meson/meson_dw_hdmi.c | 14 ++++++++++----
> 1 file changed, 10 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/video/meson/meson_dw_hdmi.c
> b/drivers/video/meson/meson_dw_hdmi.c
> index 7a1c060856..483c93f6b6 100644
> --- a/drivers/video/meson/meson_dw_hdmi.c
> +++ b/drivers/video/meson/meson_dw_hdmi.c
> @@ -361,13 +361,19 @@ static int meson_dw_hdmi_probe(struct udevice *dev)
> priv->hdmi.i2c_clk_high = 0x67;
> priv->hdmi.i2c_clk_low = 0x78;
>
> +#if CONFIG_IS_ENABLED(DM_REGULATOR)
> ret = device_get_supply_regulator(dev, "hdmi-supply", &supply);
> - if (ret)
> + if (ret && ret != -ENOENT) {
> + pr_err("Failed to get HDMI regulator\n");
> return ret;
> + }
>
> - ret = regulator_set_enable(supply, true);
> - if (ret)
> - return ret;
> + if (!ret) {
> + ret = regulator_set_enable(supply, true);
> + if (ret)
> + return ret;
> + }
> +#endif
>
> ret = reset_get_bulk(dev, &resets);
> if (ret)
> --
> 2.21.0
>
Tested-by: Mohammad Rasim <[email protected]>
_______________________________________________
U-Boot mailing list
[email protected]
https://lists.denx.de/listinfo/u-boot
