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

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@csie.org

And I've applied all three patches.

Thanks! Maxime