On Thu, Apr 19, 2012 at 10:45 AM, Dave Airlie <airlied@xxxxxxxxx> wrote: > From: Dave Airlie <airlied@xxxxxxxxxx> > > When the force changes went in back in 3.3.0, we ended up returning > disconnected in the !force case, and the connected in when forced, > as it hit the hardcoded check. > > Fix it so all exits go via the hardcoded check and stop spurious > modesets on platforms with hardcoded EDIDs. > > Reported-by: Evan McNabb (Red Hat) > Signed-off-by: Dave Airlie <airlied@xxxxxxxxxx> Reviewed-by: Alex Deucher <alexander.deucher@xxxxxxx> Should probably cc stable as well. > --- > drivers/gpu/drm/radeon/radeon_connectors.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/drivers/gpu/drm/radeon/radeon_connectors.c b/drivers/gpu/drm/radeon/radeon_connectors.c > index aa8268d..3c2e7a0 100644 > --- a/drivers/gpu/drm/radeon/radeon_connectors.c > +++ b/drivers/gpu/drm/radeon/radeon_connectors.c > @@ -1000,6 +1000,7 @@ radeon_dvi_detect(struct drm_connector *connector, bool force) > * cases the DVI port is actually a virtual KVM port connected to the service > * processor. > */ > +out: > if ((!rdev->is_atom_bios) && > (ret == connector_status_disconnected) && > rdev->mode_info.bios_hardcoded_edid_size) { > @@ -1007,7 +1008,6 @@ radeon_dvi_detect(struct drm_connector *connector, bool force) > ret = connector_status_connected; > } > > -out: > /* updated in get modes as well since we need to know if it's analog or digital */ > radeon_connector_update_scratch_regs(connector, ret); > return ret; > -- > 1.7.7.6 > > _______________________________________________ > 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