-
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
Fix readonly ubiquiti airmax #3141
base: master
Are you sure you want to change the base?
Fix readonly ubiquiti airmax #3141
Conversation
At the OpenWrt developer meetup, we've talked about this patch and while it doesn't matter for this backport, the patch should be replaced by a more selective patch based on DT-properties, as this approach might break future devices if upstream deprecates the unlock-all Kconfig symbol. |
A quick test seems promising (see updated initial post). I'm not too keen to brick (and attempt to restore the device). What tests do we want to do before readding them? I'll take the device home and can to those then. |
ebea824
to
f18f517
Compare
@herbetom Please try with this patch and add the property to the node of the nor flash:
|
f18f517
to
cb6a2ee
Compare
Okay, My patch was not the best. Please apply the patches from my staging tree and see if this fixes the WP. Also remember to revert the kernel configuration update which enabled this behavior target-wide. |
12afa0d
to
2dbb134
Compare
I've tried it and the device didn't came back up. Don't know what's gone wrong. A Power Cycle didn't change anything. |
Tried on a TL-WR842N v3 with and without the DT property. So the patch itself seems fine. |
08717ce
to
c39b4ca
Compare
The current state seems to work regarding resolvig the read only problem. Hoewer, in my tests with an Ubiquiti NanoBeam AC Gen1 (XC) i've had the problem that the device became unresponsive after some upgrades (Ethernet up, but no response). The "solution" was to power it down for an extended period of time. (A few days, haven't attempted to narrow it down. Just unpluging it for a few seconds didn't seem to help.) I'm not eaxtly sure what triggers it, but larger upgrades (upgrading from OpenWrt v2023.1.x) seems like something that should be tested a bit more. |
9e41117953 ipq40xx: fix PHY subsystem compilation (phy_interface_num_ports()) c51d49ba39 wireless-regdb: update to 2024.01.23 aa762ada80 openssl: update to 3.0.13 9ee626f945 ucode: add libjson-c/host dependency 3a23eed4dc realtek: fix zyxel-vers usage for XGS1250-12 be7d9da7be bcm27xx: base-files: fix platform_copy_config 9a869950c1 bcm27xx: improve image generation script c82ca6dc74 bcm27xx: base-files: properly detect boot partition 8a7f667fb5 kernel: 5.15: backport v6.1 PHY changes required for Aquantia 6d962cabba kernel: backport ethtool_puts 5eb578a4fb firmware-utils: bump to latest openwrt-23.05 e302172258 ath79: add Ubiquiti Rocket M XW as alternate name to Bullet M XW e5af19e955 ath79: ubnt-bullet-m-xw: fix Ethernet PHY traffic 58c9308400 ath79: ubnt,bullet-m-xw: set PHY max-speed to 100Mbps b5c728948c mbedtls: security bump to version 2.28.7 b0f3fd550b ipq807x: prpl-haze: fix sysupgrade flashing from bootloader 1ca61b7b37 uhttpd: handle reload after uhttpd-mod-ubus installation using postinst 6b7c4739c4 base-files: execute package's "postinst" after executing uci-defaults 1da896f706 bcm27xx-gpu-fw: update to latest version e05659ef59 bcm27xx-userland: update to latest version 5a4389f6ab bcm27xx: 5.15: turn on cpu erratum for A72 and disable A53 ac97ea1079 bcm27xx: config: update documentation links c55aaa7c9a ath79: generic: disable SPI-NOR write protect unconditionally 72421d973e realtek: d-link dgs-1210-10p improve sfp support 78beef6aee jsonfilter: update to Git HEAD (2024-01-23) c1615f3148 kernel: bump 5.15 to 5.15.147 7338733dc9 mediatek: filogic: add support ASUS RT-AX59U
26c24e579 mosquitto: reenable options wrongly turned off db28d99be python-ble2mqtt: update to 0.2.2 c2ae7b95d python-dbus-fast: upgrade to 2.21.1 63f6af4bc nextdns: Update to version 1.42.0 6d6e398c7 Openvpn: add missing script-security bce14a016 cloudflared: refine config.yml 892ce0645 dnsproxy: Update to 0.64.1 3ffac159c inih: Update to r58 a3c2cd8fb rclone: Update to 1.65.2 8c8ec8106 cloudflared: Update to 2024.1.5 6cb714722 dos2unix: Update to 7.5.2 8d02881d4 curl: update to 8.6.0 dcff607ba nebula: update to 1.8.2-2 905afb0ed xz: Update to 5.4.6 92a07db7d php8: update to 8.2.15 1167ef72e python-dbus-fast: update version 2.21.0 e6d3a7892 python3-bleak: add missing python3-typing-extensions dependency 0da0a501e natmap: update to 20240126 67dcaa5d7 sing-box: update to 1.8.4 f87af4985 banip: update 0.9.3-5 b61930ad7 v2ray-geodata: Update to latest version 9a26ff67e v2raya: do not allow changing config/log directory 4267c8776 dnsproxy: Update to 0.63.1 63a022a87 rclone: Update to 1.65.1 d07aa2f0f xray-core: Update to 1.8.7 d9d4acfd9 cloudflared: Update to 2024.1.4 ebddc07c1 cloudflared: Update to 2024.1.2 4da835d3e gnutls: updated to 3.8.3 5aabddf75 ocserv: updated to 1.2.4 564e6e08d Openvpn: add missing script event options 0707d98c1 htop: update to 3.3.0 ac2736542 adblock-fast: update to 1.1.1-3 eb7db48b6 yggdrasil: bump to 0.5.4 315f5c41b yggdrasil: bump to 0.5.3 be2b05436 yggdrasil: support public key in config generation 4b5d833e6 yggdrasil: overhaul package with netifd support - package is bumped to 0.5.2 - new protocol changes prevent peering with 0.4.x peers - @turretkeeper revamps package with netifd support - do not use with luci-app-yggdrasil please install luci-proto-yggdrasil
5a7cac9 batman-adv: Provide teardown hook for batadv_vlan proto 75e1614 batctl: Merge bugfixes from 2024.0
3d08b0f wgpeerselector: fix undefined variable peer
c39b4ca
to
1d81c97
Compare
I've rebased this onto #3184 |
@herbetom needs an update or rebase, doesn't it? |
closes #2939
I've installed a firmware build from the branch on a
Ubiquiti NanoBeam AC Gen1 (XC)
.I'm able to store the
/etc/dropbear/authorized_keys
file accross reboots.I haven't validated that the device was actually affected and haven't checked anything else.
dmesg