diff options
| author | Simon Horman <horms@kernel.org> | 2024-01-23 12:59:48 +0300 |
|---|---|---|
| committer | Lee Jones <lee@kernel.org> | 2024-02-23 17:58:01 +0300 |
| commit | 944e823d73d042dba6b73b6cbdb14367d38d0ca4 (patch) | |
| tree | 85303663ba324d4f1ae7fe1a85cd2042909587ea /drivers/mfd/rave-sp.c | |
| parent | 6cea614ba78d6b78e64e39f5ccdfad4aac1d0110 (diff) | |
| download | linux-944e823d73d042dba6b73b6cbdb14367d38d0ca4.tar.xz | |
mfd: rave-sp: Avoid unnecessary use of comma operator
Although it does not seem to have any untoward side-effects,
the use of ';' to separate to assignments seems more appropriate than ','.
Flagged by clang-17 -Wcomma
No functional change intended. Compile tested only.
Signed-off-by: Simon Horman <horms@kernel.org>
Link: https://lore.kernel.org/r/20240123-rave-sp-comma-v1-1-84e9b15ba205@kernel.org
Signed-off-by: Lee Jones <lee@kernel.org>
Diffstat (limited to 'drivers/mfd/rave-sp.c')
| -rw-r--r-- | drivers/mfd/rave-sp.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/mfd/rave-sp.c b/drivers/mfd/rave-sp.c index 6ff84b2600c5..ea5fbcbbe4a5 100644 --- a/drivers/mfd/rave-sp.c +++ b/drivers/mfd/rave-sp.c @@ -358,7 +358,7 @@ int rave_sp_exec(struct rave_sp *sp, ackid = atomic_inc_return(&sp->ackid); reply.ackid = ackid; - reply.code = rave_sp_reply_code((u8)command), + reply.code = rave_sp_reply_code((u8)command); mutex_lock(&sp->bus_lock); |