[PATCH v1] drivers/nouveau: change signature nv50_wndw_atomic_check_lut()

From: Denis Plotnikov
Date: Thu Dec 19 2024 - 05:10:11 EST


It always returned 0 so make it void instead of returning int.

Signed-off-by: Denis Plotnikov <den-plotnikov@xxxxxxxxxxxxxx>
---
drivers/gpu/drm/nouveau/dispnv50/wndw.c | 8 ++------
1 file changed, 2 insertions(+), 6 deletions(-)

diff --git a/drivers/gpu/drm/nouveau/dispnv50/wndw.c b/drivers/gpu/drm/nouveau/dispnv50/wndw.c
index f6be426dd5253..3ced5ebc3a897 100644
--- a/drivers/gpu/drm/nouveau/dispnv50/wndw.c
+++ b/drivers/gpu/drm/nouveau/dispnv50/wndw.c
@@ -370,7 +370,7 @@ nv50_wndw_atomic_check_acquire(struct nv50_wndw *wndw, bool modeset,
return wndw->func->acquire(wndw, asyw, asyh);
}

-static int
+static void
nv50_wndw_atomic_check_lut(struct nv50_wndw *wndw,
struct nv50_wndw_atom *armw,
struct nv50_wndw_atom *asyw,
@@ -392,7 +392,6 @@ nv50_wndw_atomic_check_lut(struct nv50_wndw *wndw,
*/
if (!(ilut = asyh->state.gamma_lut)) {
asyw->visible = false;
- return 0;
}

if (wndw->func->ilut)
@@ -436,7 +435,6 @@ nv50_wndw_atomic_check_lut(struct nv50_wndw *wndw,

/* Can't do an immediate flip while changing the LUT. */
asyh->state.async_flip = false;
- return 0;
}

static int
@@ -481,9 +479,7 @@ nv50_wndw_atomic_check(struct drm_plane *plane,
asyh->state.color_mgmt_changed ||
asyw->state.fb->format->format !=
armw->state.fb->format->format)) {
- ret = nv50_wndw_atomic_check_lut(wndw, armw, asyw, asyh);
- if (ret)
- return ret;
+ nv50_wndw_atomic_check_lut(wndw, armw, asyw, asyh);
}

/* Calculate new window state. */
--
2.34.1