Skip to content

arm64: dts: rockchip: Add radxa-cm3j-rpi-cm4-io#539

Open
LZhaoM wants to merge 4 commits intoradxa:linux-6.1-stan-rkr4.1from
LZhaoM:add-cm3j-rpi-cm4-io-dts
Open

arm64: dts: rockchip: Add radxa-cm3j-rpi-cm4-io#539
LZhaoM wants to merge 4 commits intoradxa:linux-6.1-stan-rkr4.1from
LZhaoM:add-cm3j-rpi-cm4-io-dts

Conversation

@LZhaoM
Copy link
Copy Markdown

@LZhaoM LZhaoM commented Apr 20, 2026

arm64: dts: rockchip: Add radxa-cm3j-rpi-cm4-io

Files rk3568-radxa-cm3j-rpi-cm4-io.dts and rk3568-radxa-cm3j.dtsi are
copied from [0][1][2], except the following two definitions are added to
the sdmmc2 node:
```
address-cells = <1>;
size-cells = <0>;
```
The node wifi@1 is the child of sdmmc2, and its reg is <1>, which means
there is 1 address cell and 0 size cell[3].

[0]: https://github.com/radxa/kernel/tree/linux-5.10-gen-rkr4.1
[1]: https://github.com/radxa/kernel/blob/00eb929841fd2231793b888c167594d14ae7b274/arch/arm64/boot/dts/rockchip/rk3568-radxa-cm3j-rpi-cm4-io.dts
[2]: https://github.com/radxa/kernel/blob/00eb929841fd2231793b888c167594d14ae7b274/arch/arm64/boot/dts/rockchip/rk3568-radxa-cm3j.dtsi
[3]: See Deivcetree Specification '2.3.5 #address-cells and #size-cells' and '2.3.6 reg'

Comment thread arch/arm64/boot/dts/rockchip/rk3568-radxa-cm3j-rpi-cm4-io.dts Outdated
Comment thread arch/arm64/boot/dts/rockchip/rk3568-radxa-cm3j-rpi-cm4-io.dts Outdated
Comment thread arch/arm64/boot/dts/rockchip/rk3568-radxa-cm3j.dtsi Outdated
Comment thread arch/arm64/boot/dts/rockchip/rk3568-radxa-cm3j.dtsi Outdated
Comment thread arch/arm64/boot/dts/rockchip/rk3568-radxa-cm3j-rpi-cm4-io.dts
@LZhaoM LZhaoM requested a review from RadxaYuntian April 21, 2026 08:19
Comment thread arch/arm64/boot/dts/rockchip/rk356x.dtsi Outdated
@LZhaoM LZhaoM requested a review from RadxaYuntian April 21, 2026 09:27
Copy link
Copy Markdown
Member

@RadxaYuntian RadxaYuntian left a comment

Choose a reason for hiding this comment

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

整理一下提交

LZhaoM added 3 commits April 21, 2026 17:41
The node wifi@1 is the child of sdmmc2, and its reg is <1>, which means
there is 1 address cell and 0 size cell[0]. Without the definition of
address-cells and size-cells, dtc will give the following warnings:
```
rk3568-radxa-cm3j.dtsi:576.3-13: Warning (reg_format): /mmc@fe000000/wifi@1:reg: property has invalid length (4 bytes) (#address-cells == 2, #size-cells == 1)
rk3568-radxa-cm3j-rpi-cm4-io.dtb: Warning (pci_device_reg): Failed prerequisite 'reg_format'
rk3568-radxa-cm3j-rpi-cm4-io.dtb: Warning (pci_device_bus_num): Failed prerequisite 'reg_format'
rk3568-radxa-cm3j-rpi-cm4-io.dtb: Warning (i2c_bus_reg): Failed prerequisite 'reg_format'
rk3568-radxa-cm3j-rpi-cm4-io.dtb: Warning (spi_bus_reg): Failed prerequisite 'reg_format'
rk3568-radxa-cm3j.dtsi:574.16-582.4: Warning (avoid_default_addr_size): /mmc@fe000000/wifi@1: Relying on default #address-cells value
rk3568-radxa-cm3j.dtsi:574.16-582.4: Warning (avoid_default_addr_size): /mmc@fe000000/wifi@1: Relying on default #size-cells value
```

[0]: See Deivcetree Specification '2.3.5 #address-cells and #size-cells' and '2.3.6 reg'
Define the LEDs. The heartbeat LED on CM4 IO board is set to off because
CM3J already has a heartbeat LED, it's not very useful to enable two
heartbeat LEDs at the same time.
@LZhaoM LZhaoM force-pushed the add-cm3j-rpi-cm4-io-dts branch from 6f979e4 to 814641a Compare April 21, 2026 09:55
@LZhaoM LZhaoM requested a review from RadxaYuntian April 21, 2026 09:55
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.

2 participants