* [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 related [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, other threads:[~2019-10-10 3:01 UTC | newest]
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
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).