diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2012-05-17 09:09:44 +0200 |
|---|---|---|
| committer | Dave Airlie <airlied@redhat.com> | 2012-05-17 12:42:27 +0200 |
| commit | 10f5ab04056207b900e903f1e4b50b8e3867b67e (patch) | |
| tree | a3b4e005a5b724927e6e9b6669192e79c661cdf6 /drivers | |
| parent | gma500: fix build warning (diff) | |
| download | linux-10f5ab04056207b900e903f1e4b50b8e3867b67e.tar.xz linux-10f5ab04056207b900e903f1e4b50b8e3867b67e.zip | |
drm: checking the wrong variable in savage_do_init_bci()
drm_core_ioremap() initializes ->handle. We already know
"dev->agp_buffer_map" is a valid pointer.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Dave Airlie <airlied@redhat.com>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/gpu/drm/savage/savage_bci.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/savage/savage_bci.c b/drivers/gpu/drm/savage/savage_bci.c index cb1ee4e0050a..6eb507a5d130 100644 --- a/drivers/gpu/drm/savage/savage_bci.c +++ b/drivers/gpu/drm/savage/savage_bci.c @@ -735,7 +735,7 @@ static int savage_do_init_bci(struct drm_device * dev, drm_savage_init_t * init) return -EINVAL; } drm_core_ioremap(dev->agp_buffer_map, dev); - if (!dev->agp_buffer_map) { + if (!dev->agp_buffer_map->handle) { DRM_ERROR("failed to ioremap DMA buffer region!\n"); savage_do_cleanup_bci(dev); return -ENOMEM; |