GCC warnings due to outdated CMSIS Topic is solved

[Thargon]

Hi,

GCC 7 warns when compiling ChibiOS 17.6.x and 18.2.x about unallowed 'register' storage class specifier in cmsis_gcc.h, an issue which has been fixed in the recent CMSIS 5.3 (or maybe already in 5.1 or 5.2 - I only checked 5.0 (broken) and 5.3).

For ChibiOS 17.6.x the exact file path is os/common/ext/CMSIS/include/cmsis_gcc.h and GCC complains about lines 150 (function '__get_PSP(void)') and 175 (function '__get_MSP(void)'). The error message is:

Code: Select all

warning: ISO C++1z does not allow 'register' storage class specifier [-Wregister]
   register uint32_t result;
                     ^~~~~~


For ChibiOS 18.2.x tha path is now os/common/ext/ARM/CMSIS/Core/include/cmsis_gcc.h.
In the recent version of CMSIS those 'register' keywords have been removed to fix the issue.

Maybe the CMSIS parts can be updated for ChibiOS 17.6.x and 18.2.x?

Best regards,
Thomas

PS: I don't want to start the discussion anew here, but if ChibiOS would use Git instead of SVN, the official ARM CMSIS 5 repository could be included as submodule and it would be much easier to maintain...

[Giovanni]

Hi,

Updated in trunk and 18.2 to 5.3.0.

Giovanni

[Thargon]

Hi Giovanni,

unfortunately the issue still persists. First of all, I might have made a mistake as I checked CMSIS 5.3 again and found that the issue was only resolved in 5.4
However, you said you updated to 5.3 but the cmsis_version.h file actually says 5.1. I haven't checked everything, so I can't say if its a typo in that header or if you accidentally merged 5.1 instead of 5.3.

Anyway, for CMSIS 5.4 the problem has been solved (please check yourself to make sure I don't tell bull**** again ), so maybe you can update it when there is time.

Best regards,
Thomas

[Giovanni]

Hi,

I updated to 5.4.0 in trunk, please check if it is fine now.

Giovanni

[Thargon]

Hi,

yep, now the issue is resolved. Thanks for the quick response!

- Thomas