On Tue, Feb 20, 2018 at 11:43:57AM +0200, Jani Nikula wrote: > On Tue, 20 Feb 2018, Chris Wilson <[email protected]> wrote: > > Quoting Mustaffa, Mustamin B (2018-02-20 08:44:45) > >> Hi Chris, > >> > >> Would you rather for me to use following line instead? > >> > >> + int backlight_controller = > >> intel_dp->attached_connector->panel.backlight.controller; > > > > I think so, Jani would be best to answer the question about how > > vbt/panel tie together with backlight and which is meant to be used. > > We discussed this previously, and the following sequence would lead to > panel.backlight.controller being used uninitialized in > bxt_power_sequencer_idx because it gets set at > intel_panel_setup_backlight: > > - intel_edp_init_connector > +- intel_dp_pps_init > | +- intel_dp_init_panel_power_sequencer_registers > | +- intel_pps_get_registers > | +- bxt_power_sequencer_idx > +- intel_panel_setup_backlight > > I decided it wasn't worth blocking a reasonable fix (that might warrant > cc: stable actually) on a refactoring. We can (and should) do the > refactoring later though.
Oh! please ignore the email I just sent... I hadn't noticed this discussion here before... So, with this info here: Reviewed-by: Rodrigo Vivi <[email protected]> > > BR, > Jani. > > > -- > Jani Nikula, Intel Open Source Technology Center > _______________________________________________ > Intel-gfx mailing list > [email protected] > https://lists.freedesktop.org/mailman/listinfo/intel-gfx _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
