On Tue, 17 Mar 2026, Chad Jablonski wrote:
This works in the 24/32bpp case. Before this I see a yellow background in
fluxbox, after I see gray.

I did notice that the existing big_endian_fb boolean in ati_vga_switch_mode
doesn't distinguish between big endian 32-bit and 16-bit swapping. In the 16bpp
case I see a black background in fluxbox. I did a quick and dirty test and
replaced the bswap32s with a 16-bit byte swap and then 16bpp displays the gray
background correctly.

This is a pre-existing issue but I thought it was worth pointing out. That said
this fixes things for 24/32bpp and is an improvement.

Tested-by: Chad Jablonski <[email protected]>
Reviewed-by: Chad Jablonski <[email protected]>

Thanks for testing and review. Yes you're right I should have take into account bpp too. I'll have a look to see if I can fix this and send a new version.

Regards,
BALATON Zoltan

Reply via email to