nimble/phy/cmac: Fix return value for RX late start

andrzej-kaczmarek · andrzej-kaczmarek · commit 1b09cf2a7624 · 2021-04-01T11:26:03.000+02:00
This does not affect current code, but we should return proper code if
RX was late anyway.
diff --git a/nimble/drivers/dialog_cmac/src/ble_phy.c b/nimble/drivers/dialog_cmac/src/ble_phy.c
@@ -1369,6 +1369,7 @@ ble_phy_rx_set_start_time(uint32_t cputime, uint8_t rem_usecs)
 {
     uint32_t ll_val32;
     int32_t time_till_start;
+    int rc = 0;
 
     MCU_DIAG_SER('r');
 
@@ -1404,11 +1405,12 @@ ble_phy_rx_set_start_time(uint32_t cputime, uint8_t rem_usecs)
             /* We missed start. Start now */
             CMAC->CM_EV_LINKUP_REG =
                 CMAC_CM_EV_LINKUP_REG_LU_FRAME_START_2_ASAP_Msk;
+            rc = BLE_PHY_ERR_RX_LATE;
         }
     }
     __enable_irq();
 
-    return 0;
+    return rc;
 }
 
 int

Original file line number	Diff line number	Diff line change
`@@ -1369,6 +1369,7 @@ ble_phy_rx_set_start_time(uint32_t cputime, uint8_t rem_usecs)`
`1369`	`1369`	`{`
`1370`	`1370`	`uint32_t ll_val32;`
`1371`	`1371`	`int32_t time_till_start;`
	`1372`	`+ int rc = 0;`
`1372`	`1373`
`1373`	`1374`	`MCU_DIAG_SER('r');`
`1374`	`1375`
`@@ -1404,11 +1405,12 @@ ble_phy_rx_set_start_time(uint32_t cputime, uint8_t rem_usecs)`
`1404`	`1405`	`/* We missed start. Start now */`
`1405`	`1406`	`CMAC->CM_EV_LINKUP_REG =`
`1406`	`1407`	`CMAC_CM_EV_LINKUP_REG_LU_FRAME_START_2_ASAP_Msk;`
	`1408`	`+ rc = BLE_PHY_ERR_RX_LATE;`
`1407`	`1409`	`}`
`1408`	`1410`	`}`
`1409`	`1411`	`__enable_irq();`
`1410`	`1412`
`1411`		`- return 0;`
	`1413`	`+ return rc;`
`1412`	`1414`	`}`
`1413`	`1415`
`1414`	`1416`	`int`