On Mon, 10 Aug 2015, Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> wrote: > Reviewed-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> > > On 8/10/2015 10:35 AM, Sonika Jindal wrote: >> With HPD support added for all ports including PORT_A, setting hpd_pin will >> result in enabling of hpd to edp as well. There is no need to enable HPD on >> PORT_A hence this patch removes hpd_pin update for PORT_A, where edp will >> be connected. it can be added back when required
What? You can't just go ahead and remove HPD from eDP sinks. BR, Jani. >> >> Signed-off-by: Sonika Jindal <sonika.jin...@intel.com> >> --- >> drivers/gpu/drm/i915/intel_dp.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/gpu/drm/i915/intel_dp.c >> b/drivers/gpu/drm/i915/intel_dp.c >> index fcc64e5..5a614c9 100644 >> --- a/drivers/gpu/drm/i915/intel_dp.c >> +++ b/drivers/gpu/drm/i915/intel_dp.c >> @@ -5784,7 +5784,7 @@ intel_dp_init_connector(struct intel_digital_port >> *intel_dig_port, >> /* Set up the hotplug pin. */ >> switch (port) { >> case PORT_A: >> - intel_encoder->hpd_pin = HPD_PORT_A; >> + /* Not enabling edp interrupts */ >> break; >> case PORT_B: >> intel_encoder->hpd_pin = HPD_PORT_B; > > -- > regards, > Sivakumar > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Jani Nikula, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx