From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew F. Davis Date: Tue, 14 Feb 2017 14:32:42 -0600 Subject: [U-Boot] [PATCH v4 1/6] spl: Convert CONFIG_SPL_ABORT_ON_RAW_IMAGE into a positive option In-Reply-To: <20170214201538.GO26129@bill-the-cat> References: <20170213184741.8755-1-afd@ti.com> <20170213184741.8755-2-afd@ti.com> <20170214201538.GO26129@bill-the-cat> Message-ID: <86f4a5c2-934f-e6bc-515a-3f853c60620a@ti.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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 >> --- >> 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