Skip to content

Conversation

FelixWang47831
Copy link
Contributor

Suppress "spi_bus_bridge" warning, as the flexspi is used as memory controller, the node name isn't spi@...

Copy link
Member

@decsny decsny left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rename the flexspi node name to SPI then, I guess memory-controller was the wrong choice of name from the DT specification.

The problem is due to this check:
https://github.com/dgibson/dtc/blob/9197f1ccd95c9475006677aa97bf39727c1e8aa5/checks.c#L1116

and this property:

spi-max-frequency = <DT_FREQ_M(133)>;

@FelixWang47831
Copy link
Contributor Author

rename the flexspi node name to SPI then, I guess memory-controller was the wrong choice of name from the DT specification.

The problem is due to this check: https://github.com/dgibson/dtc/blob/9197f1ccd95c9475006677aa97bf39727c1e8aa5/checks.c#L1116

and this property:

spi-max-frequency = <DT_FREQ_M(133)>;

Hi @decsny Thanks for clarification. Has updated node name.

decsny
decsny previously approved these changes Sep 24, 2025
dleach02
dleach02 previously approved these changes Sep 25, 2025
@dleach02
Copy link
Member

@FelixWang47831 you need to rebase your PR. Note that #94390 moved things around with the RT700 so your conflict needs to move changes to a different file

Rename node name to spi.

Signed-off-by: Felix Wang <fei.wang_3@nxp.com>
@FelixWang47831
Copy link
Contributor Author

@FelixWang47831 you need to rebase your PR. Note that #94390 moved things around with the RT700 so your conflict needs to move changes to a different file

Updated, please help to merge, thanks.

Copy link

@kartben kartben merged commit ca359b1 into zephyrproject-rtos:main Oct 1, 2025
28 checks passed
@FelixWang47831 FelixWang47831 deleted the FIX-DTS-warning-9-23 branch October 11, 2025 06:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants