Re: [PATCH v6 1/2] drm/bridge: Add sii902x driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Boris,

On 7 June 2016 at 12:59, Boris Brezillon
<boris.brezillon@xxxxxxxxxxxxxxxxxx> wrote:
> Add basic support for the sii902x RGB -> HDMI bridge.
> This driver does not support audio output yet.
>
> Signed-off-by: Boris Brezillon <boris.brezillon@xxxxxxxxxxxxxxxxxx>
> Tested-by: Nicolas Ferre <nicolas.ferre@xxxxxxxxx>
> ---
> Hello,
>
> This patch is only adding basic support for the sii9022 chip.
> As stated in the commit log, there's no audio support, but the
> driver also hardcodes a lot of things (like the RGB input format to
> use).
>
> Best Regards,
>
> Boris
>
> Changes in v6:
> - use HDMI_INFOFRAME_SIZE(AVI)
> - fix reset_gpio initialization
I'm afraid that this one is still busted :-( See below


> +       sii902x->reset_gpio = devm_gpiod_get_optional(dev, "reset",
> +                                                     GPIOD_OUT_LOW);
Documentation states required, above we use optional.

> +       if (IS_ERR(sii902x->reset_gpio)) {
devm_gpiod_get_optional returns NULL on error, yet we use IS_ERR.

> +               dev_err(dev, "Failed to retrieve/request reset gpio: %ld\n",
> +                       PTR_ERR(sii902x->reset_gpio));
> +               return PTR_ERR(sii902x->reset_gpio);
If the gpio is truly optional we shouldn't return here. Alternatively
the reset_gpio NULL check in sii902x_reset() can go.

Regards,
Emil
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux