[PATCH 1/2] arm: Add support for FPv4-SP floating point unit

Peter Dufault dufault at hda.com
Tue Aug 12 11:32:23 UTC 2014


On Aug 11, 2014, at 17:02 , Gedare Bloom <gedare at rtems.org> wrote:

>> +#ifdef ARM_MULTILIB_VFP
>> +       /* CPACR is located at address 0xe000ed88 */
>> +       ldr.w   r0, =0xe000ed88
> Maybe the constant should be provided by a header file? The comments
> are nice though. And I didn't see anything else in the rest, although
> my arm asm is rusty.
> 
> 

When used in a single place in a hardware specific C file I prefer a constant.  When I'm suspicious I'm comparing definitions against the reference manuals looking for transcription errors and I'd rather avoid the indirection.  Obviously it needs an appropriate constant (which Sebastian provides).

Peter
-----------------
Peter Dufault
HD Associates, Inc.      Software and System Engineering




More information about the devel mailing list