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

2014-08-12 Thread Sebastian Huber
On 08/12/2014 04:08 PM, Gedare Bloom wrote: On Tue, Aug 12, 2014 at 7:32 AM, Peter Dufault wrote: > >On Aug 11, 2014, at 17:02 , Gedare Bloom wrote: > >>>+#ifdef ARM_MULTILIB_VFP >>>+ /* CPACR is located at address 0xe000ed88 */ >>>+ ldr.w r0, =0xe000ed88 >>Maybe the constant s

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

2014-08-12 Thread Joel Sherrill
Out of curiosity, are any patches outstanding for you? I know of some gdb, one Newlib (mine), and two Ada patches outstanding. Is that it? --joel On August 12, 2014 10:07:55 AM CDT, Sebastian Huber wrote: >On 08/11/2014 11:12 PM, Joel Sherrill wrote: >> Is there any tool patch missing? I may

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

2014-08-12 Thread Sebastian Huber
On 08/11/2014 11:12 PM, Joel Sherrill wrote: Is there any tool patch missing? I may be thinking of nios and I think I committed that. No tool patch is missing. I use currently the latest GCC 4.9 without any patches. -- Sebastian Huber, embedded brains GmbH Address : Dornierstr. 4, D-82178

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

2014-08-12 Thread Gedare Bloom
On Tue, Aug 12, 2014 at 7:32 AM, Peter Dufault wrote: > > On Aug 11, 2014, at 17:02 , Gedare Bloom 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 comm

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

2014-08-12 Thread Peter Dufault
On Aug 11, 2014, at 17:02 , Gedare Bloom 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 r

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

2014-08-11 Thread Joel Sherrill
Is there any tool patch missing? I may be thinking of nios and I think I committed that. --joel On 8/11/2014 4:02 PM, Gedare Bloom wrote: > On Mon, Aug 11, 2014 at 4:48 PM, Sebastian Huber > wrote: >> This floating point unit is available in Cortex-M4F processors and >> defined by ARMv7-M. This

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

2014-08-11 Thread Gedare Bloom
On Mon, Aug 11, 2014 at 4:48 PM, Sebastian Huber wrote: > This floating point unit is available in Cortex-M4F processors and > defined by ARMv7-M. This adds basic support for other VFP-D16 variants. > --- > c/src/lib/libbsp/arm/shared/start/start.S | 20 ++ > cpukit/score/cp