projects
/
linux-block.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
| inline |
side by side
(parent:
4a8b3d4
)
mfd: ipaq-micro: Add missing break for the default case
author
Andy Shevchenko
<andriy.shevchenko@linux.intel.com>
Wed, 16 Oct 2024 13:00:23 +0000
(16:00 +0300)
committer
Lee Jones
<lee@kernel.org>
Thu, 31 Oct 2024 14:59:40 +0000
(14:59 +0000)
Even default case should have a break statement to make code robust
against changes (e.g., adding a case after the default one).
Add missing break for the default case.
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Link:
https://lore.kernel.org/r/20241016130023.872277-1-andriy.shevchenko@linux.intel.com
Signed-off-by: Lee Jones <lee@kernel.org>
drivers/mfd/ipaq-micro.c
patch
|
blob
|
blame
|
history
diff --git
a/drivers/mfd/ipaq-micro.c
b/drivers/mfd/ipaq-micro.c
index c964ea6539aa679d66da3cbce776abfdc7066e4a..2370b44e2214489ac305402f3ea097da47c3b014 100644
(file)
--- a/
drivers/mfd/ipaq-micro.c
+++ b/
drivers/mfd/ipaq-micro.c
@@
-130,6
+130,7
@@
static void micro_rx_msg(struct ipaq_micro *micro, u8 id, int len, u8 *data)
default:
dev_err(micro->dev,
"unknown msg %d [%d] %*ph\n", id, len, len, data);
+ break;
}
spin_unlock(µ->lock);
}