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

cr1000a BDFs #7

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

cr1000a BDFs #7

wants to merge 3 commits into from

Conversation

tsg2k2
Copy link

@tsg2k2 tsg2k2 commented Apr 22, 2023

No description provided.

Signed-off-by: tsg2k2 <[email protected]>
@Ansuel
Copy link
Member

Ansuel commented Apr 22, 2023

@tsg2k2 same rule of openwrt apply here...

check title for other commit
commit description is mandatory
Also Sob and author must have real name surname and real email.

Aside from these formal issue... Does the qcn bdf works with firmware 2.9 ? Please test it as currently we are using the kavlo one but from some analysis the one shipped with the board have increased power as they are not standard pcie card...

@tsg2k2
Copy link
Author

tsg2k2 commented Apr 22, 2023

Well, I think I have a catch 22 here: testing those bdfs without regdb info causes this qcn9074 to crash with 2.9 firmware. In order to test it further your instructions were to submit the bdfs to this repo for having regdb added. How could we solve it? Could I have those processed somehow without PR? Once I have the rest of device ready I'll create a formal commit.

@tsg2k2

This comment was marked as duplicate.

@Ansuel
Copy link
Member

Ansuel commented Apr 22, 2023

@tsg2k2 well i just checked and regdb is not the only problem... just as ax9000 this also require some ""love""

Will do some magic and pm you something to test. (tomorrow)

tsg2k2 added 2 commits April 24, 2023 10:54
Signed-off-by: tsg2k2 <[email protected]>
Signed-off-by: tsg2k2 <[email protected]>
@Ansuel
Copy link
Member

Ansuel commented May 7, 2023

@tsg2k2 can you test the first file you pushed here? should be the one closer

@Ansuel
Copy link
Member

Ansuel commented May 20, 2023

@tsg2k2 since this works, can you by chance rework the commit title and drop the last one?

@tsg2k2
Copy link
Author

tsg2k2 commented Nov 12, 2023

@tsg2k2 since this works, can you by chance rework the commit title and drop the last one?

This doesn't really work for 6GHz - see details here:
https://forum.openwrt.org/t/adding-support-for-verizon-cr1000a/134605/887

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