Skip to content
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

Wan is not working on Snapshot using Mi 4 A Gigabit #15273

Closed
1 task done
khanjui opened this issue Apr 26, 2024 · 8 comments
Closed
1 task done

Wan is not working on Snapshot using Mi 4 A Gigabit #15273

khanjui opened this issue Apr 26, 2024 · 8 comments
Labels
bug issue report with a confirmed bug invalid

Comments

@khanjui
Copy link

khanjui commented Apr 26, 2024

Describe the bug

It connects but does not receives any RX. Possibly related to latest MT7530 switch commits. Snapshot from a day ago is working fine. @dangowrt

OpenWrt version

...

OpenWrt release

SNAPSHOT

OpenWrt target/subtarget

ramips/mt7621

Device

Xiaomi Mi Router 4A Gigabit Edition

Image kind

Official downloaded image

Steps to reproduce

No response

Actual behaviour

No response

Expected behaviour

No response

Additional info

No response

Diffconfig

No response

Terms

  • I am reporting an issue for OpenWrt, not an unsupported fork.
@khanjui khanjui added bug issue report with a confirmed bug bug-report to-triage labels Apr 26, 2024
@khanjui khanjui changed the title Wan is not working on latest Snapshot on Mi 4 A Gigabit Wan is not working on Snapshot using Mi 4 A Gigabit Apr 26, 2024
@9uyone
Copy link

9uyone commented Apr 26, 2024

The same on Redmi AC2100. Tried yesterday

Copy link

Invalid Version reported. ...
Is this from a clean repository?

@liumingan
Copy link

The same on Phicomm K2P

@ivanich
Copy link

ivanich commented Apr 26, 2024

Happened the same, I suggest to stop accept patches from Daniel Golle, it's been 5-th time I got my device bricked after his commits.

namiltd referenced this issue Apr 26, 2024
The compatible string for the MediaTek MT7988 SoC ended up being
'mediatek,mt7988a' instead of 'mediatek,mt7988' in the now upstream
dtsi. Adapt the cpufreq driver so support for frequency scaling is
again usable.

Signed-off-by: Daniel Golle <daniel@makrotopia.org>
@namiltd
Copy link
Contributor

namiltd commented Apr 26, 2024

Are you able to compile it yourself and check exactly after which commit it stopped working?

@DragonBluep
Copy link
Contributor

Happened the same, I suggest to stop accept patches from Daniel Golle, it's been 5-th time I got my device bricked after his commits.

I have to say, this is inevitable in the process of software development. Always new features, new bugs and then new fixes. In the past few years, Daniel has contributed a lot of outstanding code to this project.

@namiltd
Copy link
Contributor

namiltd commented Apr 26, 2024

Same problem: #15279

@dangowrt
Copy link
Member

dangowrt commented Apr 26, 2024

Thank you for reporting this problem.

Probably the cause is one of the MT7530 related commits. While there are a few different version of the Mi 4 A (which are different in exactly the aspect of which port is used as WAN port and how), the Redmi AC2100 is a unique device and uses PHY-muxing of port 0 of the switch as WAN port via eth1. I'm trying to build for MT7621 myself now and test on a similar board (I don't have any board which uses PHY 0 as WAN, but I got some using PHY 4 and will try on those)

I'm currently working out which of the added pending patches is the culprit by bisecting the patches added by
commit d40691a generic: 6.1, 6.6: mt7530: import pending patches

Vladdrako pushed a commit to Vladdrako/openwrt that referenced this issue Apr 28, 2024
Importing pending patch "net: dsa: mt7530: move MT753X_MTRAP operations
for MT7530" broke WAN connectivity on most MT7621 which use PHY-muxing
to hook up either port 0 or port 4 to GMAC1.

Remove it for now until the author submits a fixed version.

Fixes: openwrt#15279
Fixes: openwrt#15273
Fixes: d40691a ("generic: 6.1, 6.6: mt7530: import pending patches")
Signed-off-by: Daniel Golle <daniel@makrotopia.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug issue report with a confirmed bug invalid
Projects
None yet
Development

No branches or pull requests

7 participants