Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

stm32cube: update stm32l4 to cube version V1.18.1 #228

Merged

Conversation

legoabram
Copy link
Contributor

Update Cube version for STM32L4xx series
on https://github.com/STMicroelectronics
from version v1.18.0
to version v1.18.1

ST included their own version of the ifndef for UNUSED, so I removed the note that it was changed.

All files were copied verbatim from a git cloned copy of 1.18.1, and then whitespace was modified to minimize the number of changes. Obviously this will need to be tested more, and it looks like I missed some whitespace, but I'll post this so it starts the review process. My team is in need of this change because it includes a fix for the STM32L496 which breaks 4bit SDMMC.

Once applied to zephyr, this should address: zephyrproject-rtos/zephyr#73398

@legoabram legoabram force-pushed the abram/stm32l4-v1.18.1 branch from 73e25c0 to 4470124 Compare August 27, 2024 21:05
@erwango
Copy link
Member

erwango commented Aug 30, 2024

@legoabram Thanks.
You might want to use Cube package update scripts documented here to be sure you didn't forget anything.

@legoabram
Copy link
Contributor Author

Dagnabbit. Of course you guys made a script for it. My bad. I'll readdress this next week.

Update Cube version for STM32L4xx series
on https://github.com/STMicroelectronics
from version v1.18.0
to version v1.18.1

Signed-off-by: Abram Early <abram.early@gmail.com>
Re - generate common_ll headers after Cube updates

Signed-off-by: Abram Early <abram.early@gmail.com>
@legoabram legoabram force-pushed the abram/stm32l4-v1.18.1 branch from 4470124 to b556aeb Compare September 6, 2024 22:08
@legoabram
Copy link
Contributor Author

legoabram commented Sep 6, 2024

@erwango So close! I missed a number of white spacing inconsistencies, and two ifdefs. It's reuploaded with only the script changes. I left the UNUSED definition as generated by the script rather than use the new ST code. Likely need to update the script to use the ST code, but I'll leave that up to you guys.

@erwango
Copy link
Member

erwango commented Sep 9, 2024

It's reuploaded with only the script changes. I left the UNUSED definition as generated by the script rather than use the new ST code. Likely need to update the script to use the ST code, but I'll leave that up to you guys.

That's expected. The UNUSED stuff is a patch over HAL that's required in Zephyr waiting it to be delivered in ST version.

Have you been able to run non regression campaign (using twister) on this version ?

@legoabram legoabram force-pushed the abram/stm32l4-v1.18.1 branch from 7f93d20 to b556aeb Compare September 9, 2024 18:45
@legoabram
Copy link
Contributor Author

That's expected. The UNUSED stuff is a patch over HAL that's required in Zephyr waiting it to be delivered in ST version.

That's what I mean: The new ST code now has it's own "patch" to not redefine UNUSED if it already has been.

Have you been able to run non regression campaign (using twister) on this version ?

I've run every twister test on my branch with three boards: nucleo_h7a3zi_q, nucleo_g01bre, and nucleo_l476rg. There were two errors that are probably unrelated:

INFO    - Zephyr version: v3.7.0-2431-g5032f099c647
INFO    - Using 'zephyr' toolchain.
INFO    - Building initial testsuite list...
INFO    - Writing JSON report /home/abram/Workspaces/zephyr-rtos/twister-out/testplan.json

Device testing on:
| Platform        | ID                       | Serial device   |
|-----------------|--------------------------|-----------------|
| nucleo_h7a3zi_q | 0052002C3038510634333935 | /dev/ttyACM0    |
| nucleo_g0b1re   | 066AFF485753667187204956 | /dev/ttyACM2    |
| nucleo_l476rg   | 0673FF3732584B3043222412 | /dev/ttyACM3    |

INFO    - JOBS: 32
INFO    - Adding tasks to the queue...
INFO    - Added initial list of jobs to queue
ERROR   - nucleo_g0b1re             tests/lib/spsc_pbuf/libraries.spsc_pbuf             FAILED : Failed
ERROR   - see: /home/abram/Workspaces/zephyr-rtos/twister-out/nucleo_g0b1re/tests/lib/spsc_pbuf/libraries.spsc_pbuf/handler.log
ERROR   - nucleo_l476rg             tests/lib/cmsis_dsp/filtering/libraries.cmsis_dsp.filtering.misc.fpu  FAILED : Timeout
ERROR   - see: /home/abram/Workspaces/zephyr-rtos/twister-out/nucleo_l476rg/tests/lib/cmsis_dsp/filtering/libraries.cmsis_dsp.filtering.misc.fpu/handler.log
INFO    - Total complete: 5838/5838  100%  skipped: 4655, failed:    2, error:    0
INFO    - 2151 test scenarios (5838 test instances) selected, 4655 configurations skipped (4458 by static filter, 197 at runtime).
INFO    - 1181 of 5838 test configurations passed (99.83%), 2 failed, 0 errored, 4655 skipped with 0 warnings in 3216.34 seconds
INFO    - In total 15705 test cases were executed, 35741 skipped on 3 out of total 3 platforms (100.00%)
INFO    - 1183 test configurations executed on platforms, 0 test configurations were only built.

@erwango
Copy link
Member

erwango commented Sep 10, 2024

That's what I mean: The new ST code now has it's own "patch" to not redefine UNUSED if it already has been.

Ok, I've missed that actually. So in this case, let's take the Cube official code and clean the patch list in README file. The interest of those patches is only for the time waiting for fixes to be officially delivered.

@erwango
Copy link
Member

erwango commented Sep 10, 2024

@legoabram Please make a zephyr branch targeting this one, so be able to launch a CI regression campaign on our end as well.
You need to update west.yml file and set, for hal_stm32: revision: pull/228/head

@legoabram
Copy link
Contributor Author

zephyrproject-rtos/zephyr#78249

I'll fix the comment compliance when I have to update it anyway.

@erwango erwango merged commit 8b15f64 into zephyrproject-rtos:main Sep 13, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants