Re: [PATCH -next 013/491] INGENIC JZ47xx SoCs: Use fallthrough;

From: Miquel Raynal
Date: Wed Mar 11 2020 - 03:40:58 EST


Hi Joe,

Joe Perches <joe@xxxxxxxxxxx> wrote on Tue, 10 Mar 2020 21:51:27 -0700:

> Convert the various uses of fallthrough comments to fallthrough;
>
> Done via script
> Link: https://lore.kernel.org/lkml/b56602fcf79f849e733e7b521bb0e17895d390fa.1582230379.git.joe.com/
>
> Signed-off-by: Joe Perches <joe@xxxxxxxxxxx>
> ---
> drivers/gpu/drm/ingenic/ingenic-drm.c | 2 +-
> drivers/mmc/host/jz4740_mmc.c | 6 ++----
> drivers/mtd/nand/raw/ingenic/ingenic_nand_drv.c | 2 +-
> drivers/mtd/nand/raw/ingenic/jz4725b_bch.c | 4 ++--
> drivers/mtd/nand/raw/ingenic/jz4780_bch.c | 4 ++--
> sound/soc/codecs/jz4770.c | 2 +-
> 6 files changed, 9 insertions(+), 11 deletions(-)

I like very much the new way to advertise for fallthrough statements,
but I am not willing to take any patch converting a single driver
anymore. I had too many from Gustavo when these comments had to be
inserted. I would really prefer a MTD-wide or a NAND-wide or at least a
raw-NAND-wide single patch (anything inside drivers/mtd/nand/raw/).

Hope you'll understand!

Thanks,
MiquÃl