-
-
Notifications
You must be signed in to change notification settings - Fork 5.4k
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
same70: Add CAN #6366
same70: Add CAN #6366
Conversation
14c5f73
to
56f0d30
Compare
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.
Thanks. Seems fine to me. I have a few very minor comments - see below.
-Kevin
src/atsam/main.c
Outdated
#if CONFIG_MACH_SAME70 | ||
if (id == MCAN0_CLOCK_ID || id == MCAN1_CLOCK_ID) | ||
return FREQ_CAN; | ||
#endif |
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 before, it would be preferable to use a regular if instead of an ifdef.
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.
This isn't feasible here, unless we also define the clock ids for other chips, which would be a bit silly. Going to keep this one.
fec3a00
to
49d6a91
Compare
Signed-off-by: Luke Vuksta <[email protected]>
49d6a91
to
c9b36e3
Compare
Thanks. -Kevin |
Signed-off-by: Luke Vuksta <[email protected]>
Signed-off-by: Luke Vuksta <[email protected]>
Add CAN to same70. Thanks to @eamaclean for the help.
Depends on #6365.