GCC warnings due to outdated CMSIS Topic is solved

Report here problems in any of ChibiOS components. This forum is NOT for support.
Thargon
Posts: 86
Joined: Wed Feb 04, 2015 5:03 pm
Location: CITEC, Bielefeld University, germany
Has thanked: 6 times
Been thanked: 12 times

GCC warnings due to outdated CMSIS  Topic is solved

Postby Thargon » Thu Jul 05, 2018 10:40 am

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...

User avatar
Giovanni
Site Admin
Posts: 11376
Joined: Wed May 27, 2009 8:48 am
Location: Salerno, Italy
Has thanked: 429 times
Been thanked: 361 times
Contact:

Re: GCC warnings due to outdated CMSIS

Postby Giovanni » Sat Jul 07, 2018 5:24 pm

Hi,

Updated in trunk and 18.2 to 5.3.0.

Giovanni

Thargon
Posts: 86
Joined: Wed Feb 04, 2015 5:03 pm
Location: CITEC, Bielefeld University, germany
Has thanked: 6 times
Been thanked: 12 times

Re: GCC warnings due to outdated CMSIS

Postby Thargon » Thu Sep 06, 2018 9:13 am

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 :oops:
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

User avatar
Giovanni
Site Admin
Posts: 11376
Joined: Wed May 27, 2009 8:48 am
Location: Salerno, Italy
Has thanked: 429 times
Been thanked: 361 times
Contact:

Re: GCC warnings due to outdated CMSIS

Postby Giovanni » Thu Sep 06, 2018 1:30 pm

Hi,

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

Giovanni

Thargon
Posts: 86
Joined: Wed Feb 04, 2015 5:03 pm
Location: CITEC, Bielefeld University, germany
Has thanked: 6 times
Been thanked: 12 times

Re: GCC warnings due to outdated CMSIS

Postby Thargon » Thu Sep 06, 2018 1:37 pm

Hi,

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

- Thomas


Return to “Bug Reports”

Who is online

Users browsing this forum: No registered users and 4 guests