public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Greg KH <greg@kroah.com>
To: Michal Nazarewicz <m.nazarewicz@samsung.com>
Cc: linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCHv8 03/13] USB: gadget: composite: Better string override handling
Date: Wed, 1 Sep 2010 14:10:17 -0700	[thread overview]
Message-ID: <20100901211017.GA8173@kroah.com> (raw)
In-Reply-To: <48e2d6507f004252a012cf25bc0a06e5cb69808f.1281625724.git.m.nazarewicz@samsung.com>

On Thu, Aug 12, 2010 at 05:43:46PM +0200, Michal Nazarewicz wrote:
> The iManufatcurer, iProduct and iSerialNumber composite module
> parameters were only used when the gadget driver registers
> strings for manufacturer, product and serial number.  If the
> gadget never bothered to set corresponding fields in USB device
> descriptors those module parameters are ignored.
> 
> This commit makes the parameters work even if the strings ID
> have not been assigned.  It also changes the way IDs are
> overridden -- what IDs are overridden is now saved in
> usb_composite_dev structure -- which makes it unnecessary to
> modify the string tables the way previous code did.
> 
> The commit also adds a iProduct and iManufatcurer fields to the
> usb_composite_device structure.  If they are set, appropriate
> strings are reserved and added to device descriptor.  This makes
> it unnecessary for gadget drivers to maintain code for setting
> those.  If iProduct is not set it defaults to
> usb_composite_device::name; if iManufatcurer is not set
> a default "<system> <release> with <gadget-name>" is used.
> 
> The last thing is that if needs_serial field of
> usb_composite_device is set and user failed to provided
> iSerialNumber parameter a warning is issued.
> 
> Signed-off-by: Michal Nazarewicz <m.nazarewicz@samsung.com>
> Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>

This patch doesn't apply on the latest tree, care to respin it, and the
ones after it as well?

thanks,

greg k-h

  parent reply	other threads:[~2010-09-01 22:13 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-08-12 15:43 [PATCHv8 00/13] Various USB improvements Michal Nazarewicz
2010-08-12 15:43 ` [PATCHv8 01/13] USB: gadget: g_multi: fixed vendor and product ID Michal Nazarewicz
2010-08-12 15:43   ` [PATCHv8 02/13] USB: gadget: g_ffs: " Michal Nazarewicz
2010-08-12 15:43     ` [PATCHv8 03/13] USB: gadget: composite: Better string override handling Michal Nazarewicz
2010-08-12 15:43       ` [PATCHv8 04/13] USB: gadget: mass_storage: moved strings handling code to composite Michal Nazarewicz
2010-08-12 15:43         ` [PATCHv8 05/13] USB: gadget: functionfs: code cleanup Michal Nazarewicz
2010-08-12 15:43           ` [PATCHv8 06/13] USB: gadget: g_multi: moved strings handling code to composite Michal Nazarewicz
2010-08-12 15:43             ` [PATCHv8 07/13] usb: gadget: storage: remove nofua file when unbinding Michal Nazarewicz
2010-08-12 15:43               ` [PATCHv8 08/13] usb: gadget: mass_storage: optional SCSI WRITE FUA bit Michal Nazarewicz
2010-08-12 15:43                 ` [PATCHv8 09/13] Revert "USB: gadget: section mismatch warning fixed" Michal Nazarewicz
2010-08-12 15:43                   ` [PATCHv8 10/13] usb gadget: don't save bind callback in struct usb_gadget_driver Michal Nazarewicz
2010-08-12 15:43                     ` [PATCHv8 11/13] usb gadget: don't save bind callback in struct usb_composite_driver Michal Nazarewicz
2010-08-12 15:43                       ` [PATCHv8 12/13] usb gadget: don't save bind callback in struct usb_configuration Michal Nazarewicz
2010-08-12 15:43                         ` [PATCHv8 13/13] init.h: add some more documentation to __ref* tags Michal Nazarewicz
2010-08-12 15:48                           ` Sam Ravnborg
2010-08-12 15:45                       ` [PATCHv8 11/13] usb gadget: don't save bind callback in struct usb_composite_driver Uwe Kleine-König
2010-08-12 22:34         ` [PATCHv8 04/13] USB: gadget: mass_storage: moved strings handling code to composite David Brownell
2010-08-13 15:15           ` Alan Stern
2010-08-13 20:30             ` David Brownell
2010-08-14  0:23               ` Alan Stern
2010-08-18  1:26                 ` Michał Nazarewicz
2010-09-01 21:10       ` Greg KH [this message]
2010-09-01 21:12         ` [PATCHv8 03/13] USB: gadget: composite: Better string override handling Greg KH
2010-09-02  0:57           ` Michał Nazarewicz

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20100901211017.GA8173@kroah.com \
    --to=greg@kroah.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=m.nazarewicz@samsung.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox