Changeset 47720


Ignore:
Timestamp:
2015-12-03T18:30:08+01:00 (2 years ago)
Author:
jow
Message:

base-files: uci-defaults-new: remove reset/enable args of ucidef_add_switch()

Out of 69 switch definitions, only 3 pass something different than "1" as
values for reset and enable, with one of those three being invalid.

This change ...

  • removes the reset and enable arguments from ucidef_add_switch()
  • unconditionally writes reset:1 and enable:1 to JSON
  • converts the three users of nonstandard values to ucidef_add_switch_attr()

Signed-off-by: Jo-Philipp Wich <jow@…>

Location:
trunk
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • trunk/package/base-files/files/lib/functions/uci-defaults-new.sh

    r47719 r47720  
    8585ucidef_add_switch() { 
    8686        local name="$1" 
    87         local reset="$2" 
    88         local enable="$3" 
    8987 
    9088        json_select_object switch 
    91  
    92         json_select_object "$name" 
    93         json_add_boolean enable "$enable" 
    94         json_add_boolean reset "$reset" 
    95         json_select .. 
    96  
     89                json_select_object "$name" 
     90                        json_add_boolean enable 1 
     91                        json_add_boolean reset 1 
     92                json_select .. 
    9793        json_select .. 
    9894} 
     
    104100 
    105101        json_select_object switch 
    106         json_select_object "$name" 
    107  
    108         case "$val" in 
    109                 [0-9]) json_add_int "$key" "$val" ;; 
    110                 *) json_add_string "$key" "$val" ;; 
    111         esac 
    112  
    113         json_select .. 
     102                json_select_object "$name" 
     103 
     104                case "$val" in 
     105                        true|false) [ "$val" != "true" ]; json_add_boolean "$key" $? ;; 
     106                        [0-9]) json_add_int "$key" "$val" ;; 
     107                        *) json_add_string "$key" "$val" ;; 
     108                esac 
     109 
     110                json_select .. 
    114111        json_select .. 
    115112} 
     
    274271 
    275272                        case "$val" in 
     273                                true|false) [ "$val" != "true" ]; json_add_boolean "$key" $? ;; 
    276274                                [0-9]) json_add_int "$key" "$val" ;; 
    277275                                *) json_add_string "$key" "$val" ;; 
  • trunk/target/linux/ar71xx/base-files/etc/board.d/02_network

    r47663 r47720  
    349349dlan-pro-1200-ac) 
    350350        ucidef_set_interface_lan "eth0" 
    351         ucidef_add_switch "switch0" "1" "0" 
     351        ucidef_add_switch "switch0" 
     352        ucidef_add_switch_attr "switch0" "reset" "false" 
    352353        ucidef_add_switch_ports "switch0" \ 
    353354                "0@eth0" "2:lan" "3:lan" "4:lan" 
  • trunk/target/linux/ramips/base-files/etc/board.d/02_network

    r47717 r47720  
    5353        wnce2001|\ 
    5454        zte-q7) 
    55                 ucidef_add_switch "switch0" "1" "0" 
     55                ucidef_add_switch "switch0" 
     56                ucidef_add_switch_attr "switch0" "reset" "false" 
    5657                ucidef_set_interface_lan "eth0" 
    5758                ;; 
     
    187188                ;; 
    188189        m2m) 
    189                 ucidef_add_switch "switch0" "4" 
     190                ucidef_add_switch "switch0" 
     191                ucidef_add_switch_attr "switch0" "reset" "false" 
    190192                ucidef_set_interface_lan "eth0" 
    191193                ;; 
Note: See TracChangeset for help on using the changeset viewer.