[PATCH 1/3] drm/sun4i: tcon: Reduce the scope of the LVDS error a bit

Maxime Ripard maxime.ripard at bootlin.com
Wed Mar 7 07:49:09 UTC 2018


Hi,

On Tue, Mar 06, 2018 at 03:38:25PM +0800, Chen-Yu Tsai wrote:
> > -       /*
> > -        * This can only be made optional since we've had DT nodes
> > -        * without the LVDS reset properties.
> > -        *
> > -        * If the property is missing, just disable LVDS, and print a
> > -        * warning.
> > -        */
> > -       if (tcon->quirks->has_lvds_alt) {
> > -               tcon->lvds_pll = devm_clk_get(dev, "lvds-alt");
> > -               if (IS_ERR(tcon->lvds_pll)) {
> > -                       if (PTR_ERR(tcon->lvds_pll) == -ENOENT) {
> > -                               has_lvds_alt = false;
> > +               /*
> > +                * This can only be made optional since we've had DT
> > +                * nodes without the LVDS reset properties.
> 
> This comment doesn't match the code following it. Otherwise,

It was there before, and it was a typo. I'll send a followup patch

> Reviewed-by: Chen-Yu Tsai <wens at csie.org>

And I've applied all three patches.

Thanks!
Maxime

-- 
Maxime Ripard, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20180307/1efe6916/attachment.sig>


More information about the dri-devel mailing list