* [RESEND PATCH] HID: hid-steam: Use clamp() macro
@ 2024-07-31 13:50 Thorsten Blum
0 siblings, 0 replies; only message in thread
From: Thorsten Blum @ 2024-07-31 13:50 UTC (permalink / raw)
To: jikos, bentiss; +Cc: linux-input, linux-kernel, Thorsten Blum
Use clamp() to actually clamp the value to the range [-32767, 32767]
(as mentioned in the function comment) instead of manually mapping
one specific value.
Compile-tested only.
Signed-off-by: Thorsten Blum <thorsten.blum@toblux.com>
---
drivers/hid/hid-steam.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/hid/hid-steam.c b/drivers/hid/hid-steam.c
index bf8b633114be..3442985d52ce 100644
--- a/drivers/hid/hid-steam.c
+++ b/drivers/hid/hid-steam.c
@@ -1341,7 +1341,7 @@ static inline s16 steam_le16(u8 *data)
{
s16 x = (s16) le16_to_cpup((__le16 *)data);
- return x == -32768 ? -32767 : x;
+ return clamp(x, -32767, 32767);
}
/*
--
2.45.2
^ permalink raw reply related [flat|nested] only message in thread
only message in thread, other threads:[~2024-07-31 13:50 UTC | newest]
Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-07-31 13:50 [RESEND PATCH] HID: hid-steam: Use clamp() macro Thorsten Blum
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).