| From e8f71f89236ef82d449991bfbc237e3cb6ea584f Mon Sep 17 00:00:00 2001 |
| From: Guenter Roeck <linux@roeck-us.net> |
| Date: Wed, 8 Sep 2021 12:08:17 -0700 |
| Subject: drm/nouveau/nvkm: Replace -ENOSYS with -ENODEV |
| |
| From: Guenter Roeck <linux@roeck-us.net> |
| |
| commit e8f71f89236ef82d449991bfbc237e3cb6ea584f upstream. |
| |
| nvkm test builds fail with the following error. |
| |
| drivers/gpu/drm/nouveau/nvkm/engine/device/ctrl.c: In function 'nvkm_control_mthd_pstate_info': |
| drivers/gpu/drm/nouveau/nvkm/engine/device/ctrl.c:60:35: error: overflow in conversion from 'int' to '__s8' {aka 'signed char'} changes value from '-251' to '5' |
| |
| The code builds on most architectures, but fails on parisc where ENOSYS |
| is defined as 251. |
| |
| Replace the error code with -ENODEV (-19). The actual error code does |
| not really matter and is not passed to userspace - it just has to be |
| negative. |
| |
| Fixes: 7238eca4cf18 ("drm/nouveau: expose pstate selection per-power source in sysfs") |
| Signed-off-by: Guenter Roeck <linux@roeck-us.net> |
| Cc: Ben Skeggs <bskeggs@redhat.com> |
| Cc: David Airlie <airlied@linux.ie> |
| Cc: Daniel Vetter <daniel@ffwll.ch> |
| Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org> |
| Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> |
| --- |
| drivers/gpu/drm/nouveau/nvkm/engine/device/ctrl.c | 2 +- |
| 1 file changed, 1 insertion(+), 1 deletion(-) |
| |
| --- a/drivers/gpu/drm/nouveau/nvkm/engine/device/ctrl.c |
| +++ b/drivers/gpu/drm/nouveau/nvkm/engine/device/ctrl.c |
| @@ -57,7 +57,7 @@ nvkm_control_mthd_pstate_info(struct nvk |
| args->v0.count = 0; |
| args->v0.ustate_ac = NVIF_CONTROL_PSTATE_INFO_V0_USTATE_DISABLE; |
| args->v0.ustate_dc = NVIF_CONTROL_PSTATE_INFO_V0_USTATE_DISABLE; |
| - args->v0.pwrsrc = -ENOSYS; |
| + args->v0.pwrsrc = -ENODEV; |
| args->v0.pstate = NVIF_CONTROL_PSTATE_INFO_V0_PSTATE_UNKNOWN; |
| } |
| |