Re: [PATCH 6/7] compressed: rename barebox target to zbarebox and zbarebox.bin

[Sascha Hauer <s.hauer@pengutronix.de>]

On Mon, Jul 23, 2012 at 10:02:08AM +0200, Jean-Christophe PLAGNIOL-VILLARD wrote:
> On 09:16 Mon 23 Jul     , Sascha Hauer wrote:
> > 
> > Does it really?
> > 
> > I see this goes back to:
> > 
> > commit 8a03bff03ebf36b7460e6ec43259bc8b40731ece
> > Author: Sascha Hauer <s.hauer@pengutronix.de>
> > Date:   Thu Jul 3 10:30:44 2008 +0200
> > 
> >     [arm] Add cleanup flags only when modules are disabled. Otherwise
> >           we remove functions from the binary which are needed for
> >           modules
> > 
> > I can't see though why this should happen. What is needed for modules
> > is defined by EXPORT_SYMBOL.
> > 
> > Unfortunately modules are quite broken atm for me anyway, so I can't
> > test.
> except you need to use the barebox file to link the ko
> 
> and that is one of the reason we do need to call the compressed barebox
> zbarebox
> 
> and I'd like to be able to detect when it's a normal barebox and a zbarebox
> so need to have start.c compiled 2 times
> 
> I really do not like this link with the whole barebox in the zbarebox
> 
> keep stuff clean and seperated is really better

Well, send patches ;)

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