<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Nov 17, 2017 at 12:29 PM, Aleksander Morgado <span dir="ltr"><<a href="mailto:aleksander@aleksander.es" target="_blank">aleksander@aleksander.es</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hey,<br>
<div><div class="h5"><br>
><br>
> I think I solved this. The Quectel BG96 does not need the QMI_QUIRK as<br>
> EC21/EC25.<br>
><br>
> # git diff<br>
> diff --git a/drivers/net/usb/qmi_wwan.c b/drivers/net/usb/qmi_wwan.c<br>
> index 8c37336..4682a93 100644<br>
> --- a/drivers/net/usb/qmi_wwan.c<br>
> +++ b/drivers/net/usb/qmi_wwan.c<br>
> @@ -1238,6 +1238,7 @@ static const struct usb_device_id products[] = {<br>
>         {QMI_FIXED_INTF(0x1e0e, 0x9001, 5)},    /* SIMCom 7230E */<br>
>         {QMI_QUIRK_SET_DTR(0x2c7c, 0x0125, 4)}, /* Quectel EC25, EC20 R2.0<br>
> Mini PCIe */<br>
>         {QMI_QUIRK_SET_DTR(0x2c7c, 0x0121, 4)}, /* Quectel EC21 Mini PCIe */<br>
> +       {QMI_FIXED_INTF(0x2c7c, 0x0296, 4)},    /* Quectel ECBG96 */<br>
><br>
>         /* 4. Gobi 1000 devices */<br>
>         {QMI_GOBI1K_DEVICE(0x05c6, 0x9212)},    /* Acer Gobi Modem Device */<br>
> diff --git a/drivers/usb/serial/option.c b/drivers/usb/serial/option.c<br>
> index db3d34c..c0ddfef 100644<br>
> --- a/drivers/usb/serial/option.c<br>
> +++ b/drivers/usb/serial/option.c<br>
> @@ -241,6 +241,7 @@ static void option_instat_callback(struct urb *urb);<br>
>  /* These Quectel products use Quectel's vendor ID */<br>
>  #define QUECTEL_PRODUCT_EC21                   0x0121<br>
>  #define QUECTEL_PRODUCT_EC25                   0x0125<br>
> +#define QUECTEL_PRODUCT_BG96                   0x0296<br>
><br>
>  #define CMOTECH_VENDOR_ID                      0x16d8<br>
>  #define CMOTECH_PRODUCT_6001                   0x6001<br>
> @@ -1185,6 +1186,8 @@ static const struct usb_device_id option_ids[] = {<br>
>           .driver_info = (kernel_ulong_t)&net_intf4_<wbr>blacklist },<br>
>         { USB_DEVICE(QUECTEL_VENDOR_ID, QUECTEL_PRODUCT_EC25),<br>
>           .driver_info = (kernel_ulong_t)&net_intf4_<wbr>blacklist },<br>
> +        { USB_DEVICE(QUECTEL_VENDOR_ID, QUECTEL_PRODUCT_BG96),<br>
> +          .driver_info = (kernel_ulong_t)&net_intf4_<wbr>blacklist },<br>
>         { USB_DEVICE(CMOTECH_VENDOR_ID, CMOTECH_PRODUCT_6001) },<br>
>         { USB_DEVICE(CMOTECH_VENDOR_ID, CMOTECH_PRODUCT_CMU_300) },<br>
>         { USB_DEVICE(CMOTECH_VENDOR_ID, CMOTECH_PRODUCT_6003),<br>
> #<br>
><br>
<br>
</div></div>Will you push these changes to the linux-usb and netdev mailing lists?<br>
<span class="HOEnZb"><font color="#888888"><br></font></span></blockquote><div><br></div><div> Yes, Björn has been kind to give me hints regarding this, so I will do it during the weekend.</div><div><br></div><div>Regards,</div><div>Sebastian</div></div><div><br></div>
</div></div>