<div dir="ltr"><br><br>On Fri, Jan 19, 2018 at 10:48 AM, Paulo Zanoni <<a href="mailto:paulo.r.zanoni@intel.com">paulo.r.zanoni@intel.com</a>> wrote:<br>><br>> From: Anusha Srivatsa <<a href="mailto:anusha.srivatsa@intel.com">anusha.srivatsa@intel.com</a>><br>><br>> ICP has two backlight controllers - similar to previous platforms like<br>> BXT -, but we only use one controller for now, so we can just reuse<br>> the CNP code.<br>><br>> v2: Remove the usage of ICP_SECOND_PPS_BACKLIGHT register.(Jani)<br>> Reuse CNP code since it is very similar.(Ville)<br>> v3 (from Paulo): Rebase.<br>> v4 (from Paulo): adjust commit message (James) and comment (Rodrigo).<br>><br>> Cc: Jani Nikula <<a href="mailto:jani.nikula@intel.com">jani.nikula@intel.com</a>><br>> Cc: Ville Syrjala <<a href="mailto:ville.syrjala@linux.intel.com">ville.syrjala@linux.intel.com</a>><br>> Cc: James Ausmus <<a href="mailto:james.ausmus@intel.com">james.ausmus@intel.com</a>><br>> Cc: Rodrigo Vivi <<a href="mailto:rodrigo.vivi@intel.com">rodrigo.vivi@intel.com</a>><br>> Reviewed-by: Paulo Zanoni <<a href="mailto:paulo.r.zanoni@intel.com">paulo.r.zanoni@intel.com</a>><br>> Signed-off-by: Anusha Srivatsa <<a href="mailto:anusha.srivatsa@intel.com">anusha.srivatsa@intel.com</a>><br>> Signed-off-by: Paulo Zanoni <<a href="mailto:paulo.r.zanoni@intel.com">paulo.r.zanoni@intel.com</a>><br><br>Looks good, thanks!<br><br>Acked-by: James Ausmus <<a href="mailto:james.ausmus@intel.com">james.ausmus@intel.com</a>><br><br> <br>><br>> ---<br>>  drivers/gpu/drm/i915/intel_panel.c | 8 ++++----<br>>  1 file changed, 4 insertions(+), 4 deletions(-)<br>><br>> diff --git a/drivers/gpu/drm/i915/intel_panel.c b/drivers/gpu/drm/i915/intel_panel.c<br>> index fa6831f8c004..e702a6487aa9 100644<br>> --- a/drivers/gpu/drm/i915/intel_panel.c<br>> +++ b/drivers/gpu/drm/i915/intel_panel.c<br>> @@ -1719,9 +1719,9 @@ cnp_setup_backlight(struct intel_connector *connector, enum pipe unused)<br>>         u32 pwm_ctl, val;<br>><br>>         /*<br>> -        * CNP has the BXT implementation of backlight, but with only<br>> -        * one controller. Future platforms could have multiple controllers<br>> -        * so let's make this extensible and prepared for the future.<br>> +        * CNP has the BXT implementation of backlight, but with only one<br>> +        * controller. TODO: ICP has multiple controllers but we only use<br>> +        * controller 0 for now.<br>>          */<br>>         panel->backlight.controller = 0;<br>><br>> @@ -1865,7 +1865,7 @@ intel_panel_init_backlight_funcs(struct intel_panel *panel)<br>>                 panel->backlight.set = bxt_set_backlight;<br>>                 panel->backlight.get = bxt_get_backlight;<br>>                 panel->backlight.hz_to_pwm = bxt_hz_to_pwm;<br>> -       } else if (HAS_PCH_CNP(dev_priv)) {<br>> +       } else if (HAS_PCH_CNP(dev_priv) || HAS_PCH_ICP(dev_priv)) {<br>>                 panel->backlight.setup = cnp_setup_backlight;<br>>                 panel->backlight.enable = cnp_enable_backlight;<br>>                 panel->backlight.disable = cnp_disable_backlight;<br>> --<br>> 2.14.3<br>><br><br><br><br>--<br><br><br>James Ausmus</div>