File 4000-GCC7-fixes-for-Wimplicit-fallthroug.patch of Package qemu-testsuite.19805

Overview Repositories Revisions Requests Users Attributes Meta

File 4000-GCC7-fixes-for-Wimplicit-fallthroug.patch of Package qemu-testsuite.19805

From: Stewart Smith <stewart@linux.vnet.ibm.com>
Date: Wed, 10 May 2017 19:33:52 +1000
Subject: GCC7: fixes for -Wimplicit-fallthrough expected regexes

It turns out GCC7 adds a useful warning and does fancy things like
parsing your comments to work out that you intended to do the fallthrough.
There's a few places where we don't match the regex. Fix them, as it's
harmless to do so.

Found by building on Fedora Rawhide in Travis.

Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
Signed-off-by: Bruce Rogers <brogers@suse.com>
---
 core/pcie-slot.c                | 3 ++-
 hw/fsp/fsp-sysparam.c           | 2 +-
 platforms/ibm-fsp/firenze-pci.c | 3 ++-
 3 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/roms/skiboot/core/pcie-slot.c b/roms/skiboot/core/pcie-slot.c
index 62933a4046f6dacb46c8cb4234e7..80972a6f55d3fb7d84bddfb895c5 100644
--- a/roms/skiboot/core/pcie-slot.c
+++ b/roms/skiboot/core/pcie-slot.c
@@ -383,7 +383,8 @@ static int64_t pcie_slot_sm_freset(struct pci_slot *slot)
 				PCI_SLOT_STATE_FRESET_POWER_OFF);
 			return pci_slot_set_sm_timeout(slot, msecs_to_tb(50));
 		}
-		/* No power state change, fall through */
+		/* No power state change, */
+		/* fallthrough */
 	case PCI_SLOT_STATE_FRESET_POWER_OFF:
 		PCIE_SLOT_DBG(slot, "FRESET: Power is off, turn on\n");
 		if (slot->ops.set_power_state)
diff --git a/roms/skiboot/hw/fsp/fsp-sysparam.c b/roms/skiboot/hw/fsp/fsp-sysparam.c
index 2dbf05c6e4387c171bd3469f3ddb..5d7ae91e8c1a838541b59f3b9601 100644
--- a/roms/skiboot/hw/fsp/fsp-sysparam.c
+++ b/roms/skiboot/hw/fsp/fsp-sysparam.c
@@ -100,7 +100,7 @@ static int fsp_sysparam_process(struct sysparam_req *r)
 	case 0x00: /* XXX Is that even possible ? */
 	case 0x11: /* Data in request */
 		memcpy(r->ubuf, &r->resp.data.words[2], len);
-		/* pass through */
+		/* fallthrough */
 	case 0x12: /* Data in TCE */
 		stlen = len;
 		break;
diff --git a/roms/skiboot/platforms/ibm-fsp/firenze-pci.c b/roms/skiboot/platforms/ibm-fsp/firenze-pci.c
index fd15e88a8104d2bdde8521aa8040..66f6efa9666765a14fb3509ad1cc 100644
--- a/roms/skiboot/platforms/ibm-fsp/firenze-pci.c
+++ b/roms/skiboot/platforms/ibm-fsp/firenze-pci.c
@@ -507,7 +507,8 @@ static int64_t firenze_pci_slot_freset(struct pci_slot *slot)
 					msecs_to_tb(FIRENZE_PCI_SLOT_DELAY));
 		}
 
-		/* Fall through: Power is off, turn it on */
+		/* Power is off, turn it on */
+		/* Fallthrough */
 	case FIRENZE_PCI_SLOT_FRESET_POWER_OFF:
 		/* Update last power status */
 		pval = (uint8_t *)(plat_slot->req->rw_buf);

Places

File 4000-GCC7-fixes-for-Wimplicit-fallthroug.patch of Package qemu-testsuite.19805

Places