From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Wed, 10 Feb 2021 12:51:43 -0500 Subject: [PATCH 13/13] arm: stm32: Disable ATAGs support In-Reply-To: <35b02e26-37a7-a798-7a0d-8af21e62385e@foss.st.com> References: <20210204022415.20589-1-trini@konsulko.com> <20210204022415.20589-13-trini@konsulko.com> <35b02e26-37a7-a798-7a0d-8af21e62385e@foss.st.com> Message-ID: <20210210175143.GI10169@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Mon, Feb 08, 2021 at 05:34:50PM +0100, Patrick DELAUNAY wrote: [snip] > Just one question for other part of generic code which can be removed.... > > bi_boot_params should be under compilation BOOTM_ENABLE_TAGS flags ? > > In include/asm-generic/u-boot.h:70 > ??? struct bd_info { > > ??? .... > ??? ??? ulong??? ??????? bi_boot_params;??? /* where this board expects > params */ > > ??? ... > > ??? }; > > > and also params global variables, only used in setup_XXX functions ? > > ??? arch/arm/lib/bootm.c:44: > ??? ??? static struct tag *params; With some further cleanup work to introduce a single symbol to say yes/no to ATAGS support, we could then cleanly condition out a few more bits of code and save some space, yes. -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 659 bytes Desc: not available URL: