On Fri, Dec 21, 2018 at 02:56:16PM +0100, Greg Kroah-Hartman wrote: > On Fri, Dec 21, 2018 at 02:40:44PM +0100, Jörgen Storvist wrote: > > Added USB serial option driver support for Fibocom NL678 series cellular > > module: VID 2cb7 and PIDs 0x0104 and 0x0105. > > Reserved network and ADB interfaces. > > > > T: Bus=01 Lev=01 Prnt=01 Port=00 Cnt=01 Dev#= 2 Spd=480 MxCh= 0 > > D: Ver= 2.00 Cls=00(>ifc ) Sub=00 Prot=00 MxPS=64 #Cfgs= 1 > > P: Vendor=2cb7 ProdID=0104 Rev=03.10 > > S: Manufacturer=Fibocom > > S: Product=Fibocom NL678-E Modem > > S: SerialNumber=12345678 > > C: #Ifs= 6 Cfg#= 1 Atr=a0 MxPwr=500mA > > I: If#= 0 Alt= 0 #EPs= 2 Cls=ff(vend.) Sub=ff Prot=ff Driver=option > > I: If#= 1 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=00 Prot=00 Driver=option > > I: If#= 2 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=00 Prot=00 Driver=option > > 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=ff Prot=ff Driver=qmi_wwan > > I: If#= 5 Alt= 0 #EPs= 2 Cls=ff(vend.) Sub=42 Prot=01 Driver=(none) > > > > T: Bus=01 Lev=01 Prnt=01 Port=00 Cnt=01 Dev#= 3 Spd=480 MxCh= 0 > > D: Ver= 2.00 Cls=00(>ifc ) Sub=00 Prot=00 MxPS=64 #Cfgs= 1 > > P: Vendor=2cb7 ProdID=0105 Rev=03.10 > > S: Manufacturer=Fibocom > > S: Product=Fibocom NL678-E Modem > > S: SerialNumber=12345678 > > C: #Ifs= 7 Cfg#= 1 Atr=a0 MxPwr=500mA > > I: If#= 0 Alt= 0 #EPs= 2 Cls=ff(vend.) Sub=ff Prot=ff Driver=option > > I: If#= 1 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=00 Prot=00 Driver=option > > I: If#= 2 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=00 Prot=00 Driver=option > > I: If#= 3 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=00 Prot=00 Driver=option > > I: If#= 4 Alt= 0 #EPs= 1 Cls=02(commc) Sub=06 Prot=00 Driver=cdc_ether > > I: If#= 5 Alt= 1 #EPs= 2 Cls=0a(data ) Sub=00 Prot=00 Driver=cdc_ether > > I: If#= 6 Alt= 0 #EPs= 2 Cls=ff(vend.) Sub=42 Prot=01 Driver=(none) > > > > Signed-off-by: Jörgen Storvist <jorgen.storvist@xxxxxxxxx> > > Johan, if you ack this, I can take this in my tree now... Looks good to me, although checkpatch reports two spaces before tabs, which perhaps you can remove when applying? Should be fine either way. Acked-by: Johan Hovold <johan@xxxxxxxxxx> Thanks, Johan
