diff options
author | Gustavo A. R. Silva <gustavo@embeddedor.com> | 2019-08-01 21:22:48 -0400 |
---|---|---|
committer | Bjorn Helgaas <bhelgaas@google.com> | 2019-08-08 16:13:34 -0400 |
commit | 5a2e340690f2908371eb9c1b4cfb82ba0b235b51 (patch) | |
tree | 2196f3d0b6dceec1e1bb09d2454f01be43bbc92c | |
parent | befa45fb5bdd514a40a19e659491193f7fd022f2 (diff) |
PCI: Mark expected switch fall-through
Mark switch cases where we are expecting to fall through.
This fixes the following warning (Building: allmodconfig i386):
drivers/pci/hotplug/ibmphp_res.c: In function ‘update_bridge_ranges’:
drivers/pci/hotplug/ibmphp_res.c:1943:16: warning: this statement may fall through [-Wimplicit-fallthrough=]
function = 0x8;
~~~~~~~~~^~~~~
drivers/pci/hotplug/ibmphp_res.c:1944:6: note: here
case PCI_HEADER_TYPE_MULTIBRIDGE:
^~~~
Link: https://lore.kernel.org/r/20190802012248.GA22622@embeddedor
Signed-off-by: Gustavo A. R. Silva <gustavo@embeddedor.com>
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
Reviewed-by: Kees Cook <keescook@chromium.org>
-rw-r--r-- | drivers/pci/hotplug/ibmphp_res.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/pci/hotplug/ibmphp_res.c b/drivers/pci/hotplug/ibmphp_res.c index 5e8caf7a4452..5c93aa14f0de 100644 --- a/drivers/pci/hotplug/ibmphp_res.c +++ b/drivers/pci/hotplug/ibmphp_res.c | |||
@@ -1941,6 +1941,7 @@ static int __init update_bridge_ranges(struct bus_node **bus) | |||
1941 | break; | 1941 | break; |
1942 | case PCI_HEADER_TYPE_BRIDGE: | 1942 | case PCI_HEADER_TYPE_BRIDGE: |
1943 | function = 0x8; | 1943 | function = 0x8; |
1944 | /* fall through */ | ||
1944 | case PCI_HEADER_TYPE_MULTIBRIDGE: | 1945 | case PCI_HEADER_TYPE_MULTIBRIDGE: |
1945 | /* We assume here that only 1 bus behind the bridge | 1946 | /* We assume here that only 1 bus behind the bridge |
1946 | TO DO: add functionality for several: | 1947 | TO DO: add functionality for several: |