video: fbdev: atari: Atari 2 bpp (STe) palette bugfix
authorMichael Schmitz <schmitzmic@gmail.com>
Wed, 16 Feb 2022 07:26:25 +0000 (20:26 +1300)
committerHelge Deller <deller@gmx.de>
Wed, 16 Feb 2022 08:11:12 +0000 (09:11 +0100)
commitc8be5edbd36ceed2ff3d6b8f8e40643c3f396ea3
tree447f6b9cd1471123102e011f0bd6eb0576b1e490
parentfdaae9fefb95b73d47fec00b40d688b2931076d1
video: fbdev: atari: Atari 2 bpp (STe) palette bugfix

The code to set the shifter STe palette registers has a long
standing operator precedence bug, manifesting as colors set
on a 2 bits per pixel frame buffer coming up with a distinctive
blue tint.

Add parentheses around the calculation of the per-color palette
data before shifting those into their respective bit field position.

This bug goes back a long way (2.4 days at the very least) so there
won't be a Fixes: tag.

Tested on ARAnyM as well on Falcon030 hardware.

Cc: stable@vger.kernel.org
Reported-by: Geert Uytterhoeven <geert@linux-m68k.org>
Link: https://lore.kernel.org/all/CAMuHMdU3ievhXxKR_xi_v3aumnYW7UNUO6qMdhgfyWTyVSsCkQ@mail.gmail.com
Tested-by: Michael Schmitz <schmitzmic@gmail.com>
Tested-by: Geert Uytterhoeven <geert@linux-m68k.org>
Signed-off-by: Michael Schmitz <schmitzmic@gmail.com>
Signed-off-by: Helge Deller <deller@gmx.de>
drivers/video/fbdev/atafb.c