diff options
| author | Simon Horman <[email protected]> | 2024-01-23 09:59:48 +0000 |
|---|---|---|
| committer | Lee Jones <[email protected]> | 2024-02-23 14:58:01 +0000 |
| commit | 944e823d73d042dba6b73b6cbdb14367d38d0ca4 (patch) | |
| tree | 85303663ba324d4f1ae7fe1a85cd2042909587ea | |
| parent | 6cea614ba78d6b78e64e39f5ccdfad4aac1d0110 (diff) | |
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 <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Lee Jones <[email protected]>
| -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); |