Skip to content

feat(comp, g4): Add COMP pin definitions and register enums #613

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

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

IvanLi-CN
Copy link

Add pin definitions for COMP1-7 in data/extra/family/STM32G4.yaml. Add enum definitions and link them in data/registers/comp_v2.yaml.

@embassy-ci
Copy link

embassy-ci bot commented Jun 3, 2025

@Dirbaio
Copy link
Member

Dirbaio commented Jun 3, 2025

Please do not add enums for "0 = disabled, 1 = enabled", they're redundant.

Add pin definitions for COMP1-7 in data/extra/family/STM32G4.yaml.
Add enum definitions and link them in data/registers/comp_v2.yaml.

Signed-off-by: Ivan Li <[email protected]>
@IvanLi-CN IvanLi-CN force-pushed the feat/g4-comp-update branch from e41202e to b739554 Compare June 3, 2025 16:08
@embassy-ci
Copy link

embassy-ci bot commented Jun 3, 2025

@Dirbaio
Copy link
Member

Dirbaio commented Jun 6, 2025

LOCK, SCALEN, BRGEN, EN enums should be deleted, they're redundant.

@IvanLi-CN
Copy link
Author

Sorry, I just noticed the notification email, please give me some time and I will correct the problem. At the moment I'm still trying to figure out what to do about embassy-stm32, when I make progress I'll change the code as you suggested and submit it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants