[PATCH] bsps/arm: fix installation broken by recent CMSIS files update
Karel Gardas
karel at functional.vision
Tue Jul 25 13:22:10 UTC 2023
Hello Christian,
the patch fixing warnings is already submitted for review.
https://lists.rtems.org/pipermail/devel/2023-July/075881.html
Thanks,
Karel
On 7/25/23 12:07, Christian MAUDERER wrote:
> Hello Karel,
>
> I tested with the patch from Chris (which is identical to yours) and I
> have no problems building and installing the imx7, imxrt1052 and
> imxrt1166-cm7-saltshaker BSP and libbsd from master for these BSPs.
>
> The only point that I have noted are a lot of new warnings when building
> the BSP. Example:
>
> ../../../../../rtems-sources/rtems/bsps/arm/include/cmsis_gcc.h:139:15:
> warning: nested extern declaration of '_start' [-Wnested-externs]
> 139 | extern void _start(void) __NO_RETURN;
>
> Best regards
>
> Christian
>
> On 2023-07-25 08:55, Karel Gardas wrote:
>>
>> Sure! Not a problem, I was just waiting for Christian to confirm its
>> also working on his side.
>>
>> Thanks,
>> Karel
>>
>> On 7/25/23 03:34, Chris Johns wrote:
>>> Hi Karel,
>>>
>>> I did not see this and made and pushed a similar patch. Thanks for
>>> this and
>>> sorry for not checking closely as I missed it on the first pass.
>>>
>>> Thanks
>>> Chris
>>>
>>> On 25/7/2023 1:53 am, Karel Gardas wrote:
>>>> ---
>>>> spec/build/bsps/arm/grp.yml | 7 ++++---
>>>> 1 file changed, 4 insertions(+), 3 deletions(-)
>>>>
>>>> diff --git a/spec/build/bsps/arm/grp.yml b/spec/build/bsps/arm/grp.yml
>>>> index 1058f58d92..a48cd80d74 100644
>>>> --- a/spec/build/bsps/arm/grp.yml
>>>> +++ b/spec/build/bsps/arm/grp.yml
>>>> @@ -10,12 +10,13 @@ includes: []
>>>> install:
>>>> - destination: ${BSP_INCLUDEDIR}
>>>> source:
>>>> + - bsps/arm/include/cachel1_armv7.h
>>>> + - bsps/arm/include/cmsis_compiler.h
>>>> - bsps/arm/include/cmsis_gcc.h
>>>> + - bsps/arm/include/cmsis_version.h
>>>> - bsps/arm/include/core_cm7.h
>>>> - bsps/arm/include/core_cm4.h
>>>> - - bsps/arm/include/core_cmFunc.h
>>>> - - bsps/arm/include/core_cmInstr.h
>>>> - - bsps/arm/include/core_cmSimd.h
>>>> + - bsps/arm/include/mpu_armv7.h
>>>> - bsps/arm/include/uart.h
>>>> - destination: ${BSP_INCLUDEDIR}/bsp
>>>> source:
>>
>> _______________________________________________
>> devel mailing list
>> devel at rtems.org
>> http://lists.rtems.org/mailman/listinfo/devel
>
More information about the devel
mailing list