diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2012-05-17 03:09:44 -0400 |
---|---|---|
committer | Dave Airlie <airlied@redhat.com> | 2012-05-17 06:42:27 -0400 |
commit | 10f5ab04056207b900e903f1e4b50b8e3867b67e (patch) | |
tree | a3b4e005a5b724927e6e9b6669192e79c661cdf6 /drivers/gpu/drm/savage/savage_bci.c | |
parent | 0333592ff9e936c6e5ae143e7418468a3198b54d (diff) |
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/gpu/drm/savage/savage_bci.c')
-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) | |||
735 | return -EINVAL; | 735 | return -EINVAL; |
736 | } | 736 | } |
737 | drm_core_ioremap(dev->agp_buffer_map, dev); | 737 | drm_core_ioremap(dev->agp_buffer_map, dev); |
738 | if (!dev->agp_buffer_map) { | 738 | if (!dev->agp_buffer_map->handle) { |
739 | DRM_ERROR("failed to ioremap DMA buffer region!\n"); | 739 | DRM_ERROR("failed to ioremap DMA buffer region!\n"); |
740 | savage_do_cleanup_bci(dev); | 740 | savage_do_cleanup_bci(dev); |
741 | return -ENOMEM; | 741 | return -ENOMEM; |