This is the mail archive of the ecos-patches@sourceware.org mailing list for the eCos project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[Bug 1001275] Cortex-M (armV7) architecture endian instructions / Applied on lwIP


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.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]