Quectel BG96

[Aleksander Morgado]

Hey,

>
> I think I solved this. The Quectel BG96 does not need the QMI_QUIRK as
> EC21/EC25.
>
> # git diff
> diff --git a/drivers/net/usb/qmi_wwan.c b/drivers/net/usb/qmi_wwan.c
> index 8c37336..4682a93 100644
> --- a/drivers/net/usb/qmi_wwan.c
> +++ b/drivers/net/usb/qmi_wwan.c
> @@ -1238,6 +1238,7 @@ static const struct usb_device_id products[] = {
>         {QMI_FIXED_INTF(0x1e0e, 0x9001, 5)},    /* SIMCom 7230E */
>         {QMI_QUIRK_SET_DTR(0x2c7c, 0x0125, 4)}, /* Quectel EC25, EC20 R2.0
> Mini PCIe */
>         {QMI_QUIRK_SET_DTR(0x2c7c, 0x0121, 4)}, /* Quectel EC21 Mini PCIe */
> +       {QMI_FIXED_INTF(0x2c7c, 0x0296, 4)},    /* Quectel ECBG96 */
>
>         /* 4. Gobi 1000 devices */
>         {QMI_GOBI1K_DEVICE(0x05c6, 0x9212)},    /* Acer Gobi Modem Device */
> diff --git a/drivers/usb/serial/option.c b/drivers/usb/serial/option.c
> index db3d34c..c0ddfef 100644
> --- a/drivers/usb/serial/option.c
> +++ b/drivers/usb/serial/option.c
> @@ -241,6 +241,7 @@ static void option_instat_callback(struct urb *urb);
>  /* These Quectel products use Quectel's vendor ID */
>  #define QUECTEL_PRODUCT_EC21                   0x0121
>  #define QUECTEL_PRODUCT_EC25                   0x0125
> +#define QUECTEL_PRODUCT_BG96                   0x0296
>
>  #define CMOTECH_VENDOR_ID                      0x16d8
>  #define CMOTECH_PRODUCT_6001                   0x6001
> @@ -1185,6 +1186,8 @@ static const struct usb_device_id option_ids[] = {
>           .driver_info = (kernel_ulong_t)&net_intf4_blacklist },
>         { USB_DEVICE(QUECTEL_VENDOR_ID, QUECTEL_PRODUCT_EC25),
>           .driver_info = (kernel_ulong_t)&net_intf4_blacklist },
> +        { USB_DEVICE(QUECTEL_VENDOR_ID, QUECTEL_PRODUCT_BG96),
> +          .driver_info = (kernel_ulong_t)&net_intf4_blacklist },
>         { USB_DEVICE(CMOTECH_VENDOR_ID, CMOTECH_PRODUCT_6001) },
>         { USB_DEVICE(CMOTECH_VENDOR_ID, CMOTECH_PRODUCT_CMU_300) },
>         { USB_DEVICE(CMOTECH_VENDOR_ID, CMOTECH_PRODUCT_6003),
> #
>

Will you push these changes to the linux-usb and netdev mailing lists?

-- 
Aleksander
https://aleksander.es