drm/msm: Fix fbdev for 16- and 24-bit modes.
authorDaniel Thompson <daniel.thompson@linaro.org>
Fri, 17 Oct 2014 15:48:54 +0000 (16:48 +0100)
committerRob Clark <robdclark@gmail.com>
Sun, 16 Nov 2014 19:22:43 +0000 (14:22 -0500)
Currently forcing the video mode from the kernel command line (for example
video=HDMI-A-1:1280x720-16@60) does not correctly set the number of bits
per pixel. This is due to a rather aggressive override in
msm_fbdev_create(). This is a particular problem for Android bring up
because the software EGL fallbacks don't support 32bpp.

Since the overrides are actually the default values anyway then this
problem can be trivially fixed by removing the overrides completely.

Change was tested by dd'ing a test image to /dev/fb0 with no video=
(still 32bpp), video=1920x1080-32@60, video=1920x1080-24@60 and
video=1920x1080-16@60 .

Signed-off-by: Daniel Thompson <daniel.thompson@linaro.org>
Cc: David Airlie <airlied@linux.ie>
Cc: Rob Clark <robdclark@gmail.com>
Signed-off-by: Rob Clark <robdclark@gmail.com>
drivers/gpu/drm/msm/msm_fbdev.c

index ab5bfd2d0ebf2e29897fa56e1df2c77c0f47aeb8..94d55e526b4e0732cba7fcf45a2ce725c2433637 100644 (file)
@@ -93,9 +93,6 @@ static int msm_fbdev_create(struct drm_fb_helper *helper,
        uint32_t paddr;
        int ret, size;
 
-       sizes->surface_bpp = 32;
-       sizes->surface_depth = 24;
-
        DBG("create fbdev: %dx%d@%d (%dx%d)", sizes->surface_width,
                        sizes->surface_height, sizes->surface_bpp,
                        sizes->fb_width, sizes->fb_height);