drivers: can: mcan: declare the loop condition variables inside the loops

Declare the loop condition variable inside the loops to limit their scope.

Signed-off-by: Henrik Brix Andersen <[email protected]>

Author: henrikbrixandersen

drivers/can/can_mcan.c

@@ -457,7 +457,6 @@ static void can_mcan_state_change_handler(const struct device *dev)
 	void *state_cb_data = data->common.state_change_cb_user_data;
 	const struct can_mcan_callbacks *cbs = config->callbacks;
 	can_tx_callback_t tx_cb;
-	uint32_t tx_idx;
 	struct can_bus_err_cnt err_cnt;
 	enum can_state state;
 	uint32_t cccr;
@@ -480,7 +479,7 @@ static void can_mcan_state_change_handler(const struct device *dev)
 		}
 
 		/* Call all TX queue callbacks with -ENETUNREACH */
-		for (tx_idx = 0U; tx_idx < cbs->num_tx; tx_idx++) {
+		for (uint32_t tx_idx = 0U; tx_idx < cbs->num_tx; tx_idx++) {
 			tx_cb = cbs->tx[tx_idx].function;
 
 			if (tx_cb != NULL) {
@@ -1075,11 +1074,10 @@ int can_mcan_add_rx_filter_std(const struct device *dev, can_rx_callback_t callb
 	};
 	int filter_id = -ENOSPC;
 	int err;
-	int i;
 
 	k_mutex_lock(&data->lock, K_FOREVER);
 
-	for (i = 0; i < cbs->num_std; i++) {
+	for (int i = 0; i < cbs->num_std; i++) {
 		if (cbs->std[i].function == NULL) {
 			filter_id = i;
 			break;
@@ -1127,11 +1125,10 @@ static int can_mcan_add_rx_filter_ext(const struct device *dev, can_rx_callback_
 	};
 	int filter_id = -ENOSPC;
 	int err;
-	int i;
 
 	k_mutex_lock(&data->lock, K_FOREVER);
 
-	for (i = 0; i < cbs->num_ext; i++) {
+	for (int i = 0; i < cbs->num_ext; i++) {
 		if (cbs->ext[i].function == NULL) {
 			filter_id = i;
 			break;