vmwgfx: Be more strict with fb depths when using screen objects
Signed-off-by: Jakob Bornecrantz <jakob@vmware.com> Reviewed-by: Thomas Hellstrom <thellstrom@vmware.com> Signed-off-by: Dave Airlie <airlied@redhat.com>
This commit is contained in:
parent
5bb39e8181
commit
c337ada7e7
|
@ -776,6 +776,33 @@ static int vmw_kms_new_framebuffer_dmabuf(struct vmw_private *dev_priv,
|
||||||
return -EINVAL;
|
return -EINVAL;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/* Limited framebuffer color depth support for screen objects */
|
||||||
|
if (dev_priv->sou_priv) {
|
||||||
|
switch (mode_cmd->depth) {
|
||||||
|
case 32:
|
||||||
|
case 24:
|
||||||
|
/* Only support 32 bpp for 32 and 24 depth fbs */
|
||||||
|
if (mode_cmd->bpp == 32)
|
||||||
|
break;
|
||||||
|
|
||||||
|
DRM_ERROR("Invalid color depth/bbp: %d %d\n",
|
||||||
|
mode_cmd->depth, mode_cmd->bpp);
|
||||||
|
return -EINVAL;
|
||||||
|
case 16:
|
||||||
|
case 15:
|
||||||
|
/* Only support 16 bpp for 16 and 15 depth fbs */
|
||||||
|
if (mode_cmd->bpp == 16)
|
||||||
|
break;
|
||||||
|
|
||||||
|
DRM_ERROR("Invalid color depth/bbp: %d %d\n",
|
||||||
|
mode_cmd->depth, mode_cmd->bpp);
|
||||||
|
return -EINVAL;
|
||||||
|
default:
|
||||||
|
DRM_ERROR("Invalid color depth: %d\n", mode_cmd->depth);
|
||||||
|
return -EINVAL;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
vfbd = kzalloc(sizeof(*vfbd), GFP_KERNEL);
|
vfbd = kzalloc(sizeof(*vfbd), GFP_KERNEL);
|
||||||
if (!vfbd) {
|
if (!vfbd) {
|
||||||
ret = -ENOMEM;
|
ret = -ENOMEM;
|
||||||
|
|
Loading…
Reference in New Issue