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>
This commit is contained in:
		
							parent
							
								
									0333592ff9
								
							
						
					
					
						commit
						10f5ab0405
					
				| @ -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; | ||||
|  | ||||
		Loading…
	
		Reference in New Issue
	
	Block a user