[LEDE-DEV] [PATCH] lantiq: dsl_notify: also restart pppoe interfaces

Valentin Spreckels Valentin.Spreckels at Informatik.Uni-Oldenburg.DE
Tue May 9 01:48:10 PDT 2017


Hi,

On 09/05/17 08:47, Felix Fietkau wrote:
> On 2017-05-09 08:46, Mathias Kresin wrote:
>> 09.05.2017 00:44, Valentin Spreckels:
>>> Signed-off-by: Valentin Spreckels <Valentin.Spreckels at Informatik.Uni-Oldenburg.DE>
>>> ---
>>
>> Thanks a lot for your patch. Would you please add a commit message which 
>> describes the issue you are trying to fix. At the moment I fail to see 
>> why this change is required.
>>
>> I rather suspect that the whole pppoa interface handling can be removed 
>> from dsl_notify.sh. Isn't it handled by netifd now days?
> pppoa needs special treatment, because it connects to ATM directly
> without going through a netdev.

May I suggest to add this information as comment in the script?

> For pppoe, this makes no sense, so this patch should be unnecessary.

I withdraw the patch.

Valentin



More information about the Lede-dev mailing list