Browse Source

qcacmn: Fix compilation error in flow control switch case logic

In flow control code switch logic break statements are not used
intentionally, but compiler is treating this as error.
So to avoid compilation error add fall through comment.

Change-Id: Ic79b75c48554182fba7c4c0f3885e3a44347e2e7
CRs-Fixed: 3090182
Karthik Kantamneni 3 years ago
parent
commit
e79b8799c1
2 changed files with 7 additions and 0 deletions
  1. 3 0
      dp/wifi3.0/dp_tx_desc.h
  2. 4 0
      dp/wifi3.0/dp_tx_flow_control.c

+ 3 - 0
dp/wifi3.0/dp_tx_desc.h

@@ -251,16 +251,19 @@ dp_tx_adjust_flow_pool_state(struct dp_soc *soc,
 		soc->pause_cb(pool->flow_pool_id,
 		soc->pause_cb(pool->flow_pool_id,
 			      WLAN_NETIF_PRIORITY_QUEUE_OFF,
 			      WLAN_NETIF_PRIORITY_QUEUE_OFF,
 			      WLAN_DATA_FLOW_CTRL_PRI);
 			      WLAN_DATA_FLOW_CTRL_PRI);
+		/* fallthrough */
 
 
 	case FLOW_POOL_VO_PAUSED:
 	case FLOW_POOL_VO_PAUSED:
 		soc->pause_cb(pool->flow_pool_id,
 		soc->pause_cb(pool->flow_pool_id,
 			      WLAN_NETIF_VO_QUEUE_OFF,
 			      WLAN_NETIF_VO_QUEUE_OFF,
 			      WLAN_DATA_FLOW_CTRL_VO);
 			      WLAN_DATA_FLOW_CTRL_VO);
+		/* fallthrough */
 
 
 	case FLOW_POOL_VI_PAUSED:
 	case FLOW_POOL_VI_PAUSED:
 		soc->pause_cb(pool->flow_pool_id,
 		soc->pause_cb(pool->flow_pool_id,
 			      WLAN_NETIF_VI_QUEUE_OFF,
 			      WLAN_NETIF_VI_QUEUE_OFF,
 			      WLAN_DATA_FLOW_CTRL_VI);
 			      WLAN_DATA_FLOW_CTRL_VI);
+		/* fallthrough */
 
 
 	case FLOW_POOL_BE_BK_PAUSED:
 	case FLOW_POOL_BE_BK_PAUSED:
 		soc->pause_cb(pool->flow_pool_id,
 		soc->pause_cb(pool->flow_pool_id,

+ 4 - 0
dp/wifi3.0/dp_tx_flow_control.c

@@ -137,21 +137,25 @@ dp_tx_flow_ctrl_reset_subqueues(struct dp_soc *soc,
 		soc->pause_cb(pool->flow_pool_id,
 		soc->pause_cb(pool->flow_pool_id,
 			      WLAN_NETIF_PRIORITY_QUEUE_ON,
 			      WLAN_NETIF_PRIORITY_QUEUE_ON,
 			      WLAN_DATA_FLOW_CTRL_PRI);
 			      WLAN_DATA_FLOW_CTRL_PRI);
+		/* fallthrough */
 
 
 	case FLOW_POOL_VO_PAUSED:
 	case FLOW_POOL_VO_PAUSED:
 		soc->pause_cb(pool->flow_pool_id,
 		soc->pause_cb(pool->flow_pool_id,
 			      WLAN_NETIF_VO_QUEUE_ON,
 			      WLAN_NETIF_VO_QUEUE_ON,
 			      WLAN_DATA_FLOW_CTRL_VO);
 			      WLAN_DATA_FLOW_CTRL_VO);
+		/* fallthrough */
 
 
 	case FLOW_POOL_VI_PAUSED:
 	case FLOW_POOL_VI_PAUSED:
 		soc->pause_cb(pool->flow_pool_id,
 		soc->pause_cb(pool->flow_pool_id,
 			      WLAN_NETIF_VI_QUEUE_ON,
 			      WLAN_NETIF_VI_QUEUE_ON,
 			      WLAN_DATA_FLOW_CTRL_VI);
 			      WLAN_DATA_FLOW_CTRL_VI);
+		/* fallthrough */
 
 
 	case FLOW_POOL_BE_BK_PAUSED:
 	case FLOW_POOL_BE_BK_PAUSED:
 		soc->pause_cb(pool->flow_pool_id,
 		soc->pause_cb(pool->flow_pool_id,
 			      WLAN_NETIF_BE_BK_QUEUE_ON,
 			      WLAN_NETIF_BE_BK_QUEUE_ON,
 			      WLAN_DATA_FLOW_CTRL_BE_BK);
 			      WLAN_DATA_FLOW_CTRL_BE_BK);
+		/* fallthrough */
 	default:
 	default:
 		break;
 		break;
 	}
 	}