Skip to content

docs(linux): Update linuxptp clone URL and branch for HSR/PRP PTP#735

Open
danish-ti wants to merge 1 commit into
TexasInstruments:masterfrom
danish-ti:prp-ptp
Open

docs(linux): Update linuxptp clone URL and branch for HSR/PRP PTP#735
danish-ti wants to merge 1 commit into
TexasInstruments:masterfrom
danish-ti:prp-ptp

Conversation

@danish-ti

Copy link
Copy Markdown
Contributor

Update the linuxptp repository URL from the bigeasy fork to the mda fork, and update the branch from hsr_prp_v2_plus to hsr_prp_ptp. Also remove the now-unnecessary config file patching steps.

Update the linuxptp repository URL from the bigeasy fork to the mda fork,
and update the branch from hsr_prp_v2_plus to hsr_prp_ptp. Also remove
the now-unnecessary config file patching steps.

Signed-off-by: MD Danish Anwar <danishanwar@ti.com>

The `linuxptp-hsr <https://git.kernel.org/pub/scm/linux/kernel/git/bigeasy/linuxptp-hsr.git>`_
implementation on the ``hsr_prp_v2_plus`` branch supports PTP (Precision Time Protocol)
The `linuxptp-hsr-prp <https://git.kernel.org/pub/scm/linux/kernel/git/mda/linuxptp-hsr-prp.git>`_

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@danish-ti, if we have to switch anyway, shouldn't we have a TI Git Repository rather than an individual fork? We have the following repository for TI's fork of linuxptp and I think you can contribute to it if the bigeasy fork is not updated:
https://git.ti.com/cgit/processor-sdk/linuxptp/

You could additionally state that we are switching to a fork of linuxptp that shall be maintained by TI for PTP support with HSR and PRP. I don't think an individual fork of an upstream repository for vendor supported features hosted on git.kernel.org is a good approach. Please check if you could switch to the TI fork instead.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The current documentation of HSR/PRP PTP is outdated as bigeasy is no longer maintained. I am changing it to my fork so that the customers can still use this feature. The TI fork is not well maintained, the last commit I see is from 5 years ago. I agree with you that instead of any custom fork, we should host this on TI fork. The longterm solution should be to switch to TI fork. And I will look into it.
For now I think we should update the documentation so that customers don't use the outdated fork. I can plan integrating this feature in the TI fork for later.

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.

7 participants