dts/boards: nordic: Fix some reg issues #93098
Conversation
nordicjm
commented
Jul 14, 2025
nordicjm
commented
Adds missing reg parameters to DTS nodes Signed-off-by: Jamie McCrae <jamie.mccrae@nordicsemi.no>
This compatible does not exist Signed-off-by: Jamie McCrae <jamie.mccrae@nordicsemi.no>
Fixes the address of this peripheral Signed-off-by: Jamie McCrae <jamie.mccrae@nordicsemi.no>
github-actions
bot
requested review from
anangl,
jaz1-nordic,
kl-cruz,
magp-nordic,
masz-nordic and
nika-nordic
| @@ -267,7 +267,7 @@ ipc0: &cpuapp_cpurad_ipc { | |||
| status = "okay"; | |||
|
|
|||
| mx25uw63: mx25uw6345g@0 { | |||
| compatible = "mxicy,mx25u", "jedec,mspi-nor"; | |||
| compatible = "jedec,mspi-nor"; | |||
There was a problem hiding this comment.
The compatible needs to stay, quirks in mspi_nor driver depend on it.
There was a problem hiding this comment.
then a compatible binding needs to be created
There was a problem hiding this comment.
and actually that isn't even right, it should be doing what the spi_nor driver does with flags: https://github.com/zephyrproject-rtos/zephyr/blob/main/dts/bindings/mtd/jedec%2Cspi-nor-common.yaml#L93
Removes compatible strings that were not defined and are not needed Signed-off-by: Jamie McCrae <jamie.mccrae@nordicsemi.no>
|
There was a problem hiding this comment.
My approval implies that the changes requested by the vscode team were done. But we do not have scope and knowledge to determine any side effects. Please make sure someone who knows how these files are used in the SDK by apps and customers; also reviews these changes
