[PATCH -next] tty: serial: imx: remove unnecessary breaks
Uwe Kleine-König
u.kleine-koenig at pengutronix.de
Mon Oct 26 12:34:59 EDT 2020
hello,
On Mon, Oct 26, 2020 at 08:51:42PM +0800, Zhang Qilong wrote:
> The 'break' is not possoble to be reached
s/possoble/possible/
> because of previous 'return', discard it.
Given that this such a patch was already sent a week ago[1], I wonder if
there is a tool that complains here? If so, mentioning it in the commit
log would be good.
Other than that I still think that there should be at least an empty
line instead of the break, as I already wrote in reply to the first
patch. And having said that I also don't think this is worth "fixing",
seeing the break in a switch statement is good for human readers.
Best regards
Uwe
[1] https://lore.kernel.org/r/20201019175915.3718-1-trix@redhat.com
--
Pengutronix e.K. | Uwe Kleine-König |
Industrial Linux Solutions | https://www.pengutronix.de/ |
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20201026/c4f7b86b/attachment-0001.sig>
More information about the linux-arm-kernel
mailing list