-
Notifications
You must be signed in to change notification settings - Fork 322
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
ramips-mt7621: add dlink-dap-1620-b1 #3124
base: master
Are you sure you want to change the base?
Conversation
Hello, I would be very grateful for any tips on how to approach this problem. 🚀
|
Hello GoliathLabs, in principal this platform should be supported. What does |
I've flashed OpenWrt 23.05.2 to test it out. Interestingly, it only shows one interface which is the 5 GHz one (association with the SSID works as expected). Logread of OpenWrt down below. EDIT to my last comment. To associate with the wifi on gluon, I had to issue the following command prior to
logread_gluon.txt |
Maybe we should try to figure out whether this broke due to a commit since openwrt/openwrt@e4c7703 but maybe it's wise to test OpenWrt master first before we start bisecting :) |
Do you have a chance to test the TP-Link RE650 v1 again to see whether it's just the DAP-1620-B1 or a general OpenWrt thing? |
What's the status here? Has the open issue been adressed? |
Issue persists to this day, even with newer builds |
Please check the EEPROM for it's validity, the information if said chip has a DBDC frontend is encoded in there. Add the dump of the EEPROM partition to this ticket, so we can have a look at it. |
Hi,
|
Thanks. Indeed This has to be worked around in the mt76 driver by overriding this field or fixed in another way. Initially i suspected the caldata offset being wrong but it is correctly defined. |
Thank you for looking into this! |
QUestion would be how to proceed with this then. Do you plan on providing upstream patches to work around the invalid EEPROM? If not, I'd close this PR, as fixing this upstream would be a hard dependency. Otherwise we could go down the route and mark the device as broken. |
I don't think I have the expertise to provide upstream patches. We could merge this PR as broken then. If someone wants to pick this up, I'd be happy to help as much as I can. |
Fine with me, please rebase this and add broken then. |
7fa1c09
to
38f9d01
Compare
Done. |
sysupgrade [-n]
,firstboot
)(
lua -e 'print(require("platform_info").get_image_name())'
)(https://gluon.readthedocs.io/en/latest/dev/hardware.html#hardware-support-in-packages)
factory reset must be performed before checking the primary MAC address, as
the setting from the old version is not reset otherwise.
--> marked as broken
(https://gluon.readthedocs.io/en/latest/features/configmode.html)
is_outdoor_device
function inpackage/gluon-core/luasrc/usr/lib/lua/gluon/platform.lua
is_cellular_device
function inpackage/gluon-core/luasrc/usr/lib/lua/gluon/platform.lua
setup_ncm_qmi
topackage/gluon-core/luasrc/lib/gluon/upgrade/250-cellular
docs/user/supported_devices.rst