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

kboot: adjust max-link-speed for 10GE Mac Mini #125

Closed
wants to merge 1 commit into from

Conversation

jannau
Copy link
Contributor

@jannau jannau commented Nov 20, 2021

Modifies just "max-link-speed" although the template ADT for J274 has
different values for "pci-max-payload-size" and "pci-aspm-default".
The "lan-10gb" sub node has an additional property
"function-pcie_port_control".

Signed-off-by: Janne Grunau j@jannau.net

@jannau
Copy link
Contributor Author

jannau commented Nov 20, 2021

A simpler alternative to this m1n1 change would be to increase "max-link-speed" unconditionally in the dts. The gigabit ethernet Mac Mini model works with that change.

Copy link
Member

@marcan marcan left a comment

Choose a reason for hiding this comment

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

I'm confused as to how this was tested, since the Linux aPCIE driver currently ignores max-link-speed entirely as far as I can tell. :)

I think that if the 1GE version doesn't have issues down-negotiating from Gen4 (with my patch in that actually makes Gen>1 work), then for now it's probably easier to just let it try up to that unconditionally, so we wouldn't need this code. However, we should certainly keep this branch around in case it becomes more useful in the future :)

src/kboot.c Outdated Show resolved Hide resolved
src/kboot.c Outdated Show resolved Hide resolved
src/kboot.c Show resolved Hide resolved
@jannau
Copy link
Contributor Author

jannau commented Nov 22, 2021

I tested only the device tree rewriting by using a dtb with max-link-speed != 1 which m1n1 corrected to 1 from the 1GE ADT. I assumed that the link speed training was implemented since the PCIe ports already used different values.

Modifies just "max-link-speed" although the template ADT for J274 has
different values for "pci-max-payload-size" and "pci-aspm-default".
The "lan-10gb" sub node has an additional property
"function-pcie_port_control".

Signed-off-by: Janne Grunau <j@jannau.net>
@jannau
Copy link
Contributor Author

jannau commented Dec 7, 2021

Closing since it is unnecessary since m1n1 sets the link speed based on the ADT since abee2ec

@jannau jannau closed this Dec 7, 2021
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.

None yet

2 participants