drm/nouveau/tegra: Fix error pointer vs NULL return in nvkm_device_tegra_resource_addr()
authorDan Carpenter <dan.carpenter@linaro.org>
Fri, 23 May 2025 16:07:58 +0000 (19:07 +0300)
committerDave Airlie <airlied@redhat.com>
Sat, 24 May 2025 04:36:07 +0000 (14:36 +1000)
The nvkm_device_tegra_resource() function returns a mix of error pointers
and NULL.  The callers only expect it to return NULL on error.  Change it
to only return NULL.

Fixes: 76b8f81a5b92 ("drm/nouveau: improve handling of 64-bit BARs")
Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org>
Reviewed-by: Timur Tabi <ttabi@nvidia.com>
Signed-off-by: Dave Airlie <airlied@redhat.com>
Link: https://lore.kernel.org/dri-devel/334404bdf60765cb5a8e855a74c688bc537531ee.camel@nvidia.com/T/#t
drivers/gpu/drm/nouveau/nvkm/engine/device/tegra.c

index 0ca2dfe996764bafbf5da97e85ba6879545a3f74..114e50ca18270c90c32ad85f8bd8469740a950cb 100644 (file)
@@ -196,7 +196,7 @@ nvkm_device_tegra_resource(struct nvkm_device *device, enum nvkm_bar_id bar)
        case NVKM_BAR1_FB : idx = 1; break;
        default:
                WARN_ON(1);
-               return ERR_PTR(-EINVAL);
+               return NULL;
        }
 
        return platform_get_resource(tdev->pdev, IORESOURCE_MEM, idx);