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: 18+ 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 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.