-
-
Notifications
You must be signed in to change notification settings - Fork 10.8k
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
lantiq: Update TP-Link TD-W89x0 flash layout #9775
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me.
Please fix the commit message.
|
Hello @hauke ,
|
It does not detect my wifi devce any more:
|
I think |
22fffab
to
7fce8df
Compare
Thanks for testing.
I checked this file and while I do think switching to nvmem is better, but I see not all of the targets can switch to nvmem because they manipulate the firmware in different ways (reverse, patch, external scripts). I just fixed the offset on the script, wifi should work now. |
Just to be clear (because I think my last comment wasn't that clear): my thought was to only switch the two TP-Link boards (which are managed in one .dtsi anyways) to nvmem, not all boards. |
Okay, I can switch to nvmem for TP-Link TD-W8970 and TP-Link TD-W8980. |
target/linux/lantiq/xrx200/base-files/etc/hotplug.d/firmware/12-ath9k-eeprom
Outdated
Show resolved
Hide resolved
Looks good to me, so:
|
The wifi driver is still not able to find the eeprom:
|
Hello, |
Force pushed to squash the commits. |
@hauke can we please merge this before 24.10 release? Currently 24.10-rc2 with default package configuration is too big for this device by 56089 bytes, with this change it barely fits. I know I can always compile from the sources, but given this is one of the rare dsl devices, I would love to have official builds for everyone to use. edit: I just saw #17303 I'm moving the changes to the |
3dc8157
to
3d52754
Compare
@@ -57,7 +57,7 @@ lantiq_setup_macs() | |||
;; | |||
tplink,tdw8970|\ | |||
tplink,tdw8980) | |||
wan_mac=$(macaddr_add "$(mtd_get_mac_binary boardconfig 0xf100)" 1) | |||
wan_mac=$(macaddr_add "$(mtd_get_mac_binary boardconfig 0x100)" 1) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This can be handled in dts, no?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh right. This is for an interface not specified in dts. nvm.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah, it's for the dsl interface.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking at this further, there’s the vdsl node. If the driver uses of_get_mac_address, it can be converted.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't know. I checked the other lantiq based devices, but I couldn't find any device that sets the mac address of the dsl port through dts.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i checked the driver. Not happening.
Does sysupgrde work without problems from an older version to this version with changed partition layout? |
What was inside this partition you are reusing? Does this make going back to the vendor firmware more complicated than before? |
The old Modifying the config is explained in the wiki: https://openwrt.org/toh/tp-link/td-w8970_v1#installation_without_opening_the_case_web_interface_hack If the user doesn't have a config file, holding down the reset button while the device is booting should reset this the config to the factory defaults if it's not already reset because of the incorrect checksum. In case it doesn't I put my config file which others can use: TD-W8970_config_partition.zip The space between
Going from an older version to this version works, even with the config. But, going from this version to another version does not. I tested this today with my only vdsl router and found out the hard way. While the flash
The device fully works in this state except you can't sysupgrade, since the firmware partition is now locked. I had to open the device and recover from serial console with tftpboot. To fix this either |
3d52754
to
d05bd44
Compare
Added |
Flash space is scarce on 8MB devices, this commit adds 124KB of usable space by reusing OEM user configuration and unused space. This commit also switches from userspace script to using nvmem for ath9k calibration data. Signed-off-by: Mustafa Can Elmacı <mustafacan@elmaci.net>
d05bd44
to
a726f69
Compare
#16555 got merged, so this is only the flash layout changes now. |
Flash space is scarce on 8MB devices, this commit adds 124KB of usable space by reusing OEM user configuration and unused space. I also split the partitions to seperate MAC Address & WPS key, Memory configuration, and Wi-Fi calibration data.