On Wed, Nov 14, 2012 at 9:32 AM, Michel Dänzer <[email protected]> wrote: > On Mit, 2012-11-14 at 09:13 -0500, [email protected] wrote: >> From: Alex Deucher <[email protected]> >> >> Fixes: >> https://bugzilla.kernel.org/show_bug.cgi?id=50431 >> >> Reported-by: David Binderman <[email protected]> >> Signed-off-by: Alex Deucher <[email protected]> >> Cc: [email protected] > > Reviewed-by: Michel Dänzer <[email protected]> > > >> drivers/gpu/drm/radeon/atombios_encoders.c | 2 +- >> 1 files changed, 1 insertions(+), 1 deletions(-) >> >> diff --git a/drivers/gpu/drm/radeon/atombios_encoders.c >> b/drivers/gpu/drm/radeon/atombios_encoders.c >> index ba498f8..010bae1 100644 >> --- a/drivers/gpu/drm/radeon/atombios_encoders.c >> +++ b/drivers/gpu/drm/radeon/atombios_encoders.c >> @@ -1625,7 +1625,7 @@ radeon_atom_encoder_dpms_dig(struct drm_encoder >> *encoder, int mode) >> atombios_dig_transmitter_setup(encoder, >> ATOM_TRANSMITTER_ACTION_SETUP, 0, 0); >> atombios_dig_transmitter_setup(encoder, >> ATOM_TRANSMITTER_ACTION_ENABLE, 0, 0); >> /* some early dce3.2 boards have a bug in their >> transmitter control table */ >> - if ((rdev->family != CHIP_RV710) || (rdev->family != >> CHIP_RV730)) >> + if ((rdev->family != CHIP_RV710) && (rdev->family != >> CHIP_RV730)) > > Ouch, the compiler didn't warn about this always evaluating to true?
Nope. At least not on any of my machines. > > > -- > Earthling Michel Dänzer | http://www.amd.com > Libre software enthusiast | Debian, X and DRI developer _______________________________________________ dri-devel mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/dri-devel
