All of lore.kernel.org
 help / color / mirror / Atom feed
From: Michal Simek <monstr@seznam.cz>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] MAKEALL patch
Date: Sun, 5 Aug 2007 16:50:38 +0200	[thread overview]
Message-ID: <001301c7d76f$fd1d5c50$0500a8c0@monstrone> (raw)
In-Reply-To: 20070805135130.2ACE3353DB4@atlas.denx.de

Hi Wolfgang,

> What is the specific bug you intend to fix here?  Using  ppc_8xx-size
> in  case  CROSS_COMPILE is not set is not worse than using the native
> "size" command, it seems. Actually, given the fact that the  majority
> supported of boards are PowerPC, it is right in most cases.
>
> So why do you think it should be changed?

I think size utility do the same function as ppc_8xx-size.
I tried 4 cross tools and the result is the same. (Logs are below)
And in man page you can find: "size can automatically recognize many 
formats" in --target option.

From my point of view is size utility better than error message.
And I think that many developers use different cross tools than ppc_8xx-.

Best regards,
Michal Simek

microblaze at monstr u-boot-microblaze]$ ./MAKEALL xupv2p
Configuring for xupv2p board...
./MAKEALL: line 351: ppc_8xx-size: command not found

[microblaze at monstr u-boot-microblaze]$ ./MAKEALL xupv2p
Configuring for xupv2p board...
   text    data     bss     dec     hex filename
  89326   21479  214916  325721   4f859 ./u-boot

[microblaze at monstr u-boot-microblaze]$ size u-boot
   text    data     bss     dec     hex filename
  89326   21479  214916  325721   4f859 u-boot

[microblaze at monstr u-boot-microblaze]$ microblaze-uclinux-size u-boot
   text    data     bss     dec     hex filename
  89326   21479  214916  325721   4f859 u-boot

[microblaze at monstr u-boot-microblaze]$ powerpc-405-linux-gnu-size u-boot
   text    data     bss     dec     hex filename
  89326   21479  214916  325721   4f859 u-boot
[microblaze at monstr u-boot-microblaze]$

[microblaze at monstr bin]$ powerpc-linux-uclibc-size  u-boot
   text    data     bss     dec     hex filename
  89326   21479  214916  325721   4f859 u-boot
[microblaze at monstr bin]$

  reply	other threads:[~2007-08-05 14:50 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-08-05 13:12 [U-Boot-Users] MAKEALL patch Michal Simek
2007-08-05 13:51 ` Wolfgang Denk
2007-08-05 14:50   ` Michal Simek [this message]
2007-08-05 19:54     ` Wolfgang Denk
2007-08-06  9:04   ` Mike Frysinger
2007-08-06 17:00     ` Michal Simek

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='001301c7d76f$fd1d5c50$0500a8c0@monstrone' \
    --to=monstr@seznam.cz \
    --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.