Re: [U-Boot] [PATCH v2 7/7] am335x_evm: Add support to boot from NOR.

2013-07-17 Thread Mark Jackson
On 11/07/13 14:06, Mark Jackson wrote: On 18/06/13 13:11, Mark Jackson wrote: On 17/06/13 15:43, Mark Jackson wrote: On 13/05/13 19:28, Tom Rini wrote: From: Steve Kipisz s-kipi...@ti.com NOR requires that s_init be within the first 4KiB of the image so that we can perform the rest of the

Re: [U-Boot] [PATCH v2 7/7] am335x_evm: Add support to boot from NOR.

2013-07-17 Thread Tom Rini
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 07/17/2013 08:58 AM, Mark Jackson wrote: On 11/07/13 14:06, Mark Jackson wrote: On 18/06/13 13:11, Mark Jackson wrote: On 17/06/13 15:43, Mark Jackson wrote: On 13/05/13 19:28, Tom Rini wrote: From: Steve Kipisz s-kipi...@ti.com NOR

Re: [U-Boot] [PATCH v2 7/7] am335x_evm: Add support to boot from NOR.

2013-07-11 Thread Mark Jackson
On 18/06/13 13:11, Mark Jackson wrote: On 17/06/13 15:43, Mark Jackson wrote: On 13/05/13 19:28, Tom Rini wrote: From: Steve Kipisz s-kipi...@ti.com NOR requires that s_init be within the first 4KiB of the image so that we can perform the rest of the required pinmuxing to talk with the rest

Re: [U-Boot] [PATCH v2 7/7] am335x_evm: Add support to boot from NOR.

2013-07-11 Thread Tom Rini
On Thu, Jul 11, 2013 at 02:06:26PM +0100, Mark Jackson wrote: On 18/06/13 13:11, Mark Jackson wrote: On 17/06/13 15:43, Mark Jackson wrote: On 13/05/13 19:28, Tom Rini wrote: From: Steve Kipisz s-kipi...@ti.com NOR requires that s_init be within the first 4KiB of the image so that we

Re: [U-Boot] [PATCH v2 7/7] am335x_evm: Add support to boot from NOR.

2013-07-11 Thread Mark Jackson
On 11/07/13 14:28, Tom Rini wrote: On Thu, Jul 11, 2013 at 02:06:26PM +0100, Mark Jackson wrote: On 18/06/13 13:11, Mark Jackson wrote: On 17/06/13 15:43, Mark Jackson wrote: Okay ... I've now got NOR booting rebased to master using the patch set below. This doesn't include my board code

Re: [U-Boot] [PATCH v2 7/7] am335x_evm: Add support to boot from NOR.

2013-07-11 Thread Albert ARIBAUD
Hi Mark, On Thu, 11 Jul 2013 14:45:08 +0100, Mark Jackson mpfj-l...@newflow.co.uk wrote: On 11/07/13 14:28, Tom Rini wrote: On Thu, Jul 11, 2013 at 02:06:26PM +0100, Mark Jackson wrote: On 18/06/13 13:11, Mark Jackson wrote: On 17/06/13 15:43, Mark Jackson wrote: Okay ... I've now got

Re: [U-Boot] [PATCH v2 7/7] am335x_evm: Add support to boot from NOR.

2013-07-11 Thread Tom Rini
On Thu, Jul 11, 2013 at 04:34:18PM +0200, Albert ARIBAUD wrote: Hi Mark, On Thu, 11 Jul 2013 14:45:08 +0100, Mark Jackson mpfj-l...@newflow.co.uk wrote: On 11/07/13 14:28, Tom Rini wrote: On Thu, Jul 11, 2013 at 02:06:26PM +0100, Mark Jackson wrote: On 18/06/13 13:11, Mark Jackson

Re: [U-Boot] [PATCH v2 7/7] am335x_evm: Add support to boot from NOR.

2013-07-11 Thread Albert ARIBAUD
Hi Tom, On Thu, 11 Jul 2013 11:54:09 -0400, Tom Rini tr...@ti.com wrote: The new in the patch board/ti/am335x/u-boot.lds needed to be re-synced with arch/arm/cpu/u-boot.lds again. Actually, I wonder why so many ARM .lds files need to exist at all, when arch/arm/cpu/u-boot[-spl].lds could be

Re: [U-Boot] [PATCH v2 7/7] am335x_evm: Add support to boot from NOR.

2013-07-11 Thread Albert ARIBAUD
On Thu, 11 Jul 2013 18:08:59 +0200, Albert ARIBAUD albert.u.b...@aribaud.net wrote: Hi Tom, On Thu, 11 Jul 2013 11:54:09 -0400, Tom Rini tr...@ti.com wrote: The new in the patch board/ti/am335x/u-boot.lds needed to be re-synced with arch/arm/cpu/u-boot.lds again. Actually, I wonder

Re: [U-Boot] [PATCH v2 7/7] am335x_evm: Add support to boot from NOR.

2013-07-11 Thread Tom Rini
On Thu, Jul 11, 2013 at 06:14:11PM +0200, Albert ARIBAUD wrote: On Thu, 11 Jul 2013 18:08:59 +0200, Albert ARIBAUD albert.u.b...@aribaud.net wrote: Hi Tom, On Thu, 11 Jul 2013 11:54:09 -0400, Tom Rini tr...@ti.com wrote: The new in the patch board/ti/am335x/u-boot.lds needed to

Re: [U-Boot] [PATCH v2 7/7] am335x_evm: Add support to boot from NOR.

2013-06-18 Thread Mark Jackson
On 17/06/13 15:43, Mark Jackson wrote: On 13/05/13 19:28, Tom Rini wrote: From: Steve Kipisz s-kipi...@ti.com NOR requires that s_init be within the first 4KiB of the image so that we can perform the rest of the required pinmuxing to talk with the rest of NOR that we are found on. When

Re: [U-Boot] [PATCH v2 7/7] am335x_evm: Add support to boot from NOR.

2013-06-17 Thread Mark Jackson
On 13/05/13 19:28, Tom Rini wrote: From: Steve Kipisz s-kipi...@ti.com NOR requires that s_init be within the first 4KiB of the image so that we can perform the rest of the required pinmuxing to talk with the rest of NOR that we are found on. When NOR_BOOT is set we save our environment in

Re: [U-Boot] [PATCH v2 7/7] am335x_evm: Add support to boot from NOR.

2013-06-17 Thread Tom Rini
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 06/17/2013 10:43 AM, Mark Jackson wrote: On 13/05/13 19:28, Tom Rini wrote: From: Steve Kipisz s-kipi...@ti.com NOR requires that s_init be within the first 4KiB of the image so that we can perform the rest of the required pinmuxing to talk

Re: [U-Boot] [PATCH v2 7/7] am335x_evm: Add support to boot from NOR.

2013-06-17 Thread Mark Jackson
On 17/06/13 15:49, Tom Rini wrote: snip Did you copy the parts that setup the pinmuxing in s_init for NOR? This bit ? #ifdef CONFIG_NOR_BOOT asm(stmfd sp!, {r2 - r4}); asm(movw r4, #0x8A4); asm(movw r3, #0x44E1); asm(orrr4, r4, r3, lsl

Re: [U-Boot] [PATCH v2 7/7] am335x_evm: Add support to boot from NOR.

2013-06-17 Thread Tom Rini
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 06/17/2013 10:59 AM, Mark Jackson wrote: On 17/06/13 15:49, Tom Rini wrote: snip Did you copy the parts that setup the pinmuxing in s_init for NOR? This bit ? #ifdef CONFIG_NOR_BOOT asm(stmfd sp!, {r2 - r4}); asm(movw r4,

Re: [U-Boot] [PATCH v2 7/7] am335x_evm: Add support to boot from NOR.

2013-06-17 Thread Mark Jackson
On 17/06/13 16:01, Tom Rini wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 06/17/2013 10:59 AM, Mark Jackson wrote: On 17/06/13 15:49, Tom Rini wrote: snip Did you copy the parts that setup the pinmuxing in s_init for NOR? This bit ? #ifdef CONFIG_NOR_BOOT asm(stmfd

Re: [U-Boot] [PATCH v2 7/7] am335x_evm: Add support to boot from NOR.

2013-06-17 Thread Mark Jackson
On 17/06/13 15:59, Mark Jackson wrote: On 17/06/13 15:49, Tom Rini wrote: snip Did you copy the parts that setup the pinmuxing in s_init for NOR? This bit ? #ifdef CONFIG_NOR_BOOT asm(stmfd sp!, {r2 - r4}); asm(movw r4, #0x8A4); asm(movw r3,

Re: [U-Boot] [PATCH v2 7/7] am335x_evm: Add support to boot from NOR.

2013-06-17 Thread Tom Rini
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 06/17/2013 11:23 AM, Mark Jackson wrote: On 17/06/13 15:59, Mark Jackson wrote: On 17/06/13 15:49, Tom Rini wrote: snip Did you copy the parts that setup the pinmuxing in s_init for NOR? This bit ? #ifdef CONFIG_NOR_BOOT asm(stmfd

Re: [U-Boot] [PATCH v2 7/7] am335x_evm: Add support to boot from NOR.

2013-06-17 Thread Stefan Roese
Hi Mark, On 17.06.2013 17:23, Mark Jackson wrote: On 17/06/13 15:59, Mark Jackson wrote: On 17/06/13 15:49, Tom Rini wrote: snip Did you copy the parts that setup the pinmuxing in s_init for NOR? This bit ? #ifdef CONFIG_NOR_BOOT asm(stmfd sp!, {r2 - r4}); asm(movw

Re: [U-Boot] [PATCH v2 7/7] am335x_evm: Add support to boot from NOR.

2013-06-17 Thread Mark Jackson
On 17/06/13 17:01, Stefan Roese wrote: Hi Mark, On 17.06.2013 17:23, Mark Jackson wrote: snip One question ... the SPL code has:- ... gd = gdata; ... But there seems to be no similar assignment when in NOR boot mode. I'm no expert in the internal workings of u-boot, so

Re: [U-Boot] [PATCH v2 7/7] am335x_evm: Add support to boot from NOR.

2013-06-17 Thread Kipisz, Steven
-Original Message- From: Mark Jackson [mailto:mpfj-l...@newflow.co.uk] Sent: Monday, June 17, 2013 11:10 AM To: Stefan Roese Cc: Rini, Tom; Kipisz, Steven; u-boot@lists.denx.de Subject: Re: [U-Boot] [PATCH v2 7/7] am335x_evm: Add support to boot from NOR. On 17/06/13 17:01

Re: [U-Boot] [PATCH v2 7/7] am335x_evm: Add support to boot from NOR.

2013-06-17 Thread Mark Jackson
On 17/06/13 18:04, Kipisz, Steven wrote: snip I'll look at that thanks. But I've taken a slightly different approach, and added the following to the start of s_init() ... void s_init(void) { unsigned short *p = (unsigned short*)0x0800; while (1) { int

Re: [U-Boot] [PATCH v2 7/7] am335x_evm: Add support to boot from NOR.

2013-05-16 Thread Peter Korsgaard
Tom == Tom Rini tr...@ti.com writes: Tom From: Steve Kipisz s-kipi...@ti.com Tom NOR requires that s_init be within the first 4KiB of the image so that Tom we can perform the rest of the required pinmuxing to talk with the rest Tom of NOR that we are found on. When NOR_BOOT is set we save

Re: [U-Boot] [PATCH v2 7/7] am335x_evm: Add support to boot from NOR.

2013-05-16 Thread Tom Rini
On Thu, May 16, 2013 at 04:36:02PM +0200, Peter Korsgaard wrote: Tom == Tom Rini tr...@ti.com writes: Tom From: Steve Kipisz s-kipi...@ti.com Tom NOR requires that s_init be within the first 4KiB of the image so that Tom we can perform the rest of the required pinmuxing to talk with the

Re: [U-Boot] [PATCH v2 7/7] am335x_evm: Add support to boot from NOR.

2013-05-16 Thread Peter Korsgaard
Tom == Tom Rini tr...@ti.com writes: Hi, Tom +#ifdef CONFIG_TI81XX Why are you adding the TI81XX dependency here? That doesn't have anything to do with nor boot, does it? Tom dmm is TI81XX-only (for this file, it's also omap4+) and since main Tom U-Boot doesn't use

Re: [U-Boot] [PATCH v2 7/7] am335x_evm: Add support to boot from NOR.

2013-05-16 Thread Tom Rini
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 05/16/2013 02:55 PM, Peter Korsgaard wrote: Tom == Tom Rini tr...@ti.com writes: Hi, Tom +#ifdef CONFIG_TI81XX Why are you adding the TI81XX dependency here? That doesn't have anything to do with nor boot, does it? Tom dmm is

Re: [U-Boot] [PATCH v2 7/7] am335x_evm: Add support to boot from NOR.

2013-05-16 Thread Peter Korsgaard
Tom == Tom Rini tr...@ti.com writes: Ok, that's interesting info which imho belongs in the commit message. Tom Reworded, thanks! Great, thanks. -- Bye, Peter Korsgaard ___ U-Boot mailing list U-Boot@lists.denx.de

[U-Boot] [PATCH v2 7/7] am335x_evm: Add support to boot from NOR.

2013-05-13 Thread Tom Rini
From: Steve Kipisz s-kipi...@ti.com NOR requires that s_init be within the first 4KiB of the image so that we can perform the rest of the required pinmuxing to talk with the rest of NOR that we are found on. When NOR_BOOT is set we save our environment in NOR at 512KiB and a redundant copy at