9607 qede: this statement may fall through

Reviewed by: Andy Fiddaman <af@citrus-it.net> Reviewed by: Andrew Stormont <andyjstormont@gmail.com> Reviewed by: Ken Mays <kmays2000@gmail.com> Approved by: Dan McDonald <danmcd@joyent.com>
illumos · Jul 30, 2018 · 0d7de61 · 0d7de61
1 parent 77b62fe
commit 0d7de61
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/usr/src/uts/common/io/qede/579xx/drivers/ecore/ecore_cxt.c b/usr/src/uts/common/io/qede/579xx/drivers/ecore/ecore_cxt.c
@@ -2225,13 +2225,12 @@ enum _ecore_status_t ecore_cxt_set_pf_params(struct ecore_hwfn *p_hwfn,
 	case ECORE_PCI_ETH_RDMA:
 	case ECORE_PCI_ETH_IWARP:
 	case ECORE_PCI_ETH_ROCE:
-	{
 		ecore_rdma_set_pf_params(p_hwfn,
 					 &p_hwfn->pf_params.rdma_pf_params,
 					 rdma_tasks);
 
 		/* no need for break since RoCE coexist with Ethernet */
-	}
+		/* FALLTHROUGH */
 	case ECORE_PCI_ETH:
 	{
 		struct ecore_eth_pf_params *p_params =

diff --git a/usr/src/uts/common/io/qede/579xx/drivers/ecore/ecore_dev.c b/usr/src/uts/common/io/qede/579xx/drivers/ecore/ecore_dev.c
@@ -2218,13 +2218,13 @@ enum _ecore_status_t ecore_hw_init(struct ecore_dev *p_dev,
 					     p_hwfn->hw_info.hw_mode);
 			if (rc != ECORE_SUCCESS)
 				break;
-			/* Fall into */
+			/* FALLTHROUGH */
 		case FW_MSG_CODE_DRV_LOAD_PORT:
 			rc = ecore_hw_init_port(p_hwfn, p_hwfn->p_main_ptt,
 						p_hwfn->hw_info.hw_mode);
 			if (rc != ECORE_SUCCESS)
 				break;
-			/* Fall into */
+			/* FALLTHROUGH */
 		case FW_MSG_CODE_DRV_LOAD_FUNCTION:
 			rc = ecore_hw_init_pf(p_hwfn, p_hwfn->p_main_ptt,
 					      p_params->p_tunn,