From mboxrd@z Thu Jan 1 00:00:00 1970 From: Allen Martin Date: Mon, 22 Oct 2012 10:26:13 -0700 Subject: [U-Boot] [PATCH V2 4/5] ARM: tegra: select between Seaboard/Ventana at compile time In-Reply-To: <50856FCA.3090409@wwwdotorg.org> References: <1350602759-22605-1-git-send-email-swarren@wwwdotorg.org> <1350602759-22605-4-git-send-email-swarren@wwwdotorg.org> <20121020055454.GE25104@badger> <50856FCA.3090409@wwwdotorg.org> Message-ID: <20121022172613.GB13201@badger> 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, Oct 22, 2012 at 09:09:46AM -0700, Stephen Warren wrote: > On 10/19/2012 11:54 PM, Allen Martin wrote: > > On Thu, Oct 18, 2012 at 04:25:58PM -0700, Stephen Warren wrote: > >> From: Stephen Warren > >> > >> Seaboard and Ventana are very similar boards, and so share the seaboard.c > >> board file. The one difference needed so far is detected at run-time by > >> calling machine_is_ventana(). This bloats the Ventana build with code > >> that is never used. Switch to detecting Ventana at compile time to remove > >> bloat. This shaves ~5K off the SPL size on Ventana, and makes the SPL fit > >> within the max size. > >> > >> Signed-off-by: Stephen Warren > >> --- > >> v2: New patch to replace modification of CONFIG_SYS_TEXT_BASE. > >> --- > >> board/nvidia/seaboard/seaboard.c | 4 ++-- > >> 1 files changed, 2 insertions(+), 2 deletions(-) > >> > >> diff --git a/board/nvidia/seaboard/seaboard.c b/board/nvidia/seaboard/seaboard.c > >> index 4e8a183..6dce57f 100644 > >> --- a/board/nvidia/seaboard/seaboard.c > >> +++ b/board/nvidia/seaboard/seaboard.c > >> @@ -35,6 +35,7 @@ > >> > >> /* TODO: Remove this code when the SPI switch is working */ > >> #ifndef CONFIG_SPI_UART_SWITCH > >> +#if CONFIG_MACH_TYPE != MACH_TYPE_VENTANA > > > > Why not roll this into the previous #ifdef intead of the back to back #ifdefs? > > I guess I could, although it's going to get line-wrapped anyway due to > length... Just a nit, your call. Either way: Acked-by: Allen Martin And the whole series: Tested-by: Allen Martin -- nvpublic