drm/cirrus: Limit modes depending on bpp option
authorTakashi Iwai <tiwai@suse.de>
Tue, 3 Feb 2015 16:51:23 +0000 (17:51 +0100)
committerDave Airlie <airlied@redhat.com>
Thu, 5 Feb 2015 00:39:00 +0000 (10:39 +1000)
commit7f551b1eee63dca6571fc44329b807aab7d5fc5c
tree4fe22dfc2ac989d274fe7ec991ddc2c4205daf50
parent86640966f6ec8ab2fdc98e916456c6fc9d13262d
drm/cirrus: Limit modes depending on bpp option

The commit [8975626ea35a: drm/cirrus: allow 32bpp framebuffers for
cirrus drm] broke X modesetting driver because cirrus driver still
provides the full list of modes up to 1280x1024 while the 32bpp can
support only up to 800x600.

We might be able to filter out the invalid modes in mode_valid
callback, but unfortunately the bpp in question can't be referred
there for now (let me know if there is a better way to retrieve the
bpp for the probed fb).

So, instead, this patch adds the bpp module option to specify the
maximal bpp explicitly and limits the resolutions in get_modes
depending on its value.

The default value is set to 24 so that the existing stuff keeps
working.  If you need a new 32bpp feature, specify cirrus.bpp=32
option explicitly.

Fixes: 8975626ea35a ('drm/cirrus: allow 32bpp framebuffers for cirrus drm')
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Signed-off-by: Dave Airlie <airlied@redhat.com>
drivers/gpu/drm/cirrus/cirrus_drv.c
drivers/gpu/drm/cirrus/cirrus_drv.h
drivers/gpu/drm/cirrus/cirrus_main.c
drivers/gpu/drm/cirrus/cirrus_mode.c