[LEDE-DEV] [PATCH] lantiq: fix network in failsafe

Mathias Kresin dev at kresin.me
Sun Jun 19 22:52:59 PDT 2016


Am 20.06.2016 um 05:15 schrieb Zhao, Gang:
> Hi, Mathias Kresin
>
> On Sun, Jun 19, 2016 at 10:30 PM, Mathias Kresin <dev at kresin.me> wrote:
>> So far the network in failsafe is setup only for one board. Use the
>> eth0 interface as lan interface for all boards for now.
>>
>> If a board has its lan interface(s) on another eth, a special
>> handling based on the board name can be added.
>>
>> Signed-off-by: Mathias Kresin <dev at kresin.me>
>> ---
>>   .../lantiq/base-files/lib/preinit/05_set_preinit_iface_lantiq  | 10 +---------
>>   1 file changed, 1 insertion(+), 9 deletions(-)
>>
>> diff --git a/target/linux/lantiq/base-files/lib/preinit/05_set_preinit_iface_lantiq b/target/linux/lantiq/base-files/lib/preinit/05_set_preinit_iface_lantiq
>> index 3d7fabf..7ed0fab 100644
>> --- a/target/linux/lantiq/base-files/lib/preinit/05_set_preinit_iface_lantiq
>> +++ b/target/linux/lantiq/base-files/lib/preinit/05_set_preinit_iface_lantiq
>> @@ -3,15 +3,7 @@
>>   . /lib/functions/lantiq.sh
>>
>>   set_preinit_iface() {
>> -
>> -       board=$(lantiq_board_name)
>> -
>> -       case "$board" in
>> -       TDW8970)
>> -               ifname=eth0
>> -               ;;
>> -       esac
>> -
>> +       ifname=eth0
>
> What about to add a default branch in case statement?
>
> *)
>      ifname=eth0
> ;;

Adding a default branch doesn't make any sense to me as long as the only 
value set in the default branch is the same as the one in the only 
conditional.

I had changed the case statement to only consists of a default case, to 
make obvious how to add a different config for a specific board. But in 
the end I dropped the whole case since it was just unused code.



More information about the Lede-dev mailing list