-
Notifications
You must be signed in to change notification settings - Fork 244
shorter registernames for stm32l4 devices of the COMP registers + add… #682
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
Conversation
You can simplify this using COMP:
COMP1_CSR:
_strip:
- "COMP1_"
COMP2_CSR:
_strip:
- "COMP2_" |
Hi burbull |
I have updated the pull request as you recomended.
Tanks a Lot, this ist a way better solution.
…
Am 28.12.2021 in 09:53, Zgarbul Andrey ***@***.***> schrieb:
You can simplify this using _strip:
COMP: COMP1_CSR: _strip: - "COMP1_" COMP2_CSR: _strip: - "COMP2_"
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
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, looks good! I noticed one typo (256 instead of 255) that I fixed.
bors r+
Merge conflict. |
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.
Rebased to fix merge conflicts.
bors r+
682: shorter registernames for stm32l4 devices of the COMP registers + add… r=adamgreig a=oldsheep68 …ing the register descriptions Changes in: crates stm32l4x1, crates stm32l4x2, crates stm32l4x3, crates stm32l4x5, crates stm32l4x6 registers of the comparator have been renamed, example: from comp1_en -> en this makes a more generic use in programming possible Furthermore, the description has been added for all comp During this work, I found the following topics, where I do not know, how to address: 1) Reference Manual RM0394: is valid for: STM32L443 and STM32L433; but I had to add the register description in the device descriptors stm32l4x3 of RM0351 2) The device STM32L471 is part of stm32l4x1, but is in RM0351 whereas all other devices are part of RM0394 Co-authored-by: oldsheep68 <oldsheep68@hotmail.com> Co-authored-by: Adam Greig <adam@adamgreig.com>
Build failed: |
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.
bors r+
Build succeeded: |
…ing the register descriptions
Changes in: crates stm32l4x1, crates stm32l4x2, crates stm32l4x3, crates stm32l4x5, crates stm32l4x6
registers of the comparator have been renamed, example: from comp1_en -> en
this makes a more generic use in programming possible
Furthermore, the description has been added for all comp
During this work, I found the following topics, where I do not know, how to address:
Reference Manual RM0394: is valid for: STM32L443 and STM32L433; but I had to add the register description in the device descriptors stm32l4x3 of RM0351
The device STM32L471 is part of stm32l4x1, but is in RM0351 whereas all other devices are part of RM0394