[LEDE-DEV] [Lantiq][ARV4518PWR01*]lan interface missing from /etc/config/network

[Mohammed Berdai]

On 05/26/2016 12:48 AM, Rafał Miłecki wrote:
> E-mail topic is missing [PATCH] and should have a proper prefix
> ("lantiq: ") and shouldn't have extra tags.
> Also topic should say what your patch does, not what is the problem.
> 
> On 26 May 2016 at 00:30, Mohammed Berdai <mohammed.berdai at gmail.com> wrote:
>> Hi,
> 
> We don't want "Hi"-s in a commit message :)
> 
> 
>> This patch fixes the missing Lan interface config in the file /etc/config/network for ARV4518PWR01/A which results in eth0 and br-lan being down and therefore no way to access the router unless via UART-TTL.
> 
> Line breaks to fit 80 chars per line would be nice.
> 
> 
>> Med Berdai (medber)
> 
> What is this? I don't think we want this in a commit message.
> 
> 
>> Signed-off-by: Mohammed Berdai <mohammed.berdai at gmail.com>
>> ---
>> diff --git a/target/linux/lantiq/base-files/etc/board.d/02_network b/target/linux/lantiq/base-files/etc/board.d/02_network
>> index b27b802..4166df0 100755
>> --- a/target/linux/lantiq/base-files/etc/board.d/02_network
>> +++ b/target/linux/lantiq/base-files/etc/board.d/02_network
>> @@ -62,6 +62,10 @@ VG3503J)
>>         ucidef_set_interface_lan 'eth0'
>>         ;;
>>
>> +ARV4518PWR01*)
>> +        ucidef_set_interface_lan 'eth0'
>> +        ;;
>> +
>>  DGN3500*)
>>         ucidef_add_switch "switch0" \
>>                 "3:lan:1" "2:lan:2" "1:lan:3" "0:lan:4" "5t at eth0"

Very clear. Thanks.

Should I resend it with the corrections you highlighted or is it ok?

Is it ok for me to ask in the ML that we should have a doc page for
the proper way to write and send mails and patches?

I myself had a bit of a challenge to setup vim write acceptable
patches and my email client (Thunderbird) not mess with break-lines.