Re: [PATCH] drm/radeon: fix logic error in atombios_encoders.c

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Nov 14, 2012 at 9:32 AM, Michel Dänzer <michel@xxxxxxxxxxx> wrote:
> On Mit, 2012-11-14 at 09:13 -0500, alexdeucher@xxxxxxxxx wrote:
>> From: Alex Deucher <alexander.deucher@xxxxxxx>
>>
>> Fixes:
>> https://bugzilla.kernel.org/show_bug.cgi?id=50431
>>
>> Reported-by: David Binderman <dcb314@xxxxxxxxxxx>
>> Signed-off-by: Alex Deucher <alexander.deucher@xxxxxxx>
>> Cc: stable@xxxxxxxxxxxxxxx
>
> Reviewed-by: Michel Dänzer <michel.daenzer@xxxxxxx>
>
>
>>  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
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel



[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux