hwmv2: port STM32L5 SoCs and boards #68706
Conversation
erwango
force-pushed
the
hwm_stm32l5
branch
7 times, most recently
from
f6a5e03
to
0a23f72
Compare
erwango
requested review from
thedjnK,
tejlmand,
nordicjm,
gautierg-st and
ajarmouni-st
erwango
requested review from
carlocaione,
povergoing,
lorc,
firscity,
sidcha,
ABOSTM,
FRASTM,
Navin-Sankar,
yonsch,
fkokosinski,
kgugala,
stephanosio,
ibirnbaum and
carlescufi
as code owners
erwango
removed request for
Navin-Sankar,
yonsch,
ifyall,
npal-cy,
greg-leach,
ibirnbaum,
MulinChao,
ChiHuaL and
firscity
Migrate STM2L5 series to new HW model. Signed-off-by: Erwan Gouriou <[email protected]>
| config BOARD_NUCLEO_L552ZE_Q_STM32L552XX_NS | ||
| bool | ||
| select SOC_STM32L552XX |
There was a problem hiding this comment.
Not resolved please amend commits in future rather than adding to the final commit with an incorrect commit
There was a problem hiding this comment.
an incorrect commit. Did you read commit description ?
This is actually a dedicated refactoring commit with a proper justification.
There was a problem hiding this comment.
soc: stm32l5: Rename overlays for nucleo_l552ze_q ns target.
File name should now be nucleo_l552ze_q_stm32l552xx_ns.overlay
No relation to this change at all
| name: nucleo_l552ze_q | ||
| vendor: st | ||
| socs: | ||
| - name: stm32l552xx |
There was a problem hiding this comment.
- name should go left by 2 spaced, then all lines below also
There was a problem hiding this comment.
This is the way it is generated by scripts/utils/board_v1_to_v2.py script.
There was a problem hiding this comment.
our .yamllint file says indentation: spaces: 2, I guess it would complain if this was wrong.
There was a problem hiding this comment.
We have .yaml files with both additional indents and without, I remember seeing a warning on a PR but can't remember which one it was but they can't both be right?
There was a problem hiding this comment.
from https://github.com/zephyrproject-rtos/zephyr/blob/main/.yamllint
indentation:
spaces: 2
indent-sequences: consistent
the lists are a bit odd for some reasons, the linter just checks that the style is coherent throughout the file
Port the board to HWMv2. Signed-off-by: Erwan Gouriou <[email protected]>
Port the board to HWMv2. Signed-off-by: Erwan Gouriou <[email protected]>
Need to update board name scheme for L5 ns targets. Signed-off-by: Erwan Gouriou <[email protected]>
Rather than configuring USE_DT_CODE_PARTITION in all ns targets, do it conditionally as part of SoC description. This allows cleaning up some BOARD dedicated symbols that are now useless. Signed-off-by: Erwan Gouriou <[email protected]>
File name should now be nucleo_l552ze_q_stm32l552xx_ns.overlay Signed-off-by: Erwan Gouriou <[email protected]>
|
CI is failing for |
It is actually failing on main branch as well. We'll fix it there. |
nordicjm
merged commit 3932ac8
into
zephyrproject-rtos:collab-hwm
No description provided.