diff options
| author | Alexandre Courbot <acourbot@nvidia.com> | 2015-03-20 10:41:05 (GMT) |
|---|---|---|
| committer | Ben Skeggs <bskeggs@redhat.com> | 2015-04-14 07:00:51 (GMT) |
| commit | bbf2e92f914019324bb4351fcf0db9f41329a7c3 (patch) | |
| tree | adb06d561022b9ac0f1026c02649f02f46c13b66 | |
| parent | 25eb3a924f8589b5ad80e5d50d743befca8a46de (diff) | |
| download | linux-bbf2e92f914019324bb4351fcf0db9f41329a7c3.tar.xz | |
drm/nouveau/platform: fix probe error path
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>
| -rw-r--r-- | drivers/gpu/drm/nouveau/nouveau_platform.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/gpu/drm/nouveau/nouveau_platform.c b/drivers/gpu/drm/nouveau/nouveau_platform.c index f83aa12..775277f 100644 --- a/drivers/gpu/drm/nouveau/nouveau_platform.c +++ b/drivers/gpu/drm/nouveau/nouveau_platform.c @@ -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; } |