<feed xmlns='http://www.w3.org/2005/Atom'>
<title>linux.git/drivers/usb/serial/ipaq.c, branch v4.12</title>
<subtitle>Linux kernel source tree</subtitle>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux.git/'/>
<entry>
<title>USB: serial: ipaq: always register a single port</title>
<updated>2017-03-28T09:00:09+00:00</updated>
<author>
<name>Johan Hovold</name>
<email>johan@kernel.org</email>
</author>
<published>2017-03-16T16:13:40+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux.git/commit/?id=204cc473bc896134870fda21cd279774b3ead024'/>
<id>204cc473bc896134870fda21cd279774b3ead024</id>
<content type='text'>
Use the calc_num_ports callback to ignore unused endpoints.

The driver binds to any interface with at least one bulk-in and one
bulk-out endpoint, but some devices can have three or more endpoints of
which only either the first or second pair of endpoints is needed.

This avoids allocating resources for unused endpoints, and specifically
a port is no longer registered for the unused first endpoint pair when
there are more than three endpoints.

Signed-off-by: Johan Hovold &lt;johan@kernel.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Use the calc_num_ports callback to ignore unused endpoints.

The driver binds to any interface with at least one bulk-in and one
bulk-out endpoint, but some devices can have three or more endpoints of
which only either the first or second pair of endpoints is needed.

This avoids allocating resources for unused endpoints, and specifically
a port is no longer registered for the unused first endpoint pair when
there are more than three endpoints.

Signed-off-by: Johan Hovold &lt;johan@kernel.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>USB: serial: ipaq: use calc_num_endpoints to verify endpoints</title>
<updated>2017-03-28T09:00:09+00:00</updated>
<author>
<name>Johan Hovold</name>
<email>johan@kernel.org</email>
</author>
<published>2017-03-16T16:13:39+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux.git/commit/?id=03b72aecadc2d7b0e8f200ec70b853e39a7e55da'/>
<id>03b72aecadc2d7b0e8f200ec70b853e39a7e55da</id>
<content type='text'>
Use the calc_num_ports rather than attach callback to determine which
interface to bind to in order to avoid allocating port-resources for
interfaces that won't be bound.

Signed-off-by: Johan Hovold &lt;johan@kernel.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Use the calc_num_ports rather than attach callback to determine which
interface to bind to in order to avoid allocating port-resources for
interfaces that won't be bound.

Signed-off-by: Johan Hovold &lt;johan@kernel.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>USB: serial: allow subdrivers to modify port-endpoint mapping</title>
<updated>2017-03-28T08:54:39+00:00</updated>
<author>
<name>Johan Hovold</name>
<email>johan@kernel.org</email>
</author>
<published>2017-03-16T16:13:30+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux.git/commit/?id=07814246dd5530860ef758fd9b2b5f2e26472aa2'/>
<id>07814246dd5530860ef758fd9b2b5f2e26472aa2</id>
<content type='text'>
Allow subdrivers to modify the port-endpoint mapping by passing the
endpoint descriptors to calc_num_ports.

The callback can now also be used to verify that the required endpoints
exists and abort probing otherwise.

This will allow us to get rid of a few hacks in subdrivers that are
already modifying the port-endpoint mapping (or aborting probe due to
missing endpoints), but only after the port structures have been setup.

Signed-off-by: Johan Hovold &lt;johan@kernel.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Allow subdrivers to modify the port-endpoint mapping by passing the
endpoint descriptors to calc_num_ports.

The callback can now also be used to verify that the required endpoints
exists and abort probing otherwise.

This will allow us to get rid of a few hacks in subdrivers that are
already modifying the port-endpoint mapping (or aborting probe due to
missing endpoints), but only after the port structures have been setup.

Signed-off-by: Johan Hovold &lt;johan@kernel.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>USB: ipaq.c: fix a timeout loop</title>
<updated>2015-12-18T17:30:34+00:00</updated>
<author>
<name>Dan Carpenter</name>
<email>dan.carpenter@oracle.com</email>
</author>
<published>2015-12-16T11:06:37+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux.git/commit/?id=abdc9a3b4bac97add99e1d77dc6d28623afe682b'/>
<id>abdc9a3b4bac97add99e1d77dc6d28623afe682b</id>
<content type='text'>
The code expects the loop to end with "retries" set to zero but, because
it is a post-op, it will end set to -1.  I have fixed this by moving the
decrement inside the loop.

Fixes: 014aa2a3c32e ('USB: ipaq: minor ipaq_open() cleanup.')
Signed-off-by: Dan Carpenter &lt;dan.carpenter@oracle.com&gt;
Cc: stable &lt;stable@vger.kernel.org&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The code expects the loop to end with "retries" set to zero but, because
it is a post-op, it will end set to -1.  I have fixed this by moving the
decrement inside the loop.

Fixes: 014aa2a3c32e ('USB: ipaq: minor ipaq_open() cleanup.')
Signed-off-by: Dan Carpenter &lt;dan.carpenter@oracle.com&gt;
Cc: stable &lt;stable@vger.kernel.org&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>usb: delete non-required instances of include &lt;linux/init.h&gt;</title>
<updated>2014-01-08T23:01:39+00:00</updated>
<author>
<name>Paul Gortmaker</name>
<email>paul.gortmaker@windriver.com</email>
</author>
<published>2014-01-08T16:08:26+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux.git/commit/?id=803a536243b3a1ed2289f41897b11b72bd083309'/>
<id>803a536243b3a1ed2289f41897b11b72bd083309</id>
<content type='text'>
None of these files are actually using any __init type directives
and hence don't need to include &lt;linux/init.h&gt;.  Most are just a
left over from __devinit and __cpuinit removal, or simply due to
code getting copied from one driver to the next.

Signed-off-by: Paul Gortmaker &lt;paul.gortmaker@windriver.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
None of these files are actually using any __init type directives
and hence don't need to include &lt;linux/init.h&gt;.  Most are just a
left over from __devinit and __cpuinit removal, or simply due to
code getting copied from one driver to the next.

Signed-off-by: Paul Gortmaker &lt;paul.gortmaker@windriver.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>USB: serial: constify device-id tables</title>
<updated>2014-01-03T20:31:46+00:00</updated>
<author>
<name>Johan Hovold</name>
<email>jhovold@gmail.com</email>
</author>
<published>2013-12-29T18:22:54+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux.git/commit/?id=5c6b98dd0437ba8c1b515bf11357784335613d65'/>
<id>5c6b98dd0437ba8c1b515bf11357784335613d65</id>
<content type='text'>
Declare device-id tables as const where possible.

Signed-off-by: Johan Hovold &lt;jhovold@gmail.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Declare device-id tables as const where possible.

Signed-off-by: Johan Hovold &lt;jhovold@gmail.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>USB: serial: remove driver version information</title>
<updated>2012-10-31T19:48:06+00:00</updated>
<author>
<name>Johan Hovold</name>
<email>jhovold@gmail.com</email>
</author>
<published>2012-10-31T10:59:52+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux.git/commit/?id=bcbec053d2197031d04b8e040c61695b5d7a949d'/>
<id>bcbec053d2197031d04b8e040c61695b5d7a949d</id>
<content type='text'>
Remove all MODULE_VERSION macros and driver-version information (except
for garmin_gps which uses it in a status reply).

It is the kernel version that matters and not some private version
scheme which rarely even gets updated.

Signed-off-by: Johan Hovold &lt;jhovold@gmail.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Remove all MODULE_VERSION macros and driver-version information (except
for garmin_gps which uses it in a status reply).

It is the kernel version that matters and not some private version
scheme which rarely even gets updated.

Signed-off-by: Johan Hovold &lt;jhovold@gmail.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>USB: serial: ipaq.c: remove debug module parameter</title>
<updated>2012-09-14T19:31:30+00:00</updated>
<author>
<name>Greg Kroah-Hartman</name>
<email>gregkh@linuxfoundation.org</email>
</author>
<published>2012-09-14T19:31:30+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux.git/commit/?id=06e454c40e67df631e2020571ab4e089d9fe0026'/>
<id>06e454c40e67df631e2020571ab4e089d9fe0026</id>
<content type='text'>
Now that the dbg() macro is no longer being used in the driver,
the debug module parameter doesn't do anything at all.  So remove
it so as to not confuse people.

CC: Rusty Russell &lt;rusty@rustcorp.com.au&gt;
CC: Johan Hovold &lt;jhovold@gmail.com&gt;
CC: Alan Stern &lt;stern@rowland.harvard.edu&gt;
CC: Mauro Carvalho Chehab &lt;mchehab@redhat.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Now that the dbg() macro is no longer being used in the driver,
the debug module parameter doesn't do anything at all.  So remove
it so as to not confuse people.

CC: Rusty Russell &lt;rusty@rustcorp.com.au&gt;
CC: Johan Hovold &lt;jhovold@gmail.com&gt;
CC: Alan Stern &lt;stern@rowland.harvard.edu&gt;
CC: Mauro Carvalho Chehab &lt;mchehab@redhat.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>USB: ipaq.c: remove dbg() usage</title>
<updated>2012-05-15T23:27:20+00:00</updated>
<author>
<name>Greg Kroah-Hartman</name>
<email>gregkh@linuxfoundation.org</email>
</author>
<published>2012-05-15T23:27:20+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux.git/commit/?id=f73e1ffa27434772f57a9a7daaa508d06c56fddf'/>
<id>f73e1ffa27434772f57a9a7daaa508d06c56fddf</id>
<content type='text'>
dbg() is a usb-serial specific macro.  This patch converts
the ipaq.c driver to use dev_dbg() instead to tie into the
dynamic debug infrastructure.

CC: Rusty Russell &lt;rusty@rustcorp.com.au&gt;
CC: Johan Hovold &lt;jhovold@gmail.com&gt;
CC: Alan Stern &lt;stern@rowland.harvard.edu&gt;
CC: Mauro Carvalho Chehab &lt;mchehab@redhat.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
dbg() is a usb-serial specific macro.  This patch converts
the ipaq.c driver to use dev_dbg() instead to tie into the
dynamic debug infrastructure.

CC: Rusty Russell &lt;rusty@rustcorp.com.au&gt;
CC: Johan Hovold &lt;jhovold@gmail.com&gt;
CC: Alan Stern &lt;stern@rowland.harvard.edu&gt;
CC: Mauro Carvalho Chehab &lt;mchehab@redhat.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>USB: serial: rework usb_serial_register/deregister_drivers()</title>
<updated>2012-05-08T22:46:14+00:00</updated>
<author>
<name>Greg Kroah-Hartman</name>
<email>gregkh@linuxfoundation.org</email>
</author>
<published>2012-05-08T22:46:14+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux.git/commit/?id=68e24113457e437b1576670f2419b77ed0531e9e'/>
<id>68e24113457e437b1576670f2419b77ed0531e9e</id>
<content type='text'>
This reworks the usb_serial_register_drivers() and
usb_serial_deregister_drivers() to not need a pointer to a struct
usb_driver anymore.  The usb_driver structure is now created dynamically
and registered and unregistered as needed.

This saves lines of code in each usb-serial driver.  All in-kernel users
of these functions were also fixed up at this time.  The pl2303 driver
was tested that everything worked properly.

Thanks for the idea to do this from Alan Stern.

Cc: Adhir Ramjiawan &lt;adhirramjiawan0@gmail.com&gt;
Cc: Alan Stern &lt;stern@rowland.harvard.edu&gt;
Cc: Al Borchers &lt;alborchers@steinerpoint.com&gt;
Cc: Aleksey Babahin &lt;tamerlan311@gmail.com&gt;
Cc: Andrew Morton &lt;akpm@linux-foundation.org&gt;
Cc: Andrew Worsley &lt;amworsley@gmail.com&gt;
Cc: Bart Hartgers &lt;bart.hartgers@gmail.com&gt;
Cc: Bill Pemberton &lt;wfp5p@virginia.edu&gt;
Cc: Dan Carpenter &lt;error27@gmail.com&gt;
Cc: Dan Williams &lt;dcbw@redhat.com&gt;
Cc: Donald Lee &lt;donald@asix.com.tw&gt;
Cc: Eric Dumazet &lt;eric.dumazet@gmail.com&gt;
Cc: "Eric W. Biederman" &lt;ebiederm@xmission.com&gt;
Cc: Felipe Balbi &lt;balbi@ti.com&gt;
Cc: Gary Brubaker &lt;xavyer@ix.netcom.com&gt;
Cc: Jesper Juhl &lt;jj@chaosbits.net&gt;
Cc: Jiri Kosina &lt;jkosina@suse.cz&gt;
Cc: Johan Hovold &lt;jhovold@gmail.com&gt;
Cc: Julia Lawall &lt;julia@diku.dk&gt;
Cc: Kautuk Consul &lt;consul.kautuk@gmail.com&gt;
Cc: Kuninori Morimoto &lt;kuninori.morimoto.gx@renesas.com&gt;
Cc: Lonnie Mendez &lt;dignome@gmail.com&gt;
Cc: Matthias Bruestle and Harald Welte &lt;support@reiner-sct.com&gt;
Cc: Matthias Urlichs &lt;smurf@smurf.noris.de&gt;
Cc: Mauro Carvalho Chehab &lt;mchehab@redhat.com&gt;
Cc: Michal Sroczynski &lt;msroczyn@gmail.com&gt;
Cc: "Michał Wróbel" &lt;michal.wrobel@flytronic.pl&gt;
Cc: Oliver Neukum &lt;oliver@neukum.name&gt;
Cc: Paul Gortmaker &lt;paul.gortmaker@windriver.com&gt;
Cc: Peter Berger &lt;pberger@brimson.com&gt;
Cc: Preston Fick &lt;preston.fick@silabs.com&gt;
Cc: "Rafael J. Wysocki" &lt;rjw@sisk.pl&gt;
Cc: Rigbert Hamisch &lt;rigbert@gmx.de&gt;
Cc: Rusty Russell &lt;rusty@rustcorp.com.au&gt;
Cc: Simon Arlott &lt;simon@fire.lp0.eu&gt;
Cc: Support Department &lt;support@connecttech.com&gt;
Cc: Thomas Tuttle &lt;ttuttle@chromium.org&gt;
Cc: Uwe Bonnes &lt;bon@elektron.ikp.physik.tu-darmstadt.de&gt;
Cc: Wang YanQing &lt;Udknight@gmail.com&gt;
Cc: William Greathouse &lt;wgreathouse@smva.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This reworks the usb_serial_register_drivers() and
usb_serial_deregister_drivers() to not need a pointer to a struct
usb_driver anymore.  The usb_driver structure is now created dynamically
and registered and unregistered as needed.

This saves lines of code in each usb-serial driver.  All in-kernel users
of these functions were also fixed up at this time.  The pl2303 driver
was tested that everything worked properly.

Thanks for the idea to do this from Alan Stern.

Cc: Adhir Ramjiawan &lt;adhirramjiawan0@gmail.com&gt;
Cc: Alan Stern &lt;stern@rowland.harvard.edu&gt;
Cc: Al Borchers &lt;alborchers@steinerpoint.com&gt;
Cc: Aleksey Babahin &lt;tamerlan311@gmail.com&gt;
Cc: Andrew Morton &lt;akpm@linux-foundation.org&gt;
Cc: Andrew Worsley &lt;amworsley@gmail.com&gt;
Cc: Bart Hartgers &lt;bart.hartgers@gmail.com&gt;
Cc: Bill Pemberton &lt;wfp5p@virginia.edu&gt;
Cc: Dan Carpenter &lt;error27@gmail.com&gt;
Cc: Dan Williams &lt;dcbw@redhat.com&gt;
Cc: Donald Lee &lt;donald@asix.com.tw&gt;
Cc: Eric Dumazet &lt;eric.dumazet@gmail.com&gt;
Cc: "Eric W. Biederman" &lt;ebiederm@xmission.com&gt;
Cc: Felipe Balbi &lt;balbi@ti.com&gt;
Cc: Gary Brubaker &lt;xavyer@ix.netcom.com&gt;
Cc: Jesper Juhl &lt;jj@chaosbits.net&gt;
Cc: Jiri Kosina &lt;jkosina@suse.cz&gt;
Cc: Johan Hovold &lt;jhovold@gmail.com&gt;
Cc: Julia Lawall &lt;julia@diku.dk&gt;
Cc: Kautuk Consul &lt;consul.kautuk@gmail.com&gt;
Cc: Kuninori Morimoto &lt;kuninori.morimoto.gx@renesas.com&gt;
Cc: Lonnie Mendez &lt;dignome@gmail.com&gt;
Cc: Matthias Bruestle and Harald Welte &lt;support@reiner-sct.com&gt;
Cc: Matthias Urlichs &lt;smurf@smurf.noris.de&gt;
Cc: Mauro Carvalho Chehab &lt;mchehab@redhat.com&gt;
Cc: Michal Sroczynski &lt;msroczyn@gmail.com&gt;
Cc: "Michał Wróbel" &lt;michal.wrobel@flytronic.pl&gt;
Cc: Oliver Neukum &lt;oliver@neukum.name&gt;
Cc: Paul Gortmaker &lt;paul.gortmaker@windriver.com&gt;
Cc: Peter Berger &lt;pberger@brimson.com&gt;
Cc: Preston Fick &lt;preston.fick@silabs.com&gt;
Cc: "Rafael J. Wysocki" &lt;rjw@sisk.pl&gt;
Cc: Rigbert Hamisch &lt;rigbert@gmx.de&gt;
Cc: Rusty Russell &lt;rusty@rustcorp.com.au&gt;
Cc: Simon Arlott &lt;simon@fire.lp0.eu&gt;
Cc: Support Department &lt;support@connecttech.com&gt;
Cc: Thomas Tuttle &lt;ttuttle@chromium.org&gt;
Cc: Uwe Bonnes &lt;bon@elektron.ikp.physik.tu-darmstadt.de&gt;
Cc: Wang YanQing &lt;Udknight@gmail.com&gt;
Cc: William Greathouse &lt;wgreathouse@smva.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</pre>
</div>
</content>
</entry>
</feed>
