[OpenWrt-Devel] [PATCH] ath79: Add LED migration for several Archer Cxx devices

[Adrian Schmutzler]

Hi,

concerning LED migration:

I have _not_ run-tested this patch, as it's closely similar to what has been done for c7-v5, which has been tested.

I own C25 and both C60, though, so if you insist I can test (but not so soon).

Concerning eth0/eth1 swap:

After https://github.com/openwrt/openwrt/pull/2066 , several devices were aligned to the ar71xx order, but also several that were "correct" before are wrong now.
This affects the TP-Link C58, C59, C60, Glint AR150, not sure about C25, and most probably several others I have not encountered.

There was a brief discussion about migration somewhere, but I do not recall how or whether it was resolved. I had a quick look into the code, and didn't find anything for eth migration (but might have overlooked it).

Technically, a migration script only would have to exchange eth0 and eth1 (with an intermediate step), so it shouldn't be too complicated, unless I overlooked something. One could do the same for board.json, although solving it like this leaves a little of a sour taste.

Maybe one should restart this discussion on the list ...

However, please note that I'm only dealing with this topic by accident, as I've found out about the missing LED migration (and with ramips LEDs, as I've destroyed them in the first place ;-) ). I'm mostly using OpenWrt to develop Freifunk firmware based on it, where we completely circumvent OpenWrt migration mechanism. So I'm definitely not an expert with this.

Best

Adrian



> -----Original Message-----
> From: openwrt-devel [mailto:openwrt-devel-bounces at lists.openwrt.org] On Behalf Of David Bauer
> Sent: Dienstag, 3. September 2019 01:09
> To: Adrian Schmutzler <freifunk at adrianschmutzler.de>
> Cc: openwrt-devel at lists.openwrt.org
> Subject: Re: [OpenWrt-Devel] [PATCH] ath79: Add LED migration for several Archer Cxx devices
> 
> Hello Adrian
> 
> On 9/2/19 11:10 AM, Adrian Schmutzler wrote:
> > Several Archer Cxx devices were using board-specific LED names in
> > ar71xx, which were changed to "tp-link:*" in ath79.
> >
> > This patch adds migration for them.
> 
> Have you run-tested this patch? For me i ran into the issue of the device
> not being reachable via Ethernet, as the Ethernet MACs were swapped.
> 
> Has anyone yet tried to find an migration approach for this? I assume this
> could be tricky, as we might need to update the board.json also in this run.
> 
> Regarding the actual patch: I assume the LEDs were migrated (as the WAN Ethernet
> got illuminated), but i want to hear your opinion on how we should proceed with
> the Ethernet migration. :)
> 
> Best wishes
> David
> 
> _______________________________________________
> openwrt-devel mailing list
> openwrt-devel at lists.openwrt.org
> https://lists.openwrt.org/mailman/listinfo/openwrt-devel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: openpgp-digital-signature.asc
Type: application/pgp-signature
Size: 834 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/openwrt-devel/attachments/20190903/862a49ad/attachment.sig>
-------------- next part --------------
_______________________________________________
openwrt-devel mailing list
openwrt-devel at lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel