Skip to content

doc: PPP sequence diagrams#181

Merged
SeppoTakalo merged 1 commit into
nrfconnect:mainfrom
SeppoTakalo:ppp_diagrams
Feb 16, 2026
Merged

doc: PPP sequence diagrams#181
SeppoTakalo merged 1 commit into
nrfconnect:mainfrom
SeppoTakalo:ppp_diagrams

Conversation

@SeppoTakalo
Copy link
Copy Markdown
Contributor

Change PPP examples to SVG diagrams to show all channels.

Added a bit more information to SVG diagrams, so that there is now a simplified view of the full PPP session and including the session termination.

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.

Awesome!

Copy link
Copy Markdown
Collaborator

@divipillai divipillai left a comment

Choose a reason for hiding this comment

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

updated .svg file to be added. Too minor changes on font size and color suggested. Thanks.

Comment thread doc/app/sm_cellular_modem.rst Outdated
Comment thread doc/app/sm_cellular_modem.rst Outdated
Comment thread doc/app/sm_cellular_modem.rst Outdated
Copy link
Copy Markdown
Contributor

@MarkusLassila MarkusLassila left a comment

Choose a reason for hiding this comment

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

Greatly approved!

Change PPP examples to SVG diagrams to show all channels.

Signed-off-by: Seppo Takalo <seppo.takalo@nordicsemi.no>
Copy link
Copy Markdown
Collaborator

@divipillai divipillai left a comment

Choose a reason for hiding this comment

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

Nice.

@SeppoTakalo SeppoTakalo merged commit 89894a8 into nrfconnect:main Feb 16, 2026
2 checks passed
@SeppoTakalo SeppoTakalo deleted the ppp_diagrams branch February 16, 2026 11:00
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