From f82718251e302fd6321224d6c3be969245d5a2d2 Mon Sep 17 00:00:00 2001 From: Chen Ni Date: Mon, 10 Mar 2025 11:11:45 +0800 Subject: [PATCH] mfd: qnap-mcu: Convert commas to semicolons in qnap_mcu_exec() Replace comma between expressions with semicolons. Using a ',' in place of a ';' can have unintended side effects. Although that is not the case here, it is seems best to use ';' unless ',' is intended. Found by inspection. No functional change intended. Compile tested only. Fixes: 998f70d1806b ("mfd: Add base driver for qnap-mcu devices") Signed-off-by: Chen Ni Reviewed-by: Heiko Stuebner Link: https://lore.kernel.org/r/20250310031145.650950-1-nichen@iscas.ac.cn Signed-off-by: Lee Jones --- drivers/mfd/qnap-mcu.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/mfd/qnap-mcu.c b/drivers/mfd/qnap-mcu.c index 4be39d8b2905..89a8a1913d42 100644 --- a/drivers/mfd/qnap-mcu.c +++ b/drivers/mfd/qnap-mcu.c @@ -158,9 +158,9 @@ int qnap_mcu_exec(struct qnap_mcu *mcu, mutex_lock(&mcu->bus_lock); - reply->data = rx, - reply->length = length, - reply->received = 0, + reply->data = rx; + reply->length = length; + reply->received = 0; reinit_completion(&reply->done); qnap_mcu_write(mcu, cmd_data, cmd_data_size);