xf86-video-geode: Changes to 'master'

Adam Jackson ajax at nwnk.net
Thu Sep 30 07:13:54 PDT 2010


On Thu, 2010-09-30 at 15:49 +0800, Huang, FrankR wrote:
> So we can keep the current git code with no change.

You can, but the code is still _wrong_.  Wrong wrong wrong.  Your
hardware has real constraints; you have finite memory bandwidth, and
finite DAC speed, and finite CRTC addressibility.  If I try this
modeline on your hardware:

atropine:~% gtf 1920 1440 120

  # 1920x1440 @ 120.00 Hz (GTF) hsync: 185.16 kHz; pclk: 497.71 MHz
  Modeline "1920x1440_120.00"  497.71  1920 2088 2304 2688  1440 1441 1444 1543  -HSync +Vsync

it will not work.  The code as you've now changed it will happily accept
this mode.

And your commit message is wrong too.  The other drivers you mention
check for failure conditions before returning MODE_OK.  You're checking
for _success_ conditions and then returning MODE_OK all the time.
Textually these look very similar; semantically they're completely
different.

Think.  Please.

- ajax
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL: <http://lists.x.org/archives/xorg-devel/attachments/20100930/7281c129/attachment.pgp>


More information about the xorg-devel mailing list