Stefano Stabellini <stefano.stabell...@eu.citrix.com> writes: > On Tue, 22 Nov 2011, Markus Armbruster wrote: >> Case 24 falls through to case 32, which overwrites all of case 24's >> work. Looks wrong. Git blames commit 0da2ea1b "fix endianness problem >> sharing the videoram buffer". >> >> Patching in a break is easy enough, but since I don't actually >> understand how the function is used, I have no idea what exactly this >> fixes. Stefano, can you help? > > You are right, there should be a break statement there. > However it doesn't matter in practice because the 24 and 32 bit settings > are the same (amask, ashift and abits don't matter much).
I'll post the obvious patch then. Thanks!
