This is the mail archive of the
ecos-patches@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: ecos-patches at ecos dot sourceware dot org
- Date: Fri, 19 Aug 2011 16:09:26 +0100
- Subject: [Bug 1001275] Cortex-M (armV7) architecture endian instructions / Applied on lwIP
- Auto-submitted: auto-generated
- References: <bug-1001275-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=1001275
--- Comment #6 from Sergei Gavrikov <sergei.gavrikov@gmail.com> 2011-08-19 16:09:22 BST ---
(In reply to comment #5)
> (In reply to comment #4)
[ snip many lines where we came to consensus, see comment 4 ]
> Personally I would like CYGARC_SWAPxx(), but since CYG_SWAPxx() are
> already present in hal_endian.h let's use them. It will propagate REV
> instructions in other parts of code that may benefit (i guess it is
> safe).
> > also I would not put the above definitions in cortexm_regs.h. In my
> > opinion CYGARC_SWAP16, CYGARC_SWAP32 can be placed in hal_arch.h.
>
> I am thinking this way too, only I didn't want to do lot's of changes
> in hal_arch.h. Now, especially if we use CYG_SWAPxx() it shouldn't be
> a problem.
So, we will not reinvent the weel and will stay on CYG_SWAPxx().
> > Well, it is my view only. I hope that I did not miss something.
>
> Thanks again for considerations. At present I am away, but today I
> return home and I am going to test during weekend. FAOD, unless you
> object I intend to use CYG_SWAPxx() naming.
I vote for CYG_SWAPxx() too. I'm sorry but this week I'll be AFK.
CYA
--
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.