All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [RFC] initcall mechanism introduction
Date: Sun, 24 May 2009 17:04:44 +0200	[thread overview]
Message-ID: <20090524150444.GI26561@game.jcrosoft.org> (raw)
In-Reply-To: <20090524145725.84A65832E416@gemini.denx.de>

On 16:57 Sun 24 May     , Wolfgang Denk wrote:
> Dear Jean-Christophe PLAGNIOL-VILLARD,
> 
> In message <20090524120041.GG26561@game.jcrosoft.org> you wrote:
> >
> > > What sort of benefit do you expect?
> > simplify the code, reduce the number of ifdef
> > reduce the size of U-Boot etc...
> 
> How would that reduce the size of the code?
just as example remove the weak function will reduce the code
as show in the patch 3 just re-implement it through it will reduce the
size by 2KiB.
> 
> > I do have some test and I've gain between 2KiB and more than 10KiB
> > by using this
> 
> And, does it still work on all boards?
for my current test more api you use more you win
as I've not test on all boards I can not certify it
> 
> > so yes I think it's great winn
> 
> I think you don't understand the complexity yet. One problem we  have
> to  solve  is  to make sure we have a very specific sequence in which
> the init routines are run. The original idea of the code was that you
> can #define the init_sequence[] table in an architecture and/or board
> config file; it's just that nobody implemented that yet.
> 
> If you changed the code in this direction, then this might actually
> make sense.
no with initcalls you will be able to do it
as it will be a matter of at which state you want to init your code
so each arch will be able to specify it and you will be able to also add
specific init level on need
we are not forced to have the same lds for all arch and/or board too
actually I've create only one init.h but we can add a arch specificity

Best Regards,
J.

  reply	other threads:[~2009-05-24 15:04 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-05-23 15:05 [U-Boot] [RFC] initcall mechanism introduction Jean-Christophe PLAGNIOL-VILLARD
     [not found] ` <1243091325-27683-1-git-send-email-plagnioj@jcrosoft.com>
2009-05-23 15:08   ` [U-Boot] [PATCH 2/3] arm: add initcalls_init Jean-Christophe PLAGNIOL-VILLARD
2009-05-23 15:08     ` [U-Boot] [PATCH 3/3] net: switch device init to initcall Jean-Christophe PLAGNIOL-VILLARD
2009-05-23 15:41       ` Ben Warren
2009-05-23 16:36         ` Jean-Christophe PLAGNIOL-VILLARD
2009-05-23 18:26           ` Wolfgang Denk
2009-05-23 18:31           ` Ben Warren
2009-05-23 18:19 ` [U-Boot] [RFC] initcall mechanism introduction Wolfgang Denk
2009-05-24 12:00   ` Jean-Christophe PLAGNIOL-VILLARD
2009-05-24 14:57     ` Wolfgang Denk
2009-05-24 15:04       ` Jean-Christophe PLAGNIOL-VILLARD [this message]
2009-05-24 17:05         ` Wolfgang Denk
2009-05-26 21:00       ` Scott Wood
2009-05-26 21:54         ` Wolfgang Denk
2009-05-26 22:07           ` Jean-Christophe PLAGNIOL-VILLARD
2009-05-26 22:25             ` Wolfgang Denk
2009-05-26 22:20           ` Scott Wood
2009-05-26 22:28             ` Wolfgang Denk
2009-05-27  7:52           ` Haavard Skinnemoen
2009-05-27  8:12             ` Stefan Roese
2009-05-27 18:48               ` Mike Frysinger

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=20090524150444.GI26561@game.jcrosoft.org \
    --to=plagnioj@jcrosoft.com \
    --cc=u-boot@lists.denx.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.