Skip to content

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

Merged
merged 4 commits into from
Jul 3, 2022

Conversation

oldsheep68
Copy link
Contributor

…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

@github-actions
Copy link

Memory map comparison

@burrbull
Copy link
Member

You can simplify this using _strip:

COMP:
  COMP1_CSR:
    _strip:
      - "COMP1_"
  COMP2_CSR:
    _strip:
      - "COMP2_"

@newAM newAM added the stm32l4 label Dec 28, 2021
@oldsheep68
Copy link
Contributor Author

Hi burbull
Thanks great solution. I am going to test the simplified version you recommend, and will check in the modified variant of register modification.

@github-actions
Copy link

Memory map comparison

@oldsheep68
Copy link
Contributor Author

oldsheep68 commented Jan 3, 2022 via email

@github-actions
Copy link

github-actions bot commented Jan 4, 2022

Memory map comparison

adamgreig
adamgreig previously approved these changes Jul 3, 2022
Copy link
Member

@adamgreig adamgreig left a 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+

@github-actions
Copy link

github-actions bot commented Jul 3, 2022

Memory map comparison

@bors
Copy link
Contributor

bors bot commented Jul 3, 2022

Merge conflict.

adamgreig
adamgreig previously approved these changes Jul 3, 2022
Copy link
Member

@adamgreig adamgreig left a 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+

bors bot added a commit that referenced this pull request Jul 3, 2022
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>
@bors
Copy link
Contributor

bors bot commented Jul 3, 2022

Build failed:

Copy link
Member

@adamgreig adamgreig left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bors r+

@github-actions
Copy link

github-actions bot commented Jul 3, 2022

Memory map comparison

@bors
Copy link
Contributor

bors bot commented Jul 3, 2022

Build succeeded:

@bors bors bot merged commit 1923975 into stm32-rs:master Jul 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants