Hi Felix,
1) Just set a non-allowed DFS channel for DE, for instance.
2) Enable wireless; it fails
3) Change to a valid channel; the wdev is not set up.
There is also a leftover on /etc/init.d/network reload. I think it should call
/sbin/wifi instead of /sbin/wifi reload.
Eduardo
On 01.07.2016 00:14, Felix Fietkau wrote:
> On 2016-06-30 16:18, Eduardo Abinader wrote:
>> Ensure wireless device struct is removed, when it fails to be set up,
>> due to a invalid configuration, hence enabling the wireless dev
>> to be set up again, when a new valid configuration is set.
>>
>> Signed-off-by: Eduardo Abinader <eduardo.abina...@riverbed.com>
>> ---
>> wireless.c | 4 ++++
>> 1 file changed, 4 insertions(+)
>>
>> diff --git a/wireless.c b/wireless.c
>> index 34dd328..bc31d4a 100644
>> --- a/wireless.c
>> +++ b/wireless.c
>> @@ -336,6 +336,10 @@ wireless_device_mark_down(struct wireless_device *wdev)
>>
>> wireless_process_kill_all(wdev, SIGTERM, true);
>>
>> + if (wdev->state == IFS_SETUP && !wdev->autostart) {
>> + wdev->config_state = IFC_REMOVE;
>> + }
> This doesn't make sense to me. Could you please provide a test case that
> reproduces the problem you're trying to fix?
>
> - Felix
>
_______________________________________________
Lede-dev mailing list
Lede-dev@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/lede-dev
