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

phy_tesla.c: Fix compilation errors for TESLA PHYs #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

MikeLooijmans
Copy link

Fix missing headers and methods that aren't critical. Undefined symbols, doubly defined methods, defined but unused symbols.

Fix missing headers and methods that aren't critical.
Undefined symbols, doubly defined methods, defined but unused symbols.
@jackyh2020
Copy link
Collaborator

#define VTSS_TESLA 1 /* Enables support of VSC8504, VSC8552 */
Seeing compile warning message with above define that is required for TELSA chip.

VTSS_TESLA_A and VTSS_TESLA_B only required for older chip revisions.
VTSS_TESLA_D is to be defined for current production revision.

@topic-buildserver
Copy link

I have no idea what you mean by that comment. To activate the TESLA PHY support you'd need to do that, yes, but without merging this PR that results in a compilation failure. This PR fixes the failures and makes the TESLA PHY work (if you #define it).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants