From: Alex BASTOS <alebas@televes.com>
Date: Tue, 18 Oct 2005 16:07:42 +0200
Subject: [U-Boot-Users] Booting MPC8272ADS from flash base
In-Reply-To: <20051018111844.35004545.ajz@cambridgebroadband.com>
References: <51DB8827D393D411BB69003048003F4601D96B83@tvesntr> <1129544985.43537d196113a@webmail.televes.com:443> <20051018111844.35004545.ajz@cambridgebroadband.com>
Message-ID: <1129644462.435501aea9962@webmail.televes.com:443>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de


> > >
> > > Index: board/mpc8260ads/config.mk
> > > =================================================================
> > > == RCS file: /newcvs/pq2-linux/u-boot/board/mpc8260ads/config.mk,v
> > > retrieving revision 1.1.1.1
> > > diff -u -r1.1.1.1 config.mk
> > > --- board/mpc8260ads/config.mk	24 Aug 2005 12:16:55 -0000	1.1.1.1
> > > +++ board/mpc8260ads/config.mk	14 Oct 2005 08:36:46 -0000
> > > @@ -29,4 +29,4 @@
> > >  # MPC8260ADS, MPC8266ADS, and PQ2FADS-ZU/VR boards
> > >  #
> > >
> > > -TEXT_BASE = 0xfff00000
> > > +TEXT_BASE = 0xff800000
> >
> > Are you sure you are using u-boot-1.1.3?
> > Patch from Jerry Van Baren already made some changes to this file
> > to allow the use of a different TEXT__BASE for lowboot in MPC8272ADS
> > selecting it from Makefile config target.
> > I have this changes already applied in 1.1.3.
> >
>
> I am using u-boot-1.1.3 from Denx ELDK 3.1.1.  There is lowboot
> support in the top level Makefile - but not for any of the 82xx ADS
> platforms.
>
>
>
>
> > > Index: include/configs/MPC8260ADS.h
> > > =================================================================
> > > == RCS file:
> > > /newcvs/pq2-linux/u-boot/include/configs/MPC8260ADS.h,v retrieving
> > > revision 1.1.1.1 diff -u -r1.1.1.1 MPC8260ADS.h
> > > --- include/configs/MPC8260ADS.h	24 Aug 2005 12:17:01 -0000
> > > 1.1.1.1+++ include/configs/MPC8260ADS.h	14 Oct 2005 08:36:46 -0000
> > > @@ -294,13 +294,8 @@
> > >  #define CFG_GBL_DATA_OFFSET	(CFG_INIT_RAM_END - CFG_GBL_DATA_SIZE)
> > >  #define CFG_INIT_SP_OFFSET	CFG_GBL_DATA_OFFSET
> > >
> > > +#define CFG_HRCW_MASTER 0x0e72b605   /* like BCSR version, but
> > > with CIP=1, BMS=1 */
> >
> >
> > In order lowboot patch from Jerry to work (which has been applied to
> > u-boot source, so I recommned you to use) in MPC8272ADS, ISB must
> > also be ISB=100, not ISB=010 as in BCSR.
> >
>
> I think either are fine, as they result in an internal register space
> that doesn't conflict with the boot code (which is initially mapped by
> the CPU to 0x0...0x01ffffff).  The startup assembly code remaps it
> anyway.
>

ISB=100 is needed when you use the MPC8272ADS_lowboot_config target
which is in the u-boot-1.1.3 branch at CVS. Jerry's patch has been
applied there and you can find many other differences.

When using this, it assumes as the initial IMMR 0xF0000000 (so ISB=100)
as this is the value expected by Linux kernel. When using BCSR, u-boot
has to change it from 0x0F000000 to 0xF0000000. So if we have the control
of HRCW, why don't do things in the simpliest way. :-)

See enclosed Jerry's patch applied 04-November-2004

Best regard,

Alex Bastos
-------------- next part --------------
A non-text attachment was scrubbed...
Name: APPLIED_1.1.3-20041104_lowboot.patch
Type: text/x-diff
Size: 4830 bytes
Desc: not available
Url : http://lists.denx.de/pipermail/u-boot/attachments/20051018/94cd97c5/attachment.patch