Re: [U-Boot] [PATCH 6/6] powerpc: Add LINK_OFF calls in early C-code.

2010-11-28 Thread Joakim Tjernlund
Kim Phillips wrote on 2010/11/28 16:12:21: > > On Wed, 24 Nov 2010 22:59:55 +0100 > Wolfgang Denk wrote: > > > Dear Joakim Tjernlund, > > > > In message > > > > you wrote: > > > > > >Are you planning to apply some of my earlier patches(not this RFC > > > series) > > >or are you wait

Re: [U-Boot] [PATCH 6/6] powerpc: Add LINK_OFF calls in early C-code.

2010-11-28 Thread Kim Phillips
On Wed, 24 Nov 2010 22:59:55 +0100 Wolfgang Denk wrote: > Dear Joakim Tjernlund, > > In message > you > wrote: > > > >Are you planning to apply some of my earlier patches(not this RFC series) > >or are you waiting for Freescale to Ack/pick up? So far nobody > >has said anything. >

Re: [U-Boot] [PATCH 6/6] powerpc: Add LINK_OFF calls in early C-code.

2010-11-24 Thread Scott Wood
On Wed, 24 Nov 2010 22:50:21 +0100 Joakim Tjernlund wrote: > Scott Wood wrote on 2010/11/24 22:41:19: > > > > > > > How about playing with BATs before entering C code, so that the > > > > > > > image > > > > > > > always appears at the same effective address? > > > > > > > > > > > > hmm, never

Re: [U-Boot] [PATCH 6/6] powerpc: Add LINK_OFF calls in early C-code.

2010-11-24 Thread Wolfgang Denk
Dear Joakim Tjernlund, In message you wrote: > >Are you planning to apply some of my earlier patches(not this RFC series) >or are you waiting for Freescale to Ack/pick up? So far nobody >has said anything. I think these are all on Freescale's (Kim Phillips') list. Best regards, W

Re: [U-Boot] [PATCH 6/6] powerpc: Add LINK_OFF calls in early C-code.

2010-11-24 Thread Joakim Tjernlund
Wolfgang Denk wrote on 2010/11/24 22:45:11: > > Dear Joakim Tjernlund, > > In message > you > wrote: > > > > Played a little with this but it seems like two BATs cannot overlap? > > IIRC they can, but the first (lower register numbers) mapping wins. This text from PPC PEM suggests otherwise:

Re: [U-Boot] [PATCH 6/6] powerpc: Add LINK_OFF calls in early C-code.

2010-11-24 Thread Joakim Tjernlund
Scott Wood wrote on 2010/11/24 22:41:19: > > > > > > How about playing with BATs before entering C code, so that the > > > > > > image > > > > > > always appears at the same effective address? > > > > > > > > > > hmm, never thought of that. The extra bonus would be that LINK_OFF > > > > > should

Re: [U-Boot] [PATCH 6/6] powerpc: Add LINK_OFF calls in early C-code.

2010-11-24 Thread Wolfgang Denk
Dear Joakim Tjernlund, In message you wrote: > > Played a little with this but it seems like two BATs cannot overlap? IIRC they can, but the first (lower register numbers) mapping wins. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB

Re: [U-Boot] [PATCH 6/6] powerpc: Add LINK_OFF calls in early C-code.

2010-11-24 Thread Scott Wood
On Wed, 24 Nov 2010 22:36:27 +0100 Joakim Tjernlund wrote: > Scott Wood wrote on 2010/11/24 18:16:56: > > > > On Wed, 24 Nov 2010 12:04:15 +0100 > > Joakim Tjernlund wrote: > > > > > > > > > > Scott Wood wrote on 2010/11/23 23:32:04: > > > > > > > > > > On Tue, 23 Nov 2010 23:14:06 +0100 > > >

Re: [U-Boot] [PATCH 6/6] powerpc: Add LINK_OFF calls in early C-code.

2010-11-24 Thread Joakim Tjernlund
Scott Wood wrote on 2010/11/24 18:16:56: > > On Wed, 24 Nov 2010 12:04:15 +0100 > Joakim Tjernlund wrote: > > > > > > > Scott Wood wrote on 2010/11/23 23:32:04: > > > > > > > > On Tue, 23 Nov 2010 23:14:06 +0100 > > > > Joakim Tjernlund wrote: > > > > > > > > > Scott Wood wrote on 2010/11/23 2

Re: [U-Boot] [PATCH 6/6] powerpc: Add LINK_OFF calls in early C-code.

2010-11-24 Thread Scott Wood
On Wed, 24 Nov 2010 12:04:15 +0100 Joakim Tjernlund wrote: > > > > Scott Wood wrote on 2010/11/23 23:32:04: > > > > > > On Tue, 23 Nov 2010 23:14:06 +0100 > > > Joakim Tjernlund wrote: > > > > > > > Scott Wood wrote on 2010/11/23 22:20:58: > > > > > "load address" being pre-relocation? Curren

Re: [U-Boot] [PATCH 6/6] powerpc: Add LINK_OFF calls in early C-code.

2010-11-24 Thread Joakim Tjernlund
> > Scott Wood wrote on 2010/11/23 23:32:04: > > > > On Tue, 23 Nov 2010 23:14:06 +0100 > > Joakim Tjernlund wrote: > > > > > Scott Wood wrote on 2010/11/23 22:20:58: > > > > "load address" being pre-relocation? Currently these must be equal > > > > (which doesn't seem particularly burdensome).

Re: [U-Boot] [PATCH 6/6] powerpc: Add LINK_OFF calls in early C-code.

2010-11-23 Thread Joakim Tjernlund
Scott Wood wrote on 2010/11/23 23:32:04: > > On Tue, 23 Nov 2010 23:14:06 +0100 > Joakim Tjernlund wrote: > > > Scott Wood wrote on 2010/11/23 22:20:58: > > > "load address" being pre-relocation? Currently these must be equal > > > (which doesn't seem particularly burdensome). > > > > Yes, but

Re: [U-Boot] [PATCH 6/6] powerpc: Add LINK_OFF calls in early C-code.

2010-11-23 Thread Graeme Russ
On Wed, Nov 24, 2010 at 10:24 AM, Scott Wood wrote: > On Wed, 24 Nov 2010 00:03:13 +0100 > Wolfgang Denk wrote: > >> Dear Scott Wood, >> >> In message <20101123163204.4f843...@udp111988uds.am.freescale.net> you wrote: >> > >> > How about playing with BATs before entering C code, so that the image

Re: [U-Boot] [PATCH 6/6] powerpc: Add LINK_OFF calls in early C-code.

2010-11-23 Thread Scott Wood
On Wed, 24 Nov 2010 00:03:13 +0100 Wolfgang Denk wrote: > Dear Scott Wood, > > In message <20101123163204.4f843...@udp111988uds.am.freescale.net> you wrote: > > > > How about playing with BATs before entering C code, so that the image > > always appears at the same effective address? > > Not al

Re: [U-Boot] [PATCH 6/6] powerpc: Add LINK_OFF calls in early C-code.

2010-11-23 Thread Wolfgang Denk
Dear Scott Wood, In message <20101123163204.4f843...@udp111988uds.am.freescale.net> you wrote: > > How about playing with BATs before entering C code, so that the image > always appears at the same effective address? Not all systems have BATs. Best regards, Wolfgang Denk -- DENX Software Engi

Re: [U-Boot] [PATCH 6/6] powerpc: Add LINK_OFF calls in early C-code.

2010-11-23 Thread Scott Wood
On Tue, 23 Nov 2010 23:14:06 +0100 Joakim Tjernlund wrote: > Scott Wood wrote on 2010/11/23 22:20:58: > > "load address" being pre-relocation? Currently these must be equal > > (which doesn't seem particularly burdensome). > > Yes, but in our case we update the boot in the field and we want to

Re: [U-Boot] [PATCH 6/6] powerpc: Add LINK_OFF calls in early C-code.

2010-11-23 Thread Joakim Tjernlund
Scott Wood wrote on 2010/11/23 22:20:58: > > On Tue, 23 Nov 2010 22:03:36 +0100 > Joakim Tjernlund wrote: > > > Scott Wood wrote on 2010/11/23 21:46:51: > > > "Only these" that you've found so far, for the board you've tried -- > > > > Yes, that is what I wrote. > > Yes, I was just emphasizing t

Re: [U-Boot] [PATCH 6/6] powerpc: Add LINK_OFF calls in early C-code.

2010-11-23 Thread Scott Wood
On Tue, 23 Nov 2010 22:03:36 +0100 Joakim Tjernlund wrote: > Scott Wood wrote on 2010/11/23 21:46:51: > > "Only these" that you've found so far, for the board you've tried -- > > Yes, that is what I wrote. Yes, I was just emphasizing that. :-) > > is it worth adding another pre-relocation lan

Re: [U-Boot] [PATCH 6/6] powerpc: Add LINK_OFF calls in early C-code.

2010-11-23 Thread Joakim Tjernlund
> > Scott Wood wrote on 2010/11/23 21:46:51: > > On Tue, 23 Nov 2010 19:48:51 +0100 > > Joakim Tjernlund wrote: > > Please document the specific circumstances in which one would need > > to use this (any data-segment pointer before relocation?). > > Any ptr needing fixups: > char *sptr = "hi"; >

Re: [U-Boot] [PATCH 6/6] powerpc: Add LINK_OFF calls in early C-code.

2010-11-23 Thread Joakim Tjernlund
Scott Wood wrote on 2010/11/23 21:46:51: > On Tue, 23 Nov 2010 19:48:51 +0100 > Joakim Tjernlund wrote: > > > Only these 2 call sites depends on fixups for my mpc8321 based > > board. > > --- > > arch/powerpc/cpu/mpc83xx/cpu_init.c |2 +- > > arch/powerpc/lib/board.c|2 +- > >

Re: [U-Boot] [PATCH 6/6] powerpc: Add LINK_OFF calls in early C-code.

2010-11-23 Thread Scott Wood
On Tue, 23 Nov 2010 19:48:51 +0100 Joakim Tjernlund wrote: > Only these 2 call sites depends on fixups for my mpc8321 based > board. > --- > arch/powerpc/cpu/mpc83xx/cpu_init.c |2 +- > arch/powerpc/lib/board.c|2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > dif

[U-Boot] [PATCH 6/6] powerpc: Add LINK_OFF calls in early C-code.

2010-11-23 Thread Joakim Tjernlund
Only these 2 call sites depends on fixups for my mpc8321 based board. --- arch/powerpc/cpu/mpc83xx/cpu_init.c |2 +- arch/powerpc/lib/board.c|2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/powerpc/cpu/mpc83xx/cpu_init.c b/arch/powerpc/cpu/mpc83xx/cpu