All of lore.kernel.org
 help / color / mirror / Atom feed
From: Kumar Gala <galak@kernel.crashing.org>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v3] TQM85xx: enable partition support, sort commands
Date: Tue, 15 Sep 2009 10:28:54 -0500	[thread overview]
Message-ID: <B1390CAD-0173-4103-9F31-A169A20AADA5@kernel.crashing.org> (raw)
In-Reply-To: <59C20CEA-A72F-4B6A-B8E2-089688EC84A4@kernel.crashing.org>


>>> Ok. I think we are see two issues interact.
>>>
>>> I've just sent a pull request w/the RESET_VECTOR_ADDRESS fix that
>>> should address the linker script issue.  These seems to build  
>>> cleanly
>>> on all 85xx platforms w/ELDK4.2 for me.
>>>
>>> Now if you apply the TQM85xx.h on top of that you will see that the
>>> enabling of new features causes the size to increase beyond what is
>>> allocated for TQM8540.
>>
>> Maybe. But this is just another manifestation of the original  
>> problem:
>> without commit ec79d33 we didn't need the RESET_VECTOR_ADDRESS stuff
>> at all, and an image size of 256 kB was sufficient without and with
>> the "TQM85xx: enable partition suppor" patch applied.
>>
>> Now, after commit ec79d33 (which needed to be fixed by the
>> RESET_VECTOR_ADDRESS stuff in the first place), we suddenly need more
>> than 256 kB.
>>
>> This obviously means that commit ec79d33 introduced changes to the
>> image layout that make it bigger.
>>
>> I consider this a bug in commit ec79d33. I don't want to waste more
>> space than needed.
>
> I don't agree.  If I revert both commits and apply the TQM config.h
> update I see build failures.
>
> I've posted a 'tqm-test' branch of the mpc85xx tree that does just
> that.  I see the following:
>
> [galak at komodo u-boot-85xx]$ ./MAKEALL TQM8540
> ... TQM8540 (MPC8540)
> Configuring for TQM85xx board...
> /opt/eldk/4.2/ppc-linux-x86/usr/bin/ppc_85xxDP-ld: section .bootpg
> [fffff000 -> fffff1df] overlaps section .data.rel.local [ffffe964 ->
> fffff6a7]
> /opt/eldk/4.2/ppc-linux-x86/usr/bin/ppc_85xxDP-ld: u-boot:
> section .bootpg lma 0xfffff000 overlaps previous sections
> /opt/eldk/4.2/ppc-linux-x86/usr/bin/ppc_85xxDP-ld: u-boot:
> section .data.rel.ro.local lma 0xfffff6a8 overlaps previous sections
> /opt/eldk/4.2/ppc-linux-x86/usr/bin/ppc_85xxDP-ld: u-boot:
> section .data.rel lma 0xfffff728 overlaps previous sections
> make: *** [u-boot] Error 1
> /opt/eldk/4.2/ppc-linux-x86/usr/bin/ppc_85xxDP-size: './u-boot': No
> such file

Wolfgang,

Any update on this?

- k

  reply	other threads:[~2009-09-15 15:28 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-09-02  8:20 [U-Boot] [PATCH] TQM85xx: enable partition support, sort commands Wolfgang Denk
2009-09-02 21:41 ` Kumar Gala
2009-09-08 14:08 ` Kumar Gala
2009-09-08 19:50   ` Wolfgang Denk
2009-09-08 19:50 ` [U-Boot] [PATCH v2] " Wolfgang Denk
2009-09-08 20:29 ` [U-Boot] [PATCH v3] " Wolfgang Denk
2009-09-10  2:02   ` Kumar Gala
2009-09-10  7:56     ` Wolfgang Denk
2009-09-10  8:57       ` Kumar Gala
2009-09-10 11:55         ` Wolfgang Denk
2009-09-10 14:41           ` Kumar Gala
2009-09-15 15:28             ` Kumar Gala [this message]
2009-09-15 19:40               ` Wolfgang Denk
2009-10-16 17:44   ` Kumar Gala
2009-10-18 21:16     ` Wolfgang Denk

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=B1390CAD-0173-4103-9F31-A169A20AADA5@kernel.crashing.org \
    --to=galak@kernel.crashing.org \
    --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.