[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