drm/omap: display: don't use dsi_get_pixel_size()
authorTomi Valkeinen <tomi.valkeinen@ti.com>
Tue, 10 Nov 2015 13:50:53 +0000 (15:50 +0200)
committerTomi Valkeinen <tomi.valkeinen@ti.com>
Mon, 3 Apr 2017 09:36:40 +0000 (12:36 +0300)
display.c uses dsi_get_pixel_size() which is implemented in the DSI
driver, and we won't have that in the omapdss-base module, to which we
want to move display.c

This patch changes display.c not to use dsi_get_pixel_size(). The call
can be replaced with a simple check for OMAP_DSS_DSI_FMT_RGB565.

We can also make dsi_get_pixel_size() static as it's no longer used
outside dsi.c.

Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
drivers/gpu/drm/omapdrm/dss/display.c
drivers/gpu/drm/omapdrm/dss/dsi.c
drivers/gpu/drm/omapdrm/dss/dss.h

index 425a5a8dff8bff04753489b38da3515463cd4fc4..333ba284ef6d29f8b44641a589736e77f3e8bc44 100644 (file)
@@ -55,10 +55,10 @@ int omapdss_default_get_recommended_bpp(struct omap_dss_device *dssdev)
                else
                        return 16;
        case OMAP_DISPLAY_TYPE_DSI:
-               if (dsi_get_pixel_size(dssdev->panel.dsi_pix_fmt) > 16)
-                       return 24;
-               else
+               if (dssdev->panel.dsi_pix_fmt == OMAP_DSS_DSI_FMT_RGB565)
                        return 16;
+               else
+                       return 24;
        case OMAP_DISPLAY_TYPE_VENC:
        case OMAP_DISPLAY_TYPE_SDI:
        case OMAP_DISPLAY_TYPE_HDMI:
index 5e51a5649efb8e0651116a8e98d262669d646f7e..f50d6fc0d92ea3a22e5f5db1b13942fd9bf7653c 100644 (file)
@@ -527,7 +527,7 @@ static inline int wait_for_bit_change(struct platform_device *dsidev,
        return !value;
 }
 
-u8 dsi_get_pixel_size(enum omap_dss_dsi_pixel_format fmt)
+static u8 dsi_get_pixel_size(enum omap_dss_dsi_pixel_format fmt)
 {
        switch (fmt) {
        case OMAP_DSS_DSI_FMT_RGB888:
index 78f6fc75948b511da2dfc85a5966200caa679391..4d568ac76826befd45061180dafd86f29bd300f9 100644 (file)
@@ -315,15 +315,7 @@ void dsi_uninit_platform_driver(void);
 void dsi_dump_clocks(struct seq_file *s);
 
 void dsi_irq_handler(void);
-u8 dsi_get_pixel_size(enum omap_dss_dsi_pixel_format fmt);
 
-#else
-static inline u8 dsi_get_pixel_size(enum omap_dss_dsi_pixel_format fmt)
-{
-       WARN(1, "%s: DSI not compiled in, returning pixel_size as 0\n",
-            __func__);
-       return 0;
-}
 #endif
 
 /* DPI */