Skip to content

Conversation

@MarkusLassila
Copy link
Contributor

If NRF91 UART is used without hw-flow-control, the RX can lose data unless hardware RX byte counting is enabled for the UART.

Add overlay for external MCU scenario, as there the customers are likely to run into this.

If NRF91 UART is used without hw-flow-control, the RX can lose
data unless hardware RX byte counting is enabled for the UART.

Add overlay for external MCU scenario, as there the customers
are likely to run into this.

Signed-off-by: Markus Lassila <[email protected]>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Although I am not really looking forward to running into VCOM0 not being enabled for 91 products, when I am switching development areas, this might be our trigger to move from UART2 to UART0 and default pins.

The issue is also present if HWFC is not set for UART0 and using that would allow us to set the values in prj.conf.

Opinions are welcome.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@SeppoTakalo opinion on this?

@MarkusLassila MarkusLassila marked this pull request as ready for review November 19, 2025 12:24
Copy link
Contributor

@SeppoTakalo SeppoTakalo left a comment

Choose a reason for hiding this comment

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

The name overlay-external-mcu.conf is misleading.

I would propose the name overlay-uart2-no-hwfc.conf because that is when it is required.

The overlay-external-mcu.overlay defines HW flow control, so this new file is not a pair for that.
Similarly, there could be overlay-uart2-no-hwfc.overlay that includes the previous one and just disables the HWFC.

@MarkusLassila MarkusLassila marked this pull request as draft December 9, 2025 08:35
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