Microchip-MPLAB-Harmony / csp

Harmony 3 Chip Support Package
https://onlinedocs.microchip.com/v2/keyword-lookup?keyword=MH3_csp&redirect=true
Other
34 stars 22 forks source link

Extended CAN template issue #54

Closed arunningpir8 closed 1 year ago

arunningpir8 commented 2 years ago

https://github.com/Microchip-MPLAB-Harmony/csp/blob/6b191b04015eaf9446280b2f81a3b88368c8f9a3/peripheral/can_01152/templates/plib_can.c.ftl#L234

should read: txMessage->msgEID = ((id & 0x3FFFF) << 10) | CAN_MSG_IDE_MASK | CAN_MSG_SRR_MASK;

this causes many issues in Extended CAN applications.

As per Microchip documentation: SRR: Substitute Remote Request bit In case of a standard message format (IDE = 0), this bit is don’t care. In case of an extended message format (IDE = 1), this bit should always be set.

jigneshmoradiya1 commented 2 years ago

@arunningpir8 , Thanks for reporting this issue. This issue will be fixed in the next csp release.

vishalnxt commented 1 year ago

@arunningpir8 , this issue is resolved in the latest CSP release v3.17.0. Hence, we are closing this issue. Feel free to reopen if you are still facing any problem.