On Fri, 09 Feb 2018, "Mustaffa, Mustamin B" <mustamin.b.musta...@intel.com> 
> 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.


Jani Nikula, Intel Open Source Technology Center
Intel-gfx mailing list

Reply via email to