>>>>> "walter" == walter harms <[email protected]> writes:
Hi,
>> static void fb_write_pixel(unsigned char *addr, unsigned pixel)
>> {
>> switch (G.bytes_per_pixel) {
>> + case 1:
>> + *addr = pixel;
>> + break;
>> case 2:
>> *(uint16_t *)addr = pixel;
>> break;
walter> this looks confusing, would you mind to add a few words why
walter> this cases are so different ? i am not an expert on
walter> fb-devices, sorry.
I'm not quite sure what's so confusing about it. It's basically just a
straight extension of the existing 16/24/32bit support. The only thing
somewhat special is that 8bit modes have palettes (color lookup tables),
that the patch sets up for RGB:332 format (E.G. 3 bits for red, 3 bits
for green and 2 bits for blue).
Could you be more specific about what you find confusing about it?
--
Bye, Peter Korsgaard
_______________________________________________
busybox mailing list
[email protected]
http://lists.busybox.net/mailman/listinfo/busybox
