Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(PRU-ICSS): Add documentation for PTP in switch mode #49

Merged
merged 1 commit into from
Nov 28, 2024

Conversation

danish-ti
Copy link
Contributor

Add documentation for running PTP to synchronise time between master and slave DUTs in switch mode.

Copy link

New warnings found with rstcheck:

source/linux/Foundational_Components/PRU-ICSS/Linux_Drivers/PRU_ICSSG_Ethernet_Switch.rst:3: (INFO/1) Enumerated list start value not ordinal-1: "2" (ordinal 2)

Copy link

New warnings found with rstcheck:

source/linux/Foundational_Components/PRU-ICSS/Linux_Drivers/PRU_ICSSG_Ethernet_Switch.rst:3: (INFO/1) Enumerated list start value not ordinal-1: "2" (ordinal 2)

Copy link
Member

@StaticRocket StaticRocket left a comment

Choose a reason for hiding this comment

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

Over 50% of this file was using the proper 3 space indentation prior to this PR. Please standardize.

In addition, the rstcheck warning is valid. Lists should be indented to more appropriately indicate where they begin an end.

@danish-ti
Copy link
Contributor Author

@StaticRocket I have standardize the indentation for the patch and fixed the rstcheck warning.

@cshilwant cshilwant dismissed StaticRocket’s stale review November 26, 2024 09:46

Author has updated the PR

Add documentation for running PTP to synchronise time between master and
slave DUTs in switch mode.

Signed-off-by: MD Danish Anwar <[email protected]>
@cshilwant cshilwant merged commit 77d91b5 into TexasInstruments:master Nov 28, 2024
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.

4 participants