All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Michał Nazarewicz" <m.nazarewicz@samsung.com>
To: Alan Stern <stern@rowland.harvard.edu>
Cc: Greg KH <greg@kroah.com>, David Brownell <david-b@pacbell.net>,
	Kyungmin Park <kyungmin.park@samsung.com>,
	Marek Szyprowski <m.szyprowski@samsung.com>,
	Kernel development list <linux-kernel@vger.kernel.org>,
	Dries Van Puymbroeck <Dries.VanPuymbroeck@dekimo.com>
Subject: Re: [PATCHv3 1/3] USB: gadget: mass/file storage: set serial number
Date: Mon, 19 Jul 2010 19:21:57 +0200	[thread overview]
Message-ID: <op.vf3o6v067p4s8u@pikus> (raw)
In-Reply-To: <Pine.LNX.4.44L0.1007191256480.1657-100000@iolanthe.rowland.org>

On Mon, 19 Jul 2010 19:06:32 +0200, Alan Stern <stern@rowland.harvard.edu> wrote:
>> 1/3: http://lkml.org/lkml/2010/7/8/317

> Ah, yes.  My personal taste would be to write fsg_string_serial_fill_n
> as an inline routine instead of as a macro, and not try to make it
> separate from fsg_string_serial_fill.

>> 2/3: http://lkml.org/lkml/2010/7/8/318

> I have only one objection to this patch: The new parameter's name
> should be "serial", not "serial_parm".  (Compare it to all the other
> parameter names.)

Will change the two tomorrow.

> As for the discussion between you and David...  I haven't tried to
> follow the details very carefully.  I get the impression that the two
> of you are talking past each other instead of coming to grips with a
> genuine issue.

I'm not entirely sure of what the issue with the patches is really.
It merely adds a serial number to the gadgets using MSF and that's all.

In any event, I am sure we will come to an agreement at one point. :)

-- 
Best regards,                                        _     _
| Humble Liege of Serenely Enlightened Majesty of  o' \,=./ `o
| Computer Science,  Michał "mina86" Nazarewicz       (o o)
+----[mina86*mina86.com]---[mina86*jabber.org]----ooO--(_)--Ooo--

  reply	other threads:[~2010-07-19 17:20 UTC|newest]

Thread overview: 51+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-07-01  9:17 [PATCH 1/2] USB: gadget: mass/file storage: set serial number Michal Nazarewicz
2010-07-01  9:17 ` [PATCH 2/2] " Michal Nazarewicz
2010-07-01 10:27 ` [PATCH 1/2] " Michał Nazarewicz
2010-07-01 13:42 ` [PATCHv2 " Michal Nazarewicz
2010-07-01 13:42   ` [PATCHv2 2/2] USB: gadget: g_multi: code clean up and refactoring Michal Nazarewicz
2010-07-08 18:34   ` [PATCHv2 1/2] USB: gadget: mass/file storage: set serial number Greg KH
2010-07-08 18:58     ` Michał Nazarewicz
2010-07-08 19:03       ` Greg KH
2010-07-08 19:31       ` David Brownell
2010-07-08 20:02         ` Michał Nazarewicz
2010-07-08 20:20           ` David Brownell
2010-07-08 20:27             ` Michał Nazarewicz
2010-07-08 20:52     ` [PATCHv3 1/3] " Michal Nazarewicz
2010-07-08 20:52       ` [PATCHv3 2/3] USB: Add a serial number parameter to g_file_storage module Michal Nazarewicz
2010-07-08 20:52         ` [PATCHv3 3/3] USB: gadget: g_multi: code clean up and refactoring Michal Nazarewicz
2010-07-09 19:04       ` [PATCHv3 1/3] USB: gadget: mass/file storage: set serial number Greg KH
2010-07-17 23:01         ` Michał Nazarewicz
2010-07-17 23:57           ` David Brownell
2010-07-19  8:58             ` Michał Nazarewicz
2010-07-19 10:08               ` David Brownell
2010-07-19 12:07                 ` Michał Nazarewicz
2010-07-19 14:19                   ` Alan Stern
2010-07-19 15:02                     ` Michał Nazarewicz
2010-07-19 16:14                       ` Alan Stern
2010-07-19 16:26                         ` Michał Nazarewicz
2010-07-19 17:06                           ` Alan Stern
2010-07-19 17:21                             ` Michał Nazarewicz [this message]
2010-07-19 17:41                               ` David Brownell
2010-07-20  8:41                                 ` Michał Nazarewicz
2010-07-20 14:07                                   ` Alan Stern
2010-07-20 14:43                                     ` Michał Nazarewicz
2010-07-20 15:01                                   ` David Brownell
2010-07-20 15:45                                     ` Michał Nazarewicz
2010-07-19 18:37                               ` Alan Stern
2010-07-20  9:57                             ` Michał Nazarewicz
2010-07-20 14:08                               ` Alan Stern
2010-07-20 14:40                                 ` Michał Nazarewicz
2010-07-20 14:52                                   ` David Brownell
2010-07-20 15:02                                   ` Alan Stern
2010-07-19 14:44                   ` David Brownell
2010-07-19 15:01                     ` Michał Nazarewicz
2010-07-22 12:16       ` [PATCHv4 1/5] USB: gadget: composite: Better string override handling Michal Nazarewicz
2010-07-22 12:16         ` [PATCHv4 2/5] USB: gadget: Use new composite features in some gadgets Michal Nazarewicz
2010-07-22 12:16           ` [PATCHv4 3/5] USB: gadget: g_multi: code clean up and refactoring Michal Nazarewicz
2010-07-22 12:16             ` [PATCHv4 4/5] USB: gadget: g_fs: code cleanup Michal Nazarewicz
2010-07-22 12:16               ` [PATCHv4 5/5] USB: gadget: file_storage: serial parameter even if not test mode Michal Nazarewicz
2010-07-22 14:14                 ` Alan Stern
2010-07-22 23:46         ` [PATCHv4 1/5] USB: gadget: composite: Better string override handling Greg KH
2010-07-23  9:04           ` Michał Nazarewicz
2010-07-27  6:53           ` David Brownell
2010-07-26 21:28         ` Greg KH

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=op.vf3o6v067p4s8u@pikus \
    --to=m.nazarewicz@samsung.com \
    --cc=Dries.VanPuymbroeck@dekimo.com \
    --cc=david-b@pacbell.net \
    --cc=greg@kroah.com \
    --cc=kyungmin.park@samsung.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=m.szyprowski@samsung.com \
    --cc=stern@rowland.harvard.edu \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.