Re: Re: HP300 support checked in

On Wed, 14 Jul 2004, Antonino A. Daplas wrote:

On Wednesday 14 July 2004 16:00, Geert Uytterhoeven wrote:

quoted

On Wed, 14 Jul 2004, Antonino A. Daplas wrote:

quoted

Or we can use the fields in var->green, var->red, var->blue, ie,  to
describe a 4-color 8-bit chunky framebuffer:

var->red.length = 2;
var->red.offset = 0;
var->green = var->blue = var->red.
var->bits_per_pixel = 8;

This breaks something different, since for pseudocolor visuals, the
*.lengths indicate the sizes of the CLUT entries (e.g. 8 bit per component
for modern 24-bit color, 6 bit for VGA-style 18-bit color).

I don't know.  For pseudocolor, color depth == CLUT size.  So if we check if all
lengths are equal and if all offsets are equal to zero, then color depth = CLUT size.
Otherwise, color depth = red.length + green.length + blue.length. Transparency
can be ignored for now.

Forget what I said. I was a bit confused, mixing code from many years ago in my
head...

Gr{oetje,eeting}s,

						Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
							    -- Linus Torvalds


-------------------------------------------------------
This SF.Net email sponsored by Black Hat Briefings & Training.
Attend Black Hat Briefings & Training, Las Vegas July 24-29 - 
digital self defense, top technical experts, no vendor pitches, 
unmatched networking opportunities. Visit www.blackhat.com