On 02/14/2017 03:04 PM, Tom Rini wrote: > On Tue, Feb 14, 2017 at 02:32:42PM -0600, Andrew F. Davis wrote: >> 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 <a...@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? > > So in this case, the way you deal with it is: > 1) Do a moveconfig.py -s and commit the results. > 2) Do the rest of your patches > 3) Drop that first patch and post. If it doesn't apply cleanly > (sometimes it in fact does), I end up dealing with the fallout. > > It's easier for me to deal with fixing defconfig fails to apply than it > is to be sure that everything is migrated correctly and as intended. >
Works for me, I'll post v5 with this change. Thanks, Andrew _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
