This is the mail archive of the
ecos-patches@sources.redhat.com
mailing list for the eCos project.
Re: ixp425: permit disabling of the RedBoot exec command
- From: Mark Salter <msalter at redhat dot com>
- To: Andrew Lunn <andrew at lunn dot ch>
- Cc: David Vrabel <dvrabel at arcom dot com>, eCos Patches <ecos-patches at ecos dot sourceware dot org>
- Date: Fri, 22 Apr 2005 12:43:09 -0400
- Subject: Re: ixp425: permit disabling of the RedBoot exec command
- Organization: Red Hat, Inc.
- References: <4267BA5F.8090103@arcom.com> <20050421182040.GE17364@lunn.ch>
On Thu, 2005-04-21 at 20:20 +0200, Andrew Lunn wrote:
> On Thu, Apr 21, 2005 at 03:36:15PM +0100, David Vrabel wrote:
> > Hi,
> >
> > This patches allows you to build a RedBoot without the exec command by
> > only requiring CYGHWR_REDBOOT_ARM_LINUX_EXEC_ADDRESS_DEFAULT if the exec
> > command is included.
>
> Thanks
> Andrew
I don't think this was the right fix. I was going to say something
yesterday, but my cable modem forced me offline for a day. Anyway,
the fix should have done something like this:
*** hal/arm/xscale/ixp425/current/cdl/hal_arm_xscale_ixp425.cdl.~1.3.~ Thu Sep 2 16:17:47 2004
--- hal/arm/xscale/ixp425/current/cdl/hal_arm_xscale_ixp425.cdl Fri Apr 22 12:36:45 2005
***************
*** 200,211 ****
configuration."
compile -library=libextras.a ixp425_redboot.c
- # RedBoot details
- requires { CYGHWR_REDBOOT_ARM_LINUX_EXEC_ADDRESS_DEFAULT != 0 }
define_proc {
- puts $::cdl_header "#define CYGHWR_REDBOOT_ARM_TRAMPOLINE_ADDRESS 0x00001f00"
puts $::cdl_header "#define HAL_FLASH_READ(a,b,c,d) hal_flash_read((a),(b),(c),(d))"
puts $::cdl_header "#define HAL_FLASH_PROGRAM(a,b,c,d) hal_flash_program((a),(b),(c),(d))"
}
}
}
--- 200,219 ----
configuration."
compile -library=libextras.a ixp425_redboot.c
define_proc {
puts $::cdl_header "#define HAL_FLASH_READ(a,b,c,d) hal_flash_read((a),(b),(c),(d))"
puts $::cdl_header "#define HAL_FLASH_PROGRAM(a,b,c,d) hal_flash_program((a),(b),(c),(d))"
}
+
+ cdl_option CYGBLD_REDBOOT_IXP4XX_LINUX_EXEC {
+ flavor none
+ no_define
+ active_if { CYGBLD_BUILD_REDBOOT_WITH_EXEC }
+ requires { CYGHWR_REDBOOT_ARM_LINUX_EXEC_ADDRESS_DEFAULT != 0 }
+
+ define_proc {
+ puts $::cdl_header "#define CYGHWR_REDBOOT_ARM_TRAMPOLINE_ADDRESS 0x00001f00"
+ }
+ }
}
}
--Mark