drm/nouveau/platform: fix probe error path
authorAlexandre Courbot <acourbot@nvidia.com>
Fri, 20 Mar 2015 10:41:05 +0000 (19:41 +0900)
committerBen Skeggs <bskeggs@redhat.com>
Tue, 14 Apr 2015 07:00:51 +0000 (17:00 +1000)
A "return 0" found its way in the middle of the error path of
nouveau_platform_probe(), remove it as it will make the kernel crash if
we try to unload the module afterwards.

While we are at it, also remove the IOMMU domain if it has been created,
as we should.

Signed-off-by: Alexandre Courbot <acourbot@nvidia.com>
Signed-off-by: Ben Skeggs <bskeggs@redhat.com>
drivers/gpu/drm/nouveau/nouveau_platform.c

index f83aa12ee5c6bd403c468cd959693aadc4d7ceae..775277f1edb0a4ae4c1a2862418827878710ddb8 100644 (file)
@@ -209,10 +209,9 @@ static int nouveau_platform_probe(struct platform_device *pdev)
 err_unref:
        drm_dev_unref(drm);
 
-       return 0;
-
 power_down:
        nouveau_platform_power_down(gpu);
+       nouveau_platform_remove_iommu(&pdev->dev, gpu);
 
        return err;
 }