public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Marek Vasut <marek.vasut@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2 0/7] Tidy up ARM link scripts
Date: Tue, 21 Feb 2012 06:53:37 +0100	[thread overview]
Message-ID: <201202210653.37444.marek.vasut@gmail.com> (raw)
In-Reply-To: <CAPnjgZ0je8voNX2YBLCVihri03JN1mK=5tqQ4pPnaR4bNRZXbA@mail.gmail.com>

> Hi Albert,
> 
> On Fri, Feb 17, 2012 at 2:47 AM, Albert ARIBAUD
> 
> <albert.u.boot@aribaud.net> wrote:
> > Hi Marek,
> > 
> > Le 17/02/2012 10:49, Marek Vasut a ?crit :
> >>> Hi Simon,
> >>> 
> >>> Le 21/11/2011 21:49, Simon Glass a ?crit :
> >>>> The ARM link scripts are very similar but each CPU has its own. This
> >>>> series adds support for a default link script across an architecture
> >>>> in arch/<arch>/cpu/u-boot.lds.
> >>> 
> >>> This patch series received no comments, and I apologize for not having
> >>> chimed in on it as it goes the right way about simplifying ARM
> >>> architecture.
> >>> 
> >>> Currently, all patches in the series do apply except patch 7, which
> >>> fails on arch/arm/cpu/pxa/u-boot.lds. As this is trivial, I'm going on
> >>> removing the file manually and testing, but can you please resubmit V3
> >>> with a fixed 1/7 patch?
> >> 
> >> Please make sure there were no changes to the linker file. I believe
> >> there were
> >> some and it might break pxa.
> >> 
> >> Thanks!
> > 
> > Thanks for pointing this out. I am not testing pxa boards now, so Simon
> > has time to submit a vV3 patch set before I get bitten by any change in
> > PXA boards.
> 
> Yes there was a change to add a 4KB MMU memory area to the link
> script, presumably to allow early MMU enable and using that cache
> space as SRAM.

Exactly, PXA250 has no SRAM so I needed some place to put stack to etc.

M
> 
> I think I will add this feature to the generic .lds file since it does
> no harm, and it's a feature I would like to implement one day (early
> MMU enable). It might help ARM11 also.
> 
> > Amicalement,
> > --
> > Albert.
> 
> Regards,
> Simon

      reply	other threads:[~2012-02-21  5:53 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-11-21 20:49 [U-Boot] [PATCH v2 0/7] Tidy up ARM link scripts Simon Glass
2011-11-21 20:49 ` [U-Boot] [PATCH v2 1/7] Allow arch directory to contain .lds without requiring Makefile Simon Glass
2011-11-21 20:49 ` [U-Boot] [PATCH v2 2/7] arm: Remove jornada link script Simon Glass
2011-11-21 20:49 ` [U-Boot] [PATCH v2 3/7] arm: Remove zipitz2 " Simon Glass
2011-11-21 20:49 ` [U-Boot] [PATCH v2 4/7] Define CPUDIR for the .lds " Simon Glass
2011-11-21 20:49 ` [U-Boot] [PATCH v2 5/7] arm: Remove unneeded setting of LDCSRIPT Simon Glass
2011-11-21 20:49 ` [U-Boot] [PATCH v2 6/7] arm: add a common .lds link script Simon Glass
2012-02-17 11:08   ` Albert ARIBAUD
2012-02-21  2:32     ` Simon Glass
2012-02-21 12:34       ` Albert ARIBAUD
2012-02-23 13:28   ` [U-Boot] [PATCH v4 " Simon Glass
2011-11-21 20:49 ` [U-Boot] [PATCH v2 7/7] arm: Use common .lds file where possible Simon Glass
2011-12-03  3:04 ` [U-Boot] [PATCH v2 0/7] Tidy up ARM link scripts Simon Glass
2012-02-17  8:58 ` Albert ARIBAUD
2012-02-17  9:49   ` Marek Vasut
2012-02-17 10:47     ` Albert ARIBAUD
2012-02-21  2:24       ` Simon Glass
2012-02-21  5:53         ` Marek Vasut [this message]

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=201202210653.37444.marek.vasut@gmail.com \
    --to=marek.vasut@gmail.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