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

ipq806x: Meraki MR52: swap lan leds #15410

Merged
merged 1 commit into from
May 7, 2024
Merged

Conversation

namiltd
Copy link
Contributor

@namiltd namiltd commented May 7, 2024

LAN Leds on Meraki MR52 are wrong and needs to be swapped to actually reflect real ports (lan1<->lan2).
Fixes: #15388

@namiltd namiltd marked this pull request as ready for review May 7, 2024 06:46
@github-actions github-actions bot added the target/ipq806x pull request/issue for ipq806x target label May 7, 2024
@castiel652
Copy link
Contributor

Should be ipq806x

@namiltd namiltd changed the title ipq8068: Meraki MR52: swap lan leds ipq806x: Meraki MR52: swap lan leds May 7, 2024
@Ansuel
Copy link
Member

Ansuel commented May 7, 2024

@namiltd

Can change and move the tag Fixes issue to Fixes and put it right up the SoB tag.

Also can you change the commit message to something more descriptive like "LAN Leds on Meraki MR52 are wrong and needs to be swapped to actually reflect real ports"

@namiltd namiltd force-pushed the patch-4 branch 3 times, most recently from 8d2885a to 41800e1 Compare May 7, 2024 11:18
Ansuel pushed a commit to namiltd/openwrt that referenced this pull request May 7, 2024
LAN Leds on Meraki MR52 are wrong and needs to be swapped to actually
reflect real ports (lan1<->lan2).

Fixes: openwrt#15388
Signed-off-by: Mieczyslaw Nalewaj <namiltd@yahoo.com>
Link: openwrt#15410
Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
Ansuel pushed a commit to namiltd/openwrt that referenced this pull request May 7, 2024
LAN Leds on Meraki MR52 are wrong and needs to be swapped to actually
reflect real ports (lan1<->lan2).

Fixes: openwrt#15388
Signed-off-by: Mieczyslaw Nalewaj <namiltd@yahoo.com>
Link: openwrt#15410
[ wrap commit to 80 columns and improve commit title ]
Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
Ansuel pushed a commit to namiltd/openwrt that referenced this pull request May 7, 2024
LAN Leds on Meraki MR52 are wrong and needs to be swapped to actually
reflect real ports (lan1<->lan2).

Fixes: openwrt#15388
Signed-off-by: Mieczyslaw Nalewaj <namiltd@yahoo.com>
Link: openwrt#15410
[ wrap commit to 80 columns and improve commit title ]
Link: openwrt#15410
Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
LAN Leds on Meraki MR52 are wrong and needs to be swapped to actually
reflect real ports (lan1<->lan2).

Fixes: openwrt#15388
Signed-off-by: Mieczyslaw Nalewaj <namiltd@yahoo.com>
Link: openwrt#15410
[ wrap commit to 80 columns and improve commit title ]
Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
@openwrt-bot openwrt-bot merged commit bd68521 into openwrt:main May 7, 2024
3 checks passed
@Ansuel
Copy link
Member

Ansuel commented May 7, 2024

@namiltd guess this should be also backported?

@namiltd namiltd deleted the patch-4 branch May 7, 2024 11:38
openwrt-bot pushed a commit that referenced this pull request May 7, 2024
LAN Leds on Meraki MR52 are wrong and needs to be swapped to actually
reflect real ports (lan1<->lan2).

Fixes: #15388
Signed-off-by: Mieczyslaw Nalewaj <namiltd@yahoo.com>
Link: #15410
[ wrap commit to 80 columns and improve commit title ]
Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
(cherry picked from commit bd68521)
@namiltd
Copy link
Contributor Author

namiltd commented May 7, 2024

@Ansuel Yes, the fix was tested on 23.05.3.

Vladdrako pushed a commit to Vladdrako/openwrt that referenced this pull request May 18, 2024
LAN Leds on Meraki MR52 are wrong and needs to be swapped to actually
reflect real ports (lan1<->lan2).

Fixes: openwrt#15388
Signed-off-by: Mieczyslaw Nalewaj <namiltd@yahoo.com>
Link: openwrt#15410
[ wrap commit to 80 columns and improve commit title ]
Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
target/ipq806x pull request/issue for ipq806x target
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Incorrect mapping of port link LED's in MR52
4 participants