This is the mail archive of the
ecos-patches@sources.redhat.com
mailing list for the eCos project.
Re: sparc leon3 hal patch
- From: Andrew Lunn <andrew at lunn dot ch>
- To: Konrad Eisele <konrad at gaisler dot com>
- Cc: ecos-patches at sources dot redhat dot com
- Date: Thu, 28 Jul 2005 12:52:29 +0200
- Subject: Re: sparc leon3 hal patch
- References: <42E73E34.1000109@gaisler.com>
On Wed, Jul 27, 2005 at 09:56:36AM +0200, Konrad Eisele wrote:
> I think Jiri Gaisler send in the patches before. He told me to
> resynchronize with the current cvs. He also sait it's ok that I sign a
> possible licence statement if needed. Let me know.
Yes, we will need a copyright assignment. Please see
http://ecos.sourceware.org/assign.html
We first need Nick to take a look at your kernel and i386 changes. If
Nick accepts them i can then take a closer look at the rest.
> --- ecos-rep-ori/packages/kernel/current/tests/tm_basic.cxx 2005-07-26 17:57:22.000000000 +0200
> +++ ecos-rep/packages/kernel/current/tests/tm_basic.cxx 2005-07-26 18:08:48.000000000 +0200
> @@ -88,7 +88,7 @@
>
> #define STACK_SIZE CYGNUM_HAL_STACK_SIZE_MINIMUM
>
> -#ifdef CYGMEM_REGION_ram_SIZE
> +#if defined(CYGMEM_REGION_ram_SIZE) && !(defined(CYGPKG_HAL_SPARC_LEON3_OPTIONS) || defined(CYGPKG_HAL_SPARC_LEON_OPTIONS))
> #define CYG_THREAD_OVERHEAD (STACK_SIZE+sizeof(cyg_thread)+(sizeof(fun_times)*2))
> #define NTEST_THREADS ((CYGMEM_REGION_ram_SIZE/16)/CYG_THREAD_OVERHEAD)
> #define CYG_MUTEX_OVERHEAD (sizeof(cyg_mutex_t)+sizeof(fun_times))
I can tell you now, this huck of the patch will be rejected. There
should be no target specific options inside the generic kernel. You
need to find another way to do this.
Andrew