[LEDE-DEV] [PATCH] ar71xx: Add TP-LINK TL-WR841N v12 support.

[Vittorio G (VittGam)]

Hello Piotr,

On 25/03/2017 23:12:58 CET, Piotr Dymacz wrote:
> Hello Vittorio,
> 
> On 25.03.2017 18:08, Vittorio Gambaletta (VittGam) wrote:
>> This router has the same hardware of TP-LINK TL-WR841N v11 (same
>> FCC ID, same TFTP image name...).
>>
>> The stock firmware web interface does not seem to accept the LEDE
>> factory image, but it can be flashed via the u-boot TFTP recovery
>> by long-pressing the reset button after power on.
> 
> Did you try factory image but with changed filename (sometimes TP-Link 
> web gui doesn't accept files with long name)?

Yes, I remembered about this problem with late versions of the 703n,
so I tried both short name (x.bin) and same name of factory image.
But it didn't work in either case.

Fortunately the TFTP client integrated in the factory u-boot accepts
the image, so opening the router to access the serial is not required.

> What kind of error are you getting with LEDE factory image?

If I remember well it was 18005, wrong filename...

> BTW. It seems that TP-Link already released v13 version of this device 
> in Europe but this time it's (surprise!) MediaTek (MT7628?) based.

Yes. But here in Italy, a big online retailer just started selling v12
(with new TP-LINK logo) instead of v11, so who knows when v13 will
arrive...

(I hope v13 will not have problems, since the 841 v9/v11/v12 today is a
very stable and reliable (and really cheap...) platform.)

>> The TFTP image name is wr841nv11_tp_recovery.bin (yes, v11, not v12).
>>
>> Signed-off-by: Vittorio Gambaletta <openwrt at vittgam.net>
>> ---
>>  target/linux/ar71xx/image/tp-link.mk |   13 ++++++++++++-
>>  1 file changed, 12 insertions(+), 1 deletion(-)
>>
>> diff --git a/target/linux/ar71xx/image/tp-link.mk b/target/linux/ar71xx/image/tp-link.mk
>> index cf2e5e7..705e400 100644
>> --- a/target/linux/ar71xx/image/tp-link.mk
>> +++ b/target/linux/ar71xx/image/tp-link.mk
>> @@ -736,6 +736,17 @@ define Device/tl-wr841-v11
>>    IMAGE/factory-eu.bin := append-rootfs | mktplinkfw factory -C EU
>>  endef
>>
>> +define Device/tl-wr841-v12
>> +  $(Device/tplink-4mlzma)
> 
> Can you use here $(Device/tl-wr841-v11) instead and then overwrite only 
> two variables which are different: DEVICE_TITLE and TPLINK_HWID?

Ok, I will send a PATCH v2 with this change.

Cheers,
Vittorio

> --
> Best regards,
> Piotr Dymacz
> 
>> +  DEVICE_TITLE := TP-LINK TL-WR841N/ND v12
>> +  BOARDNAME := TL-WR841N-v11
>> +  DEVICE_PROFILE := TLWR841
>> +  TPLINK_HWID := 0x08410012
>> +  IMAGES += factory-us.bin factory-eu.bin
>> +  IMAGE/factory-us.bin := append-rootfs | mktplinkfw factory -C US
>> +  IMAGE/factory-eu.bin := append-rootfs | mktplinkfw factory -C EU
>> +endef
>> +
>>  define Device/tl-wr842n-v1
>>    $(Device/tplink-8m)
>>    DEVICE_TITLE := TP-LINK TL-WR842N/ND v1
>> @@ -778,7 +789,7 @@ define Device/tl-wr847n-v8
>>    DEVICE_PROFILE := TLWR841
>>    TPLINK_HWID := 0x08470008
>>  endef
>> -TARGET_DEVICES += tl-wr840n-v2 tl-wr840n-v3 tl-wr841-v1.5 tl-wr841-v3 tl-wr841-v5 tl-wr841-v7 tl-wr841-v8 tl-wr841-v9 tl-wr841-v10 tl-wr841-v11 tl-wr842n-v1 tl-wr842n-v2 tl-wr842n-v3 tl-wr843nd-v1 tl-wr847n-v8
>> +TARGET_DEVICES += tl-wr840n-v2 tl-wr840n-v3 tl-wr841-v1.5 tl-wr841-v3 tl-wr841-v5 tl-wr841-v7 tl-wr841-v8 tl-wr841-v9 tl-wr841-v10 tl-wr841-v11 tl-wr841-v12 tl-wr842n-v1 tl-wr842n-v2 tl-wr842n-v3 tl-wr843nd-v1 tl-wr847n-v8
>>
>>  define Device/tl-wr941nd-v2
>>    $(Device/tplink-4m)
>>
>> _______________________________________________
>> Lede-dev mailing list
>> Lede-dev at lists.infradead.org
>> http://lists.infradead.org/mailman/listinfo/lede-dev
>>
> 
> 
> _______________________________________________
> Lede-dev mailing list
> Lede-dev at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/lede-dev