-
Notifications
You must be signed in to change notification settings - Fork 92
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
feat(PeriphDrivers): Add clock selection feature to WDT #808
Conversation
Add clock selection feature of revB WDT to MAX32655, MAX32662, MAX32670, MAX32672, MAX32675, MAX32680, MAX32690, MAX78000 and MAX78002. Signed-off-by: Furkan Akkiz <[email protected]>
fd5c532
to
97b390e
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 @hfakkiz, looks good. A couple minor comments below
Optimized code for some target numbers which has same clock sources for all instances. Signed-off-by: Furkan Akkiz <[email protected]>
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.
Thank you @hfakkiz, one last change request based on your latest comments
Thanks @hfakkiz, looks good to go. We'll get a second review then merge. |
Moved clock source check to RevB function and removed 'last_clock_source' parameter. Signed-off-by: Furkan Akkiz <[email protected]>
683953c
to
a32227c
Compare
Pull Request Template
Description
WDT peripheral of MAX32655, MAX32662, MAX32670, MAX32672, MAX32675, MAX32680, MAX32690, MAX78000 and MAX78002 supports selecting different clock sources which are given in user guide.
Added a new function to set clksel register on revB WDT driver.
Added a new function to select supported clock sources for these part numbers.
Checklist Before Requesting Review