This is the mail archive of the
ecos-patches@sources.redhat.com
mailing list for the eCos project.
Re: [ECOS] Samsung SNDS100 Eval. Board...
- From: Jonathan Larmour <jifl at eCosCentric dot com>
- To: eCos Patches List <ecos-patches at sources dot redhat dot com>
- Cc: Jay Foster <jay at systech dot com>
- Date: Sat, 19 Oct 2002 06:05:15 +0100
- Subject: Re: [ECOS] Samsung SNDS100 Eval. Board...
- References: <80B97DE95AEED311BA580050047FE98455D883@mail.systech.com>
Hi guys,
I'd like confirmation the attached patch is okay. It's to fix the below
problem, which also affects the (similar) e7t.
Just not 100% sure if there's some other more appropriate conditional.
Ordinarily I'd just go ahead, but I've got brain fade right now.
Jifl
Jay Foster wrote:
Another thing to check is if the RAM and ROM configuration registers are
getting configured correctly. Since I'm using GDB to load my RAM version of
the code, I do this with a .gdbinit file before loading the code. The
startup code for the ROM version must do this itself. I noticed that the
code that does this for the SNDS board only gets included if the GDB stubs
are compiled in (see the PLATFORM_SETUP1 definition in the
hal_platform_setup.h file for the snds platform). In fact, I had to disable
this code to get the RAM version of the RedBoot package to build (this code
can only be executed from ROM, and should only be included in the ROM
versions).
--
eCosCentric http://www.eCosCentric.com/ <info@eCosCentric.com>
--[ "You can complain because roses have thorns, or you ]--
--[ can rejoice because thorns have roses." -Lincoln ]-- Opinions==mine
Index: snds/current/ChangeLog
===================================================================
RCS file: /cvs/ecos/ecos/packages/hal/arm/snds/current/ChangeLog,v
retrieving revision 1.3
diff -u -5 -p -r1.3 ChangeLog
--- snds/current/ChangeLog 3 Oct 2002 16:19:48 -0000 1.3
+++ snds/current/ChangeLog 19 Oct 2002 05:02:25 -0000
@@ -1,5 +1,10 @@
+2002-10-19 Jonathan Larmour <jifl@eCosCentric.com>
+
+ * include/hal_platform_setup.h (PLATFORM_SETUP1): Only define if
+ ROM or ROMRAM, not if STUBS which can be RAM.
+
2002-10-03 Jonathan Larmour <jifl@eCosCentric.com>
* include/pkgconf/mlt_arm_snds_ram.ldi: Add definition of heap1.
2002-05-07 Gary Thomas <gthomas@redhat.com>
Index: snds/current/include/hal_platform_setup.h
===================================================================
RCS file: /cvs/ecos/ecos/packages/hal/arm/snds/current/include/hal_platform_setup.h,v
retrieving revision 1.2
diff -u -5 -p -r1.2 hal_platform_setup.h
--- snds/current/include/hal_platform_setup.h 23 May 2002 23:02:30 -0000 1.2
+++ snds/current/include/hal_platform_setup.h 19 Oct 2002 05:02:25 -0000
@@ -70,11 +70,11 @@
// Use relative branch since we are going to switch the address space
// around.
#define CYGSEM_HAL_ROM_RESET_USES_JUMP
-#ifdef CYGDBG_HAL_DEBUG_GDB_INCLUDE_STUBS
+#if defined(CYG_HAL_STARTUP_ROM) || defined(CYG_HAL_STARTUP_ROMRAM)
#define PLATFORM_SETUP1 \
ldr r1,=KS32C_IOPMOD ;\
ldr r2,=0xff /* set led display to output */ ;\
str r2,[r1,#0x00] ;\
LED 0xaa ;\
Index: e7t/current/ChangeLog
===================================================================
RCS file: /cvs/ecos/ecos/packages/hal/arm/e7t/current/ChangeLog,v
retrieving revision 1.6
diff -u -5 -p -r1.6 ChangeLog
--- e7t/current/ChangeLog 23 May 2002 23:01:50 -0000 1.6
+++ e7t/current/ChangeLog 19 Oct 2002 05:02:25 -0000
@@ -1,5 +1,10 @@
+2002-10-19 Jonathan Larmour <jifl@eCosCentric.com>
+
+ * include/hal_platform_setup.h (PLATFORM_SETUP1): Only define if
+ ROM or ROMRAM, not if STUBS which can be RAM.
+
2002-04-15 Jesper Skov <jskov@redhat.com>
* include/hal_platform_setup.h (PLATFORM_SETUP1): Changed ldm/stm
to ldmia/stmia.
Index: e7t/current/include/hal_platform_setup.h
===================================================================
RCS file: /cvs/ecos/ecos/packages/hal/arm/e7t/current/include/hal_platform_setup.h,v
retrieving revision 1.4
diff -u -5 -p -r1.4 hal_platform_setup.h
--- e7t/current/include/hal_platform_setup.h 23 May 2002 23:01:51 -0000 1.4
+++ e7t/current/include/hal_platform_setup.h 19 Oct 2002 05:02:25 -0000
@@ -65,11 +65,11 @@
// Override default to a more sensible value
#undef CYGNUM_HAL_COMMON_INTERRUPTS_STACK_SIZE
#define CYGNUM_HAL_COMMON_INTERRUPTS_STACK_SIZE 2048
#endif
-#ifdef CYGDBG_HAL_DEBUG_GDB_INCLUDE_STUBS
+#if defined(CYG_HAL_STARTUP_ROM) || defined(CYG_HAL_STARTUP_ROMRAM)
#define PLATFORM_SETUP1 ;\
ldr r1,=E7T_IOPMOD ;\
ldr r2,=0x0001fcf0 /* set led + seg to output */ ;\
str r2,[r1,#0x00] ;\
ldr r1,=E7T_IOPDATA ;\