ARM-software / CMSIS_5

CMSIS Version 5 Development Repository
http://arm-software.github.io/CMSIS_5/index.html
Apache License 2.0
1.33k stars 1.08k forks source link

SVDConv: New feature dsppresent #401

Open simtind opened 6 years ago

simtind commented 6 years ago

It would be useful to be able to declare the __DSP_PRESENT macro from the SVD file. I propose a new tag working similar to , either mandatory, or default not present.

ReinhardKeil commented 6 years ago

What should __DSP_PRESENT express? The availability of SIMD instructions? Every Cortex-M4/M7 has this extensions, therefore I think it is redundant.

simtind commented 6 years ago

DSP / SIMD instructions are optional in Cortex-M33. Referencing https://developer.arm.com/products/processors/cortex-m/cortex-m33 .

From CMSIS-DSP documentation: http://www.keil.com/pack/doc/CMSIS/DSP/html/index.html __DSP_PRESENT: Initialize macro __DSP_PRESENT = 1 when Armv8-M Mainline core supports DSP instructions.

ReinhardKeil commented 6 years ago

Thanks for clarifying. Yes Cortex-M33 has optional SIMD instruction set.

JonatanAntoni commented 6 years ago

We should think about adding <dspPresent>...</dspPresent> to the SVD schema and converter.

JonatanAntoni commented 5 years ago

@jkrech: We added __DSP_PRESENT and __DSP_USED to ARMCM33 device files. What about adding this to SVDConv?