diff options
author | Jakob Bornecrantz <jakob@vmware.com> | 2011-11-03 21:03:05 +0100 |
---|---|---|
committer | Dave Airlie <airlied@redhat.com> | 2011-11-07 13:15:56 +0000 |
commit | 55bde5b215ceb8356328f18cd82a77843bc4ffd3 (patch) | |
tree | 4d9f311c7d3dc9befb69a1b6cbd6ee9fd996b635 /drivers/gpu | |
parent | d982640914d74054d22508ccebeeda70ce5933e1 (diff) | |
download | op-kernel-dev-55bde5b215ceb8356328f18cd82a77843bc4ffd3.zip op-kernel-dev-55bde5b215ceb8356328f18cd82a77843bc4ffd3.tar.gz |
vmwgfx: Free prefered mode on error path
Signed-off-by: Jakob Bornecrantz <jakob@vmware.com>
Signed-off-by: Thomas Hellstrom <thellstrom@vmware.com>
Signed-off-by: Dave Airlie <airlied@redhat.com>
Diffstat (limited to 'drivers/gpu')
-rw-r--r-- | drivers/gpu/drm/vmwgfx/vmwgfx_kms.c | 17 |
1 files changed, 11 insertions, 6 deletions
diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c b/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c index 41a905d..667437b 100644 --- a/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c +++ b/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c @@ -1710,17 +1710,22 @@ int vmw_du_connector_fill_modes(struct drm_connector *connector, mode->hdisplay = du->pref_width; mode->vdisplay = du->pref_height; vmw_guess_mode_timing(mode); + if (vmw_kms_validate_mode_vram(dev_priv, mode->hdisplay * 2, mode->vdisplay)) { drm_mode_probed_add(connector, mode); + } else { + drm_mode_destroy(dev, mode); + mode = NULL; + } - if (du->pref_mode) { - list_del_init(&du->pref_mode->head); - drm_mode_destroy(dev, du->pref_mode); - } - - du->pref_mode = mode; + if (du->pref_mode) { + list_del_init(&du->pref_mode->head); + drm_mode_destroy(dev, du->pref_mode); } + + /* mode might be null here, this is intended */ + du->pref_mode = mode; } for (i = 0; vmw_kms_connector_builtin[i].type != 0; i++) { |