Skip to content

Conversation

@SnakeOilSalesman
Copy link
Contributor

Apply fixes required after recent changes.
ATDF file was refreshed as well.

And I also found that step 8 in the README.md is no longer required since there is no table there any more.

Copy link
Owner

@Rahix Rahix left a comment

Choose a reason for hiding this comment

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

Hey, thanks a lot for the contribution! Changes look good, but I have one question for clarification, see below.

<bitfield caption="USART Data Register Empty" mask="0x20" name="UDRE" rw="R"/>
<bitfield caption="Framing Error" mask="0x10" name="FE" rw="R"/>
<bitfield caption="Data Overrun" mask="0x08" name="DOR" rw="R"/>
<bitfield caption="Parity Error" mask="0x04" name="UPE" rw="R"/>
Copy link
Owner

Choose a reason for hiding this comment

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

Just for my understanding, this was an update published by Microchip?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yep. Just downloaded a fresh pack via Mplab X IDE. These changes were there.
Should I revert them?

To be honest, I haven't tested USART with this MCU yet.

Copy link
Owner

Choose a reason for hiding this comment

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

No, totally fine as it is :)

Just wanted to make sure these changes really came from upstream. The ATDFs should not be modified locally in this project, we want to mirror whatever gets released by the vendor.

@Rahix Rahix merged commit 466d415 into Rahix:main Dec 7, 2025
2 checks passed
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