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:
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 >
This Thread
  • No further messages