swconfig: parse "switch_vlan" before "switch_port"
authorRafał Miłecki <rafal@milecki.pl>
Tue, 5 Apr 2022 06:49:58 +0000 (08:49 +0200)
committerRafał Miłecki <rafal@milecki.pl>
Wed, 15 Jun 2022 08:44:32 +0000 (10:44 +0200)
commitd75bb744eaa1edd613612af0399f318fae841ae6
tree22741d014179cfd1d6344b5f359b270456b3bf1b
parentc4b499bc03ab0e2eea643c46d1d781ab64e78931
swconfig: parse "switch_vlan" before "switch_port"

Before this change UCI sections of both types were parsed in order as
specified in UCI. That didn't work well with all drivers (e.g. b53).

It seems that VLAN setup can reset / overwrite previously set ports
parameters. It resulted in "switch_port" options defined above
"switch_vlan"s being silently ignored.

Ideally swconfig & all drivers should be improved to handle that
properly but it'd be a waste of time at this point as DSA replaces
swconfig. Use this minor parsing change as a quick fix.

Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
package/network/config/swconfig/src/uci.c