Hi Mathias, Sorry about the malformed patch. You’re right about the RouterStation Pro. I was wrongly changing it too. It has a different switch config from RB450G. I will split them and leave the RouterStation Pro unchanged. Please let me test it first before resubmit the patch. > On 17 Nov 2016, at 21:13, Mathias Kresin <d...@kresin.me> wrote: > > Hey João, > > both patches you have send are whitespace mangled by your mail program and do > not apply: > > $ git am LEDE-DEV-ar71xx-enable-serial-console-on-mikrotik-devices.patch > > Applying: ar71xx: enable serial console on mikrotik devices > error: patch failed: > target/linux/ar71xx/patches-4.4/701-MIPS-ath79-add-routerboard-detection.patch:1 > error: > target/linux/ar71xx/patches-4.4/701-MIPS-ath79-add-routerboard-detection.patch: > patch does not apply > Patch failed at 0001 ar71xx: enable serial console on mikrotik devices > > Please consider using 'git send-email' to prevent such issues. You can find > your broken patches in out patchwork at > https://patchwork.ozlabs.org/project/lede/list/?submitter=70405&state=* to > test it by yourself. > > 16.11.2016 18:17, João Chaínho: >> This patch fixes switch initial config on Mikrotik RB450/G >> Signed-off-by: João Chaínho <joaochai...@gmail.com> >> --- >> diff --git a/target/linux/ar71xx/base-files/etc/board.d/02_network >> b/target/linux/ar71xx/base-files/etc/board.d/02_network >> index df87c96..daf9495 100755 >> --- a/target/linux/ar71xx/base-files/etc/board.d/02_network >> +++ b/target/linux/ar71xx/base-files/etc/board.d/02_network >> @@ -343,13 +343,13 @@ ar71xx_setup_interfaces() >> rb-450) >> ucidef_set_interfaces_lan_wan "eth1" "eth0" >> ucidef_add_switch "switch0" \ >> - "0:lan" "1:lan" "2:lan" "3:lan" "5@eth1" >> + "0:lan:4" "1:lan:3" "2:lan:2" "3:lan:1" "5@eth1" >> ;; >> rb-450g|\ >> routerstation-pro) >> ucidef_set_interfaces_lan_wan "eth1" "eth0" >> ucidef_add_switch "switch0" \ >> - "0@eth1" "2:lan:3" "3:lan:2" "4:lan:1" >> + "0@eth1" "1:lan:1" "2:lan:4" "3:lan:3" "4:lan:2" > > You are changing the default switch config for the routerstation-pro as well. > Is this tested? It seams to me you're fixing one board and breaking another. > > Mathias
_______________________________________________ Lede-dev mailing list Lede-dev@lists.infradead.org http://lists.infradead.org/mailman/listinfo/lede-dev
