[Xorg-driver-geode] First patch for you review

Chris Ball cjb at laptop.org
Wed Jun 9 07:20:47 PDT 2010


Hi,

   > Put the first patch here.  If that is ok, I'll generate other
   > patches.

Looks great, thanks!  Please do send the rest, inline.

A really minor point is that you sometimes use more parens than
necessary -- you don't need parens around each condition in an if
statement -- but I understand that it's better to have too many
than too few.  :-)

- Chris.
-- 
Chris Ball   <cjb at laptop.org>
One Laptop Per Child


More information about the Xorg-driver-geode mailing list