Linux-USB Archive on lore.kernel.org
 help / color / Atom feed
* [PATCH 1/1] net: usb: qmi_wwan: add Telit 0x1050 composition
@ 2019-10-09  9:07 Daniele Palmas
  2019-10-09 10:17 ` Bjørn Mork
  2019-10-10  3:01 ` Jakub Kicinski
  0 siblings, 2 replies; 4+ messages in thread
From: Daniele Palmas @ 2019-10-09  9:07 UTC (permalink / raw)
  To: Bjørn Mork; +Cc: netdev, David S . Miller, linux-usb, Daniele Palmas

This patch adds support for Telit FN980 0x1050 composition

0x1050: tty, adb, rmnet, tty, tty, tty, tty

Signed-off-by: Daniele Palmas <dnlplm@gmail.com>
---
Hi Bjørn,

please find below usb-devices output

T:  Bus=03 Lev=01 Prnt=01 Port=06 Cnt=02 Dev#= 10 Spd=480 MxCh= 0
D:  Ver= 2.10 Cls=00(>ifc ) Sub=00 Prot=00 MxPS=64 #Cfgs=  1
P:  Vendor=1bc7 ProdID=1050 Rev=04.14
S:  Manufacturer=Telit Wireless Solutions
S:  Product=FN980m
S:  SerialNumber=270b8241
C:  #Ifs= 7 Cfg#= 1 Atr=80 MxPwr=500mA
I:  If#= 0 Alt= 0 #EPs= 2 Cls=ff(vend.) Sub=ff Prot=30 Driver=option
I:  If#= 1 Alt= 0 #EPs= 2 Cls=ff(vend.) Sub=42 Prot=01 Driver=usbfs
I:  If#= 2 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=ff Prot=ff Driver=qmi_wwan
I:  If#= 3 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=00 Prot=00 Driver=option
I:  If#= 4 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=00 Prot=00 Driver=option
I:  If#= 5 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=00 Prot=00 Driver=option
I:  If#= 6 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=00 Prot=00 Driver=option

Thanks,
Daniele
---
 drivers/net/usb/qmi_wwan.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/net/usb/qmi_wwan.c b/drivers/net/usb/qmi_wwan.c
index 3d77cd402ba9..596428ec71df 100644
--- a/drivers/net/usb/qmi_wwan.c
+++ b/drivers/net/usb/qmi_wwan.c
@@ -1327,6 +1327,7 @@ static const struct usb_device_id products[] = {
 	{QMI_FIXED_INTF(0x2357, 0x0201, 4)},	/* TP-LINK HSUPA Modem MA180 */
 	{QMI_FIXED_INTF(0x2357, 0x9000, 4)},	/* TP-LINK MA260 */
 	{QMI_QUIRK_SET_DTR(0x1bc7, 0x1040, 2)},	/* Telit LE922A */
+	{QMI_QUIRK_SET_DTR(0x1bc7, 0x1050, 2)},	/* Telit FN980 */
 	{QMI_FIXED_INTF(0x1bc7, 0x1100, 3)},	/* Telit ME910 */
 	{QMI_FIXED_INTF(0x1bc7, 0x1101, 3)},	/* Telit ME910 dual modem */
 	{QMI_FIXED_INTF(0x1bc7, 0x1200, 5)},	/* Telit LE920 */
-- 
2.17.1


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH 1/1] net: usb: qmi_wwan: add Telit 0x1050 composition
  2019-10-09  9:07 [PATCH 1/1] net: usb: qmi_wwan: add Telit 0x1050 composition Daniele Palmas
@ 2019-10-09 10:17 ` Bjørn Mork
  2019-10-09 14:49   ` Daniele Palmas
  2019-10-10  3:01 ` Jakub Kicinski
  1 sibling, 1 reply; 4+ messages in thread
From: Bjørn Mork @ 2019-10-09 10:17 UTC (permalink / raw)
  To: Daniele Palmas; +Cc: netdev, David S . Miller, linux-usb

Daniele Palmas <dnlplm@gmail.com> writes:

> This patch adds support for Telit FN980 0x1050 composition
>
> 0x1050: tty, adb, rmnet, tty, tty, tty, tty

Great!  I must admit I have been a bit curious about this since you
submitted the option patch.  And I'm still curious about what to expect
from X55 modems in general.  There was a lot of discussion about future
modems using PCIe instead of USB etc.  I'd appreciate any info you have
on relative performance, quirks, firmware workarounds etc.  If you are
allowed to share any of it..

Acked-by: Bjørn Mork" <bjorn@mork.no>

> please find below usb-devices output
>
> T:  Bus=03 Lev=01 Prnt=01 Port=06 Cnt=02 Dev#= 10 Spd=480 MxCh= 0

480 Mbps is a bit slow for this device, isn't it? :-)

I assume you've tested with higher bus speeds?  Not that it matters for
this patch.  Just curious again.

> D:  Ver= 2.10 Cls=00(>ifc ) Sub=00 Prot=00 MxPS=64 #Cfgs=  1
> P:  Vendor=1bc7 ProdID=1050 Rev=04.14
> S:  Manufacturer=Telit Wireless Solutions
> S:  Product=FN980m
> S:  SerialNumber=270b8241
> C:  #Ifs= 7 Cfg#= 1 Atr=80 MxPwr=500mA
> I:  If#= 0 Alt= 0 #EPs= 2 Cls=ff(vend.) Sub=ff Prot=30 Driver=option
> I:  If#= 1 Alt= 0 #EPs= 2 Cls=ff(vend.) Sub=42 Prot=01 Driver=usbfs
> I:  If#= 2 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=ff Prot=ff Driver=qmi_wwan
> I:  If#= 3 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=00 Prot=00 Driver=option
> I:  If#= 4 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=00 Prot=00 Driver=option
> I:  If#= 5 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=00 Prot=00 Driver=option
> I:  If#= 6 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=00 Prot=00 Driver=option
>
> Thanks,
> Daniele
> ---
>  drivers/net/usb/qmi_wwan.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/drivers/net/usb/qmi_wwan.c b/drivers/net/usb/qmi_wwan.c
> index 3d77cd402ba9..596428ec71df 100644
> --- a/drivers/net/usb/qmi_wwan.c
> +++ b/drivers/net/usb/qmi_wwan.c
> @@ -1327,6 +1327,7 @@ static const struct usb_device_id products[] = {
>  	{QMI_FIXED_INTF(0x2357, 0x0201, 4)},	/* TP-LINK HSUPA Modem MA180 */
>  	{QMI_FIXED_INTF(0x2357, 0x9000, 4)},	/* TP-LINK MA260 */
>  	{QMI_QUIRK_SET_DTR(0x1bc7, 0x1040, 2)},	/* Telit LE922A */
> +	{QMI_QUIRK_SET_DTR(0x1bc7, 0x1050, 2)},	/* Telit FN980 */
>  	{QMI_FIXED_INTF(0x1bc7, 0x1100, 3)},	/* Telit ME910 */
>  	{QMI_FIXED_INTF(0x1bc7, 0x1101, 3)},	/* Telit ME910 dual modem */
>  	{QMI_FIXED_INTF(0x1bc7, 0x1200, 5)},	/* Telit LE920 */


Acked-by: Bjørn Mork <bjorn@mork.no>

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH 1/1] net: usb: qmi_wwan: add Telit 0x1050 composition
  2019-10-09 10:17 ` Bjørn Mork
@ 2019-10-09 14:49   ` Daniele Palmas
  0 siblings, 0 replies; 4+ messages in thread
From: Daniele Palmas @ 2019-10-09 14:49 UTC (permalink / raw)
  To: Bjørn Mork; +Cc: netdev, David S . Miller, linux-usb

Il giorno mer 9 ott 2019 alle ore 12:17 Bjørn Mork <bjorn@mork.no> ha scritto:
>
> Daniele Palmas <dnlplm@gmail.com> writes:
>
> > This patch adds support for Telit FN980 0x1050 composition
> >
> > 0x1050: tty, adb, rmnet, tty, tty, tty, tty
>
> Great!  I must admit I have been a bit curious about this since you
> submitted the option patch.  And I'm still curious about what to expect
> from X55 modems in general.  There was a lot of discussion about future
> modems using PCIe instead of USB etc.  I'd appreciate any info you have
> on relative performance, quirks, firmware workarounds etc.  If you are
> allowed to share any of it..
>

I was having issues with my hw sample related to the data connection
setup with rmnet: now I verified that is working properly with basic
tests, so I sent the patch.

I've reports of peak UDP DL throughput in loopback in the range of
6Gbps (not in Linux), more than 6 times the value I had with qmi_wwan
and LM960 (1.1Gbps), where the modem limit was reached.

For the sample I have it's probably a bit early to perform throughput
tests, but it is something that I hope to do in the next weeks.

The PCIe 3 EP 2-lanes mode is declared faster, but it won't be easy to
test it on official kernel releases, since drivers are missing.

The idea is to start with the patchset available at
https://lore.kernel.org/lkml/001601d52148$bd852840$388f78c0$@codeaurora.org/T/,
since it is the only public codebase I could find, but I don't expect
to have something working soon...

> Acked-by: Bjørn Mork" <bjorn@mork.no>
>
> > please find below usb-devices output
> >
> > T:  Bus=03 Lev=01 Prnt=01 Port=06 Cnt=02 Dev#= 10 Spd=480 MxCh= 0
>
> 480 Mbps is a bit slow for this device, isn't it? :-)
>
> I assume you've tested with higher bus speeds?  Not that it matters for
> this patch.  Just curious again.
>

Agree, wrong cable just used for getting the usb-devices output :-) A
better one:

T:  Bus=04 Lev=01 Prnt=01 Port=05 Cnt=01 Dev#=  4 Spd=5000 MxCh= 0
D:  Ver= 3.20 Cls=00(>ifc ) Sub=00 Prot=00 MxPS= 9 #Cfgs=  1
P:  Vendor=1bc7 ProdID=1050 Rev=04.14
S:  Manufacturer=Telit Wireless Solutions
S:  Product=FN980m

Thanks,
Daniele

> > D:  Ver= 2.10 Cls=00(>ifc ) Sub=00 Prot=00 MxPS=64 #Cfgs=  1
> > P:  Vendor=1bc7 ProdID=1050 Rev=04.14
> > S:  Manufacturer=Telit Wireless Solutions
> > S:  Product=FN980m
> > S:  SerialNumber=270b8241
> > C:  #Ifs= 7 Cfg#= 1 Atr=80 MxPwr=500mA
> > I:  If#= 0 Alt= 0 #EPs= 2 Cls=ff(vend.) Sub=ff Prot=30 Driver=option
> > I:  If#= 1 Alt= 0 #EPs= 2 Cls=ff(vend.) Sub=42 Prot=01 Driver=usbfs
> > I:  If#= 2 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=ff Prot=ff Driver=qmi_wwan
> > I:  If#= 3 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=00 Prot=00 Driver=option
> > I:  If#= 4 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=00 Prot=00 Driver=option
> > I:  If#= 5 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=00 Prot=00 Driver=option
> > I:  If#= 6 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=00 Prot=00 Driver=option
> >
> > Thanks,
> > Daniele
> > ---
> >  drivers/net/usb/qmi_wwan.c | 1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git a/drivers/net/usb/qmi_wwan.c b/drivers/net/usb/qmi_wwan.c
> > index 3d77cd402ba9..596428ec71df 100644
> > --- a/drivers/net/usb/qmi_wwan.c
> > +++ b/drivers/net/usb/qmi_wwan.c
> > @@ -1327,6 +1327,7 @@ static const struct usb_device_id products[] = {
> >       {QMI_FIXED_INTF(0x2357, 0x0201, 4)},    /* TP-LINK HSUPA Modem MA180 */
> >       {QMI_FIXED_INTF(0x2357, 0x9000, 4)},    /* TP-LINK MA260 */
> >       {QMI_QUIRK_SET_DTR(0x1bc7, 0x1040, 2)}, /* Telit LE922A */
> > +     {QMI_QUIRK_SET_DTR(0x1bc7, 0x1050, 2)}, /* Telit FN980 */
> >       {QMI_FIXED_INTF(0x1bc7, 0x1100, 3)},    /* Telit ME910 */
> >       {QMI_FIXED_INTF(0x1bc7, 0x1101, 3)},    /* Telit ME910 dual modem */
> >       {QMI_FIXED_INTF(0x1bc7, 0x1200, 5)},    /* Telit LE920 */
>
>
> Acked-by: Bjørn Mork <bjorn@mork.no>

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH 1/1] net: usb: qmi_wwan: add Telit 0x1050 composition
  2019-10-09  9:07 [PATCH 1/1] net: usb: qmi_wwan: add Telit 0x1050 composition Daniele Palmas
  2019-10-09 10:17 ` Bjørn Mork
@ 2019-10-10  3:01 ` Jakub Kicinski
  1 sibling, 0 replies; 4+ messages in thread
From: Jakub Kicinski @ 2019-10-10  3:01 UTC (permalink / raw)
  To: Daniele Palmas; +Cc: Bjørn Mork, netdev, David S . Miller, linux-usb

On Wed,  9 Oct 2019 11:07:18 +0200, Daniele Palmas wrote:
> This patch adds support for Telit FN980 0x1050 composition
> 
> 0x1050: tty, adb, rmnet, tty, tty, tty, tty
> 
> Signed-off-by: Daniele Palmas <dnlplm@gmail.com>

Applied, thanks!

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, back to index

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-10-09  9:07 [PATCH 1/1] net: usb: qmi_wwan: add Telit 0x1050 composition Daniele Palmas
2019-10-09 10:17 ` Bjørn Mork
2019-10-09 14:49   ` Daniele Palmas
2019-10-10  3:01 ` Jakub Kicinski

Linux-USB Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-usb/0 linux-usb/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-usb linux-usb/ https://lore.kernel.org/linux-usb \
		linux-usb@vger.kernel.org linux-usb@archiver.kernel.org
	public-inbox-index linux-usb

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.linux-usb


AGPL code for this site: git clone https://public-inbox.org/ public-inbox