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