From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from metis.ext.pengutronix.de ([2001:67c:670:201:290:27ff:fe1d:cc33]) by bombadil.infradead.org with esmtps (Exim 4.85_2 #1 (Red Hat Linux)) id 1baGrK-0002In-0N for barebox@lists.infradead.org; Thu, 18 Aug 2016 06:31:22 +0000 Date: Thu, 18 Aug 2016 08:31:00 +0200 From: Sascha Hauer Message-ID: <20160818063059.GD20657@pengutronix.de> References: MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "barebox" Errors-To: barebox-bounces+u.kleine-koenig=pengutronix.de@lists.infradead.org Subject: Re: Fwd: Shouldn't boot_board be called from boot instead of init? To: Guillermo Rodriguez Garcia Cc: barebox@lists.infradead.org Hi, On Tue, Aug 16, 2016 at 10:42:32AM +0200, Guillermo Rodriguez Garcia wrote: > Hello all, > > Currently, for defaultenv v1, the /env/bin/boot_board script is called > from /env/bin/init. > > However this means boot_board will not be run if booting manually (by > running 'boot' from the barebox console). > > Shouldn't this script be called from /env/bin/boot instead? If a board > needs any specific stuff to be done when booting, this probably > applies both when autobooting and when booting manually (otherwise, > anything that only applies only when autobooting could also be done > from init_board instead of boot_board). The only boot_board script we have is arch/arm/boards/at91sam9m10g45ek/env/bin/boot_board. Here a menu is built which I think makes sense at that stage and not at init_board. However, I would be glad to get rid of defaultenv-1 rather sooner than later. Sascha -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ barebox mailing list barebox@lists.infradead.org http://lists.infradead.org/mailman/listinfo/barebox