occam25 opened a new pull request, #15952:
URL: https://github.com/apache/nuttx/pull/15952

   ## Summary
   
   Fix timestamp in socket CAN. Right now the timestamp is only generated if 
there is no reader and the frame is stored in the read ahead list. This is 
solved by moving the timestamp generation before the code flow branch.
   
   ## Impact
   
   This has no impact other than having the timestamp working again.
   
   ## Testing
   
   As discussed in the mailing list, I have tested this fix in our current 
branch that is older than main. I can't test it on main because our whole 
system would not even compile. Updating the system to the latest Nuttx is a 
pending task in our backlog but it will take quite some time. Nevertheless, 
since this is a small change that doesn't add any new code but just moves the 
timestamp generation it may be worth to consider it.
   
   
   


-- 
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

Reply via email to