On Thu, 20 Oct 2016, Imre Deak <imre.deak@xxxxxxxxx> wrote: > On Thu, 2016-10-20 at 21:20 +0300, Jani Nikula wrote: >> On Thu, 20 Oct 2016, Imre Deak <imre.deak@xxxxxxxxx> wrote: >> > + bool desc_valid; >> > + struct intel_dp_desc desc; >> >> I guess we could cache the desc in intel_dp directly. Independent of >> this patch. > > It's not used anywhere else, but I can move it to intel_dp. > >> >> Also, I'm wondering if we could stick with just aux here, and read >> something else from dpcd instead. > > Not sure either, I picked desc since we read it out anyway during init. That was my point with putting it in intel_dp. If it's read out anyway, lspcon or not, just cache it in intel_dp. BR, Jani. -- Jani Nikula, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
