diff options
| author | Alexandre Courbot <acourbot@nvidia.com> | 2016-01-28 12:30:06 +0900 |
|---|---|---|
| committer | Ben Skeggs <bskeggs@redhat.com> | 2016-02-25 13:22:09 +1000 |
| commit | f36c6fd9e1b7308b2b92f613c5945015a3147097 (patch) | |
| tree | 740f50bd40bd47b9f7ea6286972feabd3df22646 | |
| parent | a12523d997bb2f5be68514e2b91148d6c3cb3eae (diff) | |
| download | nouveau-f36c6fd9e1b7308b2b92f613c5945015a3147097.tar.gz | |
device/tegra: fix uninitialized IRQ number
nvkm_device_tegra_new initializes the irq member of the Tegra device
to -1 in order to signal that it is uninitialized. However,
nvkm_device_tegra_fini tests it against 0 to check whether an IRQ has
been allocated or not. This leads to free_irq being called on -1 during
device initialization.
Fix this by using 0 as the uninitialized value everywhere.
Signed-off-by: Alexandre Courbot <acourbot@nvidia.com>
Signed-off-by: Ben Skeggs <bskeggs@redhat.com>
| -rw-r--r-- | drm/nouveau/nvkm/engine/device/tegra.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drm/nouveau/nvkm/engine/device/tegra.c b/drm/nouveau/nvkm/engine/device/tegra.c index 7f8a42721..6d89416f0 100644 --- a/drm/nouveau/nvkm/engine/device/tegra.c +++ b/drm/nouveau/nvkm/engine/device/tegra.c @@ -255,7 +255,6 @@ nvkm_device_tegra_new(const struct nvkm_device_tegra_func *func, *pdevice = &tdev->device; tdev->func = func; tdev->pdev = pdev; - tdev->irq = -1; tdev->vdd = devm_regulator_get(&pdev->dev, "vdd"); if (IS_ERR(tdev->vdd)) |