[PATCH v3 0/2] riscv: dts: starfive: Add OrangePi RV board

E Shattow posted 2 patches 1 week, 1 day ago
.../devicetree/bindings/riscv/starfive.yaml   |  1 +
arch/riscv/boot/dts/starfive/Makefile         |  1 +
.../boot/dts/starfive/jh7110-orangepi-rv.dts  | 76 +++++++++++++++++++
3 files changed, 78 insertions(+)
create mode 100644 arch/riscv/boot/dts/starfive/jh7110-orangepi-rv.dts
[PATCH v3 0/2] riscv: dts: starfive: Add OrangePi RV board
Posted by E Shattow 1 week, 1 day ago
Icenowy is busy and so with their permission I have applied v2 review
feedback. Additionally I revise the commit messages. This should now be in
good shape for -next

- E

Changes since v2:
- enable active high mmc1 card detect gpio
- drop delete-node from mmc0 reset pins following commit fa939a287224
  ("dts: starfive: jh7110-common: split out mmc0 reset pins from common 
  into boards")
- sort phy0 properties rx before tx
- revise commit description of SoC peripherals as list
- revise commit description of compatible binding

Changes in v2:
- Property order change mentioned in the review of v1.
- Added Wi-Fi (along with the always on VCC3V3_PCIE regulator, which is
  used to power up WIFI_3V3). The OOB IRQ is still not possible to use
  because of some incompatibility between StarFive pinctrl driver and
  brcmfmac.
- Removed the LED because it's in common DTSI.

Icenowy Zheng (2):
  dt-bindings: riscv: starfive: add xunlong,orangepi-rv
  riscv: dts: starfive: add Orange Pi RV

 .../devicetree/bindings/riscv/starfive.yaml   |  1 +
 arch/riscv/boot/dts/starfive/Makefile         |  1 +
 .../boot/dts/starfive/jh7110-orangepi-rv.dts  | 76 +++++++++++++++++++
 3 files changed, 78 insertions(+)
 create mode 100644 arch/riscv/boot/dts/starfive/jh7110-orangepi-rv.dts


base-commit: 1088d49b626487777913079bef0db3adef4bfb4a
-- 
2.50.0
Re: [PATCH v3 0/2] riscv: dts: starfive: Add OrangePi RV board
Posted by Conor Dooley 6 days, 2 hours ago
From: Conor Dooley <conor.dooley@microchip.com>

On Sun, 23 Nov 2025 14:50:43 -0800, E Shattow wrote:
> Icenowy is busy and so with their permission I have applied v2 review
> feedback. Additionally I revise the commit messages. This should now be in
> good shape for -next
> 
> - E
> 
> Changes since v2:
> - enable active high mmc1 card detect gpio
> - drop delete-node from mmc0 reset pins following commit fa939a287224
>   ("dts: starfive: jh7110-common: split out mmc0 reset pins from common
>   into boards")
> - sort phy0 properties rx before tx
> - revise commit description of SoC peripherals as list
> - revise commit description of compatible binding
> 
> [...]

Applied to riscv-dt-for-next, thanks! I know I mentioned waiting for
Emil, but this stuff is actually fairly simple so I just went ahead
with grabbing it.

[1/2] dt-bindings: riscv: starfive: add xunlong,orangepi-rv
      https://git.kernel.org/conor/c/d94ebab404b0
[2/2] riscv: dts: starfive: add Orange Pi RV
      https://git.kernel.org/conor/c/5b70764e1019

Thanks,
Conor.
Re: [PATCH v3 0/2] riscv: dts: starfive: Add OrangePi RV board
Posted by Emil Renner Berthing 5 days, 10 hours ago
Quoting Conor Dooley (2025-11-25 23:24:37)
> From: Conor Dooley <conor.dooley@microchip.com>
>
> On Sun, 23 Nov 2025 14:50:43 -0800, E Shattow wrote:
> > Icenowy is busy and so with their permission I have applied v2 review
> > feedback. Additionally I revise the commit messages. This should now be in
> > good shape for -next
> >
> > - E
> >
> > Changes since v2:
> > - enable active high mmc1 card detect gpio
> > - drop delete-node from mmc0 reset pins following commit fa939a287224
> >   ("dts: starfive: jh7110-common: split out mmc0 reset pins from common
> >   into boards")
> > - sort phy0 properties rx before tx
> > - revise commit description of SoC peripherals as list
> > - revise commit description of compatible binding
> >
> > [...]
>
> Applied to riscv-dt-for-next, thanks! I know I mentioned waiting for
> Emil, but this stuff is actually fairly simple so I just went ahead
> with grabbing it.

That's fine thanks. I was under the impression that this still needed some work,
but it seems like you've fixed it up when merging.

In case you still want it here is my

Acked-by: Emil Renner Berthing <emil.renner.berthing@canonical.com>