Re: [PATCH] Fix modem doesn't auto register to network issue

[Denis Kenzior <denkenz@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 2165 bytes --]

Hi Caiwen,

On 09/11/2013 09:48 PM, Zhang, Caiwen wrote:
> Hi Denis,
>
>>> Some modem may return registration status whitch is out of the range
>>> (0-5). it will cause the modem don't auto register to network when
>>> network atom initializes.
>>
>> oFono core will not validate values outside what is defined in 27.007.
>> If your driver reports such values, then it is an obvious bug in the driver.
>>
>> Please fix your modem driver.
>>
>
> (status == NETWORK_REGISTRATION_STATUS_NOT_REGISTERED ||
> 		status == NETWORK_REGISTRATION_STATUS_DENIED ||
> 		status == NETWORK_REGISTRATION_STATUS_UNKNOWN)
>
> Above checking tries to list the cases that should trigger auto network registration but for the modems which
> extend network registration status may miss some cases.
>
> With the new checking:
> 		(status != NETWORK_REGISTRATION_STATUS_REGISTERED &&
> 			status != NETWORK_REGISTRATION_STATUS_SEARCHING &&
> 			status != NETWORK_REGISTRATION_STATUS_ROAMING))
> it tries to exclude the cases that not required to trigger auto network registration. In this way, it can avoid
> above issue.

Since we do not know what the meaning of the extended value actually is, 
we can be wrong in either situation.  The core only knows about values 
defined in 27.007 or values it defines explicitly inside include/*.  If 
we start having the core deal with every single vendor extension then we 
will quickly go insane and the core becomes unreadable.

I have been and will continue to be quite adamant here.  Fix. Your. Driver.

>
> I am not sure whether other modem has the same issue. This way should be more make sense.
>
> By the way, it should be better to change the status from "int" to " enum network_registration_status" to avoid modem driver pass
> in value we don't expect.

How does this help you?  If your modem reports issues outside the range, 
you still need to sanitize the values you send to the core.  If your 
modem is compliant, then converting them to an enum just leads to you 
writing more code that does nothing useful.

We've tried this already and abandoned it long ago.

Regards,
-Denis