diff options
| author | Daniel Vetter <daniel.vetter@ffwll.ch> | 2012-01-08 22:42:27 +0100 |
|---|---|---|
| committer | Dave Airlie <airlied@redhat.com> | 2012-01-09 13:11:39 +0100 |
| commit | b5215ef1a877cd81a2558f3ddbcf784671a27588 (patch) | |
| tree | 9bef1f5e9814906da9144ff8dc62006bc9e163cb /drivers/gpu/drm/sis | |
| parent | drm/radeon/kms: remove pointless CS flags priority struct (diff) | |
| download | linux-b5215ef1a877cd81a2558f3ddbcf784671a27588.tar.xz linux-b5215ef1a877cd81a2558f3ddbcf784671a27588.zip | |
drm/sis|via: don't return stack garbage from free_mem ioctl
Fallout from my "kill drm_sman" refactor. Unfortunately gcc seems to
have failed me and not warned about this.
Tested-by: Jakob Bornecrantz <wallbraker@gmail.com> (on via)
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: Dave Airlie <airlied@redhat.com>
Diffstat (limited to 'drivers/gpu/drm/sis')
| -rw-r--r-- | drivers/gpu/drm/sis/sis_mm.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/gpu/drm/sis/sis_mm.c b/drivers/gpu/drm/sis/sis_mm.c index 46cb2bce6cc5..dd4a316c3d74 100644 --- a/drivers/gpu/drm/sis/sis_mm.c +++ b/drivers/gpu/drm/sis/sis_mm.c @@ -172,7 +172,6 @@ static int sis_drm_free(struct drm_device *dev, void *data, struct drm_file *fil drm_sis_private_t *dev_priv = dev->dev_private; drm_sis_mem_t *mem = data; struct sis_memblock *obj; - int ret; mutex_lock(&dev->struct_mutex); obj = idr_find(&dev_priv->object_idr, mem->free); @@ -193,7 +192,7 @@ static int sis_drm_free(struct drm_device *dev, void *data, struct drm_file *fil mutex_unlock(&dev->struct_mutex); DRM_DEBUG("free = 0x%lx\n", mem->free); - return ret; + return 0; } static int sis_fb_alloc(struct drm_device *dev, void *data, |