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

u-boot LSDK-20.12: Missing patch for DPL load location #60

Open
Thermi opened this issue Mar 4, 2021 · 5 comments
Open

u-boot LSDK-20.12: Missing patch for DPL load location #60

Thermi opened this issue Mar 4, 2021 · 5 comments

Comments

@Thermi
Copy link

Thermi commented Mar 4, 2021

We got an issue with loading DPL files in LSDK 20.12 in uboot. We suspect it's due to the location the DPL file gets loaded into.
In LSDK 20.04 with the patches, that is different from LSDK 20.12 with patches.
The specific patch that changes that is 0007-lx2160a-load-dpl-into-0x80001000-instead-of-0x80d000.patch.

We haven't tested yet if that helps the issue, but we're pretty certain it's that.

@rabeeh
Copy link
Member

rabeeh commented Mar 15, 2021

I just re-tested from a clean build; DPL is loaded correctly.
Which SERDES configuration are you trying to use?

@Thermi
Copy link
Author

Thermi commented Mar 15, 2021

We're trying to use SERDES config 4-5-2.

@rabeeh
Copy link
Member

rabeeh commented Mar 15, 2021

Did you write your own DPL / DPC?
Using SD1=4 as SGMII should function without setting PLLS <-- PLLF
The only issue we previously had is setting more than 6 DPNI in runtime due to some resource allocation issues.

If you were able to fix that resource issue please let me know how you have done that.

@Thermi
Copy link
Author

Thermi commented Mar 15, 2021

No, we used the ones provided.

@Thermi
Copy link
Author

Thermi commented Jun 25, 2021

We have working DPC and DPL files for this use case. The problem was solved by dividing up 96 of the 8 prio channels to make enough channels with 2 prios. It's the same fix as described in a commit by Ionut Carlig titled ""Changes for Serdes1 protocol 8 and 13".

The DPNIs are ordered in reverse because at least with our kernel, DPNIs with a higher ID are assigned interface names first.

dpl and dpc 4-5-2.zip

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

No branches or pull requests

2 participants