Skip to contentSkip to conversation

Power On Self Tests

U-Boot-Users Pierre AUBERT 2 participants 5 messages Nov 21, 2002

lore
Share & tools
lore
#0 lore raw

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

#1 lore raw

Dear Pierre,

Quoted 8 lines · in message <[email protected]> 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

Signature · 5 lines
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
#2 lore raw
Quoted 10 lines · Wolfgang Denk

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.

README 2002-11-21 14:03:17.000000000 +0100 renamed from README 2002-11-21 13:50:01.000000000 +0100 +3 -0
@@ -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.

#3 lore raw
Quoted 2 lines · In message <[email protected]> you wrote:

Please find attached the patch for the README file.

...

Review

Quoted 6 lines

+- CFG_CPM_POST_WORD_ADDR:

README 2002-11-21 14:03:17.000000000 +0100 renamed from README 2002-11-21 13:50:01.000000000 +0100 +2 -0
+       (CPM8xx, CPM8260)
+       Offset of the bootmode word in DPRAM used by post (Power On Self

Tests)

(quoted diff) +2 -0
+       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

Signature · 5 lines
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_
#4 lore raw
Quoted 11 lines · Wolfgang Denk
Quoted 2 lines · In message <[email protected]> you wrote:

Please find attached the patch for the README file.

...

Review

Quoted 6 lines

+- CFG_CPM_POST_WORD_ADDR:

README 2002-11-21 14:03:17.000000000 +0100 renamed from README 2002-11-21 13:50:01.000000000 +0100 +2 -0
+       (CPM8xx, CPM8260)
+       Offset of the bootmode word in DPRAM used by post (Power On Self

Tests)

(quoted diff) +2 -0
+       This definition overrides hardcoded #define in commproc.h or
+       cpm_8260.h.
Quoted 4 lines

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

Quoted 7 lines

Best regards,

Wolfgang Denk

--
Software Engineering: Embedded and Realtime Systems, Embedded Linux