Power On Self Tests
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
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 incommproc.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
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.
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 SelfTests)
(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_
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 SelfTests)
(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