From: Tom Rini <trini@konsulko.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] config USB_STORAGE: defconfig vs include header
Date: Thu, 8 Sep 2016 14:13:21 -0400 [thread overview]
Message-ID: <20160908181321.GI4990@bill-the-cat> (raw)
In-Reply-To: <4ffdaf01-571b-8636-5261-90ba4568e349@wwwdotorg.org>
On Thu, Sep 08, 2016 at 09:58:13AM -0600, Stephen Warren wrote:
> On 09/07/2016 07:29 PM, Masahiro Yamada wrote:
> >Hi Stephen
> >
> >
> >2016-09-08 1:15 GMT+09:00 Stephen Warren <swarren@wwwdotorg.org>:
> >>Masahiro,
> >>
> >>In patch 6e7e9294d321 "usb: add basic USB configs in Kconfig", you added
> >>"config USB_STORAGE" to drivers/usb/Kconfig. However, it's still just
> >>#defined by many include/configs/*.h rather than being defined in
> >>configs/*_defconfig. Is that a problem? It seems to work in practice, but
> >>leads people adding new boards to put the definition in configs/*_defconfig
> >>which then may be inconsistent with similar existing boards which have it
> >>defined in include/configs/*.h.
> >
> >Once we create an entry in Kconfig,
> >all the defines in include/configs/*.h should be moved.
>
> That's what I imagined. The commit above didn't do that though; are
> you planning on sending a fix?
Since those are often conflict-happy I'll take care of it. I probably
intended to, even.
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160908/8a240dd8/attachment.sig>
next prev parent reply other threads:[~2016-09-08 18:13 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-09-07 16:15 [U-Boot] config USB_STORAGE: defconfig vs include header Stephen Warren
2016-09-08 1:29 ` Masahiro Yamada
2016-09-08 15:58 ` Stephen Warren
2016-09-08 18:13 ` Tom Rini [this message]
2016-09-08 20:17 ` Tom Rini
2016-09-09 1:09 ` Masahiro Yamada
2016-09-09 1:13 ` Tom Rini
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=20160908181321.GI4990@bill-the-cat \
--to=trini@konsulko.com \
--cc=u-boot@lists.denx.de \
/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