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 <w...@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

Attachment: signature.asc
Description: PGP signature

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

Reply via email to