public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 1/2] OMAP3: Remove unused board-types
Date: Sat, 25 Apr 2009 14:50:33 +0200	[thread overview]
Message-ID: <20090425125033.GD25381@game.jcrosoft.org> (raw)
In-Reply-To: <20090424210125.72DCA83420E8@gemini.denx.de>

On 23:01 Fri 24 Apr     , Wolfgang Denk wrote:
> Dear Jean-Christophe PLAGNIOL-VILLARD,
> 
> In message <20090424200323.GD2953@game.jcrosoft.org> you wrote:
> >
> > > What exactly do you mean by "move the STD API"? 
> > > 
> > > In which way should the "STD API" be moved, and what exactly is the
> > > "STD API" you are referring to?
> > 
> > extract of arm init function
> > 
> > #if defined(CONFIG_DISPLAY_CPUINFO)
> > 	print_cpuinfo,		/* display cpu info (and speed) */
> > #endif
> > #if defined(CONFIG_DISPLAY_BOARDINFO)
> > 	checkboard,		/* display board info */
> > #endif
> > 
> > I want we use the current API and not re-invent a new API for an arch only
> 
> Well, if you conside rthis the "standard API", this should (1) be
> documented somewhere, and (2) it must be fixed - at the moment, the
> code reads:
> 
> 	lib_arm/board.c:int print_cpuinfo (void); /* test-only */
> 
> I would not dare to use such a function in my code given the
> "test-only" comment.
sorry I've no time to clean every part of the arm as noone else are
interrested in old code

so yes it will be cleanup but later asI work on other part of the arm actually
which I will finish first

Best Regards,
J.

  reply	other threads:[~2009-04-25 12:50 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-04-24 14:51 [U-Boot] [PATCH 1/2] OMAP3: Remove unused board-types Sanjeev Premi
2009-04-24 16:44 ` Jean-Christophe PLAGNIOL-VILLARD
2009-04-24 16:59   ` Dirk Behme
2009-04-24 20:04     ` Jean-Christophe PLAGNIOL-VILLARD
2009-04-24 19:34   ` Wolfgang Denk
2009-04-24 20:03     ` Jean-Christophe PLAGNIOL-VILLARD
2009-04-24 21:01       ` Wolfgang Denk
2009-04-25 12:50         ` Jean-Christophe PLAGNIOL-VILLARD [this message]
2009-04-26  4:56           ` Dirk Behme
2009-04-26 18:58             ` Wolfgang Denk
2009-04-24 16:47 ` Dirk Behme

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=20090425125033.GD25381@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox