CANBus RX interrupts of HAL

[wu ryan]

This is my RX callback,it only can run one time, anything I set wrong?

I use it in LOOPBACK mode.

void HAL_CAN_RxCpltCallback(CAN_HandleTypeDef* hcan)

{

 Response=(uint8_t *)&hcan->pRxMsg->Data;

HAL_UART_Transmit(&huart4,Response, 8, 5);

HAL_CAN_Receive_IT(hcan,CAN_FIFO0);

}

static void MX_CAN_Init(void)

{

 hcan.Instance = CAN;

 hcan.Init.Prescaler = 2;

 hcan.Init.Mode = CAN_MODE_LOOPBACK;

 hcan.Init.SJW = CAN_SJW_1TQ;

 hcan.Init.BS1 = CAN_BS1_14TQ;

 hcan.Init.BS2 = CAN_BS2_5TQ;

 hcan.Init.TTCM = DISABLE;

 hcan.Init.ABOM = DISABLE;

 hcan.Init.AWUM = DISABLE;

 hcan.Init.NART = DISABLE;

 hcan.Init.RFLM = DISABLE;

 hcan.Init.TXFP = DISABLE;

 if (HAL_CAN_Init(&hcan) != HAL_OK)

 {

  _Error_Handler(__FILE__, __LINE__);

 }

}

[T J]

ABOM enable

use a transceiver, you need two physical CanBus controllers on the bus for correct operation, not loopback

[T J]

maybe you could purchase a USB Can Transceiver, then work your receiver code first.

I used a CanDo unit from UK, set it to transmit every second, it took two days to get the RxTx interrupts all working correctly.

[RN_it]

can you post your tx code ?

also error check UART TRANSMIT .

[T J]

void checkCanTxMsgs(void) {
 // this is a non-ordered packet transmitter.
//int canMsgCounter =0;
if(!canTxMsgTableEMPTY) {
 if ((canTxMsgOUT != canTxMsgIN) || canTxMsgTableFULL) // non-ordered packet transmitter
 { 	
 
 
 if ((CAN->TSR & CAN_TSR_TME0) == CAN_TSR_TME0) 														// (1) 
 {
 if (showRxCanFrames)
 sendCanTxFrametoConsole();
 
 CAN->sTxMailBox[0].TDTR = canTxMsgLength[canTxMsgOUT]; 							// (2) length
 CAN->sTxMailBox[0].TDLR = canTxMsgBottomWord[canTxMsgOUT]; 					// (3) 4bytes
 CAN->sTxMailBox[0].TDHR = canTxMsgTopWord[canTxMsgOUT]; 					 // (3) 4bytes
 CAN->sTxMailBox[0].TIR = ((uint32_t)canTxMsgID[canTxMsgOUT] << 21 | CAN_TI0R_TXRQ); 	// (4)	// send it now if the line is idle
 
 // destroy old data to be sure we only transmit fresh data
 // not needed
 		canTxMsgLength[canTxMsgOUT] 	= 0;			
 canTxMsgBottomWord[canTxMsgOUT] = 0; 	
 canTxMsgTopWord[canTxMsgOUT]	= 0; 		
 canTxMsgID[canTxMsgOUT++]		= 0;
					
 canTxMsgOUT &= 0x3F; 						// 64 buffer elements
 canTxMsgTableFULL = false;
 canTxMsgOverrun = false;
 //canMsgCounter ++;
				
 }
 
 if ((canTxMsgOUT != canTxMsgIN) || canTxMsgTableFULL) 
 if ((CAN->TSR & CAN_TSR_TME1) == CAN_TSR_TME1) 					 				// (1) 
 {
 if (showRxCanFrames)
 sendCanTxFrametoConsole();				
 CAN->sTxMailBox[1].TDTR = canTxMsgLength[canTxMsgOUT]; 						// (2) length
 CAN->sTxMailBox[1].TDLR = canTxMsgBottomWord[canTxMsgOUT]; 				// (3) 4bytes
 CAN->sTxMailBox[1].TDHR = canTxMsgTopWord[canTxMsgOUT]; 				 	// (3) 4bytes
 CAN->sTxMailBox[1].TIR = ((uint32_t) canTxMsgID[canTxMsgOUT] << 21 | CAN_TI1R_TXRQ); 	// (4)	// send it now if the line is idle
 
 // destroy old data to be sure we only transmit fresh data
 // not needed
 		canTxMsgLength[canTxMsgOUT] 	= 0;			
 canTxMsgBottomWord[canTxMsgOUT] = 0; 	
 canTxMsgTopWord[canTxMsgOUT]	= 0; 		
 canTxMsgID[canTxMsgOUT++]		= 0;
					
 canTxMsgOUT &= 0x3F; 					// 64 buffer depth
 canTxMsgOverrun = false;
 canTxMsgTableFULL = false;
 //canMsgCounter ++;
 }
 if ((canTxMsgOUT != canTxMsgIN) || canTxMsgTableFULL) 
 if ((CAN->TSR & CAN_TSR_TME2) == CAN_TSR_TME2) 											// (1) 
 {
 if (showRxCanFrames)
 sendCanTxFrametoConsole();				
 CAN->sTxMailBox[2].TDTR = canTxMsgLength[canTxMsgOUT]; 							// (2) length
 CAN->sTxMailBox[2].TDLR = canTxMsgBottomWord[canTxMsgOUT]; 					// (3) 4bytes
 CAN->sTxMailBox[2].TDHR = canTxMsgTopWord[canTxMsgOUT]; 						// (3) 4bytes
 CAN->sTxMailBox[2].TIR = ((uint32_t) canTxMsgID[canTxMsgOUT] << 21 | CAN_TI2R_TXRQ); 	// (4)	// send it now if the line is idle
 
 // destroy old data to be sure we only transmit fresh data
 // not needed
 		canTxMsgLength[canTxMsgOUT] 	= 0;			
 canTxMsgBottomWord[canTxMsgOUT] = 0; 	
 canTxMsgTopWord[canTxMsgOUT]	= 0; 		
 canTxMsgID[canTxMsgOUT++]		= 0;
					
 
 canTxMsgOUT &= 0x3F; 		// 64 buffer elements
 canTxMsgOverrun = false;
 canTxMsgTableFULL = false;
 //canMsgCounter ++;
 }
 if (canTxMsgOUT == canTxMsgIN)	canTxMsgTableEMPTY = true;
 }
 }
}