Re: [U-Boot] Problems with a P2020 board

2012-08-30 Thread Joakim Tjernlund
Hi. We have a new board which we are suing with U-boot. The CPU is a P2020. I am having a few minor problems, and I was hoping to get some help. - Boot Count. We are keen to use the boot counter feature, but I am struggling to find a suitable register in the P2020. The file

Re: [U-Boot] [PATCHv2 1/2] mpc85xx: Initial SP alignment is wrong.

2012-08-23 Thread Joakim Tjernlund
Andy Fleming aflem...@gmail.com wrote on 2012/08/22 23:08:45: On Mon, Jul 23, 2012 at 3:58 PM, Joakim Tjernlund joakim.tjernl...@transmode.se wrote: PowerPC mandates SP to be 16 bytes aligned. Furthermore, a stack frame is added, pointing to the reset vector which may in the way when gdb

Re: [U-Boot] [PATCHv2 1/2] mpc85xx: Initial SP alignment is wrong.

2012-08-23 Thread Joakim Tjernlund
Scott Wood scottw...@freescale.com wrote on 2012/08/23 18:53:14: On 08/23/2012 02:21 AM, Joakim Tjernlund wrote: Andy Fleming aflem...@gmail.com wrote on 2012/08/22 23:08:45: On Mon, Jul 23, 2012 at 3:58 PM, Joakim Tjernlund joakim.tjernl...@transmode.se wrote: PowerPC mandates SP

Re: [U-Boot] [PATCHv2 1/2] mpc85xx: Initial SP alignment is wrong.

2012-08-15 Thread Joakim Tjernlund
Scott Wood scottw...@freescale.com wrote on 2012/08/14 23:01:47: On 08/14/2012 03:55 PM, Joakim Tjernlund wrote: Ping? PowerPC mandates SP to be 16 bytes aligned. Furthermore, a stack frame is added, pointing to the reset vector which may in the way when gdb is walking the stack

Re: [U-Boot] [PATCHv2 1/2] mpc85xx: Initial SP alignment is wrong.

2012-08-15 Thread Joakim Tjernlund
Kumar Gala ga...@kernel.crashing.org wrote on 2012/08/14 23:28:45: On Jul 23, 2012, at 3:58 PM, Joakim Tjernlund wrote: PowerPC mandates SP to be 16 bytes aligned. Furthermore, a stack frame is added, pointing to the reset vector which may in the way when gdb is walking the stack because

Re: [U-Boot] [PATCHv2 1/2] mpc85xx: Initial SP alignment is wrong.

2012-08-14 Thread Joakim Tjernlund
intermediate values. Signed-off-by: Joakim Tjernlund joakim.tjernl...@transmode.se --- v2 - Address Scott Wood's comments arch/powerpc/cpu/mpc85xx/start.S | 16 +--- 1 files changed, 5 insertions(+), 11 deletions(-) diff --git arch/powerpc/cpu/mpc85xx/start.S arch/powerpc

Re: [U-Boot] [PATCHv2 2/2] powerpc: Stack Pointer not properly aligned

2012-08-14 Thread Joakim Tjernlund
Ping? The code first aligns the SP to 16 then subtract 8, making it 8 bytes aligned. Furthermore the initial stack frame not quite correct either. Signed-off-by: Joakim Tjernlund joakim.tjernl...@transmode.se --- v2 - Address Scott Wood's comments arch/powerpc/lib/board.c |5

Re: [U-Boot] [PATCH 2/2] mpc85xx: Initial SP alignment is wrong.

2012-07-23 Thread Joakim Tjernlund
Scott Wood scottw...@freescale.com wrote on 2012/07/20 23:11:33: On 07/20/2012 04:20 AM, Joakim Tjernlund wrote: PowerPC mandates SP to be 16 bytes aligned. Furthermore, a stack frame is added, pointing to the reset vector which is just get in the way when gdb is walking the stack

Re: [U-Boot] [PATCH 2/2] mpc85xx: Initial SP alignment is wrong.

2012-07-23 Thread Joakim Tjernlund
Scott Wood scottw...@freescale.com wrote on 2012/07/23 18:52:28: From: Scott Wood scottw...@freescale.com To: Joakim Tjernlund joakim.tjernl...@transmode.se, Cc: u-boot@lists.denx.de Date: 2012/07/23 18:52 Subject: Re: [U-Boot] [PATCH 2/2] mpc85xx: Initial SP alignment is wrong. On 07/21

[U-Boot] [PATCHv2 1/2] mpc85xx: Initial SP alignment is wrong.

2012-07-23 Thread Joakim Tjernlund
values. Signed-off-by: Joakim Tjernlund joakim.tjernl...@transmode.se --- v2 - Address Scott Wood's comments arch/powerpc/cpu/mpc85xx/start.S | 16 +--- 1 files changed, 5 insertions(+), 11 deletions(-) diff --git arch/powerpc/cpu/mpc85xx/start.S arch/powerpc/cpu/mpc85xx/start.S

[U-Boot] [PATCHv2 2/2] powerpc: Stack Pointer not properly aligned

2012-07-23 Thread Joakim Tjernlund
The code first aligns the SP to 16 then subtract 8, making it 8 bytes aligned. Furthermore the initial stack frame not quite correct either. Signed-off-by: Joakim Tjernlund joakim.tjernl...@transmode.se --- v2 - Address Scott Wood's comments arch/powerpc/lib/board.c |5 ++--- 1 files

Re: [U-Boot] [PATCH 2/2] mpc85xx: Initial SP alignment is wrong.

2012-07-23 Thread Joakim Tjernlund
Scott Wood scottw...@freescale.com wrote on 2012/07/23 19:28:20: On 07/23/2012 12:11 PM, Joakim Tjernlund wrote: ...but I was a bit confused when I thought it would help terminate things. The NULL LR only helps prevent finding something worse, if something happens to do a backtrace

Re: [U-Boot] [PATCH 2/2] mpc85xx: Initial SP alignment is wrong.

2012-07-21 Thread Joakim Tjernlund
Scott Wood scottw...@freescale.com wrote on 2012/07/20 23:11:33: On 07/20/2012 04:20 AM, Joakim Tjernlund wrote: PowerPC mandates SP to be 16 bytes aligned. Furthermore, a stack frame is added, pointing to the reset vector which is just get in the way when gdb is walking the stack because

Re: [U-Boot] [PATCH 1/2] powerpc: Stack Pointer must be 16 aligned

2012-07-21 Thread Joakim Tjernlund
Scott Wood scottw...@freescale.com wrote on 2012/07/20 23:12:49: From: Scott Wood scottw...@freescale.com To: Joakim Tjernlund joakim.tjernl...@transmode.se, Cc: u-boot@lists.denx.de Date: 2012/07/20 23:12 Subject: Re: [U-Boot] [PATCH 1/2] powerpc: Stack Pointer must be 16 aligned On 07

[U-Boot] [PATCH 1/2] powerpc: Stack Pointer must be 16 aligned

2012-07-20 Thread Joakim Tjernlund
The PowerPC ABI mandates the SP to be 16 bytes aligned, make is so. Signed-off-by: Joakim Tjernlund joakim.tjernl...@transmode.se --- arch/powerpc/lib/board.c |5 ++--- 1 files changed, 2 insertions(+), 3 deletions(-) diff --git arch/powerpc/lib/board.c arch/powerpc/lib/board.c index

[U-Boot] [PATCH 2/2] mpc85xx: Initial SP alignment is wrong.

2012-07-20 Thread Joakim Tjernlund
intermediate values. Signed-off-by: Joakim Tjernlund joakim.tjernl...@transmode.se --- arch/powerpc/cpu/mpc85xx/start.S | 16 +--- 1 files changed, 5 insertions(+), 11 deletions(-) diff --git arch/powerpc/cpu/mpc85xx/start.S arch/powerpc/cpu/mpc85xx/start.S index 8d66cf1..8c75af9 100644

[U-Boot] [PATCH] ppc: Create a stack frame for wait_ticks()

2012-07-17 Thread Joakim Tjernlund
wait_ticks() calls get_ticks() without building a back chain which makes gdb unhappy when doing back trace. This can also cause improper memory accesses. Signed-off-by: Joakim Tjernlund joakim.tjernl...@transmode.se --- arch/powerpc/lib/ticks.S |7 +-- 1 files changed, 5 insertions(+), 2

[U-Boot] Hidden BPs i gdb

2012-05-31 Thread Joakim Tjernlund
Just stumbled on a (mis) feature in gdb that I figured u-boot devs should know about: powerpc-softfloat_4.5.3-linux-gnu-gdb u-boot GNU gdb (Gentoo 7.4 p1) 7.4 Copyright (C) 2012 Free Software Foundation, Inc. License GPLv3+: GNU GPL version 3 or later http://gnu.org/licenses/gpl.html This is

Re: [U-Boot] [PATCH V2] MPC8xxx: Define cache ops for USB

2012-05-25 Thread Joakim Tjernlund
Hello Marek, On 5/25/2012 7:44 PM, Marek Vasut wrote: This patch conditionally defines flush_dcache_range() and invalidate_dcache_range() on MPC8xxx, to avoid EHCI complaining, resulting in the following output: snip + +void invalidate_dcache_range(unsigned long start, unsigned long

Re: [U-Boot] Default LAWBAR mapping at reset for mpc85xx?

2012-05-03 Thread Joakim Tjernlund
Schneider, Kolja kolja.schnei...@men.de wrote on 2012/05/03 10:46:55: If you mean the BDI is trying to initialize things rather than leave the system in its default state, don't do that. Yes, trying to get there. Easily done, just throw out everything in the [init] section. Or is

Re: [U-Boot] Default LAWBAR mapping at reset for mpc85xx?

2012-05-03 Thread Joakim Tjernlund
Wolfgang Denk w...@denx.de wrote on 2012/05/03 15:11:33: Dear Joakim, In message of35057017.5788f889-onc12579f3.00427b35-c12579f3.00442...@transmode.se you wrote: I found the problem, u-boot is somewhat flawed, it requires CONFIG_SYS_MONITOR_BASE to be base 2 even, must land on a

Re: [U-Boot] [PATCH] powerpc/85xx: fix NAND boot linker scripts for -fpic

2012-05-02 Thread Joakim Tjernlund
...@freescale.com Loks good Acked-by: Joakim Tjernlund joakim.tjernl...@transmode.se ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] Default LAWBAR mapping at reset for mpc85xx?

2012-05-02 Thread Joakim Tjernlund
Still trying to wrap my head around the P2010 cpu and its boot sequence(NOR) We can run the full u-boot it we use the BDI emulator but without emulator it won't boot. We have 64 MB NOR flash and have based out design on the P1020RDB and we boot from NOR. Using the emulator but with minimal

Re: [U-Boot] Default LAWBAR mapping at reset for mpc85xx?

2012-05-02 Thread Joakim Tjernlund
Scott Wood scottw...@freescale.com wrote on 2012/05/02 21:34:18: On 05/02/2012 02:04 PM, Joakim Tjernlund wrote: Still trying to wrap my head around the P2010 cpu and its boot sequence(NOR) Yeah, it's a bit convoluted. Yes, fully agreed. I miss the sane 0 address boot vector too. We

Re: [U-Boot] [PATCH] PowerPC: Change -fpic flag to -fPIC flag in the config.mk

2012-04-27 Thread Joakim Tjernlund
Scott Wood scottw...@freescale.com wrote on 2012/04/26 23:26:52: On 04/26/2012 01:53 AM, Joakim Tjernlund wrote: Scott Wood scottw...@freescale.com wrote on 2012/04/25 21:01:00: On 04/18/2012 05:07 AM, Joakim Tjernlund wrote: Your linker file is buggy I think. I found u-boot-nand_spl.lds

Re: [U-Boot] [PATCH] PowerPC: Change -fpic flag to -fPIC flag in the config.mk

2012-04-27 Thread Joakim Tjernlund
Scott Wood scottw...@freescale.com wrote on 2012/04/26 23:26:52: On 04/26/2012 01:53 AM, Joakim Tjernlund wrote: Scott Wood scottw...@freescale.com wrote on 2012/04/25 21:01:00: On 04/18/2012 05:07 AM, Joakim Tjernlund wrote: Your linker file is buggy I think. I found u-boot

Re: [U-Boot] [PATCH] PowerPC: Change -fpic flag to -fPIC flag in the config.mk

2012-04-26 Thread Joakim Tjernlund
Scott Wood scottw...@freescale.com wrote on 2012/04/25 21:01:00: On 04/18/2012 05:07 AM, Joakim Tjernlund wrote: Your linker file is buggy I think. I found u-boot-nand_spl.lds, is that the one? That's the one for the SPL part. OK, good. Check out that files reloc part: .reloc

Re: [U-Boot] [PATCH 2/4][v2] powerpc/85xx:Fix MSR[DE] bit in MSR to support debugger

2012-04-26 Thread Joakim Tjernlund
Scott Wood scottw...@freescale.com wrote on 2012/04/25 20:59:22: On 04/25/2012 01:55 PM, Joakim Tjernlund wrote: Scott Wood scottw...@freescale.com wrote on 2012/04/25 20:43:22: On 04/25/2012 05:57 AM, Joakim Tjernlund wrote: Debugging of e500 and e500v1 processer requires MSR[DE

Re: [U-Boot] [PATCH 2/4][v2] powerpc/85xx:Fix MSR[DE] bit in MSR to support debugger

2012-04-25 Thread Joakim Tjernlund
Debugging of e500 and e500v1 processer requires MSR[DE] bit to be set always. Where MSR = Machine State register Make sure of MSR[DE] bit is set uniformaly across the different execution address space i.e. AS0 and AS1. Hi We are trying to bring up our custom P2010 RDB based board. boot is

Re: [U-Boot] [PATCH 2/4][v2] powerpc/85xx:Fix MSR[DE] bit in MSR to support debugger

2012-04-25 Thread Joakim Tjernlund
Scott Wood scottw...@freescale.com wrote on 2012/04/25 20:43:22: On 04/25/2012 05:57 AM, Joakim Tjernlund wrote: Debugging of e500 and e500v1 processer requires MSR[DE] bit to be set always. Where MSR = Machine State register Make sure of MSR[DE] bit is set uniformaly across

Re: [U-Boot] [PATCH] PowerPC: Change -fpic flag to -fPIC flag in the config.mk

2012-04-18 Thread Joakim Tjernlund
Wolfgang Denk wrote: Dear Chunhe Lan, In message 1334719161-3500-1-git-send-email-chunhe@freescale.com you wrote: The -fPIC/-fpic flag belongs with -mrelocatable. The -fpic flag can limit the size of the GOT and produce smaller binaries, so it causes some GOT entries to be

Re: [U-Boot] [PATCH] Prevent malloc with size 0

2012-04-02 Thread Joakim Tjernlund
Graeme Russ graeme.r...@gmail.com wrote on 2012/04/02 05:05:51: Hi Marek, On Mon, Apr 2, 2012 at 12:51 PM, Marek Vasut marek.va...@gmail.com wrote: Dear Graeme Russ, Hi Marek, Bottom line is, we could do either and we would be 100% compliant with the C standard The question is,

Re: [U-Boot] [PATCH] Prevent malloc with size 0

2012-04-02 Thread Joakim Tjernlund
Hi Marek, On Mon, Apr 2, 2012 at 1:36 PM, Marek Vasut marek.va...@gmail.com wrote: Dear Mike Frysinger, On Sunday 01 April 2012 20:25:44 Graeme Russ wrote: b) The code calling malloc(0) is making a perfectly legitimate assumption    based on how glibc handles malloc(0) not

Re: [U-Boot] [PATCH] Prevent malloc with size 0

2012-04-02 Thread Joakim Tjernlund
Hi Grame Graeme Russ graeme.r...@gmail.com wrote on 2012/04/02 09:17:44: Hi Joakim, On Apr 2, 2012 4:55 PM, Joakim Tjernlund joakim.tjernl...@transmode.se wrote: Hi Marek, On Mon, Apr 2, 2012 at 1:36 PM, Marek Vasut marek.va...@gmail.com wrote: Dear Mike Frysinger

Re: [U-Boot] [PATCH] Prevent malloc with size 0

2012-04-02 Thread Joakim Tjernlund
Marek Vasut marek.va...@gmail.com wrote on 2012/04/02 16:05:13: Dear Joakim Tjernlund, Hi Grame Graeme Russ graeme.r...@gmail.com wrote on 2012/04/02 09:17:44: Hi Joakim, On Apr 2, 2012 4:55 PM, Joakim Tjernlund joakim.tjernl...@transmode.se wrote: Hi Marek

Re: [U-Boot] [PATCH] Prevent malloc with size 0

2012-04-02 Thread Joakim Tjernlund
Marek Vasut marek.va...@gmail.com wrote on 2012/04/02 16:42:30: Dear Joakim Tjernlund, Marek Vasut marek.va...@gmail.com wrote on 2012/04/02 16:05:13: Dear Joakim Tjernlund, Hi Grame Graeme Russ graeme.r...@gmail.com wrote on 2012/04/02 09:17:44: Hi Joakim

Re: [U-Boot] [PATCH] Prevent malloc with size 0

2012-04-02 Thread Joakim Tjernlund
Marek Vasut marek.va...@gmail.com wrote on 2012/04/02 17:23:03: Dear Joakim Tjernlund, Marek Vasut marek.va...@gmail.com wrote on 2012/04/02 16:42:30: Dear Joakim Tjernlund, Marek Vasut marek.va...@gmail.com wrote on 2012/04/02 16:05:13: Dear Joakim Tjernlund, Hi

Re: [U-Boot] [PATCH] Prevent malloc with size 0

2012-04-02 Thread Joakim Tjernlund
Marek Vasut marek.va...@gmail.com wrote on 2012/04/02 18:39:33: From: Marek Vasut marek.va...@gmail.com Dear Joakim Tjernlund, Marek Vasut marek.va...@gmail.com wrote on 2012/04/02 17:23:03: Dear Joakim Tjernlund, Marek Vasut marek.va...@gmail.com wrote on 2012/04/02 16:42:30

Re: [U-Boot] [PATCH] Prevent malloc with size 0

2012-04-02 Thread Joakim Tjernlund
Marek Vasut marek.va...@gmail.com wrote on 2012/04/02 20:00:03: Dear Joakim Tjernlund, Marek Vasut marek.va...@gmail.com wrote on 2012/04/02 18:39:33: From: Marek Vasut marek.va...@gmail.com Dear Joakim Tjernlund, Marek Vasut marek.va...@gmail.com wrote on 2012/04/02 17:23:03

Re: [U-Boot] [PATCH] Prevent malloc with size 0

2012-04-02 Thread Joakim Tjernlund
Graeme Russ graeme.r...@gmail.com wrote on 2012/04/02 22:28:46: From: Graeme Russ graeme.r...@gmail.com On 04/02/2012 05:40 PM, Joakim Tjernlund wrote: Hi Grame Graeme Russ graeme.r...@gmail.com wrote on 2012/04/02 09:17:44: Hi Joakim, On Apr 2, 2012 4:55 PM, Joakim Tjernlund

Re: [U-Boot] [PATCH] Prevent malloc with size 0

2012-04-02 Thread Joakim Tjernlund
vapierfil...@gmail.com wrote on 2012/04/02 21:14:14: On Mon, Apr 2, 2012 at 14:40, Joakim Tjernlund wrote: Marek Vasut marek.va...@gmail.com wrote on 2012/04/02 20:00:03: Dear Joakim Tjernlund, Depends, if writing generic code for lots of OS:es you cannot rely malloc(0). Writing kernel

Re: [U-Boot] [PATCH] Prevent malloc with size 0

2012-04-02 Thread Joakim Tjernlund
Graeme Russ graeme.r...@gmail.com wrote on 2012/04/02 22:59:57: On Apr 3, 2012 6:57 AM, Joakim Tjernlund joakim.tjernl...@transmode.se wrote: Graeme Russ graeme.r...@gmail.com wrote on 2012/04/02 22:28:46: From: Graeme Russ graeme.r...@gmail.com On 04/02/2012 05:40 PM, Joakim

Re: [U-Boot] [PATCH] Prevent malloc with size 0

2012-04-01 Thread Joakim Tjernlund
Dear Mike Frysinger, On Thursday, October 21, 2010 17:10:31 Graeme Russ wrote: On 22/10/10 06:51, Mike Frysinger wrote: have u-boot return an error. Is NULL what you consider to be an error yes Besides, is not free(NULL) valid (does nothing) as well? yes, free(NULL)

Re: [U-Boot] [PATCH] Prevent malloc with size 0

2012-04-01 Thread Joakim Tjernlund
Marek Vasut marek.va...@gmail.com wrote on 2012/04/01 16:01:56: Dear Joakim Tjernlund, Dear Mike Frysinger, On Thursday, October 21, 2010 17:10:31 Graeme Russ wrote: On 22/10/10 06:51, Mike Frysinger wrote: have u-boot return an error. Is NULL what you consider

Re: [U-Boot] [PATCH 2/2] Powerpc/i2c: Force i2c to become bus master out of reset

2011-10-27 Thread Joakim Tjernlund
From: chang-ming.hu...@freescale.com From: Jerry Huang chang-ming.hu...@freescale.com It is sometimes necessary to force the I2C module to become the I2C bus master out of reset and drive SCL(even though SDA may already be driven, which indicates that the bus is busy). This can occur when a

Re: [U-Boot] [PATCH 2/2] Powerpc/i2c: Force i2c to become bus master out of reset

2011-10-27 Thread Joakim Tjernlund
Huang Changming-R66093 r66...@freescale.com wrote on 2011/10/27 09:40:23: Thanks and Best Regards Jerry Huang -Original Message- From: Joakim Tjernlund [mailto:joakim.tjernl...@transmode.se] Sent: Thursday, October 27, 2011 3:26 PM To: Huang Changming-R66093 Cc: u-boot

Re: [U-Boot] [PATCH 2/2] Powerpc/i2c: Force i2c to become bus master out of reset

2011-10-27 Thread Joakim Tjernlund
PM To: Joakim Tjernlund Cc: u-boot@lists.denx.de Subject: Re: [U-Boot] [PATCH 2/2] Powerpc/i2c: Force i2c to become bus master out of reset Thanks and Best Regards Jerry Huang -Original Message- From: Joakim Tjernlund [mailto:joakim.tjernl...@transmode.se

Re: [U-Boot] [PATCH 2/2] Powerpc/i2c: Force i2c to become bus master out of reset

2011-10-27 Thread Joakim Tjernlund
Huang Changming-R66093 r66...@freescale.com wrote on 2011/10/27 11:26:04: The above sequence is different than the kernel version, why? I don't know the kernel version, but I write the u-boot code according to the Reference Manual of PowerPC, e.g p1022: 11.5.6 Generation of

Re: [U-Boot] the processor performs an operation overflows,U-boot-based platform

2011-05-18 Thread Joakim Tjernlund
unsigned long get_tbus (void) { unsigned long tbl=0; unsigned long tbu1=0, tbu2=0; unsigned long us=0; unsigned long long tmp=0; unsigned long tbclk = os_get_tbclk(); // get the timebase ticks do { __asm__ __volatile__ (mftbu %0:=r (tbu1):);

Re: [U-Boot] [PATCH] MPC8260: Fix compile problems with hymod board

2011-05-01 Thread Joakim Tjernlund
Wolfgang Denk w...@denx.de wrote on 2011/05/01 16:35:06: Commit 9d8fbd1 powerpc, 8xx: Fixup all 8xx u-boot.lds scripts broke building of the MPC8260 based hymod board. Fix this. Signed-off-by: Wolfgang Denk w...@denx.de Cc: Murray Jensen murray.jen...@csiro.au Cc: Joakim Tjernlund

Re: [U-Boot] [PATCH 2/3] Fix a few gcc warnings.

2011-04-25 Thread Joakim Tjernlund
vapierfil...@gmail.com wrote on 2011/04/25 06:13:20: On Sun, Apr 24, 2011 at 7:42 PM, Joakim Tjernlund wrote: vapierfil...@gmail.com wrote on 2011/04/25 00:38:31: On Sun, Apr 24, 2011 at 6:14 PM, Wolfgang Denk wrote: Joakim Tjernlund wrote: --- a/examples/standalone/timer.c +++ b

Re: [U-Boot] [PATCH 2/3] Fix a few gcc warnings.

2011-04-25 Thread Joakim Tjernlund
vapierfil...@gmail.com wrote on 2011/04/25 07:40:00: On Mon, Apr 25, 2011 at 12:13 AM, Mike Frysinger wrote: however, that doesnt change my original point ... we shouldnt be fixing things like this that have no relevance in the u-boot world. disable the warning flags in the build system.

Re: [U-Boot] [PATCH 3/3] powerpc, 8xx: adapt a few linker scripts so it will build.

2011-04-25 Thread Joakim Tjernlund
Wolfgang Denk w...@denx.de wrote on 2011/04/25 10:35:25: Dear Joakim Tjernlund, In message of102cc99d.b86b21b6-onc125787c.007cb439-c125787c.0080f...@transmode.se you wrote: Well, your gcc is different from mine and I know nothing about it as you won't say anything about

[U-Boot] [PATCH] Fix a few gcc warnings.

2011-04-25 Thread Joakim Tjernlund
': codec.c:585: warning: array subscript is above array bounds codec.c: In function 'codsp_write_sop_int': codec.c:512: warning: array subscript is above array bounds Signed-off-by: Joakim Tjernlund joakim.tjernl...@transmode.se --- -V3 use puts() and constify usage string in timer.c board/netta

Re: [U-Boot] [PATCH 3/3] powerpc, 8xx: adapt a few linker scripts so it will build.

2011-04-25 Thread Joakim Tjernlund
Wolfgang Denk w...@denx.de wrote on 2011/04/25 12:17:11: Dear Joakim Tjernlund, In message of522299d7.a02484a0-onc125787d.003187d6-c125787d.0032f...@transmode.se you wrote: If in doubt, use ELDK. All examples were compiled with ELDK 4.2 (GCC 4.2.2) for ppc_8xx. Have you tried

Re: [U-Boot] [PATCH 2/3] Fix a few gcc warnings.

2011-04-25 Thread Joakim Tjernlund
vapierfil...@gmail.com wrote on 2011/04/25 19:53:50: On Mon, Apr 25, 2011 at 13:45, Scott Wood wrote: On Mon, 25 Apr 2011 00:13:20 -0400 Mike Frysinger wrote: On Sun, Apr 24, 2011 at 7:42 PM, Joakim Tjernlund wrote: vapierfil...@gmail.com wrote on 2011/04/25 00:38:31: probably want

Re: [U-Boot] [PATCH] disable security warning flags when possible

2011-04-25 Thread Joakim Tjernlund
Mike Frysinger vap...@gentoo.org wrote on 2011/04/25 20:06:40: Some toolchains enable security warning flags by default, but these don't really make sense in the u-boot world. Such as forcing changes like: -printf(foo); +printf(%s, foo); So disable the flags when the compiler

[U-Boot] [PATCH 0/3] Make 8xx build with -msingle-pic-base

2011-04-24 Thread Joakim Tjernlund
With these patches 8xx willbuild again for me with a number of gcc's even with Add support for -msingle-pic-base I hope WD's gcc will also work. Joakim Tjernlund (3): powerpc, 8xx: Fix fallout from Fixup all 8xx u-boot.lds scripts Fix a few gcc warnings. powerpc, 8xx: adapt a few linker

[U-Boot] [PATCH 1/3] powerpc, 8xx: Fix fallout from Fixup all 8xx u-boot.lds scripts

2011-04-24 Thread Joakim Tjernlund
Two linker scripts for 8xx was missed. Signed-off-by: Joakim Tjernlund joakim.tjernl...@transmode.se --- board/matrix_vision/mvsmr/u-boot.lds |2 +- board/rsdproto/u-boot.lds|7 --- 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/board/matrix_vision/mvsmr/u

[U-Boot] [PATCH 2/3] Fix a few gcc warnings.

2011-04-24 Thread Joakim Tjernlund
Noticed while building all of mpc8xx Signed-off-by: Joakim Tjernlund joakim.tjernl...@transmode.se --- board/netta/codec.c |6 +++--- board/siemens/IAD210/atm.c |4 ++-- examples/standalone/timer.c |4 ++-- 3 files changed, 7 insertions(+), 7 deletions(-) diff --git a/board

[U-Boot] [PATCH 3/3] powerpc, 8xx: adapt a few linker scripts so it will build.

2011-04-24 Thread Joakim Tjernlund
Seems like recent linker/fpic changes broke a few 8xx boards. This makes them build again. Signed-off-by: Joakim Tjernlund joakim.tjernl...@transmode.se --- board/LEOX/elpt860/u-boot.lds |2 +- board/etx094/u-boot.lds |2 +- board/svm_sc8xx/u-boot.lds|2 +- board/tqc/tqm8xx

Re: [U-Boot] [PATCH 0/3] Make 8xx build with -msingle-pic-base

2011-04-24 Thread Joakim Tjernlund
Wolfgang Denk w...@denx.de wrote on 2011/04/24 16:30:14: Dear Joakim Tjernlund, In message 1303633774-22961-1-git-send-email-joakim.tjernl...@transmode.se you wrote: With these patches 8xx willbuild again for me with a number of gcc's even with Add support for -msingle-pic-base I hope

Re: [U-Boot] [PATCH 3/3] powerpc, 8xx: adapt a few linker scripts so it will build.

2011-04-24 Thread Joakim Tjernlund
Wolfgang Denk w...@denx.de wrote on 2011/04/24 22:48:09: Dear Joakim Tjernlund, In message 1303633774-22961-4-git-send-email-joakim.tjernl...@transmode.se you wrote: Seems like recent linker/fpic changes broke a few 8xx boards. This makes them build again. Signed-off-by: Joakim

Re: [U-Boot] [PATCH 2/3] Fix a few gcc warnings.

2011-04-24 Thread Joakim Tjernlund
Wolfgang Denk w...@denx.de wrote on 2011/04/25 00:14:58: Dear Joakim Tjernlund, In message 1303633774-22961-3-git-send-email-joakim.tjernl...@transmode.se you wrote: Noticed while building all of mpc8xx Please include in the commit message what the compiler warnings were, and which

Re: [U-Boot] [PATCH 2/3] Fix a few gcc warnings.

2011-04-24 Thread Joakim Tjernlund
vapierfil...@gmail.com wrote on 2011/04/25 00:38:31: On Sun, Apr 24, 2011 at 6:14 PM, Wolfgang Denk wrote: Joakim Tjernlund wrote: --- a/examples/standalone/timer.c +++ b/examples/standalone/timer.c @@ -186,7 +186,7 @@ int timer (int argc, char * const argv[])       /* clear all events

Re: [U-Boot] [PATCH] PowerPC: Move -fPIC flag to common place

2011-04-22 Thread Joakim Tjernlund
Wolfgang Denk w...@denx.de wrote on 2011/04/20 22:13:35: Dear Joakim Tjernlund, In message of8b32e14f.150b51b9-onc1257878.0044927e-c1257878.00452...@transmode.se you wrote: OK, I managed to script the change, patch last in mail. Thanks. However CROSS_COMPILE=powerpc-softfloat

Re: [U-Boot] [PATCH] PowerPC: Move -fPIC flag to common place

2011-04-20 Thread Joakim Tjernlund
Wolfgang Denk w...@denx.de wrote on 2011/04/20 00:41:01: Dear Joakim Tjernlund, In message ofbc9c03bc.436c27c7-onc1257877.00797190-c1257877.007a0...@transmode.se you wrote: Yes, but you yorself pointed out that commit 337f5f5 missed a large number of boards, leaving the tree

Re: [U-Boot] [PATCH] PowerPC: Move -fPIC flag to common place

2011-04-19 Thread Joakim Tjernlund
Wolfgang Denk w...@denx.de wrote on 2011/04/19 22:25:25: Dear Joakim Tjernlund, In message of484eff4d.45d9734b-onc1257870.002e25b2-c1257870.002f0...@transmode.se you wrote: It seems this change increases the code size somehow - which I would not expect from your comment above

Re: [U-Boot] [PATCH] PowerPC: Move -fPIC flag to common place

2011-04-19 Thread Joakim Tjernlund
Wolfgang Denk w...@denx.de wrote on 2011/04/19 23:35:40: Dear Joakim Tjernlund, In message ofc3d6185b.85dce339-onc1257877.007406b0-c1257877.00746...@transmode.se you wrote: Can you please provide a fix? I'd like to get rid of this build error ASAP. We could, of course, simply

Re: [U-Boot] [PATCH] PowerPC: Move -fPIC flag to common place

2011-04-19 Thread Joakim Tjernlund
Wolfgang Denk w...@denx.de wrote on 2011/04/20 00:05:27: From: Wolfgang Denk w...@denx.de To: Joakim Tjernlund joakim.tjernl...@transmode.se Cc: u-boot@lists.denx.de Date: 2011/04/20 00:05 Subject: Re: [U-Boot] [PATCH] PowerPC: Move -fPIC flag to common place Dear Joakim Tjernlund

Re: [U-Boot] [PATCH] PowerPC: Move -fPIC flag to common place

2011-04-19 Thread Joakim Tjernlund
Wolfgang Denk w...@denx.de wrote on 2011/04/20 00:41:01: Dear Joakim Tjernlund, In message ofbc9c03bc.436c27c7-onc1257877.00797190-c1257877.007a0...@transmode.se you wrote: Yes, but you yorself pointed out that commit 337f5f5 missed a large number of boards, leaving the tree

Re: [U-Boot] [PATCH] JFFS2: accelerate scanning.

2011-04-13 Thread Joakim Tjernlund
Hi Baidu, diff --git a/fs/jffs2/jffs2_1pass.c b/fs/jffs2/jffs2_1pass.c index dfb1745..f38f755 100644 --- a/fs/jffs2/jffs2_1pass.c +++ b/fs/jffs2/jffs2_1pass.c @@ -1441,7 +1441,7 @@ dump_dirents(struct b_lists *pL) } #endif -#define DEFAULT_EMPTY_SCAN_SIZE 4096 +#define

Re: [U-Boot] [PATCH] PowerPC: Move -fPIC flag to common place

2011-04-12 Thread Joakim Tjernlund
Dear Joakim Tjernlund, In message 1291642606-4290-1-git-send-email-joakim.tjernl...@transmode.se you wrote: The -fPIC flag belongs with -mrelocatable, move it there. Also change -fPIC to -fpic as this produces smaller binaries. However, currently -mrelocatable promotes -fpic to -fPIC

Re: [U-Boot] Booting kernel from NAND flash on AT91SAM9 custom board using fsload

2011-03-30 Thread Joakim Tjernlund
Dear Nicholas Kinar, In message 4d92428e.6030...@usask.ca you wrote: (1) Does replacing jffs2_1pass.c with jffs2_nand_1pass.c in the fs/jffs2 directory influence the robustness of the fsload code? JFFS2 is more or less deprecated these days. FOr new projects, we recommend to use UBI/

Re: [U-Boot] [PATCH] mpc83xx: Use correct register to calculate clocks.

2011-02-03 Thread Joakim Tjernlund
-by: Joakim Tjernlund joakim.tjernl...@transmode.se --- arch/powerpc/cpu/mpc83xx/speed.c | 15 --- include/mpc83xx.h|7 +++ 2 files changed, 15 insertions(+), 7 deletions(-) diff --git a/arch/powerpc/cpu/mpc83xx/speed.c b/arch/powerpc/cpu/mpc83xx/speed.c

Re: [U-Boot] [PATCH] i2c: Fix chip address.0

2011-01-30 Thread Joakim Tjernlund
Heiko Schocher h...@denx.de wrote on 2011/01/31 08:18:56: Hello Joakim, Joakim Tjernlund write: According to the i2c command alen 0 should be allowed but this is not allowed in the implementation, fix. Also cleanup all cmd_usage cases while at it. It is cleaner to return cmd_usage

[U-Boot] [PATCH] i2c: Fix chip address.0

2011-01-28 Thread Joakim Tjernlund
According to the i2c command alen 0 should be allowed but this is not allowed in the implementation, fix. Also cleanup all cmd_usage cases while at it. It is cleaner to return cmd_usage(cmdtp); instead of { cmd_usage(cmdtp); return 1; } Signed-off-by: Joakim Tjernlund joakim.tjernl

[U-Boot] [PATCH] mpc83xx: Use correct register to calculate clocks.

2011-01-27 Thread Joakim Tjernlund
Use SPMR instead of HRCWL when calculating clocks as HCRWL may be changed and the CPU will not pick up all changes until there is a POR. u-boot will think SPMF has changed and get the clocks wrong. Signed-off-by: Joakim Tjernlund joakim.tjernl...@transmode.se --- arch/powerpc/cpu/mpc83xx/speed.c

Re: [U-Boot] [PATCH] Top config.mk: add include/config.mk

2011-01-23 Thread Joakim Tjernlund
Seems to that the top level config.mk should include the auto generated include/config.mk so that all Makefile's pickup those definitions. Signed-off-by: Joakim Tjernlund joakim.tjernl...@transmode.se --- config.mk |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git

Re: [U-Boot] [PATCH] Top config.mk: add include/config.mk

2011-01-23 Thread Joakim Tjernlund
Wolfgang Denk w...@denx.de wrote on 2011/01/23 20:35:48: Dear Joakim Tjernlund, In message of749bbc82.7aa2f037-onc1257821.0069eda0-c1257821.0069f...@transmode.se you wrote: Seems to that the top level config.mk should include the auto generated include/config.mk so that all

Re: [U-Boot] [PATCH] Top config.mk: add include/config.mk

2011-01-23 Thread Joakim Tjernlund
Wolfgang Denk w...@denx.de wrote on 2011/01/23 23:50:03: Dear Joakim Tjernlund, In message of0949c72e.59c831f7-onc1257821.006ee01d-c1257821.006ff...@transmode.se you wrote: What is the exact problem you are trying to fix? Do you have a test case? Example from my board/config.mk

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

2011-01-18 Thread Joakim Tjernlund
Scott Wood scottw...@freescale.com wrote on 2011/01/18 18:27:49: On Tue, 18 Jan 2011 01:18:34 +0100 Joakim Tjernlund joakim.tjernl...@transmode.se wrote: How do you think a solution for all boards would look like? The only other method is can think of is some MMU trickery and I don't even

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

2011-01-18 Thread Joakim Tjernlund
Scott Wood scottw...@freescale.com wrote on 2011/01/18 21:24:16: On Tue, 18 Jan 2011 21:06:34 +0100 Wolfgang Denk w...@denx.de wrote: Dear Joakim Tjernlund, In message of287114d8.013b6e51-onc125781c.00667dcd-c125781c.00673...@transmode.se you wrote: boards which I don't think

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

2011-01-18 Thread Joakim Tjernlund
Wolfgang Denk w...@denx.de wrote on 2011/01/18 21:06:34: Dear Joakim Tjernlund, In message of287114d8.013b6e51-onc125781c.00667dcd-c125781c.00673...@transmode.se you wrote: You don't need to make the MMU trick work on all boards, just your board (or cpu family), because it wouldn't

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

2011-01-18 Thread Joakim Tjernlund
Wolfgang Denk w...@denx.de wrote on 2011/01/18 22:25:04: Dear Joakim Tjernlund, In message ofbbe91305.b19350d9-onc125781c.007192fc-c125781c.00742...@transmode.se you wrote: Ah, finally you make sense to me. I actually tested this with mainline on my board so it is not completely

[U-Boot] [PATCH] Top config.mk: add include/config.mk

2011-01-17 Thread Joakim Tjernlund
Seems to that the top level config.mk should include the auto generated include/config.mk so that all Makefile's pickup those definitions. Signed-off-by: Joakim Tjernlund joakim.tjernl...@transmode.se --- config.mk |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git

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

2011-01-17 Thread Joakim Tjernlund
Wolfgang Denk w...@denx.de wrote on 2011/01/17 23:11:59: Dear Joakim Tjernlund, In message of6a573b77.9119deb0-onc1257814.0069a574-c1257814.00753...@transmode.se you wrote: It would break only if link address != load address. That is, if you want to use my new CONFIG_SYS_TRUE_PIC

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

2011-01-17 Thread Joakim Tjernlund
Wolfgang Denk w...@denx.de wrote on 2011/01/18 00:42:10: Dear Joakim Tjernlund, In message of71b40f2e.5c935ba8-onc125781b.007cb9e6-c125781b.007e5...@transmode.se you wrote: No other board is broken. This new function is neutral to other boards. Well, I see this differntly

Re: [U-Boot] [PATCH 2/2] PowerPC, nand_spl: Add relocation support for -fpic

2011-01-12 Thread Joakim Tjernlund
Wolfgang Denk w...@denx.de wrote on 2010/12/17 17:13:47: Dear Joakim Tjernlund, In message of3301f88e.e61f6369-onc12577fc.00573ebc-c12577fc.00576...@transmode.se you wrote: Great! then we got: [PATCH] PowerPC: Move -fPIC flag to common place [PATCH] PowerPC: Add

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

2011-01-10 Thread Joakim Tjernlund
Scott Wood scottw...@freescale.com wrote on 2011/01/10 19:24:02: On Sun, 9 Jan 2011 21:48:47 +0100 Joakim Tjernlund joakim.tjernl...@transmode.se wrote: Wolfgang Denk w...@denx.de wrote on 2011/01/09 21:29:04: Dear Joakim Tjernlund, In message 1292838435-14958-4-git-send

Re: [U-Boot] [PATCH 1/4] mpc83xx: Make start.S true PIC

2011-01-09 Thread Joakim Tjernlund
Wolfgang Denk w...@denx.de wrote on 2011/01/09 21:44:08: Dear Joakim Tjernlund, In message 1292838435-14958-1-git-send-email-joakim.tjernl...@transmode.se you wrote: Remove dependencies on link address. Use GOT and add an new function to calculate the actual address. Signed-off

Re: [U-Boot] [PATCH 3/4] mpc83xx: Add true PIC support.

2011-01-09 Thread Joakim Tjernlund
Wolfgang Denk w...@denx.de wrote on 2011/01/09 21:54:34: Dear Joakim Tjernlund, In message 1292838435-14958-3-git-send-email-joakim.tjernl...@transmode.se you wrote: By copying the GOT to the end of the INIT_RAM(dcache) and relocating it there, it is much esier to support true PIC on u

Re: [U-Boot] [PATCH 1/4] mpc83xx: Make start.S true PIC

2011-01-09 Thread Joakim Tjernlund
Wolfgang Denk w...@denx.de wrote on 2011/01/10 00:25:59: Dear Joakim Tjernlund, In message of7f8bae70.ce1486c7-onc1257813.00727b82-c1257813.0072b...@transmode.se you wrote: But your new code does not reference CONFIG_SYS_MONITOR_BASE at all, but uses CONFIG_SYS_FLASH_BASE instead

Re: [U-Boot] [PATCH RFC] armv7: fixloop: don't fixup if location is NULL

2011-01-08 Thread Joakim Tjernlund
Le 08/01/2011 11:49, Albert ARIBAUD a écrit : In my investigations for 'NULL fixup' (- see http://thread.gmane.org/gmane.comp.boot-loaders.u-boot/89864/focus=89906) pointed out that only symbols in 'absolute fixup' loop could be 'NULL' if there is a not aliased/empty weakly linked

Re: [U-Boot] [PATCH v2, RFC] armv7: fixloop: don't fixup if location is invalid on RAM

2011-01-04 Thread Joakim Tjernlund
Need to check that location is valid on RAM before the fixup. Why this change? The [PATCH RFC] armv7: fixloop: don't fixup if location is NULL should be what you need. You could mention what causes these NULL fixups, here is commit entry for powerpc: powerpc: do not fixup NULL ptrs The

Re: [U-Boot] [PATCH v2, RFC] armv7: fixloop: don't fixup if location is invalid on RAM

2011-01-04 Thread Joakim Tjernlund
Minkyu Kang proms...@gmail.com wrote on 2011/01/04 11:04:57: Dear Joakim Tjernlund, On 4 January 2011 18:49, Joakim Tjernlund joakim.tjernl...@transmode.se wrote: Need to check that location is valid on RAM before the fixup. Why this change? The [PATCH RFC] armv7: fixloop: don't

Re: [U-Boot] [PATCH v2, RFC] armv7: fixloop: don't fixup if location is invalid on RAM

2011-01-04 Thread Joakim Tjernlund
Minkyu Kang proms...@gmail.com wrote on 2011/01/04 12:02:29: Dear Joakim Tjernlund, On 4 January 2011 19:31, Joakim Tjernlund joakim.tjernl...@transmode.se wrote: Minkyu Kang proms...@gmail.com wrote on 2011/01/04 11:04:57: Dear Joakim Tjernlund, On 4 January 2011 18:49, Joakim

[U-Boot] [PATCH 1/4] mpc83xx: Make start.S true PIC

2010-12-20 Thread Joakim Tjernlund
Remove dependencies on link address. Use GOT and add an new function to calculate the actual address. Signed-off-by: Joakim Tjernlund joakim.tjernl...@transmode.se --- arch/powerpc/cpu/mpc83xx/start.S | 35 +++ 1 files changed, 27 insertions(+), 8 deletions

[U-Boot] [PATCH 3/4] mpc83xx: Add true PIC support.

2010-12-20 Thread Joakim Tjernlund
-base option to gcc. Signed-off-by: Joakim Tjernlund joakim.tjernl...@transmode.se --- arch/powerpc/cpu/mpc83xx/start.S | 36 1 files changed, 36 insertions(+), 0 deletions(-) diff --git a/arch/powerpc/cpu/mpc83xx/start.S b/arch/powerpc/cpu/mpc83xx/start.S

[U-Boot] [PATCH 2/4] mpc83xx: Add link vs. load address calculation

2010-12-20 Thread Joakim Tjernlund
link_off calculates the difference between link address and actual load address. This is a must for true PIC u-boot. Signed-off-by: Joakim Tjernlund joakim.tjernl...@transmode.se --- arch/powerpc/cpu/mpc83xx/start.S | 26 ++ include/common.h |7

<    1   2   3   4   5   6   7   8   >