On Fri, May 4, 2012 at 11:06 AM, <j.glisse@xxxxxxxxx> wrote: > From: Jerome Glisse <jglisse@xxxxxxxxxx> > > It seems imac pannel doesn't like whe we change the hot plug setup > and then refuse to work. This help but doesn't fully fix: > https://bugzilla.redhat.com/show_bug.cgi?id=726143 How does it help? Does it fix the the aux problems, but the monitor still doesn't train? What's the working value of the relevant DC_HPD*_CONTROL register? Alex > > v2: fix typo and improve commit message > > Signed-off-by: Matthew Garrett <mjg@xxxxxxxxxx> > Signed-off-by: Jerome Glisse <jglisse@xxxxxxxxxx> > --- > drivers/gpu/drm/radeon/r600.c | 8 ++++++++ > 1 file changed, 8 insertions(+) > > diff --git a/drivers/gpu/drm/radeon/r600.c b/drivers/gpu/drm/radeon/r600.c > index 694b6b2..a304c9d 100644 > --- a/drivers/gpu/drm/radeon/r600.c > +++ b/drivers/gpu/drm/radeon/r600.c > @@ -713,6 +713,14 @@ void r600_hpd_init(struct radeon_device *rdev) > list_for_each_entry(connector, &dev->mode_config.connector_list, head) { > struct radeon_connector *radeon_connector = to_radeon_connector(connector); > > + if (connector->connector_type == DRM_MODE_CONNECTOR_eDP || > + connector->connector_type == DRM_MODE_CONNECTOR_LVDS) { > + /* don't try to enable hpd on eDP or LVDS avoid breaking the > + * aux dp channel on imac and help (but not completely fix) > + * https://bugzilla.redhat.com/show_bug.cgi?id=726143 > + */ > + continue; > + } > if (ASIC_IS_DCE3(rdev)) { > u32 tmp = DC_HPDx_CONNECTION_TIMER(0x9c4) | DC_HPDx_RX_INT_TIMER(0xfa); > if (ASIC_IS_DCE32(rdev)) > -- > 1.7.9.3 > > _______________________________________________ > dri-devel mailing list > dri-devel@xxxxxxxxxxxxxxxxxxxxx > http://lists.freedesktop.org/mailman/listinfo/dri-devel _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel