ChibiOS Free Embedded RTOS

Regarding to the comment, I'm providing the patch here:

Hi,

Usually we do not fix vendor files, the fix would be lost next time we update the headers. Perhaps the best workaround is to define the x8 as an xB in board.h.

Giovanni

Report issues with CMSIS files here:
https://github.com/ARM-software/CMSIS_5

Also, check if the latest trunk code already has the fix.

Submit bug report to STMicro, on their forums perhaps?

faisal wrote:Report issues with CMSIS files here:
https://github.com/ARM-software/CMSIS_5

Also, check if the latest trunk code already has the fix.

The problem is in ST files, those are not handled there I believe.

Giovanni

Giovanni wrote:Hi,

Usually we do not fix vendor files, the fix would be lost next time we update the headers.
Giovanni

That's reasonable.

Perhaps the best workaround is to define the x8 as an xB in board.h.

Current examples in ChibiOS/demos stick to this approach. However, I find that quite confusing because it becomes very hard to remember which definition encapsulates what other definitions. In order to prevent typing errors (I'm not quite sure of this name) definitions in the header files should be same as the included files in the makefiles.

Another approach might be maintaining another repository for this files, like https://github.com/modm-io/cmsis-header-stm32 and applying this type of patches in that repository. When vendor makes an update, we can simply copy and paste all files into our "vendor" branch, either cherry pick all of our patches or create a "vendor update" patch, review them and finally apply everything to the "master" branch. This may seem a little bit complex, but it would consume a tiny amount of time per vendor update once it's put in order.

If you still find above approach cumbersome, I may propagate my own patches along with a ST-patches folder within chibi-project, my boilerplate for ChibiOS projects.