* [PATCH] MIPS: fix some more fall through errors in arch/mips
@ 2019-07-15 12:46 Stephen Rothwell
2019-07-16 11:42 ` Paul Burton
0 siblings, 1 reply; 2+ messages in thread
From: Stephen Rothwell @ 2019-07-15 12:46 UTC (permalink / raw)
To: Ralf Baechle, Paul Burton, James Hogan
Cc: Florian Fainelli, Kees Cook, Gustavo A. R. Silva, linux-mips,
linux-kernel, bcm-kernel-feedback-list, John Crispin,
Thomas Gleixner, Matthias Schiffer, Petr Štetiar,
linux-arm-kernel
[-- Attachment #1.1: Type: text/plain, Size: 3070 bytes --]
Fix these errors:
arch/mips/cavium-octeon/executive/cvmx-pko.c:489:7: error: this statement may fall through [-Werror=implicit-fallthrough=]
arch/mips/bcm63xx/dev-flash.c:89:3: error: this statement may fall through [-Werror=implicit-fallthrough=]
arch/mips/ath79/setup.c:155:17: error: this statement may fall through [-Werror=implicit-fallthrough=]
arch/mips/ar7/setup.c:50:3: error: this statement may fall through [-Werror=implicit-fallthrough=]
Cc: Florian Fainelli <f.fainelli@gmail.com>
Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: John Crispin <john@phrozen.org>
Cc: Matthias Schiffer <mschiffer@universe-factory.net>
Cc: "Petr Štetiar" <ynezz@true.cz>
Signed-off-by: Stephen Rothwell <sfr@canb.auug.org.au>
---
arch/mips/ar7/setup.c | 1 +
arch/mips/ath79/setup.c | 2 +-
arch/mips/bcm63xx/dev-flash.c | 1 +
arch/mips/cavium-octeon/executive/cvmx-pko.c | 2 +-
4 files changed, 4 insertions(+), 2 deletions(-)
These are the remaining errors in Linus' tree.
Again, not even tested, but pretty straight forward.
diff --git a/arch/mips/ar7/setup.c b/arch/mips/ar7/setup.c
index ba32825ae58a..b3ffe7c898eb 100644
--- a/arch/mips/ar7/setup.c
+++ b/arch/mips/ar7/setup.c
@@ -57,6 +57,7 @@ const char *get_system_type(void)
case TITAN_CHIP_1060:
return "TI AR7 (TNETV1060)";
}
+ /* fall through */
default:
return "TI AR7 (unknown)";
}
diff --git a/arch/mips/ath79/setup.c b/arch/mips/ath79/setup.c
index f22538cae0ab..ea385a865781 100644
--- a/arch/mips/ath79/setup.c
+++ b/arch/mips/ath79/setup.c
@@ -153,7 +153,7 @@ static void __init ath79_detect_sys_type(void)
case REV_ID_MAJOR_QCA9533_V2:
ver = 2;
ath79_soc_rev = 2;
- /* drop through */
+ /* fall through */
case REV_ID_MAJOR_QCA9533:
ath79_soc = ATH79_SOC_QCA9533;
diff --git a/arch/mips/bcm63xx/dev-flash.c b/arch/mips/bcm63xx/dev-flash.c
index 172dd8397178..a1093934c616 100644
--- a/arch/mips/bcm63xx/dev-flash.c
+++ b/arch/mips/bcm63xx/dev-flash.c
@@ -94,6 +94,7 @@ static int __init bcm63xx_detect_flash_type(void)
case STRAPBUS_6368_BOOT_SEL_PARALLEL:
return BCM63XX_FLASH_TYPE_PARALLEL;
}
+ /* fall through */
default:
return -EINVAL;
}
diff --git a/arch/mips/cavium-octeon/executive/cvmx-pko.c b/arch/mips/cavium-octeon/executive/cvmx-pko.c
index 676fab50dd2b..b077597c668a 100644
--- a/arch/mips/cavium-octeon/executive/cvmx-pko.c
+++ b/arch/mips/cavium-octeon/executive/cvmx-pko.c
@@ -485,11 +485,11 @@ cvmx_pko_status_t cvmx_pko_config_port(uint64_t port, uint64_t base_queue,
config.s.qos_mask = 0xff;
break;
case CVMX_PKO_QUEUE_STATIC_PRIORITY:
- /* Pass 1 will fall through to the error case */
if (!cvmx_octeon_is_pass1()) {
config.s.qos_mask = 0xff;
break;
}
+ /* fall through - to the error case, when Pass 1 */
default:
cvmx_dprintf("ERROR: cvmx_pko_config_port: Invalid "
"priority %llu\n",
--
2.22.0
--
Cheers,
Stephen Rothwell
[-- Attachment #1.2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
[-- Attachment #2: Type: text/plain, Size: 176 bytes --]
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] MIPS: fix some more fall through errors in arch/mips
2019-07-15 12:46 [PATCH] MIPS: fix some more fall through errors in arch/mips Stephen Rothwell
@ 2019-07-16 11:42 ` Paul Burton
0 siblings, 0 replies; 2+ messages in thread
From: Paul Burton @ 2019-07-16 11:42 UTC (permalink / raw)
To: Stephen Rothwell
Cc: Paul Burton, Florian Fainelli, Kees Cook, Gustavo A. R. Silva,
James Hogan, linux-mips, Ralf Baechle, linux-kernel,
bcm-kernel-feedback-list, John Crispin, Thomas Gleixner,
Matthias Schiffer, Petr Štetiar ,
linux-arm-kernel
Hello,
Stephen Rothwell wrote:
> Fix these errors:
>
> arch/mips/cavium-octeon/executive/cvmx-pko.c:489:7: error: this statement may fall through [-Werror=implicit-fallthrough=]
> arch/mips/bcm63xx/dev-flash.c:89:3: error: this statement may fall through [-Werror=implicit-fallthrough=]
> arch/mips/ath79/setup.c:155:17: error: this statement may fall through [-Werror=implicit-fallthrough=]
> arch/mips/ar7/setup.c:50:3: error: this statement may fall through [-Werror=implicit-fallthrough=]
>
> Cc: Florian Fainelli <f.fainelli@gmail.com>
> Cc: Thomas Gleixner <tglx@linutronix.de>
> Cc: John Crispin <john@phrozen.org>
> Cc: Matthias Schiffer <mschiffer@universe-factory.net>
> Cc: "Petr Å tetiar" <ynezz@true.cz>
> Signed-off-by: Stephen Rothwell <sfr@canb.auug.org.au>
Applied to mips-next.
Thanks,
Paul
[ This message was auto-generated; if you believe anything is incorrect
then please email paul.burton@mips.com to report it. ]
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2019-07-16 11:42 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-07-15 12:46 [PATCH] MIPS: fix some more fall through errors in arch/mips Stephen Rothwell
2019-07-16 11:42 ` Paul Burton
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).