From: Tom Rini <trini@konsulko.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2 1/6] arm: am57xx: reintroduce the CONFIG_AM57XX symbol
Date: Tue, 14 Feb 2017 14:20:17 -0500 [thread overview]
Message-ID: <20170214192017.GL26129@bill-the-cat> (raw)
In-Reply-To: <CAGo_u6rEq47Kw6cvPzEm1Np-TPrycqHEL2y8LWpYJ_T9EZ7XWA@mail.gmail.com>
On Tue, Feb 14, 2017 at 01:18:20PM -0600, menon.nishanth at gmail.com wrote:
> On Mon, Feb 13, 2017 at 7:27 AM, Tom Rini <trini@konsulko.com> wrote:
> > On Mon, Feb 13, 2017 at 01:34:52PM +0200, Uri Mashiach wrote:
> >
> >> The SOC family symbol CONFIG_AM57XX was removed by the commit
> >> 3891a54: "ARM: DRA7x/AM57xx: Get rid of CONFIG_AM57XX".
> >>
> >> The symbol is needed by the XHCI OMAP USB driver.
> >> Without the symbol all the AM57x targets symbols should be ored in the
> >> ifdef'ery.
> >>
> >> Signed-off-by: Uri Mashiach <uri.mashiach@compulab.co.il>
> >
> > We need to introduce it into Kconfig files now, thanks!
>
> DRA7 and AM57xx are literally the same. the #ifdeffery in USB is more
> to do which port to use rather than having anything to do with DRA7 Vs
> AM57xx.
Yes, agreed. And we need to solve this, and introduce some Kconfig
CONFIG symbol to deal with it. Maybe the better way to look at this is,
can we have DRA7xx boards configured in the way Uri wants to be able to
configure the AM57xx board he's working with? And vice versa? The
answer to that will help drive what the right CONFIG_xxx name is.
--
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/20170214/dc5284df/attachment.sig>
next prev parent reply other threads:[~2017-02-14 19:20 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-02-13 11:34 [U-Boot] [PATCH v2 1/6] arm: am57xx: reintroduce the CONFIG_AM57XX symbol Uri Mashiach
2017-02-13 13:27 ` Tom Rini
2017-02-14 19:18 ` menon.nishanth at gmail.com
2017-02-14 19:20 ` Tom Rini [this message]
2017-02-14 19:26 ` Nishanth Menon
2017-02-14 20:12 ` Tom Rini
2017-02-15 8:24 ` Roger Quadros
2017-02-15 11:30 ` Roger Quadros
2017-02-15 13:50 ` Uri Mashiach
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=20170214192017.GL26129@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