Mailinglist Archive: radeonhd (312 mails)
| < Previous | Next > |
Re: [radeonhd] Bad switch statement in rhd_atombios.c
- From: Egbert Eich <eich@xxxxxxx>
- Date: Wed, 1 Apr 2009 09:39:59 +0200
- Message-id: <18899.6735.677314.667681@xxxxxxxxxxxxxx>
Matthias Hopf writes:
Yes, I just pushed the fix.
Cheers,
Egbert.
--
To unsubscribe, e-mail: radeonhd+unsubscribe@xxxxxxxxxxxx
For additional commands, e-mail: radeonhd+help@xxxxxxxxxxxx
On Mar 24, 09 16:32:41 -0600, Ales Fiala wrote:
The file rhd_atombios.c has the following statement around line 1553:
switch (Config->u.lvds2.SpatialDither)
lvds->ucSpatial |= PANEL_ENCODER_SPATIAL_DITHER_EN;
My compiler rejects the statement inside the switch as "unreachable".
It's clearly a bug.
Egbert, should that be an 'if'? Or are there multiple potential values?
Yes, I just pushed the fix.
Cheers,
Egbert.
--
To unsubscribe, e-mail: radeonhd+unsubscribe@xxxxxxxxxxxx
For additional commands, e-mail: radeonhd+help@xxxxxxxxxxxx
| < Previous | Next > |