Skip to content

tplink-safeloader: add support for TPLink Archer A7 v5 (RU) #3683

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

Closed

Conversation

alxkv
Copy link
Contributor

@alxkv alxkv commented Dec 13, 2020

Signed-off-by: Alexey Kunitskiy alexey.kv@gmail.com

I have Archer A7 v5 hardware that I can't flash using official openwrt build. I've noticed this guide https://forum.openwrt.org/t/support-for-tp-link-archer-c7-v5-ru/28402 and that worked for me - I successfully flashed my variant (RU) of router. So I contributing my fix. It would be great to have it soon in stable revisions of latest 19.07*

@adschm
Copy link
Member

adschm commented Dec 13, 2020

This has formal issues. Please add prefix, shorten commit title and add commit message.

@adschm adschm added needs changes target/ath79 pull request/issue for ath79 target labels Dec 13, 2020
@alxkv alxkv force-pushed the tplink-archer-a7-ru-safeloader-fix branch from e6e3665 to f10c2ab Compare December 16, 2020 15:41
@alxkv
Copy link
Contributor Author

alxkv commented Dec 16, 2020

This has formal issues. Please add prefix, shorten commit title and add commit message.

Updated commit. Is it okay now?

@adschm
Copy link
Member

adschm commented Dec 16, 2020

Formatting seems fine. Now we just have to get rid of the "should" in "should be the same" somehow ...

@@ -726,6 +726,7 @@ static struct device_info boards[] = {
"{product_name:Archer A7,product_ver:5.0.0,special_id:43410000}\n"
"{product_name:Archer A7,product_ver:5.0.0,special_id:4A500000}\n"
"{product_name:Archer A7,product_ver:5.0.0,special_id:54570000}\n",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That comma needs to be removed.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Omg, thanks

This adds support for another variant of TPLink Archer A7 v5 (RU) which
should be the same as Archer C7 v5. Other special_ids from C7 may be considered
but I have only this one tested on my hardware

Signed-off-by: Alexey Kunitskiy <alexey.kv@gmail.com>
@alxkv alxkv force-pushed the tplink-archer-a7-ru-safeloader-fix branch from f10c2ab to 2d5b574 Compare December 16, 2020 18:47
@alxkv
Copy link
Contributor Author

alxkv commented Dec 16, 2020

Now we just have to get rid of the "should" in "should be the same" somehow ...

According to the wiki page https://openwrt.org/toh/tp-link/archer_a7_v5 :
"The Archer A7 is believed to be identical to the Archer C7, but it has its own set of firmware. "
So my decision was made based on this statement

@alxkv alxkv changed the title Extend TPLink Archer A7 support by adding special_id for RU variant tplink-safeloader: add support for TPLink Archer A7 v5 (RU) Dec 21, 2020
@adschm
Copy link
Member

adschm commented Feb 5, 2021

Thanks, will be merged later today.
I checked the firmware files and they are actually identical for EU and RU ...

@adschm adschm closed this Feb 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
target/ath79 pull request/issue for ath79 target
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants