topic Re: LPC546 MCANRxFifo assert in LPC Microcontrollers

LPC546 MCANRxFifo assert

Hector_Jimenez — Fri, 21 Feb 2025 13:24:37 GMT

I'm using MCAN module in LPC54616 MCU in a very unstable environment, which causes assertion in MCAN_ReadRxFifo function when receiving null frames (rxFrame = NULL).

fsl_mcan.c

[...]
status_t MCAN_ReadRxFifo(CAN_Type *base, uint8_t fifoBlock, mcan_rx_buffer_frame_t *rxFrame)
{
/* Assertion. */
assert((0U == fifoBlock) || (1U == fifoBlock));
assert(NULL != rxFrame);
[...]

leading into MCU continiously stalling.

Is this a real assertion reason or could it be sorted out by skipping that reading?

Perhaps it's already been solved in later versions of SDK, can anyone confirm this? My SDK version is 2.10.

Thank you!

Re: LPC546 MCANRxFifo assert

Harry_Zhang — Mon, 24 Feb 2025 08:04:19 GMT

Hi @Hector_Jimenez

In the MCAN_ReadRxFifo function,

assert(NULL != rxFrame) is a necessary condition.

So it could not be sorted out by skipping that reading.

Harry

Re: LPC546 MCANRxFifo assert

Hector_Jimenez — Mon, 24 Feb 2025 08:18:14 GMT

Hello @Harry_Zhang.

What if we ack the read and return failure? This return value could be checked when calling to MCAN_ReadRxFifo function and set a different status for CAN bus, like kStatus_MCAN_RxFifo0Lost.

status_t MCAN_ReadRxFifo(CAN_Type *base, uint8_t fifoBlock, mcan_rx_buffer_frame_t *rxFrame) { /* Assertion. */ assert((0U == fifoBlock) || (1U == fifoBlock)); if (NULL == rxFrame) { /* Acknowledge the read. */ if (0U == fifoBlock) { base->RXF0A = (base->RXF0S & CAN_RXF0S_F0GI_MASK) >> CAN_RXF0S_F0GI_SHIFT; } else { base->RXF1A = (base->RXF1S & CAN_RXF1S_F1GI_MASK) >> CAN_RXF1S_F1GI_SHIFT; } return kStatus_Fail; } [...]

Kind regards

Re: LPC546 MCANRxFifo assert

Harry_Zhang — Wed, 26 Feb 2025 03:41:13 GMT

Hi @Hector_Jimenez

Yes, you can add related code.

Or you can add a new function.

status_t CheckFrame(CAN_Type *base, uint8_t fifoBlock, mcan_rx_buffer_frame_t *rxFrame) { /* Assertion. */ assert((0U == fifoBlock) || (1U == fifoBlock)); if (NULL == rxFrame) { /* Acknowledge the read. */ if (0U == fifoBlock) { base->RXF0A = (base->RXF0S & CAN_RXF0S_F0GI_MASK) >> CAN_RXF0S_F0GI_SHIFT; } else { base->RXF1A = (base->RXF1S & CAN_RXF1S_F1GI_MASK) >> CAN_RXF1S_F1GI_SHIFT; } return kStatus_Fail; } else { MCAN_ReadRxFifo(base, fifoBlock, rxFrame); } [...]

Harry

Re: LPC546 MCANRxFifo assert

Hector_Jimenez — Fri, 28 Feb 2025 07:27:08 GMT

Hi, @Harry_Zhang .

We are testing the code I posted and it seems to work fine in these unstable conditions. Some frames are lost, but MCU doesn't get stalled.

Thanks for your help.

Regards!