drm/i915: fixup fb bpp computation in pipe_config_set_bpp
authorDaniel Vetter <daniel.vetter@ffwll.ch>
Thu, 28 Mar 2013 15:38:08 +0000 (16:38 +0100)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Thu, 28 Mar 2013 15:55:25 +0000 (16:55 +0100)
commitd42264b1fcc0f7b2fc472af6694b098e86e63be0
tree102c5ae0bf92c4248ac1239dabb9cc0c8bf74592
parent72f4901e3c9b138d4ba5fd51e93eb97207c6eaca
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>
drivers/gpu/drm/i915/intel_display.c