Hi,

On Wed, Oct 31, 2018 at 7:54 AM Nandini Rebello
<[email protected]> wrote:
>
> Newer intel modems enumerate with different subsystem numbers,
> adding code to detect newer 7xxx modules.
>
> Plan to add patch for backward compatibility of intels modems soon,
> to based on interface number instead of subsystem name string.
> ---
>  plugins/udevng.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/plugins/udevng.c b/plugins/udevng.c
> index 11338f7..bf0b1c4 100644
> --- a/plugins/udevng.c
> +++ b/plugins/udevng.c
> @@ -1194,10 +1194,10 @@ static gboolean setup_xmm7xxx(struct modem_info 
> *modem)
>                                 info->sysattr, info->subsystem);
>
>                 if (g_strcmp0(info->subsystem, "tty") == 0) {
> -                       if (g_strcmp0(info->number, "02") == 0)
> +                       if (g_strcmp0(info->number, "00") == 0)

but then all current users of the models using the usb device 02 (and
00 below) will be cut off if they update to this patch.
Maybe you need to add the switching logic you mentioned above based on
interface number in the same patch.

>                                 mdm = info->devnode;
>                 } else if (g_strcmp0(info->subsystem, "net") == 0) {
> -                       if (g_strcmp0(info->number, "00") == 0)
> +                       if (g_strcmp0(info->number, "06") == 0)
>                                 net = info->devnode;
>                 }
>         }
> --
> 2.7.4
>
> _______________________________________________
> ofono mailing list
> [email protected]
> https://lists.ofono.org/mailman/listinfo/ofono

regards,
Giacinto
_______________________________________________
ofono mailing list
[email protected]
https://lists.ofono.org/mailman/listinfo/ofono

Reply via email to