diff options
| author | Deepak R Varma <drv@mailo.com> | 2023-01-25 18:07:14 +0300 |
|---|---|---|
| committer | Lyude Paul <lyude@redhat.com> | 2023-01-26 00:50:31 +0300 |
| commit | da2b1a0a40d8ea8f23f9d13e10ec3160ef92178d (patch) | |
| tree | 234f27f6bc2ff6177392df6b9c508b3f4407d42d /drivers/gpu/drm/nouveau/nvkm/subdev/devinit/priv.h | |
| parent | 8d71c78e1aeec37eca4f0a4c190fd3e9548721de (diff) | |
| download | linux-da2b1a0a40d8ea8f23f9d13e10ec3160ef92178d.tar.xz | |
drm/nouveau/devinit: Convert function disable() to be void
The current design of callback function disable() of struct
nvkm_devinit_func is defined to return a u64 value. In its implementation
in the driver modules, the function always returns a fixed value 0. Hence
the design and implementation of this function should be enhanced to return
void instead of a fixed value. This change also eliminates untouched
return variables.
The change is identified using the returnvar.cocci Coccinelle semantic
patch script.
Signed-off-by: Deepak R Varma <drv@mailo.com>
Reviewed-by: Lyude Paul <lyude@redhat.com>
Signed-off-by: Lyude Paul <lyude@redhat.com>
Link: https://patchwork.freedesktop.org/patch/msgid/Y9FFoooIXjlr+UP1@ubun2204.myguest.virtualbox.org
Diffstat (limited to 'drivers/gpu/drm/nouveau/nvkm/subdev/devinit/priv.h')
| -rw-r--r-- | drivers/gpu/drm/nouveau/nvkm/subdev/devinit/priv.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/nouveau/nvkm/subdev/devinit/priv.h b/drivers/gpu/drm/nouveau/nvkm/subdev/devinit/priv.h index dd8b038a8cee..a648482d06e9 100644 --- a/drivers/gpu/drm/nouveau/nvkm/subdev/devinit/priv.h +++ b/drivers/gpu/drm/nouveau/nvkm/subdev/devinit/priv.h @@ -12,7 +12,7 @@ struct nvkm_devinit_func { u32 (*mmio)(struct nvkm_devinit *, u32); void (*meminit)(struct nvkm_devinit *); int (*pll_set)(struct nvkm_devinit *, u32 type, u32 freq); - u64 (*disable)(struct nvkm_devinit *); + void (*disable)(struct nvkm_devinit *); }; void nvkm_devinit_ctor(const struct nvkm_devinit_func *, struct nvkm_device *, |