Skip to content
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

Merged
merged 3 commits into from
Dec 6, 2023

Conversation

hfakkiz
Copy link
Contributor

@hfakkiz hfakkiz commented Nov 24, 2023

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

  • PR Title follows correct guidelines.
  • Description of changes and all other relevant information.
  • (Optional) Link any related GitHub issues using a keyword
  • (Optional) Provide info on any relevant functional testing/validation. For API changes or significant features, this is not optional.

@hfakkiz hfakkiz added MAX32655 Related to the MAX32655 (ME17) MAX32670 Related to the MAX32670 (ME15) MAX32672 Related to the MAX32672 (ME21) MAX32675 Related to the MAX32675 (ME16) MAX32680 Related to the MAX32680 (ME20) MAX32690 Related to the MAX32690 (ME18) MAX78000 Related to the MAX78000 (AI85) MAX78002 Related to the MAX78002 (AI87) MAX32662 Related to the MAX32662 (ME12) labels Nov 24, 2023
Add clock selection feature of revB WDT to MAX32655, MAX32662, MAX32670, MAX32672, MAX32675, MAX32680, MAX32690, MAX78000 and MAX78002.

Signed-off-by: Furkan Akkiz <hasanfurkan.akkiz@analog.com>
@hfakkiz hfakkiz force-pushed the feature/wdt-select-clock-source branch from fd5c532 to 97b390e Compare November 24, 2023 11:42
Copy link
Contributor

@Jake-Carter Jake-Carter left a 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

Libraries/PeriphDrivers/Source/WDT/wdt_ai87.c Outdated Show resolved Hide resolved
Libraries/PeriphDrivers/Source/WDT/wdt_me12.c Outdated Show resolved Hide resolved
Libraries/PeriphDrivers/Source/WDT/wdt_me17.c Outdated Show resolved Hide resolved
Libraries/PeriphDrivers/Source/WDT/wdt_me18.c Show resolved Hide resolved
Optimized code for some target numbers which has same clock sources for all instances.

Signed-off-by: Furkan Akkiz <hasanfurkan.akkiz@analog.com>
Copy link
Contributor

@Jake-Carter Jake-Carter left a 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

Libraries/PeriphDrivers/Source/WDT/wdt_me15.c Outdated Show resolved Hide resolved
Libraries/PeriphDrivers/Source/WDT/wdt_revb.c Show resolved Hide resolved
Libraries/PeriphDrivers/Source/WDT/wdt_me18.c Show resolved Hide resolved
@Jake-Carter
Copy link
Contributor

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 <hasanfurkan.akkiz@analog.com>
@hfakkiz hfakkiz force-pushed the feature/wdt-select-clock-source branch from 683953c to a32227c Compare December 5, 2023 10:23
@Jake-Carter Jake-Carter merged commit 6035f1a into main Dec 6, 2023
12 of 13 checks passed
@Jake-Carter Jake-Carter deleted the feature/wdt-select-clock-source branch December 6, 2023 14:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
MAX32655 Related to the MAX32655 (ME17) MAX32662 Related to the MAX32662 (ME12) MAX32670 Related to the MAX32670 (ME15) MAX32672 Related to the MAX32672 (ME21) MAX32675 Related to the MAX32675 (ME16) MAX32680 Related to the MAX32680 (ME20) MAX32690 Related to the MAX32690 (ME18) MAX78000 Related to the MAX78000 (AI85) MAX78002 Related to the MAX78002 (AI87)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants