-
Notifications
You must be signed in to change notification settings - Fork 67
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
DESMAKERS-3827: enable 11/29Bits id frame send and receive #284
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As usual, minor clue about the CAN details. But if the corresponding (manual) tests are performed.
I can just approve :)
libraries/CAN/src/CANXMC.cpp
Outdated
/* Configuration of CAN Node and enable the clock */ | ||
#if (UC_FAMILY == XMC4) | ||
#define CAN_CLOCK_SOURCE ((XMC_CAN_CANCLKSRC_t)XMC_CAN_CANCLKSRC_FPERI) | ||
#define CAN_CLOCK_SOURCE ((XMC_CAN_CANCLKSRC_t)XMC_CAN_CANCLKSRC_FPERI) // synchronous clock for XMC4000 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All good, but if we have a lot of conditional compiling, in the future we have to check if some of these things might be better defined in the each board variant file...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, I agree - the XMC4000 and XMC1000 have some different settings. But also not board different... Maybe in the future a more reasonable set of variants could be considered
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe this should be defined in variants files
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I changed the variants files 😄
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
look good for me, but please test on hardware.
By creating this pull request you agree to the terms in CONTRIBUTING.md.
https://github.com/Infineon/.github/blob/master/CONTRIBUTING.md
--- DO NOT DELETE ANYTHING ABOVE THIS LINE ---
CONTRIBUTING.md also tells you what to expect in the PR process.
Description
Now sendpacket() support both 11 & 29 bits identifier. same for receive.
Functions that are not the same as the official API can be removed, updating the README and example sketches.
Related Issue
N/A
Context
Once it's up and running, it works ;)