Skip to content

Enhance PTP capabilities in l2discovery#25

Merged
jzding merged 2 commits into
redhat-cne:mainfrom
edcdavid:fix-wpc1
May 8, 2026
Merged

Enhance PTP capabilities in l2discovery#25
jzding merged 2 commits into
redhat-cne:mainfrom
edcdavid:fix-wpc1

Conversation

@edcdavid
Copy link
Copy Markdown

@edcdavid edcdavid commented May 7, 2026

  • Added support for retrieving PTP pins and GNSS device information in the getIfs function.
  • Updated PTPCaps struct to include PhcIndex, HasPtpPins, and GnssDevice fields.
  • Improved IsWpcNic and IsWPCNicWrapper functions to check for PTP pins and valid PHC index.

These changes enhance the PTP capabilities detection and provide more detailed interface information.

Assisted-By: Cursor

edcdavid added 2 commits May 7, 2026 14:47
- Added support for retrieving PTP pins and GNSS device information in the getIfs function.
- Updated PTPCaps struct to include PhcIndex, HasPtpPins, and GnssDevice fields.
- Improved IsWpcNic and IsWPCNicWrapper functions to check for PTP pins and valid PHC index.

These changes enhance the PTP capabilities detection and provide more detailed interface information.
- Reordered installation commands in Dockerfile for clarity.
- Enhanced PTP capabilities detection in l2discovery by adjusting the getIfs, hasPtpPins, and getGnssDevice functions to utilize host paths.
- Updated IsWpcNic and IsWPCNicWrapper functions to consider both PTP pins and GNSS devices for WPC NIC identification.

These changes improve the robustness of PTP capabilities detection and streamline the Dockerfile setup.
Copy link
Copy Markdown
Member

@jzding jzding left a comment

Choose a reason for hiding this comment

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

/lgtm

@jzding jzding merged commit 556ef5d into redhat-cne:main May 8, 2026
4 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.

2 participants