On Fri, 09 Feb 2018, "Mustaffa, Mustamin B" <mustamin.b.mustaffa@xxxxxxxxx> wrote: > May I know why the need to use connector as connector wasn't > initialized in parent function ' intel_pps_get_registers'? Oh, right, I overlooked that. > While ' dev_priv' already initialized which also already initialized > to the VBT value. So it make sense to me to use 'dev_priv' structure > to read the VBT value instead of connector. Okay, I think the right thing to do in the long run is to refactor the code to initialize the value in the connector earlier. We'll want to use the VBT only for initialization once. But for now, I think this is fine. Please rebase the patch against current upstream so we get fresh CI results, and resend. Thanks, Jani. -- Jani Nikula, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
