Skip to content

Conversation

@kloenk
Copy link

@kloenk kloenk commented Feb 7, 2026

Use the numaker reset driver for the reset in the numicro uart driver.

@kloenk
Copy link
Author

kloenk commented Feb 12, 2026

Changes in V2

  • Remove unnecessary parentheses from copied part of the BSP in the reset header
  • Replace (xUL << 24) with a constant in the BSP to see which is IPRSTx

Copy link
Member

@fabiobaltieri fabiobaltieri left a comment

Choose a reason for hiding this comment

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

looks good, CI is catching a couple things, it'll run automatically once this is in though

@kloenk
Copy link
Author

kloenk commented Feb 12, 2026

The long line comment lint is also from the BSP. but guess we also want to alter the file there?

Use the numaker reset driver for the reset in the numicro uart driver.

Signed-off-by: Fiona Behrens <[email protected]>
@kloenk kloenk force-pushed the nuvoton-m48x-reset-uart branch from be0afde to b248ba3 Compare February 12, 2026 16:25
@kloenk
Copy link
Author

kloenk commented Feb 12, 2026

Not sure if the doxygen lint is required in this case? as this are just board constants to be used in devicetrees and self documenting for that usecase/documented in the nuvoton TRM?

Changes in v3

  • Change line length of block copied from the BSP to less then 100 lines

@sonarqubecloud
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: Boards/SoCs area: Devicetree Binding PR modifies or adds a Device Tree binding area: Reset area: UART Universal Asynchronous Receiver-Transmitter platform: Nuvoton NPCM platform: Nuvoton NPCX Nuvoton NPCX platform: Nuvoton Numicro Numaker Nuvoton Technology Corporation, Numicro Numaker

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants