mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
From: Michael Olbrich <m.olbrich@pengutronix.de>
To: ptxdist@pengutronix.de
Subject: Re: [ptxdist] [PATCH] add patch for building at91bootstrap with gcc 4.6.x
Date: Tue, 31 Jan 2012 20:29:53 +0100	[thread overview]
Message-ID: <20120131192953.GC32382@pengutronix.de> (raw)
In-Reply-To: <19561ec9a868fab957c3ca64a164602e@idefix.lespocky.dyndns.org>

On Tue, Jan 31, 2012 at 05:54:41PM +0100, Alexander Dahl wrote:
> Am 30.01.2012 15:20, schrieb Michael Olbrich:
> >> See my mail from January 13th, where I described the problem. It occurs
> >> with the OSELAS toolchain 2011.11.0 compiling for arm-v5te.
> > 
> > Yes, I tried that and didn't get any build error. Which at91bootstrap
> > config do I select to this error?
> 
> Okay I try to give some more information. Meanwhile I switched to the
> 2011.11.1 toolchain, but the error can be reproduced with this:
> 
> arm-v5te-linux-gnueabi-gcc -nostartfiles -nostdlib
> -Wl,-Map=nandflash_at91sam9g20ek.map,--cref -T
> ./../../elf32-littlearm.lds -Ttext 0x200000 -n -o
> nandflash_at91sam9g20ek.elf crt0_gnu.o at91sam9g20ek.o main.o gpio.o
> pmc.o debug.o sdramc.o nandflash.o _udivsi3.o _umodsi3.o div0.o udiv.o
> string.o
> /home/adahl/opt/OSELAS.Toolchain-2011.11.1/arm-v5te-linux-gnueabi/gcc-4.6.2-glibc-2.14.1-binutils-2.21.1a-kernel-2.6.39-sanitized/bin/../lib/gcc/arm-v5te-linux-gnueabi/4.6.2/../../../../arm-v5te-linux-gnueabi/bin/ld:
> section .text.startup loaded at [00200e84,00200eab] overlaps section
> data loaded at [00200e84,00200f47]
> collect2: ld returned 1 exit status
> 
> You see the toolchain is for arm-v5te-linux-gnueabi and uses gcc 4.6,
> it's the toolchain build from the tag v2011.11.1 from the git repo
> http://git.pengutronix.de/?p=OSELAS.Toolchain.git;a=summary and the
> toolchain is built on the same up-to-date i686 Debian Squeeze VMware
> virtual machine where I try to compile the at91bootstrap.
> 
> In the platformconfig I chose this: 
> 
> % grep -irn bootstrap configs
> configs/arm-ncl/platformconfig:124:PTXCONF_AT91BOOTSTRAP=y
> configs/arm-ncl/platformconfig:125:PTXCONF_AT91BOOTSTRAP_VERSION="1.16"
> configs/arm-ncl/platformconfig:126:PTXCONF_AT91BOOTSTRAP_MD5="2d222312cf0af81c56db8747d6a38c7c"
> configs/arm-ncl/platformconfig:127:PTXCONF_AT91BOOTSTRAP_CONFIG="at91sam9g20ek"
> configs/arm-ncl/platformconfig:128:#
> PTXCONF_AT91BOOTSTRAP_BOOT_DATAFLASH is not set
> configs/arm-ncl/platformconfig:129:PTXCONF_AT91BOOTSTRAP_BOOT_NAND=y
> configs/arm-ncl/platformconfig:130:# PTXCONF_AT91BOOTSTRAP2 is not set
> 
> I did a `ptxdist clean at91bootstrap` before and made sure there are no
> customized rules or patches used, it's at91bootstrap as it comes with
> ptxdist 2012.01.0, which is the version I use for testing this. Do you
> need any more information to reproduce this? I attached the
> platformconfig, hope this is helpful somehow.

Yes thank you. Now I can reproduce this. It only happens with NAND and the
default in ptxdist is dataflash...
I've looked at the patch more closely. The file is indented with tabs and
your patch uses spaces. Please fix that. Also fix the patch header as I
described in my last mail. See e.g. the other patch to see how it should
look like.

Michael

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

-- 
ptxdist mailing list
ptxdist@pengutronix.de

  reply	other threads:[~2012-01-31 19:29 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-01-26 15:04 Alexander Dahl
2012-01-26 15:36 ` Michael Olbrich
2012-01-27  9:25   ` Alexander Dahl
2012-01-30 14:20     ` Michael Olbrich
2012-01-31 16:54       ` Alexander Dahl
2012-01-31 19:29         ` Michael Olbrich [this message]
2012-02-01 11:54           ` [ptxdist] [PATCH] add patch fixing at91bootstrap build with gcc 4.6.x, and recreate patch series Alexander Dahl
2012-02-01 12:20             ` Michael Olbrich

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=20120131192953.GC32382@pengutronix.de \
    --to=m.olbrich@pengutronix.de \
    --cc=ptxdist@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