[U-Boot] [PATCH v4 1/6] spl: Convert CONFIG_SPL_ABORT_ON_RAW_IMAGE into a positive option

From: Andrew F. Davis <afd@ti.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v4 1/6] spl: Convert CONFIG_SPL_ABORT_ON_RAW_IMAGE into a positive option
Date: Tue, 14 Feb 2017 14:32:42 -0600	[thread overview]
Message-ID: <86f4a5c2-934f-e6bc-515a-3f853c60620a@ti.com> (raw)
In-Reply-To: <20170214201538.GO26129@bill-the-cat>

On 02/14/2017 02:15 PM, Tom Rini wrote:
> On Mon, Feb 13, 2017 at 12:47:36PM -0600, Andrew F. Davis wrote:
> 
>> CONFIG_SPL_ABORT_ON_RAW_IMAGE causes SPL to abort and move on when it
>> encounters RAW images, express this same functionality as a positive
>> option enabling support for RAW images: CONFIG_SPL_RAW_IMAGE_SUPPORT
>>
>> Signed-off-by: Andrew F. Davis <afd@ti.com>
>> ---
>>  Kconfig                           |  7 +++++++
>>  README                            |  4 ----
>>  common/spl/spl.c                  | 10 ++++++----
>>  include/configs/imx6_spl.h        |  4 ++--
>>  include/configs/socfpga_de1_soc.h |  2 +-
> 
> OK, with this in Kconfig, we need to migrate the appropriate configs/
> files rather than tweak include/configs/ files again.  I think
> moveconfig.py will be reasonable to run here given that you can pass a
> list of defconfigs to look at (anything that sets CONFIG_IMX6 and so
> on).  Thanks!
> 

It does, and it removes the symbols I tweaked in the include/configs/
automatically. The issue is that it also caused a lot of churn in the
defconfig files as a lot of symbols have become out of order and get
re-ordered. With other similar patches re-ordering defconfig symbols
this would be trouble for you to merge and for me to rebase if it needs
a re-spin. Maybe moveconfig.py could be done on your side when patches
like this are taken?

I'll post the patch generated by
tools/moveconfig.py -c -v SPL_RAW_IMAGE_SUPPORT
but it may be very large and I cannot guarantee it will not have +300
merge conflicts for you to handle if you take it with other defconfig
fixup patches :)

Andrew