This is the mail archive of the ecos-patches@sources.redhat.com mailing list for the eCos project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: redboot w/o gdb stubs


On Mon, 2003-12-08 at 08:06, Jani Monoses wrote:
> > 
> > I'm interested to know what patch to the PowerPC code you had to make.
> > 
> 
> Just this minor one as seen in the ARM hal. Otherwise it won't link.

I guess it depends on the version of GCC you're using.  Mine worked
fine with GCC-3.2.2

In any case, I think that the attached CDL handles it for RedBoot.
If you want to disable GDB, just turn off CYGDBG_HAL_DEBUG_GDB_INCLUDE_STUBS

If this works for you, I'll commit it.

-- 
Gary Thomas <gary@mlbassoc.com>
MLB Associates
Index: redboot/current/cdl/redboot.cdl
===================================================================
RCS file: /misc/cvsfiles/ecos/packages/redboot/current/cdl/redboot.cdl,v
retrieving revision 1.60
diff -u -5 -p -r1.60 redboot.cdl
--- redboot/current/cdl/redboot.cdl	2 Dec 2003 14:40:19 -0000	1.60
+++ redboot/current/cdl/redboot.cdl	8 Dec 2003 15:09:43 -0000
@@ -77,25 +77,32 @@ cdl_package CYGPKG_REDBOOT {
     cdl_component CYGBLD_BUILD_REDBOOT {
         display "Build Redboot ROM ELF image"
         default_value 0
         requires CYGPKG_INFRA
         requires CYGPKG_ISOINFRA
-# Someday the tools will handle this
-#        requires { CYGINT_HAL_DEBUG_GDB_STUBS implies CYGDBG_HAL_DEBUG_GDB_INCLUDE_STUBS }
-#        requires { CYGINT_HAL_DEBUG_GDB_STUBS implies CYGDBG_HAL_DEBUG_GDB_BREAK_SUPPORT }
-#
-        requires { !CYGINT_HAL_DEBUG_GDB_STUBS || CYGDBG_HAL_DEBUG_GDB_INCLUDE_STUBS }
-        requires { !CYGINT_HAL_DEBUG_GDB_STUBS || CYGDBG_HAL_DEBUG_GDB_BREAK_SUPPORT }
-#
+
         requires ! CYGDBG_HAL_DEBUG_GDB_CTRLC_SUPPORT
         requires ! CYGDBG_HAL_COMMON_INTERRUPTS_SAVE_MINIMUM_CONTEXT
         requires ! CYGDBG_HAL_COMMON_CONTEXT_SAVE_MINIMUM
         requires CYGSEM_HAL_VIRTUAL_VECTOR_SUPPORT
 
         requires CYGINT_ISO_STRING_MEMFUNCS 
         requires CYGINT_ISO_STRING_STRFUNCS
         requires CYGINT_REDBOOT_LOAD_METHOD
+
+        cdl_option CYGBLD_BUILD_REDBOOT_WITH_GDB {
+            display       "Include GDB support in RedBoot"
+            no_define
+            default_value 1
+            active_if     CYGINT_HAL_DEBUG_GDB_STUBS
+            requires      CYGDBG_HAL_DEBUG_GDB_INCLUDE_STUBS
+            requires      CYGDBG_HAL_DEBUG_GDB_BREAK_SUPPORT
+            description "
+              RedBoot normally includes support for the GDB debugging protocols.
+              This option allows this to be disabled which may yield a substantial
+              savings in terms of code and memory usage by RedBoot."
+        }
 
         cdl_option CYGBLD_BUILD_REDBOOT_WITH_THREADS {
             display "Threads debugging support"
             no_define
             description "

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]