[U-Boot] [PATCH 2/5] dm: video: bridge: don't fail to activate bridge if sleep gpio is missing

Vasily Khoruzhick anarsoul at gmail.com
Sun Sep 30 06:02:54 UTC 2018


Hi Vagrant,

On Sat, Sep 29, 2018 at 4:46 PM Vagrant Cascadian <vagrant at debian.org> wrote:
>
> From: Vasily Khoruzhick <anarsoul at gmail.com>
>
> Sleep gpio is optional, so it's possible to have reset gpio, but no sleep gpio.
> We shouldn't fail early in case of missing sleep gpio, otherwise we won't
> deassert reset.
>
> Signed-off-by: Vasily Khoruzhick <anarsoul at gmail.com>
> Signed-off-by: Vagrant Cascadian <vagrant at debian.org>
> ---
>
>  drivers/video/bridge/video-bridge-uclass.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/video/bridge/video-bridge-uclass.c b/drivers/video/bridge/video-bridge-uclass.c
> index cd4959cc71..46936a0626 100644
> --- a/drivers/video/bridge/video-bridge-uclass.c
> +++ b/drivers/video/bridge/video-bridge-uclass.c
> @@ -110,7 +110,7 @@ int video_bridge_set_active(struct udevice *dev, bool active)
>
>         debug("%s: %d\n", __func__, active);
>         ret = dm_gpio_set_value(&uc_priv->sleep, !active);
> -       if (ret)
> +       if (ret != -ENOENT)

It should be 'if (ret && ret != -ENOENT)'. Btw, I fixed it in
pinebook-wip-20180909 branch.

>                 return ret;
>         if (active) {
>                 ret = dm_gpio_set_value(&uc_priv->reset, true);
> @@ -120,7 +120,7 @@ int video_bridge_set_active(struct udevice *dev, bool active)
>                 ret = dm_gpio_set_value(&uc_priv->reset, false);
>         }
>
> -       return ret;
> +       return 0;
>  }
>
>  UCLASS_DRIVER(video_bridge) = {
> --
> 2.11.0
>


More information about the U-Boot mailing list