Skip to content

telco-features: document a PTP BC on GNR-D#1130

Merged
mchiappero merged 1 commit into
suse-edge:mainfrom
mchiappero:perla-ptp
May 20, 2026
Merged

telco-features: document a PTP BC on GNR-D#1130
mchiappero merged 1 commit into
suse-edge:mainfrom
mchiappero:perla-ptp

Conversation

@mchiappero
Copy link
Copy Markdown
Contributor

Introduce a new section, following the existing PTP ones, about running a Boundary Clock on an Intel Granite Rapids-D server. It takes into account the specifics of these new designs, routing synchronization signals internally across NAC, Timing Module and Add-In Cards.

In particular, this commit document the steps required to have a working static BC setup.

@mchiappero mchiappero marked this pull request as draft May 11, 2026 17:37
@mchiappero
Copy link
Copy Markdown
Contributor Author

Marked as WIP due to the still missing "pmc" commands, but it's otherwise ready for review. Any input is very welcome.

@mchiappero mchiappero force-pushed the perla-ptp branch 3 times, most recently from f363631 to fe63b7f Compare May 18, 2026 10:05
@mchiappero mchiappero marked this pull request as ready for review May 18, 2026 10:12
Comment thread asciidoc/product/atip-features.adoc Outdated
@mchiappero
Copy link
Copy Markdown
Contributor Author

Marked as WIP due to the still missing "pmc" commands, but it's otherwise ready for review. Any input is very welcome.

I have added the missing sections. I might still refine the script a little bit, maybe sourcing the input files from command line or env variables, but the text will almost certainly stay the same.

hardys
hardys previously approved these changes May 18, 2026
Copy link
Copy Markdown
Contributor

@hardys hardys 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, I wonder if we should consider re-structuring this page in future though e.g we could have a page per feature - that's unrelated to the content here though and could be handled in a future PR

Comment thread asciidoc/product/atip-features.adoc Outdated
Copy link
Copy Markdown
Collaborator

@ranjinimn ranjinimn left a comment

Choose a reason for hiding this comment

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

Hey @mchiappero I had a few suggestions. Please let me know if you have any questions I would be happy to do another round of review.

Comment thread asciidoc/product/atip-features.adoc Outdated
Comment thread asciidoc/product/atip-features.adoc Outdated
Comment thread asciidoc/product/atip-features.adoc Outdated
Comment thread asciidoc/product/atip-features.adoc Outdated
Comment thread asciidoc/product/atip-features.adoc Outdated
Comment thread asciidoc/product/atip-features.adoc Outdated
Comment thread asciidoc/product/atip-features.adoc Outdated
Comment thread asciidoc/product/atip-features.adoc Outdated
Comment thread asciidoc/product/atip-features.adoc Outdated
Comment thread asciidoc/product/atip-features.adoc Outdated
@mchiappero mchiappero force-pushed the perla-ptp branch 2 times, most recently from 6a33bb6 to ef7d2dc Compare May 19, 2026 15:37
@mchiappero
Copy link
Copy Markdown
Contributor Author

Hey @mchiappero I had a few suggestions. Please let me know if you have any questions I would be happy to do another round of review.

Thank you so much! I have applied all of your changes.

@mchiappero
Copy link
Copy Markdown
Contributor Author

Besides incorporating the changes from the comments, I have made very few modifications, plus a final rebase on main. I guess you can just have a look at the delta. Thanks!

Comment thread asciidoc/product/atip-features.adoc
Copy link
Copy Markdown
Collaborator

@ranjinimn ranjinimn left a comment

Choose a reason for hiding this comment

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

There was a small indentation issue which I must have missed earlier.
Was there anything specific I had to review on this PR apart from the changes suggested previously?

@mchiappero
Copy link
Copy Markdown
Contributor Author

There was a small imdentation issue which I must have missed earlier. Was there anything specific I had to review on this PR apart from the changes suggested previously?

No, only the changes here, if you haven't done so: https://github.com/suse-edge/suse-edge.github.io/compare/7ecee716c41e0c2d0b18d8b538bc80133bd89349..6a33bb682cfc948fa8a0d123d7bbb96f2d918f7c.

Otherwise, thank you very much for your effort!

ranjinimn
ranjinimn previously approved these changes May 19, 2026
Copy link
Copy Markdown
Collaborator

@ranjinimn ranjinimn left a comment

Choose a reason for hiding this comment

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

Cool. Looks good :)

Copy link
Copy Markdown
Contributor

@hardys hardys left a comment

Choose a reason for hiding this comment

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

One small adjustment then I think we can merge, thanks!

Comment thread asciidoc/product/atip-features.adoc Outdated
Introduce a new section, following the existing PTP ones, about running
a Boundary Clock on an Intel Granite Rapids-D server. It takes into
account the specifics of these new designs, routing synchronization
signals internally across NAC, Timing Module and Add-In Cards.

In particular, this commit documents the steps required to have a working
static BC setup.

Reviewed-by: Ranjini M N <ranjini.n@suse.com>
Signed-off-by: Marco Chiappero <marco.chiappero@suse.com>
Copy link
Copy Markdown
Contributor

@hardys hardys left a comment

Choose a reason for hiding this comment

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

Thanks!

@mchiappero mchiappero merged commit 3397ac2 into suse-edge:main May 20, 2026
12 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.

3 participants