From: Tom Rini <trini@konsulko.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] Kconfig: Enable FIT support by default for TI platforms
Date: Thu, 20 Apr 2017 11:48:04 -0400 [thread overview]
Message-ID: <20170420154804.GT19487@bill-the-cat> (raw)
In-Reply-To: <df882a69-27ff-c573-2928-b590b500d52c@ti.com>
On Thu, Apr 20, 2017 at 10:22:24AM -0500, Andrew F. Davis wrote:
> On 04/20/2017 10:14 AM, Tom Rini wrote:
> > On Thu, Apr 20, 2017 at 09:52:37AM -0500, Andrew F. Davis wrote:
> >
> >> Almost all TI defconfigs enable this already, add this as a default
> >> and remove the explicit assignment.
> >>
> >> Signed-off-by: Andrew F. Davis <afd@ti.com>
> >> ---
> >> Kconfig | 1 +
> > [snip]
> >> diff --git a/Kconfig b/Kconfig
> >> index 1cf990dfce..396c600385 100644
> >> --- a/Kconfig
> >> +++ b/Kconfig
> >> @@ -144,6 +144,7 @@ menu "Boot images"
> >>
> >> config FIT
> >> bool "Support Flattened Image Tree"
> >> + default y if ARCH_KEYSTONE || ARCH_OMAP2
> >> select MD5
> >> help
> >> This option allows you to boot the new uImage structure,
> >
> > This is where I think we start to want to use imply, now that we have
> > it. As imply does not let us 'default n' or 'default
> > hex-int-or-string', we can't use it in those cases. But imply works for
> > 'default y' cases, so we should have ARCH_OMAP2 and ARCH_KEYSTONE imply
> > FIT. This keeps much of the changes out of the higher level Kconfig
> > files where we may get conflicts between patches. Thanks!
>
> I can make that change, but in this case I was thinking that at some
> point this will become just a default y for all platforms, so we
> wouldn't need "imply FIT" for every ARCH_*.
At some point in the future, yes, we might make FIT a default y and then
we would drop the various imply lines. But that's a future clean-up :)
Thanks!
--
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/20170420/3b23fef8/attachment.sig>
prev parent reply other threads:[~2017-04-20 15:48 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-04-20 14:52 [U-Boot] [PATCH] Kconfig: Enable FIT support by default for TI platforms Andrew F. Davis
2017-04-20 15:14 ` Tom Rini
2017-04-20 15:22 ` Andrew F. Davis
2017-04-20 15:48 ` Tom Rini [this message]
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=20170420154804.GT19487@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 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.