hollalinux
/
linux-sg2042
mirror of https://github.com/sophgo/linux-riscv.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

drm/i915: fixup fb bpp computation in pipe_config_set_bpp 

Ville pointed out that my assumption that no unsupported pixel format
can get past the pipe config computation stage to the platform
update_plane callbacks is wrong. The reason is that this function
still checks the old fb->depth value instead of the new pixel_format.

While checking with all the other places that use this I've noticed
that intel_framebuffer_init already has all the platform checks we
need, so replace those checks with a WARN_ON.

Since fb->depth isn't set for YUV pixel formats and since we already
can't create an fb with an rgb layout not support on the running
platform I /think/ this patch doesn't fix any bug.

But it surely looks better!

v2: BGR formats are also only gen4+, so add the corresponding WARN_ON,
too (Ville).

Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
This commit is contained in:
Daniel Vetter 2013-03-28 16:38:08 +01:00
parent 72f4901e3c
commit d42264b1fc
1 changed files with 21 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

31
drivers/gpu/drm/i915/intel_display.c
View File

@ -7394,23 +7394,34 @@ pipe_config_set_bpp(struct drm_crtc *crtc,
struct drm_connector *connector; struct drm_connector *connector;
int bpp; int bpp;
switch (fb->depth) { switch (fb->pixel_format) {
case 8: case DRM_FORMAT_C8:
bpp = 8*3; /* since we go through a colormap */ bpp = 8*3; /* since we go through a colormap */
break; break;
case 15: case DRM_FORMAT_XRGB1555:
case 16: case DRM_FORMAT_ARGB1555:
/* checked in intel_framebuffer_init already */
if (WARN_ON(INTEL_INFO(dev)->gen > 3))
return -EINVAL;
case DRM_FORMAT_RGB565:
bpp = 6*3; /* min is 18bpp */ bpp = 6*3; /* min is 18bpp */
break; break;
case 24: case DRM_FORMAT_XBGR8888:
case DRM_FORMAT_ABGR8888:
/* checked in intel_framebuffer_init already */
if (WARN_ON(INTEL_INFO(dev)->gen < 4))
return -EINVAL;
case DRM_FORMAT_XRGB8888:
case DRM_FORMAT_ARGB8888:
bpp = 8*3; bpp = 8*3;
break; break;
case 30: case DRM_FORMAT_XRGB2101010:
if (INTEL_INFO(dev)->gen < 4) { case DRM_FORMAT_ARGB2101010:
DRM_DEBUG_KMS("10 bpc not supported on gen2/3\n"); case DRM_FORMAT_XBGR2101010:
case DRM_FORMAT_ABGR2101010:
/* checked in intel_framebuffer_init already */
if (WARN_ON(INTEL_INFO(dev)->gen < 4))
return -EINVAL; return -EINVAL;
}
bpp = 10*3; bpp = 10*3;
break; break;
/* TODO: gen4+ supports 16 bpc floating point, too. */ /* TODO: gen4+ supports 16 bpc floating point, too. */