This is the mail archive of the
ecos-bugs@sourceware.org
mailing list for the eCos project.
[Bug 1001275] Cortex-M (armV7) architecture endian instructions /Applied on lwIP
- From: bugzilla-daemon at bugs dot ecos dot sourceware dot org
- To: unassigned at bugs dot ecos dot sourceware dot org
- Date: Tue, 23 Aug 2011 15:18:26 +0100
- Subject: [Bug 1001275] Cortex-M (armV7) architecture endian instructions /Applied on lwIP
- Auto-submitted: auto-generated
- References: <bug-1001275-777@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=1001275
--- Comment #18 from Sergei Gavrikov <sergei.gavrikov@gmail.com> 2011-08-23 15:18:22 BST ---
(In reply to comment #16)
> (In reply to comment #15)
>
> > I will very appreciate your final patches for lwipopts.h, and config
> > files.
>
> Here's what I got today.
>
> Cortex-M CDL: Disabling CYGIMP_HAL_ARCH_ENDIAN also suppresses
> generation of cortexm_endian.h so we have somplete global wipe-out.
>
> --- ecos/packages/hal/cortexm/arch/current/cdl/hal_cortexm.cdl 2011-04-06
> 18:54:16.000000000 +0200
> +++ ecos_kin/packages/hal/cortexm/arch/current/cdl/hal_cortexm.cdl
> 2011-08-23 14:54:14.000000000 +0200
> @@ -138,6 +138,19 @@
> Higher priorities are reserved for the DEBUG and SVC traps."
> }
>
> + cdl_option CYGIMP_HAL_ARCH_ENDIAN {
> + display "Architecture optimized endian functions"
> + flavor bool
> + default_value 1
> + description "..."
> +
> + define_proc {
> + puts $::cdl_system_header \
> + "#define CYGBLD_HAL_ENDIAN_H <cyg/hal/cortexm_endian.h>"
> + }
> + }
> +
> cdl_option CYGBLD_LINKER_SCRIPT {
> display "Linker script"
> flavor data
>
> lwIP CDL (lwipopt.h changed accordingly):
>
> --- ecos/packages/net/lwip_tcpip/current/cdl/lwip_net.cdl 2011-03-17
> 12:40:35.000000000 +0100
> +++ ecos_kin/packages/net/lwip_tcpip/current/cdl/lwip_net.cdl 2011-08-23
> 13:06:35.000000000 +0200
> @@ -127,6 +127,17 @@
> compile ecos/sio.c
> }
>
> + cdl_option CYGIMP_LWIP_ENDIAN_BY_HAL {
> + display "HAL optimized endian functions"
> + flavor bool
> + active_if CYGIMP_HAL_ARCH_ENDIAN
> + default_value CYGIMP_HAL_ARCH_ENDIAN
> +
> + description "..."
> + }
> +
> cdl_component CYGPKG_LWIP_APIS {
> display "APIs"
> flavor none
Ilija, for me these CDLs look great! IMHO, you picked clear names.
> I tested following cases:
> 1. CYGIMP_HAL_ARCH_ENDIAN absent (what we have in other architectures)
> 2. CYGIMP_HAL_ARCH_ENDIAN disabled (global disable)
> 3. CYGIMP_HAL_ARCH_ENDIAN enabled and CYGIMP_LWIP_ENDIAN_BY_HAL disabled
> 4 Both CYGIMP_HAL_ARCH_ENDIAN and CYGIMP_LWIP_ENDIAN_BY_HAL enabled
Many thanks for testing.
> If you are happy, I would submit.
Of course, please, submit your patch(es).
Thank you,
Sergei
>
> Ilija
--
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.