[source] base-files: board.json's switch reset means existence, not argument

LEDE Commits lede-commits at lists.infradead.org
Sun Jun 11 03:51:04 PDT 2017


jogo pushed a commit to source.git, branch master:
https://git.lede-project.org/fd952c7a83e5357f5dcab28c0bcaca756e27eaa0

commit fd952c7a83e5357f5dcab28c0bcaca756e27eaa0
Author: Jonas Gorski <jonas.gorski at gmail.com>
AuthorDate: Sun Jun 11 12:50:14 2017 +0200

    base-files: board.json's switch reset means existence, not argument
    
    Don't pass the value unconditionally to swconfig as a parameter but
    instead only call reset if it is 1.
    
    Signed-off-by: Jonas Gorski <jonas.gorski at gmail.com>
---
 package/base-files/files/lib/preinit/10_indicate_preinit | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/package/base-files/files/lib/preinit/10_indicate_preinit b/package/base-files/files/lib/preinit/10_indicate_preinit
index a30bf34..43bd04d 100644
--- a/package/base-files/files/lib/preinit/10_indicate_preinit
+++ b/package/base-files/files/lib/preinit/10_indicate_preinit
@@ -43,7 +43,10 @@ preinit_config_switch() {
 			json_select ..
 
 			if [ "$device" = "$lan_if" ]; then
-				swconfig dev $name set reset $reset
+				if [ "$reset" -eq "1" ]; then
+					swconfig dev $name set reset
+				fi
+
 				swconfig dev $name set enable_vlan $enable
 				swconfig dev $name vlan $role set ports "$ports"
 				swconfig dev $name set apply



More information about the lede-commits mailing list