From: Gregor Herburger <hidden> Date: 2026-09-04 09:24:43
The Raspberry Pi 5 routes the spi0 of the rp1 chip to the pin header.
Add the rp1_spi0 node and its pinctrl node to the devicetree to enable
the spi interface.
Signed-off-by: Gregor Herburger <redacted>
---
The RP1 chip, found on the raspberry Pi 5, has nine spi controllers. The
spi0 is routed to the pin header. Add the rp1 nodes and enable it for
the Raspberry Pi 5.
---
Changes in v2:
- Drop first patch as it got already applied: https://lore.kernel.org/lkml/20260903194624.2038562-1-florian.fainelli@broadcom.com/
- add cs gpios to rp1_gpio for pinctrl
- Link to v1: https://patch.msgid.link/20260811-bcm2712-spi-v1-0-4c1708d354bb@linutronix.de
---
arch/arm64/boot/dts/broadcom/bcm2712-rpi-5-b.dts | 22 ++++++++++++++++++++++
1 file changed, 22 insertions(+)
Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Low] Device tree compilation fails due to an undefined phandle reference to `rp1_spi0`.
--
commit 330478d6863965c56268ce6533d5e3b957169da2
Author: Gregor Herburger [off-list ref]
arm64: dts: broadcom: bcm2712-rpi-5-b: add spi0 interface
This commit adds the rp1_spi0 node and its pinctrl node to the bcm2712-rpi-5-b
devicetree to route spi0 to the pin header.
Link to v1: https://patch.msgid.link/20260811-bcm2712-spi-v1-0-4c1708d354bb@linutronix.de
[Severity: Low]
Will this cause the device tree compiler (dtc) to fail with a "Reference to
non-existent node or label" error?
It looks like the rp1_spi0 label is undefined in this revision. The patch
notes mention dropping the first patch because it was already applied, but it
seems the prerequisite patch providing the rp1_spi0 base node is missing from
the tree at this revision.
From: Andrea della Porta <andrea.porta@suse.com> Date: 2026-09-04 10:30:30
Hi Gregor,
On 11:24 Fri 04 Sep , Gregor Herburger wrote:
The Raspberry Pi 5 routes the spi0 of the rp1 chip to the pin header.
Add the rp1_spi0 node and its pinctrl node to the devicetree to enable
the spi interface.
Just a nit-pick: in the comment it's stated that the spi interface is enabled,
which is not true since rp1_usb inherit status="disabled" from its node
definition. This is fine because the user would probably want to enable the
spi interface through some overlay, so no need to resend the patch (at least
for me), so:
Reviewed-by: Andrea della Porta <andrea.porta@suse.com>
Thanks!
Andrea
From: Gregor Herburger <hidden> Date: 2026-09-04 10:54:56
Hi Andrea,
On Fri, Sep 04, 2026 at 12:33:51PM +0200, Andrea della Porta wrote:
Hi Gregor,
On 11:24 Fri 04 Sep , Gregor Herburger wrote:
quoted
The Raspberry Pi 5 routes the spi0 of the rp1 chip to the pin header.
Add the rp1_spi0 node and its pinctrl node to the devicetree to enable
the spi interface.
Just a nit-pick: in the comment it's stated that the spi interface is enabled,
which is not true since rp1_usb inherit status="disabled" from its node
definition. This is fine because the user would probably want to enable the
spi interface through some overlay, so no need to resend the patch (at least
for me), so:
Indeed. It could be rephrased to:
Add the rp1_spi0 node and its pinctrl nodes to the devicetree to make
the spi interface available.
@Florian: Should I send another version or do you want to change it yourself?
Best regards,
--
Gregor Herburger
Linutronix GmbH | Bahnhofstrasse 3 | D-88690 Uhldingen-Mühlhofen
Phone: +49 7556 25 999 35; Fax.: +49 7556 25 999 99
Hinweise zum Datenschutz finden Sie hier (Informations on data privacy
can be found here): https://linutronix.de/legal/data-protection.php
Linutronix GmbH | Firmensitz (Registered Office): Uhldingen-Mühlhofen |
Registergericht (Registration Court): Amtsgericht Freiburg i.Br., HRB700
806 | Geschäftsführer (Managing Directors): Dr. Wilfried Wessner,
Katharina Kopp, Alexander Gieringer
From: Florian Fainelli <f.fainelli@gmail.com>
On Fri, 04 Sep 2026 11:24:24 +0200, Gregor Herburger [off-list ref] wrote:
The Raspberry Pi 5 routes the spi0 of the rp1 chip to the pin header.
Add the rp1_spi0 node and its pinctrl node to the devicetree to enable
the spi interface.
Signed-off-by: Gregor Herburger <redacted>
---
Hi Andrea,
On Fri, Sep 04, 2026 at 12:33:51PM +0200, Andrea della Porta wrote:
quoted
Hi Gregor,
On 11:24 Fri 04 Sep , Gregor Herburger wrote:
quoted
The Raspberry Pi 5 routes the spi0 of the rp1 chip to the pin header.
Add the rp1_spi0 node and its pinctrl node to the devicetree to enable
the spi interface.
Just a nit-pick: in the comment it's stated that the spi interface is enabled,
which is not true since rp1_usb inherit status="disabled" from its node
definition. This is fine because the user would probably want to enable the
spi interface through some overlay, so no need to resend the patch (at least
for me), so:
Indeed. It could be rephrased to:
Add the rp1_spi0 node and its pinctrl nodes to the devicetree to make
the spi interface available.
@Florian: Should I send another version or do you want to change it yourself?
I used that as the new version of your second paragraph, please let me
know if I should be making further adjustments.
Thanks!
--
Florian