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

[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-kipisz2 at 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 768KiB.  We avoid
> >>> using SPL for this case and u-boot.bin is written directly to the start
> >>> of NOR.
> >>
> >> I'm trying to get this up and running our NanoBone platform, but I'm having
> >> no success.
> > 
> > Just an update ... I now have it working ... yay !!
> > 
> > The main (and possibly the *only*) culprit was that SYSBOOT[8] (which selects
> > 8/16bit mode) was being incorrectly influenced by a hardware fault.
> > 
> > I have pulled quite a few patches from various places, some of which might
> > not be required.  Now I have a working base, I'll work out what is needed
> > on top of Tom's original patch set.
> 
> Okay ... I've now got NOR booting rebased to master using the patch set below.
> This doesn't include my board code as that's of no interest until NOR booting
> is added to the mainline code.
> 
> Sorry, Tom, I'm no git expert, so I'm not sure how to compare this to your
> previous patches ... I hope this is useful in some way ??

I'm just starting to cycle back to the NOR patches again myself and
fixup the few remaining comments.  But in your series:

> diff --git a/arch/arm/config.mk b/arch/arm/config.mk
> index e80e1ed..ba310cb 100644
> --- a/arch/arm/config.mk
> +++ b/arch/arm/config.mk
> @@ -111,6 +111,6 @@ endif
>  endif
>  
>  # check that only R_ARM_RELATIVE relocations are generated
> -ifneq ($(CONFIG_SPL_BUILD),y)
> +ifneq ($(CONFIG_SPL_BUILD)$(CONFIG_NOR_BOOT),y)
>  ALL-y	+= checkarmreloc
>  endif

What toolchain are you using?  This seems worrying at best.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20130711/0dae9706/attachment.pgp>