Re: [PATCH] Documentation: usb: serial: fixed how to provide vendor and product id
On Thu, Apr 02, 2015 at 04:25:50PM +0200, Dr. H. Nikolaus Schaller wrote: > But it is outdated. The parameters vendor= and product= are no longer > available since ca. 3.12 which means that documentation is lagging behind > quite some time. > >>> > >>> This is simply not true. The vendor and module parameters are still > >>> there for usbserial (i.e. the generic driver). > >>> > >>> Would you mind updating this patch to document the sysfs-method as an > >>> alternative instead? Using sysfs is preferred as that functionality can > >>> be used with any usb-serial driver and the drivers can also be compiled > >>> in. > >> > >> Hm. For me it did not work as described and therefore I searched for > >> and found the solution cited below, which appears to confirm that it does > >> not work any more. Only with these hints to use the /sys approach I > >> could make it working immediately. > >> > >> Therefore I assumed that the documentation is outdated. > >> > >> But it might also be a hidden bug with the parameter passing. > >> I had tested on 4.0-rc3. > >> > >> I can try a little more but it might be waste of time to find out why the > >> old > >> does not work, if there is a preferred method now. > > > > Please do try to figure out if it doesn't work as we have not deprecated > > this functionality yet. It should still work. > > Good news: > > I have tried again (this time with 4.0-rc5) and was successful. I have no > clue why it did not work with my tests with 4.0-rc3. Great. > So we will edit the patch to describe > * that the sysfs method is in addition, and preferred > * and works even if driver is compiled into the kernel (which I > haven’t tested in my setup but assume it works as well) > > Ok? That should do it (and, yes, it works when the module is compiled in). Thanks, Johan -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] Documentation: usb: serial: fixed how to provide vendor and product id
Hi, Am 30.03.2015 um 12:08 schrieb Johan Hovold : > On Mon, Mar 30, 2015 at 11:32:39AM +0200, Dr. H. Nikolaus Schaller wrote: >> Hi Johan, >> >> Am 30.03.2015 um 09:45 schrieb Johan Hovold : >> >>> On Sat, Mar 28, 2015 at 02:37:57PM +0100, Marek Belisko wrote: From: "H. Nikolaus Schaller" While trying to test the Pyra GSM/GPS/3G module I had reconfigured the USB interface by mistake and therefore needed to run a different USB driver than CSC-ACM. It turned out that I need the "usbserial" driver. >>> >>> CDC-ACM >> >> oh, thanks. There is always at least one more typo in any document… >> >>> This file is an official description how to use it: Documentation/usb/usb-serial.txt But it is outdated. The parameters vendor= and product= are no longer available since ca. 3.12 which means that documentation is lagging behind quite some time. >>> >>> This is simply not true. The vendor and module parameters are still >>> there for usbserial (i.e. the generic driver). >>> >>> Would you mind updating this patch to document the sysfs-method as an >>> alternative instead? Using sysfs is preferred as that functionality can >>> be used with any usb-serial driver and the drivers can also be compiled >>> in. >> >> Hm. For me it did not work as described and therefore I searched for >> and found the solution cited below, which appears to confirm that it does >> not work any more. Only with these hints to use the /sys approach I >> could make it working immediately. >> >> Therefore I assumed that the documentation is outdated. >> >> But it might also be a hidden bug with the parameter passing. >> I had tested on 4.0-rc3. >> >> I can try a little more but it might be waste of time to find out why the old >> does not work, if there is a preferred method now. > > Please do try to figure out if it doesn't work as we have not deprecated > this functionality yet. It should still work. Good news: I have tried again (this time with 4.0-rc5) and was successful. I have no clue why it did not work with my tests with 4.0-rc3. Here is the log: > gta04 login: root > Password: > Last login: Thu Apr 2 14:06:13 UTC 2015 on ttyO2 > Linux gta04 4.0.0-rc5-gta04+ #1022 SMP Mon Mar 23 08:13:29 CET 2015 armv7l > > The programs included with the Debian GNU/Linux system are free software; > the exact distribution terms for each program are described in the > individual files in /usr/share/doc/*/copyright. > > Debian GNU/Linux comes with ABSOLUTELY NO WARRANTY, to the extent > permitted by applicable law. > root@gta04:~# lsusb > Bus 001 Device 002: ID 0424:3503 Standard Microsystems Corp. > Bus 001 Device 003: ID 0424:9730 Standard Microsystems Corp. > Bus 001 Device 001: ID 1d6b:0002 Linux Foundation 2.0 root hub > root@gta04:~# rfkill unblock wwan > [ 40.443892] wwan_on_off_rfkill_set_block: blocked: 0 > [ 40.449109] modem: set_power 0 > [ 40.452318] state 1 > [ 40.454687] modem: send impulse > [ 41.174775] modem: done > root@gta04:~# [ 42.874901] usb 1-2.2: new high-speed USB device number 4 > using ehci-omap > [ 43.008009] usb 1-2.2: New USB device found, idVendor=1e2d, idProduct=0053 > [ 43.015211] usb 1-2.2: New USB device strings: Mfr=3, Product=2, > SerialNumber=0 > [ 43.022859] usb 1-2.2: Product: PH8 > [ 43.026525] usb 1-2.2: Manufacturer: Cinterion > > root@gta04:~# lsusb > Bus 001 Device 002: ID 0424:3503 Standard Microsystems Corp. > Bus 001 Device 003: ID 0424:9730 Standard Microsystems Corp. > Bus 001 Device 001: ID 1d6b:0002 Linux Foundation 2.0 root hub > Bus 001 Device 004: ID 1e2d:0053 > root@gta04:~# insmod > /lib/modules/4.0.0-rc5-gta04+/kernel/drivers/usb/serial/usbserial.ko > vendor=0x1e2d product=0x0053 > [ 172.699426] usbcore: registered new interface driver usbserial > [ 172.705714] usbcore: registered new interface driver usbserial_generic > [ 172.715819] usbserial: USB Serial support registered for generic > [ 172.722237] usbserial_generic 1-2.2:1.0: The "generic" usb-serial driver > is only for testing and one-off prototypes. > [ 172.733328] usbserial_generic 1-2.2:1.0: Tell linux-...@vger.kernel.org to > add your device to a proper driver. > [ 172.743848] usbserial_generic 1-2.2:1.0: generic converter detected > [ 172.754681] usb 1-2.2: generic converter now attached to ttyUSB0 > [ 172.761543] usbserial_generic 1-2.2:1.1: The "generic" usb-serial driver > is only for testing and one-off prototypes. > [ 172.772640] usbserial_generic 1-2.2:1.1: Tell linux-...@vger.kernel.org to > add your device to a proper driver. > [ 172.783154] usbserial_generic 1-2.2:1.1: generic converter detected > [ 172.790765] usb 1-2.2: generic converter now attached to ttyUSB1 > [ 172.797235] usbserial_generic 1-2.2:1.2: The "generic" usb-serial driver > is only for testing and one-off prototypes. > [ 172.808296] usbserial_generic 1-2.2:1.2: Tell linux-...@vger.kernel.org to > add your device to a proper driver. > [
Re: [PATCH] Documentation: usb: serial: fixed how to provide vendor and product id
On Thu, 2 Apr 2015 12:35:09 +0200 Johan Hovold wrote: > This patch was incorrect and needed to be updated as discussed in the > thread. Can you drop it or is your tree immutable? Sigh...and I thought I was on top of that. Dropped, sorry for the confusion. jon -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] Documentation: usb: serial: fixed how to provide vendor and product id
Am 02.04.2015 um 12:35 schrieb Johan Hovold : > On Thu, Apr 02, 2015 at 12:12:37PM +0200, Jonathan Corbet wrote: >> On Sat, 28 Mar 2015 14:37:57 +0100 >> Marek Belisko wrote: >> >>> This patch changes the documentation file as needed. >> >> Applied to the docs tree (with the changelog typo fixed). > > This patch was incorrect and needed to be updated as discussed in the > thread. Can you drop it or is your tree immutable? Yes, please. I did not find time to test Johan’s valuable comments. BR, Nikolaus -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] Documentation: usb: serial: fixed how to provide vendor and product id
On Thu, Apr 02, 2015 at 12:12:37PM +0200, Jonathan Corbet wrote: > On Sat, 28 Mar 2015 14:37:57 +0100 > Marek Belisko wrote: > > > This patch changes the documentation file as needed. > > Applied to the docs tree (with the changelog typo fixed). This patch was incorrect and needed to be updated as discussed in the thread. Can you drop it or is your tree immutable? Thanks, Johan -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] Documentation: usb: serial: fixed how to provide vendor and product id
On Sat, 28 Mar 2015 14:37:57 +0100 Marek Belisko wrote: > This patch changes the documentation file as needed. Applied to the docs tree (with the changelog typo fixed). Thanks, jon -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] Documentation: usb: serial: fixed how to provide vendor and product id
On Sat, 28 Mar 2015 14:37:57 +0100 Marek Belisko ma...@goldelico.com wrote: This patch changes the documentation file as needed. Applied to the docs tree (with the changelog typo fixed). Thanks, jon -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] Documentation: usb: serial: fixed how to provide vendor and product id
Am 02.04.2015 um 12:35 schrieb Johan Hovold jo...@kernel.org: On Thu, Apr 02, 2015 at 12:12:37PM +0200, Jonathan Corbet wrote: On Sat, 28 Mar 2015 14:37:57 +0100 Marek Belisko ma...@goldelico.com wrote: This patch changes the documentation file as needed. Applied to the docs tree (with the changelog typo fixed). This patch was incorrect and needed to be updated as discussed in the thread. Can you drop it or is your tree immutable? Yes, please. I did not find time to test Johan’s valuable comments. BR, Nikolaus -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] Documentation: usb: serial: fixed how to provide vendor and product id
On Thu, Apr 02, 2015 at 12:12:37PM +0200, Jonathan Corbet wrote: On Sat, 28 Mar 2015 14:37:57 +0100 Marek Belisko ma...@goldelico.com wrote: This patch changes the documentation file as needed. Applied to the docs tree (with the changelog typo fixed). This patch was incorrect and needed to be updated as discussed in the thread. Can you drop it or is your tree immutable? Thanks, Johan -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] Documentation: usb: serial: fixed how to provide vendor and product id
On Thu, 2 Apr 2015 12:35:09 +0200 Johan Hovold jo...@kernel.org wrote: This patch was incorrect and needed to be updated as discussed in the thread. Can you drop it or is your tree immutable? Sigh...and I thought I was on top of that. Dropped, sorry for the confusion. jon -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] Documentation: usb: serial: fixed how to provide vendor and product id
Hi, Am 30.03.2015 um 12:08 schrieb Johan Hovold jo...@kernel.org: On Mon, Mar 30, 2015 at 11:32:39AM +0200, Dr. H. Nikolaus Schaller wrote: Hi Johan, Am 30.03.2015 um 09:45 schrieb Johan Hovold jo...@kernel.org: On Sat, Mar 28, 2015 at 02:37:57PM +0100, Marek Belisko wrote: From: H. Nikolaus Schaller h...@goldelico.com While trying to test the Pyra GSM/GPS/3G module I had reconfigured the USB interface by mistake and therefore needed to run a different USB driver than CSC-ACM. It turned out that I need the usbserial driver. CDC-ACM oh, thanks. There is always at least one more typo in any document… This file is an official description how to use it: Documentation/usb/usb-serial.txt But it is outdated. The parameters vendor= and product= are no longer available since ca. 3.12 which means that documentation is lagging behind quite some time. This is simply not true. The vendor and module parameters are still there for usbserial (i.e. the generic driver). Would you mind updating this patch to document the sysfs-method as an alternative instead? Using sysfs is preferred as that functionality can be used with any usb-serial driver and the drivers can also be compiled in. Hm. For me it did not work as described and therefore I searched for and found the solution cited below, which appears to confirm that it does not work any more. Only with these hints to use the /sys approach I could make it working immediately. Therefore I assumed that the documentation is outdated. But it might also be a hidden bug with the parameter passing. I had tested on 4.0-rc3. I can try a little more but it might be waste of time to find out why the old does not work, if there is a preferred method now. Please do try to figure out if it doesn't work as we have not deprecated this functionality yet. It should still work. Good news: I have tried again (this time with 4.0-rc5) and was successful. I have no clue why it did not work with my tests with 4.0-rc3. Here is the log: gta04 login: root Password: Last login: Thu Apr 2 14:06:13 UTC 2015 on ttyO2 Linux gta04 4.0.0-rc5-gta04+ #1022 SMP Mon Mar 23 08:13:29 CET 2015 armv7l The programs included with the Debian GNU/Linux system are free software; the exact distribution terms for each program are described in the individual files in /usr/share/doc/*/copyright. Debian GNU/Linux comes with ABSOLUTELY NO WARRANTY, to the extent permitted by applicable law. root@gta04:~# lsusb Bus 001 Device 002: ID 0424:3503 Standard Microsystems Corp. Bus 001 Device 003: ID 0424:9730 Standard Microsystems Corp. Bus 001 Device 001: ID 1d6b:0002 Linux Foundation 2.0 root hub root@gta04:~# rfkill unblock wwan [ 40.443892] wwan_on_off_rfkill_set_block: blocked: 0 [ 40.449109] modem: set_power 0 [ 40.452318] state 1 [ 40.454687] modem: send impulse [ 41.174775] modem: done root@gta04:~# [ 42.874901] usb 1-2.2: new high-speed USB device number 4 using ehci-omap [ 43.008009] usb 1-2.2: New USB device found, idVendor=1e2d, idProduct=0053 [ 43.015211] usb 1-2.2: New USB device strings: Mfr=3, Product=2, SerialNumber=0 [ 43.022859] usb 1-2.2: Product: PH8 [ 43.026525] usb 1-2.2: Manufacturer: Cinterion root@gta04:~# lsusb Bus 001 Device 002: ID 0424:3503 Standard Microsystems Corp. Bus 001 Device 003: ID 0424:9730 Standard Microsystems Corp. Bus 001 Device 001: ID 1d6b:0002 Linux Foundation 2.0 root hub Bus 001 Device 004: ID 1e2d:0053 root@gta04:~# insmod /lib/modules/4.0.0-rc5-gta04+/kernel/drivers/usb/serial/usbserial.ko vendor=0x1e2d product=0x0053 [ 172.699426] usbcore: registered new interface driver usbserial [ 172.705714] usbcore: registered new interface driver usbserial_generic [ 172.715819] usbserial: USB Serial support registered for generic [ 172.722237] usbserial_generic 1-2.2:1.0: The generic usb-serial driver is only for testing and one-off prototypes. [ 172.733328] usbserial_generic 1-2.2:1.0: Tell linux-...@vger.kernel.org to add your device to a proper driver. [ 172.743848] usbserial_generic 1-2.2:1.0: generic converter detected [ 172.754681] usb 1-2.2: generic converter now attached to ttyUSB0 [ 172.761543] usbserial_generic 1-2.2:1.1: The generic usb-serial driver is only for testing and one-off prototypes. [ 172.772640] usbserial_generic 1-2.2:1.1: Tell linux-...@vger.kernel.org to add your device to a proper driver. [ 172.783154] usbserial_generic 1-2.2:1.1: generic converter detected [ 172.790765] usb 1-2.2: generic converter now attached to ttyUSB1 [ 172.797235] usbserial_generic 1-2.2:1.2: The generic usb-serial driver is only for testing and one-off prototypes. [ 172.808296] usbserial_generic 1-2.2:1.2: Tell linux-...@vger.kernel.org to add your device to a proper driver. [ 172.818797] usbserial_generic 1-2.2:1.2: generic converter detected [ 172.826135] usb 1-2.2: generic converter now attached to ttyUSB2 [
Re: [PATCH] Documentation: usb: serial: fixed how to provide vendor and product id
On Thu, Apr 02, 2015 at 04:25:50PM +0200, Dr. H. Nikolaus Schaller wrote: But it is outdated. The parameters vendor= and product= are no longer available since ca. 3.12 which means that documentation is lagging behind quite some time. This is simply not true. The vendor and module parameters are still there for usbserial (i.e. the generic driver). Would you mind updating this patch to document the sysfs-method as an alternative instead? Using sysfs is preferred as that functionality can be used with any usb-serial driver and the drivers can also be compiled in. Hm. For me it did not work as described and therefore I searched for and found the solution cited below, which appears to confirm that it does not work any more. Only with these hints to use the /sys approach I could make it working immediately. Therefore I assumed that the documentation is outdated. But it might also be a hidden bug with the parameter passing. I had tested on 4.0-rc3. I can try a little more but it might be waste of time to find out why the old does not work, if there is a preferred method now. Please do try to figure out if it doesn't work as we have not deprecated this functionality yet. It should still work. Good news: I have tried again (this time with 4.0-rc5) and was successful. I have no clue why it did not work with my tests with 4.0-rc3. Great. So we will edit the patch to describe * that the sysfs method is in addition, and preferred * and works even if driver is compiled into the kernel (which I haven’t tested in my setup but assume it works as well) Ok? That should do it (and, yes, it works when the module is compiled in). Thanks, Johan -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] Documentation: usb: serial: fixed how to provide vendor and product id
On Mon, Mar 30, 2015 at 11:32:39AM +0200, Dr. H. Nikolaus Schaller wrote: > Hi Johan, > > Am 30.03.2015 um 09:45 schrieb Johan Hovold : > > > On Sat, Mar 28, 2015 at 02:37:57PM +0100, Marek Belisko wrote: > >> From: "H. Nikolaus Schaller" > >> > >> While trying to test the Pyra GSM/GPS/3G module I had reconfigured > >> the USB interface by mistake and therefore needed to run a different > >> USB driver than CSC-ACM. It turned out that I need the "usbserial" driver. > > > > CDC-ACM > > oh, thanks. There is always at least one more typo in any document… > > > > >> This file is an official description how to use it: > >> Documentation/usb/usb-serial.txt > >> > >> But it is outdated. The parameters vendor= and product= are no longer > >> available since ca. 3.12 which means that documentation is lagging behind > >> quite some time. > > > > This is simply not true. The vendor and module parameters are still > > there for usbserial (i.e. the generic driver). > > > > Would you mind updating this patch to document the sysfs-method as an > > alternative instead? Using sysfs is preferred as that functionality can > > be used with any usb-serial driver and the drivers can also be compiled > > in. > > Hm. For me it did not work as described and therefore I searched for > and found the solution cited below, which appears to confirm that it does > not work any more. Only with these hints to use the /sys approach I > could make it working immediately. > > Therefore I assumed that the documentation is outdated. > > But it might also be a hidden bug with the parameter passing. > I had tested on 4.0-rc3. > > I can try a little more but it might be waste of time to find out why the old > does not work, if there is a preferred method now. Please do try to figure out if it doesn't work as we have not deprecated this functionality yet. It should still work. Johan -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] Documentation: usb: serial: fixed how to provide vendor and product id
Hi Johan, Am 30.03.2015 um 09:45 schrieb Johan Hovold : > On Sat, Mar 28, 2015 at 02:37:57PM +0100, Marek Belisko wrote: >> From: "H. Nikolaus Schaller" >> >> While trying to test the Pyra GSM/GPS/3G module I had reconfigured >> the USB interface by mistake and therefore needed to run a different >> USB driver than CSC-ACM. It turned out that I need the "usbserial" driver. > > CDC-ACM oh, thanks. There is always at least one more typo in any document… > >> This file is an official description how to use it: >> Documentation/usb/usb-serial.txt >> >> But it is outdated. The parameters vendor= and product= are no longer >> available since ca. 3.12 which means that documentation is lagging behind >> quite some time. > > This is simply not true. The vendor and module parameters are still > there for usbserial (i.e. the generic driver). > > Would you mind updating this patch to document the sysfs-method as an > alternative instead? Using sysfs is preferred as that functionality can > be used with any usb-serial driver and the drivers can also be compiled > in. Hm. For me it did not work as described and therefore I searched for and found the solution cited below, which appears to confirm that it does not work any more. Only with these hints to use the /sys approach I could make it working immediately. Therefore I assumed that the documentation is outdated. But it might also be a hidden bug with the parameter passing. I had tested on 4.0-rc3. I can try a little more but it might be waste of time to find out why the old does not work, if there is a preferred method now. > >> Here was the solution: >> >> https://bbs.archlinux.org/viewtopic.php?id=175499 >> >> insmod usbserial vendor=0x product=0x >> >> becomes (first is vendor, second is product) >> >> modprobe usbserial >> echo >/sys/bus/usb-serial/drivers/generic/new_id >> >> This patch changes the documentation file as needed. >> >> Signed-off-by: H. Nikolaus Schaller >> Signed-off-by: Marek Belisko >> --- >> Documentation/usb/usb-serial.txt | 12 +++- >> 1 file changed, 7 insertions(+), 5 deletions(-) >> >> diff --git a/Documentation/usb/usb-serial.txt >> b/Documentation/usb/usb-serial.txt >> index 947fa62..9f184d8 100644 >> --- a/Documentation/usb/usb-serial.txt >> +++ b/Documentation/usb/usb-serial.txt >> @@ -466,11 +466,13 @@ Generic Serial driver >> is required of your device is that it has at least one bulk in endpoint, >> or one bulk out endpoint. >> >> - To enable the generic driver to recognize your device, build the driver >> - as a module and load it by the following invocation: >> -insmod usbserial vendor=0x product=0x >> - where the is replaced with the hex representation of your device's >> - vendor id and product id. >> + To enable the generic driver to recognize your device, configure >> + USB_SERIAL_GENERIC=y and build the driver as a module. Then load it by the > > You don't need to build the driver as a module when using the sysfs > method. Ok, that is right and my formulation is too narrow on that. So we just s/ as a module//. > >> + following invocation: >> +modprobe usbserial >> +echo >/sys/bus/usb-serial/drivers/generic/new_id >> + where the the first is replaced with the hex representation of your >> + device's vendor id and the second by the product id. >> >> This driver has been successfully used to connect to the NetChip USB >> development board, providing a way to develop USB firmware without > > Thanks, > Johan Thanks as well, Nikolaus -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] Documentation: usb: serial: fixed how to provide vendor and product id
On Sat, Mar 28, 2015 at 02:37:57PM +0100, Marek Belisko wrote: > From: "H. Nikolaus Schaller" > > While trying to test the Pyra GSM/GPS/3G module I had reconfigured > the USB interface by mistake and therefore needed to run a different > USB driver than CSC-ACM. It turned out that I need the "usbserial" driver. CDC-ACM > This file is an official description how to use it: > Documentation/usb/usb-serial.txt > > But it is outdated. The parameters vendor= and product= are no longer > available since ca. 3.12 which means that documentation is lagging behind > quite some time. This is simply not true. The vendor and module parameters are still there for usbserial (i.e. the generic driver). Would you mind updating this patch to document the sysfs-method as an alternative instead? Using sysfs is preferred as that functionality can be used with any usb-serial driver and the drivers can also be compiled in. > Here was the solution: > > https://bbs.archlinux.org/viewtopic.php?id=175499 > > insmod usbserial vendor=0x product=0x > > becomes (first is vendor, second is product) > > modprobe usbserial > echo >/sys/bus/usb-serial/drivers/generic/new_id > > This patch changes the documentation file as needed. > > Signed-off-by: H. Nikolaus Schaller > Signed-off-by: Marek Belisko > --- > Documentation/usb/usb-serial.txt | 12 +++- > 1 file changed, 7 insertions(+), 5 deletions(-) > > diff --git a/Documentation/usb/usb-serial.txt > b/Documentation/usb/usb-serial.txt > index 947fa62..9f184d8 100644 > --- a/Documentation/usb/usb-serial.txt > +++ b/Documentation/usb/usb-serial.txt > @@ -466,11 +466,13 @@ Generic Serial driver >is required of your device is that it has at least one bulk in endpoint, >or one bulk out endpoint. > > - To enable the generic driver to recognize your device, build the driver > - as a module and load it by the following invocation: > - insmod usbserial vendor=0x product=0x > - where the is replaced with the hex representation of your device's > - vendor id and product id. > + To enable the generic driver to recognize your device, configure > + USB_SERIAL_GENERIC=y and build the driver as a module. Then load it by the You don't need to build the driver as a module when using the sysfs method. > + following invocation: > + modprobe usbserial > + echo >/sys/bus/usb-serial/drivers/generic/new_id > + where the the first is replaced with the hex representation of your > + device's vendor id and the second by the product id. > >This driver has been successfully used to connect to the NetChip USB >development board, providing a way to develop USB firmware without Thanks, Johan -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] Documentation: usb: serial: fixed how to provide vendor and product id
On Sat, Mar 28, 2015 at 02:37:57PM +0100, Marek Belisko wrote: From: H. Nikolaus Schaller h...@goldelico.com While trying to test the Pyra GSM/GPS/3G module I had reconfigured the USB interface by mistake and therefore needed to run a different USB driver than CSC-ACM. It turned out that I need the usbserial driver. CDC-ACM This file is an official description how to use it: Documentation/usb/usb-serial.txt But it is outdated. The parameters vendor= and product= are no longer available since ca. 3.12 which means that documentation is lagging behind quite some time. This is simply not true. The vendor and module parameters are still there for usbserial (i.e. the generic driver). Would you mind updating this patch to document the sysfs-method as an alternative instead? Using sysfs is preferred as that functionality can be used with any usb-serial driver and the drivers can also be compiled in. Here was the solution: https://bbs.archlinux.org/viewtopic.php?id=175499 insmod usbserial vendor=0x product=0x becomes (first is vendor, second is product) modprobe usbserial echo /sys/bus/usb-serial/drivers/generic/new_id This patch changes the documentation file as needed. Signed-off-by: H. Nikolaus Schaller h...@goldelico.com Signed-off-by: Marek Belisko ma...@goldelico.com --- Documentation/usb/usb-serial.txt | 12 +++- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/Documentation/usb/usb-serial.txt b/Documentation/usb/usb-serial.txt index 947fa62..9f184d8 100644 --- a/Documentation/usb/usb-serial.txt +++ b/Documentation/usb/usb-serial.txt @@ -466,11 +466,13 @@ Generic Serial driver is required of your device is that it has at least one bulk in endpoint, or one bulk out endpoint. - To enable the generic driver to recognize your device, build the driver - as a module and load it by the following invocation: - insmod usbserial vendor=0x product=0x - where the is replaced with the hex representation of your device's - vendor id and product id. + To enable the generic driver to recognize your device, configure + USB_SERIAL_GENERIC=y and build the driver as a module. Then load it by the You don't need to build the driver as a module when using the sysfs method. + following invocation: + modprobe usbserial + echo /sys/bus/usb-serial/drivers/generic/new_id + where the the first is replaced with the hex representation of your + device's vendor id and the second by the product id. This driver has been successfully used to connect to the NetChip USB development board, providing a way to develop USB firmware without Thanks, Johan -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] Documentation: usb: serial: fixed how to provide vendor and product id
On Mon, Mar 30, 2015 at 11:32:39AM +0200, Dr. H. Nikolaus Schaller wrote: Hi Johan, Am 30.03.2015 um 09:45 schrieb Johan Hovold jo...@kernel.org: On Sat, Mar 28, 2015 at 02:37:57PM +0100, Marek Belisko wrote: From: H. Nikolaus Schaller h...@goldelico.com While trying to test the Pyra GSM/GPS/3G module I had reconfigured the USB interface by mistake and therefore needed to run a different USB driver than CSC-ACM. It turned out that I need the usbserial driver. CDC-ACM oh, thanks. There is always at least one more typo in any document… This file is an official description how to use it: Documentation/usb/usb-serial.txt But it is outdated. The parameters vendor= and product= are no longer available since ca. 3.12 which means that documentation is lagging behind quite some time. This is simply not true. The vendor and module parameters are still there for usbserial (i.e. the generic driver). Would you mind updating this patch to document the sysfs-method as an alternative instead? Using sysfs is preferred as that functionality can be used with any usb-serial driver and the drivers can also be compiled in. Hm. For me it did not work as described and therefore I searched for and found the solution cited below, which appears to confirm that it does not work any more. Only with these hints to use the /sys approach I could make it working immediately. Therefore I assumed that the documentation is outdated. But it might also be a hidden bug with the parameter passing. I had tested on 4.0-rc3. I can try a little more but it might be waste of time to find out why the old does not work, if there is a preferred method now. Please do try to figure out if it doesn't work as we have not deprecated this functionality yet. It should still work. Johan -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] Documentation: usb: serial: fixed how to provide vendor and product id
Hi Johan, Am 30.03.2015 um 09:45 schrieb Johan Hovold jo...@kernel.org: On Sat, Mar 28, 2015 at 02:37:57PM +0100, Marek Belisko wrote: From: H. Nikolaus Schaller h...@goldelico.com While trying to test the Pyra GSM/GPS/3G module I had reconfigured the USB interface by mistake and therefore needed to run a different USB driver than CSC-ACM. It turned out that I need the usbserial driver. CDC-ACM oh, thanks. There is always at least one more typo in any document… This file is an official description how to use it: Documentation/usb/usb-serial.txt But it is outdated. The parameters vendor= and product= are no longer available since ca. 3.12 which means that documentation is lagging behind quite some time. This is simply not true. The vendor and module parameters are still there for usbserial (i.e. the generic driver). Would you mind updating this patch to document the sysfs-method as an alternative instead? Using sysfs is preferred as that functionality can be used with any usb-serial driver and the drivers can also be compiled in. Hm. For me it did not work as described and therefore I searched for and found the solution cited below, which appears to confirm that it does not work any more. Only with these hints to use the /sys approach I could make it working immediately. Therefore I assumed that the documentation is outdated. But it might also be a hidden bug with the parameter passing. I had tested on 4.0-rc3. I can try a little more but it might be waste of time to find out why the old does not work, if there is a preferred method now. Here was the solution: https://bbs.archlinux.org/viewtopic.php?id=175499 insmod usbserial vendor=0x product=0x becomes (first is vendor, second is product) modprobe usbserial echo /sys/bus/usb-serial/drivers/generic/new_id This patch changes the documentation file as needed. Signed-off-by: H. Nikolaus Schaller h...@goldelico.com Signed-off-by: Marek Belisko ma...@goldelico.com --- Documentation/usb/usb-serial.txt | 12 +++- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/Documentation/usb/usb-serial.txt b/Documentation/usb/usb-serial.txt index 947fa62..9f184d8 100644 --- a/Documentation/usb/usb-serial.txt +++ b/Documentation/usb/usb-serial.txt @@ -466,11 +466,13 @@ Generic Serial driver is required of your device is that it has at least one bulk in endpoint, or one bulk out endpoint. - To enable the generic driver to recognize your device, build the driver - as a module and load it by the following invocation: -insmod usbserial vendor=0x product=0x - where the is replaced with the hex representation of your device's - vendor id and product id. + To enable the generic driver to recognize your device, configure + USB_SERIAL_GENERIC=y and build the driver as a module. Then load it by the You don't need to build the driver as a module when using the sysfs method. Ok, that is right and my formulation is too narrow on that. So we just s/ as a module//. + following invocation: +modprobe usbserial +echo /sys/bus/usb-serial/drivers/generic/new_id + where the the first is replaced with the hex representation of your + device's vendor id and the second by the product id. This driver has been successfully used to connect to the NetChip USB development board, providing a way to develop USB firmware without Thanks, Johan Thanks as well, Nikolaus -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] Documentation: usb: serial: fixed how to provide vendor and product id
On Thu, Mar 26, 2015 at 11:41:24AM +0100, Belisko Marek wrote: > On Thu, Mar 26, 2015 at 10:53 AM, Greg KH wrote: > > On Wed, Mar 25, 2015 at 09:53:34PM +0100, Marek Belisko wrote: > >> From: "H. Nikolaus Schaller" > >> > >> While trying to test the Pyra GSM/GPS/3G module I had reconfigured > >> the USB interface by mistake and therefore needed to run a different > >> USB driver than CSC-ACM. It turned out that I need the "usbserial" driver. > >> > >> This file is an official description how to use it: > >> Documentation/usb/usb-serial.txt > >> > >> But it is outdated. The parameters vendor= and product= are no longer > >> available since ca. 3.12 which means that documentation is lagging behind > >> quite some time. > >> > >> Here was the solution: > >> > >> https://bbs.archlinux.org/viewtopic.php?id=175499 > >> > >> insmod usbserial vendor=0x product=0x > >> > >> becomes (first is vendor, second is product) > >> > >> modprobe usbserial > >> echo >/sys/bus/usb-serial/drivers/generic/new_id > >> > >> This patch changes the documentation file as needed. > >> > >> Signed-off-by: H. Nikolaus Schaller > > > > You are forwarding on a patch from someone else, why didn't you also > > sign-off on it? Do you not agree with it? > I wasn't involved in patch development I just review it and forward. If you reviewed and forwarded it, why did you not add a signed-off-by: to it? Do you not think it is acceptable for merging? If not, then we shouldn't either :) thanks, greg k-h -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] Documentation: usb: serial: fixed how to provide vendor and product id
Am 26.03.2015 um 11:41 schrieb Belisko Marek : > On Thu, Mar 26, 2015 at 10:53 AM, Greg KH wrote: >> On Wed, Mar 25, 2015 at 09:53:34PM +0100, Marek Belisko wrote: >>> From: "H. Nikolaus Schaller" >>> >>> While trying to test the Pyra GSM/GPS/3G module I had reconfigured >>> the USB interface by mistake and therefore needed to run a different >>> USB driver than CSC-ACM. It turned out that I need the "usbserial" driver. >>> >>> This file is an official description how to use it: >>> Documentation/usb/usb-serial.txt >>> >>> But it is outdated. The parameters vendor= and product= are no longer >>> available since ca. 3.12 which means that documentation is lagging behind >>> quite some time. >>> >>> Here was the solution: >>> >>> https://bbs.archlinux.org/viewtopic.php?id=175499 >>> >>> insmod usbserial vendor=0x product=0x >>> >>> becomes (first is vendor, second is product) >>> >>> modprobe usbserial >>> echo >/sys/bus/usb-serial/drivers/generic/new_id >>> >>> This patch changes the documentation file as needed. >>> >>> Signed-off-by: H. Nikolaus Schaller >> >> You are forwarding on a patch from someone else, why didn't you also >> sign-off on it? Do you not agree with it? > I wasn’t involved in patch development I just review it and forward. Yes, I asked Marek to submit this patch (from our local repo). BR, Nikolaus >> >> thanks, >> >> greg k-h > > BR, > > marek > > -- > as simple and primitive as possible > - > Marek Belisko - OPEN-NANDRA > Freelance Developer > > Ruska Nova Ves 219 | Presov, 08005 Slovak Republic > Tel: +421 915 052 184 > skype: marekwhite > twitter: #opennandra > web: http://open-nandra.com -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] Documentation: usb: serial: fixed how to provide vendor and product id
On Thu, Mar 26, 2015 at 10:53 AM, Greg KH wrote: > On Wed, Mar 25, 2015 at 09:53:34PM +0100, Marek Belisko wrote: >> From: "H. Nikolaus Schaller" >> >> While trying to test the Pyra GSM/GPS/3G module I had reconfigured >> the USB interface by mistake and therefore needed to run a different >> USB driver than CSC-ACM. It turned out that I need the "usbserial" driver. >> >> This file is an official description how to use it: >> Documentation/usb/usb-serial.txt >> >> But it is outdated. The parameters vendor= and product= are no longer >> available since ca. 3.12 which means that documentation is lagging behind >> quite some time. >> >> Here was the solution: >> >> https://bbs.archlinux.org/viewtopic.php?id=175499 >> >> insmod usbserial vendor=0x product=0x >> >> becomes (first is vendor, second is product) >> >> modprobe usbserial >> echo >/sys/bus/usb-serial/drivers/generic/new_id >> >> This patch changes the documentation file as needed. >> >> Signed-off-by: H. Nikolaus Schaller > > You are forwarding on a patch from someone else, why didn't you also > sign-off on it? Do you not agree with it? I wasn't involved in patch development I just review it and forward. > > thanks, > > greg k-h BR, marek -- as simple and primitive as possible - Marek Belisko - OPEN-NANDRA Freelance Developer Ruska Nova Ves 219 | Presov, 08005 Slovak Republic Tel: +421 915 052 184 skype: marekwhite twitter: #opennandra web: http://open-nandra.com -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] Documentation: usb: serial: fixed how to provide vendor and product id
On Wed, Mar 25, 2015 at 09:53:34PM +0100, Marek Belisko wrote: > From: "H. Nikolaus Schaller" > > While trying to test the Pyra GSM/GPS/3G module I had reconfigured > the USB interface by mistake and therefore needed to run a different > USB driver than CSC-ACM. It turned out that I need the "usbserial" driver. > > This file is an official description how to use it: > Documentation/usb/usb-serial.txt > > But it is outdated. The parameters vendor= and product= are no longer > available since ca. 3.12 which means that documentation is lagging behind > quite some time. > > Here was the solution: > > https://bbs.archlinux.org/viewtopic.php?id=175499 > > insmod usbserial vendor=0x product=0x > > becomes (first is vendor, second is product) > > modprobe usbserial > echo >/sys/bus/usb-serial/drivers/generic/new_id > > This patch changes the documentation file as needed. > > Signed-off-by: H. Nikolaus Schaller You are forwarding on a patch from someone else, why didn't you also sign-off on it? Do you not agree with it? thanks, greg k-h -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] Documentation: usb: serial: fixed how to provide vendor and product id
On Wed, Mar 25, 2015 at 09:53:34PM +0100, Marek Belisko wrote: From: H. Nikolaus Schaller h...@goldelico.com While trying to test the Pyra GSM/GPS/3G module I had reconfigured the USB interface by mistake and therefore needed to run a different USB driver than CSC-ACM. It turned out that I need the usbserial driver. This file is an official description how to use it: Documentation/usb/usb-serial.txt But it is outdated. The parameters vendor= and product= are no longer available since ca. 3.12 which means that documentation is lagging behind quite some time. Here was the solution: https://bbs.archlinux.org/viewtopic.php?id=175499 insmod usbserial vendor=0x product=0x becomes (first is vendor, second is product) modprobe usbserial echo /sys/bus/usb-serial/drivers/generic/new_id This patch changes the documentation file as needed. Signed-off-by: H. Nikolaus Schaller h...@goldelico.com You are forwarding on a patch from someone else, why didn't you also sign-off on it? Do you not agree with it? thanks, greg k-h -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] Documentation: usb: serial: fixed how to provide vendor and product id
On Thu, Mar 26, 2015 at 11:41:24AM +0100, Belisko Marek wrote: On Thu, Mar 26, 2015 at 10:53 AM, Greg KH gre...@linuxfoundation.org wrote: On Wed, Mar 25, 2015 at 09:53:34PM +0100, Marek Belisko wrote: From: H. Nikolaus Schaller h...@goldelico.com While trying to test the Pyra GSM/GPS/3G module I had reconfigured the USB interface by mistake and therefore needed to run a different USB driver than CSC-ACM. It turned out that I need the usbserial driver. This file is an official description how to use it: Documentation/usb/usb-serial.txt But it is outdated. The parameters vendor= and product= are no longer available since ca. 3.12 which means that documentation is lagging behind quite some time. Here was the solution: https://bbs.archlinux.org/viewtopic.php?id=175499 insmod usbserial vendor=0x product=0x becomes (first is vendor, second is product) modprobe usbserial echo /sys/bus/usb-serial/drivers/generic/new_id This patch changes the documentation file as needed. Signed-off-by: H. Nikolaus Schaller h...@goldelico.com You are forwarding on a patch from someone else, why didn't you also sign-off on it? Do you not agree with it? I wasn't involved in patch development I just review it and forward. If you reviewed and forwarded it, why did you not add a signed-off-by: to it? Do you not think it is acceptable for merging? If not, then we shouldn't either :) thanks, greg k-h -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] Documentation: usb: serial: fixed how to provide vendor and product id
On Thu, Mar 26, 2015 at 10:53 AM, Greg KH gre...@linuxfoundation.org wrote: On Wed, Mar 25, 2015 at 09:53:34PM +0100, Marek Belisko wrote: From: H. Nikolaus Schaller h...@goldelico.com While trying to test the Pyra GSM/GPS/3G module I had reconfigured the USB interface by mistake and therefore needed to run a different USB driver than CSC-ACM. It turned out that I need the usbserial driver. This file is an official description how to use it: Documentation/usb/usb-serial.txt But it is outdated. The parameters vendor= and product= are no longer available since ca. 3.12 which means that documentation is lagging behind quite some time. Here was the solution: https://bbs.archlinux.org/viewtopic.php?id=175499 insmod usbserial vendor=0x product=0x becomes (first is vendor, second is product) modprobe usbserial echo /sys/bus/usb-serial/drivers/generic/new_id This patch changes the documentation file as needed. Signed-off-by: H. Nikolaus Schaller h...@goldelico.com You are forwarding on a patch from someone else, why didn't you also sign-off on it? Do you not agree with it? I wasn't involved in patch development I just review it and forward. thanks, greg k-h BR, marek -- as simple and primitive as possible - Marek Belisko - OPEN-NANDRA Freelance Developer Ruska Nova Ves 219 | Presov, 08005 Slovak Republic Tel: +421 915 052 184 skype: marekwhite twitter: #opennandra web: http://open-nandra.com -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] Documentation: usb: serial: fixed how to provide vendor and product id
Am 26.03.2015 um 11:41 schrieb Belisko Marek marek.beli...@gmail.com: On Thu, Mar 26, 2015 at 10:53 AM, Greg KH gre...@linuxfoundation.org wrote: On Wed, Mar 25, 2015 at 09:53:34PM +0100, Marek Belisko wrote: From: H. Nikolaus Schaller h...@goldelico.com While trying to test the Pyra GSM/GPS/3G module I had reconfigured the USB interface by mistake and therefore needed to run a different USB driver than CSC-ACM. It turned out that I need the usbserial driver. This file is an official description how to use it: Documentation/usb/usb-serial.txt But it is outdated. The parameters vendor= and product= are no longer available since ca. 3.12 which means that documentation is lagging behind quite some time. Here was the solution: https://bbs.archlinux.org/viewtopic.php?id=175499 insmod usbserial vendor=0x product=0x becomes (first is vendor, second is product) modprobe usbserial echo /sys/bus/usb-serial/drivers/generic/new_id This patch changes the documentation file as needed. Signed-off-by: H. Nikolaus Schaller h...@goldelico.com You are forwarding on a patch from someone else, why didn't you also sign-off on it? Do you not agree with it? I wasn’t involved in patch development I just review it and forward. Yes, I asked Marek to submit this patch (from our local repo). BR, Nikolaus thanks, greg k-h BR, marek -- as simple and primitive as possible - Marek Belisko - OPEN-NANDRA Freelance Developer Ruska Nova Ves 219 | Presov, 08005 Slovak Republic Tel: +421 915 052 184 skype: marekwhite twitter: #opennandra web: http://open-nandra.com -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
