From: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
To: Sascha Hauer <s.hauer@pengutronix.de>
Cc: barebox@lists.infradead.org
Subject: Re: [PATCH 6/7] compressed: rename barebox target to zbarebox and zbarebox.bin
Date: Mon, 23 Jul 2012 10:18:07 +0200 [thread overview]
Message-ID: <20120723081807.GQ22657@game.jcrosoft.org> (raw)
In-Reply-To: <20120723080507.GY30009@pengutronix.de>
On 10:05 Mon 23 Jul , Sascha Hauer wrote:
> 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 ;)
THis is what this patch do
Best Regards,
J.
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
next prev parent reply other threads:[~2012-07-23 8:17 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-07-23 4:20 [PATCH 0/7 V2] compressed image update Jean-Christophe PLAGNIOL-VILLARD
2012-07-23 4:25 ` [PATCH 1/7] stddev: make it selectable via Kconfig Jean-Christophe PLAGNIOL-VILLARD
2012-07-23 4:25 ` [PATCH 2/7] decompress_unlzo: define decompress_unlzo as decompress Jean-Christophe PLAGNIOL-VILLARD
2012-07-23 4:25 ` [PATCH 3/7] compressed image: factorise compressor type Jean-Christophe PLAGNIOL-VILLARD
2012-07-23 4:26 ` [PATCH 4/7] decompressor: import malloc/free implementation for linux 3.4 Jean-Christophe PLAGNIOL-VILLARD
2012-07-23 4:26 ` [PATCH 5/7] ARM: add early malloc support needed by the decompressor Jean-Christophe PLAGNIOL-VILLARD
2012-07-23 4:26 ` [PATCH 6/7] compressed: rename barebox target to zbarebox and zbarebox.bin Jean-Christophe PLAGNIOL-VILLARD
2012-07-23 6:35 ` Sascha Hauer
2012-07-23 6:41 ` Jean-Christophe PLAGNIOL-VILLARD
2012-07-23 7:16 ` Sascha Hauer
2012-07-23 8:02 ` Jean-Christophe PLAGNIOL-VILLARD
2012-07-23 8:05 ` Sascha Hauer
2012-07-23 8:18 ` Jean-Christophe PLAGNIOL-VILLARD [this message]
2012-07-23 8:22 ` Sascha Hauer
2012-07-23 9:08 ` Jean-Christophe PLAGNIOL-VILLARD
2012-07-23 9:05 ` Jean-Christophe PLAGNIOL-VILLARD
2012-07-23 4:26 ` [PATCH 7/7] compressed image: add gzip support Jean-Christophe PLAGNIOL-VILLARD
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20120723081807.GQ22657@game.jcrosoft.org \
--to=plagnioj@jcrosoft.com \
--cc=barebox@lists.infradead.org \
--cc=s.hauer@pengutronix.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox