[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