Benjamin Herrenschmidt wrote:

Well, the interesting thing is does X using "fbdev" has proper colors
(or X using "ati" driver with UseFBDev option). If yes, then that code
is right. It's the console/blit code that is wrong.
 

Confirmed using fbtest.  With that section commented out fbtest's colors 
are messed up.

Hmmm....  Just found something.  Here's an updated patch.  Notice the 6 
changing to a 5 in the 16 bit case.  This makes the console and fbtest 
look right.... except for the penguins.  Their colors are messed up 
depending on the test order.

John