This is the mail archive of the
ecos-patches@sourceware.org
mailing list for the eCos project.
[Bug 1001125] New HAL for the Stellaris EK-LM3S811 board
- From: bugzilla-daemon at bugs dot ecos dot sourceware dot org
- To: ecos-patches at ecos dot sourceware dot org
- Date: Mon, 31 Jan 2011 10:48:34 +0000
- Subject: [Bug 1001125] New HAL for the Stellaris EK-LM3S811 board
- Auto-submitted: auto-generated
- References: <bug-1001125-104@http.bugs.ecos.sourceware.org/>
Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001125
--- Comment #8 from Sergei Gavrikov <sergei.gavrikov@gmail.com> 2011-01-31 10:48:33 GMT ---
(In reply to comment #7)
> You are correct, the mlt.h should have:
>
> #define CYGMEM_REGION_flash_SIZE (0x00010000)
>
> also in i2c_lm3s.c the external definition of hal_cortexm_systick_clock is
> not required and shall be removed.
>
> __externC cyg_uint32 hal_cortexm_systick_clock;
I'm sorry for the delay in response. The above has been done.
Christophe, yet more minor notes about HAL sources, but
hal/cortexm/lm3s/var/current/include/hal_cache.h:
----
The header contains many commented lines with the definitions. Do you really
need those memos? If you need it, no issue.
hal/cortexm/lm3s/lm3s8xx/current/tests/timers.c:
----
I would remove that rudiment '#if 0' from the source. Ok?
hal/cortexm/lm3s/var/current/src/lm3s_misc.c:hal_variant_init():
----
Commented #if !defined(CYG_HAL_STARTUP_RAM) looks good for me, I would
revert it back. What's wrong with it?
--
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.