summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorHeiko Stuebner <heiko@sntech.de>2025-11-13 17:52:16 +0100
committerLee Jones <lee@kernel.org>2025-11-20 10:14:06 +0000
commitc94fce30e190555d74e2769b5fe4a932d0ad432e (patch)
tree00353862bcd44a5ef15278f03fa9d7f3b9de0250
parentb4881070a02b017aea84592c424d5a980ed261c4 (diff)
mfd: qnap-mcu: Use EPROTO in stead of EIO on checksum errors
EPROTO stands for protocol error and a lot of driver already use it to designate errors in the sent or received data from a peripheral. So use it in the qnap-mcu as well for checksum errors. Signed-off-by: Heiko Stuebner <heiko@sntech.de> Link: https://patch.msgid.link/20251113165218.449616-3-heiko@sntech.de Signed-off-by: Lee Jones <lee@kernel.org>
-rw-r--r--drivers/mfd/qnap-mcu.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/mfd/qnap-mcu.c b/drivers/mfd/qnap-mcu.c
index 4cd5319fc6cb..1f4741cad875 100644
--- a/drivers/mfd/qnap-mcu.c
+++ b/drivers/mfd/qnap-mcu.c
@@ -178,7 +178,7 @@ int qnap_mcu_exec(struct qnap_mcu *mcu,
crc = qnap_mcu_csum(rx, reply->received - QNAP_MCU_CHECKSUM_SIZE);
if (crc != rx[reply->received - QNAP_MCU_CHECKSUM_SIZE]) {
dev_err(&mcu->serdev->dev, "Invalid Checksum received\n");
- return -EIO;
+ return -EPROTO;
}
memcpy(reply_data, rx, reply_data_size);