Re: [PATCH] Documentation: usb: serial: fixed how to provide vendor and product id

[Johan Hovold <johan@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 <johan@kernel.org>:
> 
> > On Sat, Mar 28, 2015 at 02:37:57PM +0100, Marek Belisko wrote:
> >> From: "H. Nikolaus Schaller" <hns@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