On Mon, Jul 10, 2023 at 11:22:18AM +0330, Mohsen Tahmasebi wrote:
> Add Quectel EC200A "DIAG, AT, MODEM":
>
> 0x6005: ECM + RNDIS + DIAG + AT + MODEM
This device appears to only have four ports. Can you switch between ECM
and RNDIS using some AT command?
Does the device use the same PID in RNDIS mode?
Or is RNDIS perhaps not at all supported?
I've applied the patch after dropping RNDIS above and with the minor
changes below, but please clarify this so I can update the commit
message if needed (e.g. if this should have said "ECM/RNDIS" or
similar).
> T: Bus=01 Lev=01 Prnt=02 Port=05 Cnt=01 Dev#= 8 Spd=480 MxCh= 0
> D: Ver= 2.00 Cls=ef(misc ) Sub=02 Prot=01 MxPS=64 #Cfgs= 1
> P: Vendor=2c7c ProdID=6005 Rev=03.18
> S: Manufacturer=Android
> S: Product=Android
> S: SerialNumber=0000
> C: #Ifs= 5 Cfg#= 1 Atr=e0 MxPwr=500mA
> I: If#= 0 Alt= 0 #EPs= 1 Cls=02(commc) Sub=06 Prot=00 Driver=cdc_ether
> E: Ad=87(I) Atr=03(Int.) MxPS= 64 Ivl=4096ms
> I: If#= 1 Alt= 1 #EPs= 2 Cls=0a(data ) Sub=00 Prot=00 Driver=cdc_ether
> E: Ad=0c(O) Atr=02(Bulk) MxPS= 512 Ivl=0ms
> E: Ad=83(I) Atr=02(Bulk) MxPS= 512 Ivl=0ms
> I: If#= 2 Alt= 0 #EPs= 2 Cls=ff(vend.) Sub=00 Prot=00 Driver=option
> E: Ad=0b(O) Atr=02(Bulk) MxPS= 512 Ivl=0ms
> E: Ad=82(I) Atr=02(Bulk) MxPS= 512 Ivl=0ms
> I: If#= 3 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=00 Prot=00 Driver=option
> E: Ad=0f(O) Atr=02(Bulk) MxPS= 512 Ivl=0ms
> E: Ad=86(I) Atr=02(Bulk) MxPS= 512 Ivl=0ms
> E: Ad=89(I) Atr=03(Int.) MxPS= 64 Ivl=4096ms
> I: If#= 4 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=00 Prot=00 Driver=option
> E: Ad=0a(O) Atr=02(Bulk) MxPS= 512 Ivl=0ms
> E: Ad=81(I) Atr=02(Bulk) MxPS= 512 Ivl=0ms
> E: Ad=88(I) Atr=03(Int.) MxPS= 64 Ivl=4096ms
>
> Signed-off-by: Mohsen Tahmasebi <moh53n@xxxxxxxxx>
> Tested-by: Mostafa Ghofrani <mostafaghrr@xxxxxxxxx>
> ---
> V1 -> V2: Add proper changelog text
>
> drivers/usb/serial/option.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/usb/serial/option.c b/drivers/usb/serial/option.c
> index 288a96a74266..ed4405efc18f 100644
> --- a/drivers/usb/serial/option.c
> +++ b/drivers/usb/serial/option.c
> @@ -271,6 +271,7 @@ static void option_instat_callback(struct urb *urb);
> #define QUECTEL_PRODUCT_EM061K_LWW 0x6008
> #define QUECTEL_PRODUCT_EM061K_LCN 0x6009
> #define QUECTEL_PRODUCT_EC200T 0x6026
> +#define QUECTEL_PRODUCT_EC200A 0x6005
Next time, please keep the PID defines sorted by value.
> #define QUECTEL_PRODUCT_RM500K 0x7001
>
> #define CMOTECH_VENDOR_ID 0x16d8
> @@ -1228,6 +1229,7 @@ static const struct usb_device_id option_ids[] = {
> { USB_DEVICE_AND_INTERFACE_INFO(QUECTEL_VENDOR_ID, QUECTEL_PRODUCT_EC200U, 0xff, 0, 0) },
> { USB_DEVICE_AND_INTERFACE_INFO(QUECTEL_VENDOR_ID, QUECTEL_PRODUCT_EC200S_CN, 0xff, 0, 0) },
> { USB_DEVICE_AND_INTERFACE_INFO(QUECTEL_VENDOR_ID, QUECTEL_PRODUCT_EC200T, 0xff, 0, 0) },
> + { USB_DEVICE_AND_INTERFACE_INFO(QUECTEL_VENDOR_ID, QUECTEL_PRODUCT_EC200A, 0xff, 0, 0) },
And the device-id entries sorted alphabetically.
> { USB_DEVICE_AND_INTERFACE_INFO(QUECTEL_VENDOR_ID, QUECTEL_PRODUCT_RM500K, 0xff, 0x00, 0x00) },
>
> { USB_DEVICE(CMOTECH_VENDOR_ID, CMOTECH_PRODUCT_6001) },
Johan
