Thread (11 messages) 11 messages, 2 authors, 2013-12-04

Re: [PATCH v2] offb: make the screen properties endian safe

From: Cedric Le Goater <hidden>
Date: 2013-12-04 16:47:50

On 11/23/2013 10:04 PM, Benjamin Herrenschmidt wrote:
On Sat, 2013-11-23 at 18:38 +0100, Cedric Le Goater wrote:
quoted
So, 32bpp "works" but 16 is broken ... I guess my palette fix is just a lucky
hack and I need to dig deeper in fb code to have a better understanding of
the color map. 

I should have provided you two patches in the first place. Do you want the 
device tree data fixes for the frame buffer screen properties ? It helps to 
have a display for little endian guests even if the colors are wrong.
Before you pull your hair out, check if it works in BE :-)
I spent some more time on this topic and I have a few more patches to 
send you.

The first patch is straight forward. It fixes host endian order issues 
in the offb code when accessing the OF device tree properties. 

The following patch is more of an attempt to fix the pseudo-palette 
entries on little endian. Hopefully this is the good direction. If not, 
I still have some hair to pull out :)

Cheers,

C.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help