[U-Boot] [PATCH] video: meson: hdmi-supply regulator should be optional

Neil Armstrong narmstrong at baylibre.com
Wed Jun 5 07:24:06 UTC 2019


On 04/06/2019 22:26, 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 <mohammad.rasim96 at gmail.com>
> Signed-off-by: Maxime Jourdan <mjourdan at baylibre.com>
> ---
>  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)
> 

Good catch !

Reviewed-by: Neil Armstrong <narmstrong at baylibre.com>


More information about the U-Boot mailing list