video: da8xx-fb: fix 24bpp raster configuration

Based on original patch by: Manjunathappa, Prakash <prakash.pm@ti.com>
	and Afzal Mohammed <afzal@ti.com>

Set only LCD_V2_TFT_24BPP_MODE bit for 24bpp and LCD_V2_TFT_24BPP_UNPACK
bit along with LCD_V2_TFT_24BPP_MODE for 32bpp configuration.

Patch is tested on am335x-evm for 24bpp and da850-evm for 16bpp
configurations.

v2: removes confusing fall through in case statement for pixel
	depth configuration.

Signed-off-by: Darren Etheridge <detheridge@ti.com>
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>

diff --git a/drivers/video/da8xx-fb.c b/drivers/video/da8xx-fb.c
index 93753828eb08fbacf1936ce2d0bb7fdbe61d20cb..893aefe2547899f6c65690565d17e073737aa80a 100644 (file)
--- a/drivers/video/da8xx-fb.c
+++ b/drivers/video/da8xx-fb.c
@@ -554,10 +554,11 @@ static int lcd_cfg_frame_buffer(struct da8xx_fb_par *par, u32 width, u32 height,
                break;
        case 24:
                reg |= LCD_V2_TFT_24BPP_MODE;
+               break;
        case 32:
+               reg |= LCD_V2_TFT_24BPP_MODE;
                reg |= LCD_V2_TFT_24BPP_UNPACK;
                break;
-
        case 8:
                par->palette_sz = 256 * 2;
                break;