From: "Michał Nazarewicz" <m.nazarewicz@samsung.com>
To: Andy Shevchenko <andy.shevchenko@gmail.com>
Cc: Greg KH <greg@kroah.com>, David Brownell <david-b@pacbell.net>,
linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCHv5 3/3] USB: gadget: storage_common: fixed warning building mass storage function
Date: Wed, 28 Jul 2010 15:02:01 +0200 [thread overview]
Message-ID: <op.vgj05nev7p4s8u@pikus> (raw)
In-Reply-To: <AANLkTikW8eSn3fgRqJsXwp=+ZMjKgojHDP6+DsDMtg4y@mail.gmail.com>
On Wed, 28 Jul 2010 14:25:59 +0200, Andy Shevchenko <andy.shevchenko@gmail.com> wrote:
> On Wed, Jul 28, 2010 at 3:13 PM, Michal Nazarewicz
> <m.nazarewicz@samsung.com> wrote:
>> The "nofua" attribute is used in the File-Storage Gadget but the
>> functions handling it are defined in storage_common.c (which may be
>> not a bad thing if anyone will want to port tho "nofua" attribute to
>> mass storage function) which causes the following warnings:
>>
>> drivers/usb/gadget/storage_common.c:718: \
>> warning: ‘fsg_show_nofua’ defined but not used
>> drivers/usb/gadget/storage_common.c:782: \
>> warning: ‘fsg_store_nofua’ defined but not used
>>
>> Adding __maybe_unused fixes this issue.
> It's a bit odd.
>
> Why the warning happens for those two only?
> There are two more parameters (four methods) to handle 'ro' and 'file'
> which are done in the same way.
Those are the only ones that are not used by the mass storage function.
The warning is issued when drivers using mass storage function are built.
File-Storage Gadget compiles fine.
--
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--
next prev parent reply other threads:[~2010-07-28 13:00 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-07-28 12:13 [PATCHv5 1/3] USB: gadget: composite: Better string override handling Michal Nazarewicz
2010-07-28 12:13 ` [PATCHv5 2/3] USB: gadget: Use new composite features in some gadgets Michal Nazarewicz
2010-07-28 12:13 ` [PATCHv5 3/3] USB: gadget: storage_common: fixed warning building mass storage function Michal Nazarewicz
2010-07-28 12:25 ` Andy Shevchenko
2010-07-28 13:02 ` Michał Nazarewicz [this message]
2010-07-28 13:42 ` Andy Shevchenko
2010-07-28 14:02 ` Michał Nazarewicz
2010-07-29 22:21 ` [PATCHv5 2/3] USB: gadget: Use new composite features in some gadgets David Brownell
2010-07-30 16:48 ` Michał Nazarewicz
2010-07-30 16:54 ` Greg KH
2010-07-30 18:57 ` David Brownell
2010-07-30 21:21 ` Greg KH
2010-07-30 22:01 ` David Brownell
2010-07-30 22:16 ` Greg KH
2010-07-30 23:58 ` Xiaofan Chen
2010-08-02 17:14 ` Michał Nazarewicz
2010-08-02 22:52 ` Greg KH
2010-08-04 9:21 ` Michal Nazarewicz
2010-08-01 19:05 ` [PATCHv5 1/3] USB: gadget: composite: Better string override handling David Brownell
2010-08-02 9:48 ` Michał Nazarewicz
2010-08-02 11:26 ` David Brownell
2010-08-02 12:47 ` 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=op.vgj05nev7p4s8u@pikus \
--to=m.nazarewicz@samsung.com \
--cc=andy.shevchenko@gmail.com \
--cc=david-b@pacbell.net \
--cc=greg@kroah.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-usb@vger.kernel.org \
/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