| From 234f0849345cd93e79a904444957e457287b50f2 Mon Sep 17 00:00:00 2001 |
| From: Sasha Levin <sashal@kernel.org> |
| Date: Sun, 15 Aug 2021 12:18:52 -0700 |
| Subject: bus: ti-sysc: Add break in switch statement in sysc_init_soc() |
| |
| From: Nathan Chancellor <nathan@kernel.org> |
| |
| [ Upstream commit e879f855e590b40fe3c79f2fbd8f65ca3c724120 ] |
| |
| After commit a6d90e9f2232 ("bus: ti-sysc: AM3: RNG is GP only"), clang |
| with -Wimplicit-fallthrough enabled warns: |
| |
| drivers/bus/ti-sysc.c:2958:3: warning: unannotated fall-through between |
| switch labels [-Wimplicit-fallthrough] |
| default: |
| ^ |
| drivers/bus/ti-sysc.c:2958:3: note: insert 'break;' to avoid |
| fall-through |
| default: |
| ^ |
| break; |
| 1 warning generated. |
| |
| Clang's version of this warning is a little bit more pedantic than |
| GCC's. Add the missing break to satisfy it to match what has been done |
| all over the kernel tree. |
| |
| Fixes: a6d90e9f2232 ("bus: ti-sysc: AM3: RNG is GP only") |
| Signed-off-by: Nathan Chancellor <nathan@kernel.org> |
| Reviewed-by: Nick Desaulniers <ndesaulniers@google.com> |
| Signed-off-by: Tony Lindgren <tony@atomide.com> |
| Signed-off-by: Sasha Levin <sashal@kernel.org> |
| --- |
| drivers/bus/ti-sysc.c | 1 + |
| 1 file changed, 1 insertion(+) |
| |
| diff --git a/drivers/bus/ti-sysc.c b/drivers/bus/ti-sysc.c |
| index 159b57c6dc4d..d2b7338c073f 100644 |
| --- a/drivers/bus/ti-sysc.c |
| +++ b/drivers/bus/ti-sysc.c |
| @@ -2922,6 +2922,7 @@ static int sysc_init_soc(struct sysc *ddata) |
| break; |
| case SOC_AM3: |
| sysc_add_disabled(0x48310000); /* rng */ |
| + break; |
| default: |
| break; |
| }; |
| -- |
| 2.33.0 |
| |