[U-Boot-Users] Power On Self Tests

[U-Boot-Users] Power On Self Tests

[Pierre AUBERT]

Hello Everybody,

During the power on self tests, the bootmode of the board is stored in
the processor DPRAM. This
location is hardcoded in commproc.h. The following patch adds the
opportunity to override this value
in the board config file. The position of this value in DPRAM depends on
the usage of the DPRAM
made by the operating system.

For example, on an MPC860T, I've add
#define CFG_CPM_POST_WORD_ADDR    0x0ffc
in the board config file to have the bootmode being the last word of the
DPRAM.


--- ppcboot-2.0.0/include/commproc.h    2002-10-16 13:31:18.000000000
+0200
+++ ppcboot-2.0.0-m/include/commproc.h  2002-11-21 10:42:31.000000000
+0100
@@ -77,7 +77,11 @@

 #endif

+#ifndef CFG_CPM_POST_WORD_ADDR
 #define CPM_POST_WORD_ADDR     0x07FC
+#else
+#define CPM_POST_WORD_ADDR     CFG_CPM_POST_WORD_ADDR
+#endif

 #define BD_IIC_START   ((uint) 0x0400) // <- please use CPM_I2C_BASE !!






Best regards

[U-Boot-Users] Power On Self Tests

[Wolfgang Denk]

Dear Pierre,

in message <3DDCABA9.DD718682@staubli.com> you wrote:
> 
> During the power on self tests, the bootmode of the board is stored in
> the processor DPRAM. This
> location is hardcoded in commproc.h. The following patch adds the
> opportunity to override this value
> in the board config file. The position of this value in DPRAM depends on
> the usage of the DPRAM
> made by the operating system.

Thanks.

Can you please also send the correspoding CHANGELOG entry,  plus  the
patch for README to describe the new CFG_* variable?

Best regards,

Wolfgang Denk

-- 
Software Engineering:  Embedded and Realtime Systems,  Embedded Linux
Phone: (+49)-8142-4596-87  Fax: (+49)-8142-4596-88  Email: wd at denx.de
Common sense and a sense of humor  are  the  same  thing,  moving  at
different speeds.  A sense of humor is just common sense, dancing.
- Clive James

[U-Boot-Users] Power On Self Tests

[Pierre AUBERT]

Wolfgang Denk wrote:

> Dear Pierre,
>
>
> Can you please also send the correspoding CHANGELOG entry,  plus  the
> patch for README to describe the new CFG_* variable?
>
> Best regards,
>
> Wolfgang Denk
>

Please find attached the patch for the README file.

--- ppcboot-2.0.0-cvs/README    2002-11-21 13:50:01.000000000 +0100
+++ ppcboot-2.0.0-m/README      2002-11-21 14:03:17.000000000 +0100
@@ -1653,6 +1653,13 @@ Low Level (hardware related) configurati
                wrong setting might damage your board. Read
                doc/README.MBX before setting this variable!

+- CFG_CPM_POST_WORD_ADDR:
+       (CPM8xx, CPM8260)
+       Offset of the bootmode word in DPRAM used by post (Power On Self
Tests)
+       This definition overrides hardcoded #define in commproc.h or
+       cpm_8260.h.
+
+
 Building the Software:
 ======================


CHANGELOG:
* Patch by Pierre Aubert , 21 Nov 2002
  Add CFG_CPM_POST_WORD_ADDR so the offset of the bootmode
  word in DPRAM can be redefined in the board config file.



Best regards.

[U-Boot-Users] Power On Self Tests

[Wolfgang Denk]

In message <3DDCDA0A.AF0E75FE@staubli.com> you wrote:
>
> Please find attached the patch for the README file.
...
> +- CFG_CPM_POST_WORD_ADDR:
> +       (CPM8xx, CPM8260)
> +       Offset of the bootmode word in DPRAM used by post (Power On Self
> Tests)
> +       This definition overrides hardcoded #define in commproc.h or
> +       cpm_8260.h.

Aha! This points out a shortcoming of your patch: you fixed  it  only
for  MPC8xx,  but  not  for  MPC8260  (although  your  comment claims
otherwise). OK, I added the code for 8260.

Best regards,

Wolfgang Denk

-- 
Software Engineering:  Embedded and Realtime Systems,  Embedded Linux
Phone: (+49)-8142-4596-87  Fax: (+49)-8142-4596-88  Email: wd at denx.de
This is a story about sex and drugs and Music with Rocks In.  Well...
...one out of three ain't bad.  Actually, it's only thirty-three per-
cent, but it could be worse.          - Terry Pratchett, _Soul Music_

[U-Boot-Users] Power On Self Tests

[Pierre AUBERT]

Wolfgang Denk wrote:

> In message <3DDCDA0A.AF0E75FE@staubli.com> you wrote:
> >
> > Please find attached the patch for the README file.
> ...
> > +- CFG_CPM_POST_WORD_ADDR:
> > +       (CPM8xx, CPM8260)
> > +       Offset of the bootmode word in DPRAM used by post (Power On Self
> > Tests)
> > +       This definition overrides hardcoded #define in commproc.h or
> > +       cpm_8260.h.
>
> Aha! This points out a shortcoming of your patch: you fixed  it  only
> for  MPC8xx,  but  not  for  MPC8260  (although  your  comment claims
> otherwise). OK, I added the code for 8260.
>

You're right, I've forgotten the cpm_8260.h part. Sorry

>
> Best regards,
>
> Wolfgang Denk
>
> --
> Software Engineering:  Embedded and Realtime Systems,  Embedded Linux