From mboxrd@z Thu Jan  1 00:00:00 1970
From: Wolfgang Denk <wd@denx.de>
Date: Mon, 20 Oct 2008 09:28:36 +0200
Subject: [U-Boot] [PATCH v2] bootm: Move to using a function pointer
	table for the boot os function
In-Reply-To: <6FA5D356-5762-4185-BD0C-1CE3BDEDE2D9@kernel.crashing.org>
References: <1224439644-10648-1-git-send-email-galak@kernel.crashing.org>
	<20081019184405.GA9455@game.jcrosoft.org>
	<6FA5D356-5762-4185-BD0C-1CE3BDEDE2D9@kernel.crashing.org>
Message-ID: <20081020072836.2C017835697A@gemini.denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Dear Kumar Gala,

In message <6FA5D356-5762-4185-BD0C-1CE3BDEDE2D9@kernel.crashing.org> you wrote:
> 
> >> This removes a bit of code and makes it easier for the upcoming sub  
> >> bootm
> >> command support to call into the proper OS specific handler.
> >>
> >> Signed-off-by: Kumar Gala <galak@kernel.crashing.org>
> >> ---
...
> >> @@ -119,6 +119,22 @@ int do_bootelf (cmd_tbl_t *cmdtp, int flag,  
> >> int argc, char *argv[]);
> >> static boot_os_fn do_bootm_integrity;
> >> #endif
> >>
> >> +boot_os_fn * boot_os[] = {
> >> +	[IH_OS_LINUX] = do_bootm_linux,
> >> +	[IH_OS_NETBSD] = do_bootm_netbsd,
> > why not add config to ative or not netbsd, linux and rterms to  
> > reduce U-Boot
> > size?
> 
> ative?
> 
> that would be an orthogonal change.

One that we should implement, I think.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Drawing on my fine command of language, I said nothing.