Skip to content

app: Add nRF91M1 overlay for pin configuration#234

Merged
MarkusLassila merged 1 commit into
nrfconnect:mainfrom
MarkusLassila:91M1-overlay
Mar 30, 2026
Merged

app: Add nRF91M1 overlay for pin configuration#234
MarkusLassila merged 1 commit into
nrfconnect:mainfrom
MarkusLassila:91M1-overlay

Conversation

@MarkusLassila
Copy link
Copy Markdown
Contributor

@MarkusLassila MarkusLassila commented Mar 26, 2026

Add overlay and documentation for nRF91M1 pin configuration for v2.x pre-releases.

Jira: SM-249

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds support material for the nRF91M1 (pre-programmed nRF9151 SiP) UART/pin configuration by documenting the mapping and introducing a devicetree overlay intended to configure UART pins for DK/external-host scenarios.

Changes:

  • Document nRF91M1 UART pin mapping and usage notes (incl. DTR-to-GND requirement).
  • Add a new overlay-91m1.overlay devicetree overlay for the nRF91M1 pin configuration.
  • Add a new documentation link for nRF9151 LGA pin assignments and reference the new UART docs from SM configuration docs.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

File Description
doc/uart_configuration.rst Adds a new nRF91M1 UART/pin mapping section and operational notes.
doc/links.txt Adds a link target for nRF9151 LGA pin assignments.
doc/app/sm_configuration.rst Lists the new overlay and links to the UART pin details section.
app/overlay-91m1.overlay Introduces a devicetree overlay intended to configure UART0/UART1 pins for nRF91M1.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread app/overlay-nrf91m1.overlay
Comment thread app/overlay-nrf91m1.overlay
Comment thread doc/uart_configuration.rst Outdated
Comment thread doc/uart_configuration.rst Outdated
Copy link
Copy Markdown
Collaborator

@trantanen trantanen left a comment

Choose a reason for hiding this comment

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

Looks very good!

Comment thread doc/uart_configuration.rst Outdated
Comment thread doc/uart_configuration.rst Outdated
Comment thread doc/app/sm_configuration.rst Outdated
@MarkusLassila MarkusLassila force-pushed the 91M1-overlay branch 2 times, most recently from de61938 to 0fc47f3 Compare March 27, 2026 09:28
@MarkusLassila MarkusLassila requested a review from trantanen March 27, 2026 09:28
@trantanen trantanen requested a review from divipillai March 27, 2026 10:29
Copy link
Copy Markdown
Collaborator

@trantanen trantanen left a comment

Choose a reason for hiding this comment

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

I'm basically ready with this. Let's wait for other comments.

Comment thread doc/uart_configuration.rst Outdated
Comment thread app/sample.yaml Outdated
Comment thread app/sample.yaml Outdated
Comment thread app/overlay-nrf91m1.overlay
Comment thread doc/uart_configuration.rst Outdated
Comment thread app/overlay-91m1.overlay Outdated
@MarkusLassila MarkusLassila changed the title app: Add 91M1 overlay for pin configuration app: Add nRF91M1 overlay for pin configuration Mar 27, 2026
@MarkusLassila MarkusLassila requested a review from trantanen March 27, 2026 11:03
Comment thread doc/app/sm_configuration.rst Outdated
Comment thread doc/uart_configuration.rst Outdated
Comment thread doc/uart_configuration.rst Outdated
Comment thread doc/uart_configuration.rst Outdated
Comment thread doc/uart_configuration.rst Outdated
Add overlay and documentation for nRF91M1 pin configuration
for v2.x pre-releases.

Signed-off-by: Markus Lassila <markus.lassila@nordicsemi.no>
@MarkusLassila MarkusLassila merged commit c109a32 into nrfconnect:main Mar 30, 2026
3 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.

4 participants