Re: [PR] Add MSS CAN driver for MPFS [nuttx]

[via GitHub]

acassis commented on PR #17161:
URL: https://github.com/apache/nuttx/pull/17161#issuecomment-3461440492

   > @tinnedkarma , after testing, I encounter this weird issue with corrupted 
frame received from upperhalf. I would say about 10% defection. The CAN ID is 
always either corrupted to 0x0 or 0xC2008020. I initially tried with 
netpkt_copyout and current use netpkt_getdata but both have the same issue. 
Initially, I use quota value 1 for both TX and RX. I increased upto max number 
of hw buffers of 32 but still doesnt help.
   > 
   > ```
   > saluki> ifup can0
   > ifup can0...OK
   > saluki> cansend can0 123#deadbeef
   > [CPU2] mpfs_transmit: TX Frame: ID=0x123, DLC=4
   > [CPU2] mpfs_transmit: TX Debug: cf.can_id=0x123, msg_id=0x48C0000, 
shifted=0x24600000, DLC=4
   > [CPU2] mpfs_transmit: TX HW Write: addr=0x2010C020, ID_reg=0x24600000, 
data_high=0xDEADBEEF, data_low=0x0
   > [CPU0] mpfs_transmit: Sending CAN message (ID: 0x123, DLC: 4) via buffer 0
   > saluki> 
   > saluki> 
   > saluki> 
   > saluki> 
   > saluki> 
   > saluki> cansend can0 123#deadbeef
   > [CPU0] mpfs_transmit: TX Frame: ID=0x0, DLC=0
   > [CPU0] mpfs_transmit: TX Debug: cf.can_id=0x0, msg_id=0x0, shifted=0x0, 
DLC=0
   > [CPU0] mpfs_transmit: TX HW Write: addr=0x2010C020, ID_reg=0x0, 
data_high=0x0, data_low=0x0
   > [CPU1] mpfs_transmit: Sending CAN message (ID: 0x0, DLC: 0) via buffer 0
   > saluki> 
   > saluki> 
   > saluki> cansend can0 123#deadbeef
   > [CPU1] mpfs_transmit: TX Frame: ID=0x123, DLC=4
   > [CPU1] mpfs_transmit: TX Debug: cf.can_id=0x123, msg_id=0x48C0000, 
shifted=0x24600000, DLC=4
   > [CPU2] mpfs_transmit: TX HW Write: addr=0x2010C020, ID_reg=0x24600000, 
data_high=0xDEADBEEF, data_low=0x0
   > [CPU0] mpfs_transmit: Sending CAN message (ID: 0x123, DLC: 4) via buffer 0
   > saluki> 
   > saluki> 
   > saluki> cansend can0 123#deadbeef
   > [CPU0] mpfs_transmit: TX Frame: ID=0x123, DLC=4
   > [CPU0] mpfs_transmit: TX Debug: cf.can_id=0x123, msg_id=0x48C0000, 
shifted=0x24600000, DLC=4
   > [CPU1] mpfs_transmit: TX HW Write: addr=0x2010C020, ID_reg=0x24600000, 
data_high=0xDEADBEEF, data_low=0x0
   > [CPU0] mpfs_transmit: Sending CAN message (ID: 0x123, DLC: 4) via buffer 0
   > saluki> 
   > saluki> cansend can0 123#deadbeef
   > [CPU0] mpfs_transmit: TX Frame: ID=0x123, DLC=4
   > [CPU2] mpfs_transmit: TX Debug: cf.can_id=0x123, msg_id=0x48C0000, 
shifted=0x24600000, DLC=4
   > [CPU3] mpfs_transmit: TX HW Write: addr=0x2010C020, ID_reg=0x24600000, 
data_high=0xDEADBEEF, data_low=0x0
   > [CPU3] mpfs_transmit: Sending CAN message (ID: 0x123, DLC: 4) via buffer 0
   > ```
   > 
   > <img alt="image" width="710" height="672" 
src="https://private-user-images.githubusercontent.com/17795294/507085289-b13ce31c-84b0-4278-9618-d9367d03ae30.png?jwt=eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3NjE3NDI5MzgsIm5iZiI6MTc2MTc0MjYzOCwicGF0aCI6Ii8xNzc5NTI5NC81MDcwODUyODktYjEzY2UzMWMtODRiMC00Mjc4LTk2MTgtZDkzNjdkMDNhZTMwLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNTEwMjklMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjUxMDI5VDEyNTcxOFomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTk2ZWFkMzk0OTg3MDZmNGE0YzIzZWU0YWRlOGYyOWFhMzIzYzM0ZDVhMjk4NTJhODlmYzAzOThkNjc2ZmY5OWYmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0.Ge7gERTo4rPpTiY1RRCc3U0AR3TlfKfO_lSpikBIyY4";>
   
   @haitomatic That is strange, did you try to debug it to discover the root 
cause? It could be some driver buffer misalignment or some code writing beyond 
its expected position.
   
   Please consider to contribute your canfd_test to be included inside 
nuttx-apps/testing/drivers/


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org