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

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

 



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@xxxxxxxx>

And I've applied all three patches.

Thanks!
Maxime

-- 
Maxime Ripard, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com

Attachment: signature.asc
Description: PGP signature

_______________________________________________
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