[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <b8db0cdf-163e-416d-94b8-c9e1f10c8011@freeshell.de>
Date: Sun, 5 Oct 2025 11:47:29 -0700
From: E Shattow <e@...eshell.de>
To: Icenowy Zheng <uwu@...nowy.me>, Rob Herring <robh@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley
<conor+dt@...nel.org>, Paul Walmsley <paul.walmsley@...ive.com>,
Palmer Dabbelt <palmer@...belt.com>, Albert Ou <aou@...s.berkeley.edu>,
Alexandre Ghiti <alex@...ti.fr>, Emil Renner Berthing <kernel@...il.dk>,
Michael Zhu <michael.zhu@...rfivetech.com>,
Drew Fustini <drew@...gleboard.org>, Yao Zi <ziyao@...root.org>
Cc: devicetree@...r.kernel.org, linux-riscv@...ts.infradead.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2 2/2] riscv: dts: starfive: add DT for Orange Pi RV
Hi Icenowy,
On 9/30/25 08:51, Icenowy Zheng wrote:
> 在 2025-09-30星期二的 18:03 +0800,Icenowy Zheng写道:
>> Orange Pi RV is a newly released SBC with JH7110 SoC, single GbE port
>> (connected to JH7110 GMAC0 via a YT8531 PHY), 4 USB ports (via a
>> VL805
>> PCIe USB controller connected to JH7110 PCIE0), a M.2 M-key slot
>> (connected to JH7110 PCIE1), a HDMI video output, a 3.5mm audio
>> output
>> and a microSD slot.
>>
>> Other Onboard peripherals contain a SPI NOR (which contains the U-
>> Boot
>> firmware), a 24c02 EEPROM storing some StarFive-specific information
>> (factory programmed and read only by default) and an Ampak AP6256
>> SDIO
>> Wi-Fi module.
>>
>> Signed-off-by: Icenowy Zheng <uwu@...nowy.me>
>> ---
>> 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.
>>
>> arch/riscv/boot/dts/starfive/Makefile | 1 +
>> .../boot/dts/starfive/jh7110-orangepi-rv.dts | 87
>> +++++++++++++++++++
>> 2 files changed, 88 insertions(+)
>> create mode 100644 arch/riscv/boot/dts/starfive/jh7110-orangepi-
>> rv.dts
>>
>> diff --git a/arch/riscv/boot/dts/starfive/Makefile
>> b/arch/riscv/boot/dts/starfive/Makefile
>> index b3bb12f78e7d5..24f1a44828350 100644
>> --- a/arch/riscv/boot/dts/starfive/Makefile
>> +++ b/arch/riscv/boot/dts/starfive/Makefile
>> @@ -10,6 +10,7 @@ dtb-$(CONFIG_ARCH_STARFIVE) += jh7100-starfive-
>> visionfive-v1.dtb
>>
>> dtb-$(CONFIG_ARCH_STARFIVE) += jh7110-deepcomputing-fml13v01.dtb
>> dtb-$(CONFIG_ARCH_STARFIVE) += jh7110-milkv-mars.dtb
>> +dtb-$(CONFIG_ARCH_STARFIVE) += jh7110-orangepi-rv.dtb
>> dtb-$(CONFIG_ARCH_STARFIVE) += jh7110-pine64-star64.dtb
>> dtb-$(CONFIG_ARCH_STARFIVE) += jh7110-starfive-visionfive-2-
>> v1.2a.dtb
>> dtb-$(CONFIG_ARCH_STARFIVE) += jh7110-starfive-visionfive-2-
>> v1.3b.dtb
>> diff --git a/arch/riscv/boot/dts/starfive/jh7110-orangepi-rv.dts
>> b/arch/riscv/boot/dts/starfive/jh7110-orangepi-rv.dts
>> new file mode 100644
>> index 0000000000000..5a917b7db6f78
>> --- /dev/null
>> +++ b/arch/riscv/boot/dts/starfive/jh7110-orangepi-rv.dts
>> @@ -0,0 +1,87 @@
>> +// SPDX-License-Identifier: GPL-2.0 OR MIT
>> +/*
>> + * Copyright (C) 2025 Icenowy Zheng <uwu@...nowy.me>
>> + */
>> +
>> +/dts-v1/;
>> +#include "jh7110-common.dtsi"
>> +
>> +/ {
>> + model = "Xunlong Orange Pi RV";
>> + compatible = "xunlong,orangepi-rv", "starfive,jh7110";
>> +
>> + /* This regulator is always on by hardware */
>> + reg_vcc3v3_pcie: regulator-vcc3v3-pcie {
>> + compatible = "regulator-fixed";
>> + regulator-name = "vcc3v3-pcie";
>> + regulator-min-microvolt = <3300000>;
>> + regulator-max-microvolt = <3300000>;
>> + regulator-always-on;
>> + };
>> +
>> + wifi_pwrseq: wifi-pwrseq {
>> + compatible = "mmc-pwrseq-simple";
>> + reset-gpios = <&sysgpio 62 GPIO_ACTIVE_LOW>;
>> + };
>> +};
>> +
>> +&gmac0 {
>> + assigned-clocks = <&aoncrg JH7110_AONCLK_GMAC0_TX>;
>> + assigned-clock-parents = <&aoncrg
>> JH7110_AONCLK_GMAC0_RMII_RTX>;
>> + starfive,tx-use-rgmii-clk;
>> + status = "okay";
>> +};
>> +
>> +&mmc0 {
>> + #address-cells = <1>;
>> + #size-cells = <0>;
>> + cap-sd-highspeed;
>> + mmc-pwrseq = <&wifi_pwrseq>;
>> + vmmc-supply = <®_vcc3v3_pcie>;
>> + vqmmc-supply = <&vcc_3v3>;
>> + status = "okay";
>> +
>> + ap6256: wifi@1 {
>> + compatible = "brcm,bcm43456-fmac", "brcm,bcm4329-
>> fmac";
>> + reg = <1>;
>> + /* TODO: out-of-band IRQ on GPIO21 */
>> + };
>> +};
>> +
>> +&mmc0_pins {
>> + /*
>> + * As the MMC0 bus is used to connect a SDIO Wi-Fi card
>> instead of
>> + * an eMMC card, and the eMMC RST is repurposed to be an
>> enablement
>> + * pin of the SDIO Wi-Fi, remove it from the pinctrl node and
>> manage
>> + * it as a GPIO instead.
>> + */
>> + /delete-node/ rst-pins;
>> +};
>> +
Listed on the schematic [1] as:
Default function SDIO0 RSTn GPIO62 for eMMC:J9
Highlighted (non-default?) function GPIO62 D17 << WIFI_EN_H_GPIO62
WIFI_EN_H_GPIO62 >> WIFI_PWREN (pin 12 WL_REG_ON of module AP6256)
I've sent a patch [2] to portion out mmc0 reset pins from jh7110-common.dtsi
>> +&mmc1 {
>> + /delete-property/ cd-gpios;
>> + broken-cd;
>
> Well it's found that the card detect is working, although with
> different polarity with other boards.
>
> Here should be:
> ```
> cd-gpios = <&sysgpio 41 GPIO_ACTIVE_HIGH>;
> ```
>
> Will be fixed in the next revision.
Yes, listed on the schematic [1] as:
SD SDIO0 CD GPIO41 for MicroSD:J10
There is not a mention of active high or active low on the schematic
label, however there is listed a 10Kohm pull-up to +Vdd1.833 for the
circuit diagram of the Micro SD Card. The card holder is referenced to
ground and could reasonably be N/O or N/C switch operation depending on
the exact part selected for manufacture.
>
>> +};
>> +
>> +&pcie0 {
>> + status = "okay";
>> +};
>> +
>> +&pcie1 {
>> + status = "okay";
>> +};
>> +
>> +&phy0 {
>> + rx-internal-delay-ps = <1500>;
>> + tx-internal-delay-ps = <1500>;
>> + motorcomm,tx-clk-adj-enabled;
>> + motorcomm,tx-clk-10-inverted;
>> + motorcomm,tx-clk-100-inverted;
>> + motorcomm,tx-clk-1000-inverted;
>> + motorcomm,rx-clk-drv-microamp = <3970>;
>> + motorcomm,rx-data-drv-microamp = <2910>;
>> +};
'motorcomm,rx' before 'motorcomm,tx' in `LANG=C sort` of vendor-specific
properties.
>> +
>> +&pwmdac {
>> + status = "okay";
>> +};
>
Additional non-default GPIO as listed in the Orange Pi design:
GPIO21 WIFI_WAKE_HOST_H /* default vf2 function PCIE_PWREN_H_GPIO21 */
GPIO22 >> BT_UART_RXD /* default vf2 function MIPI_PWR_EN */
GPIO23 << BT_UART_TXD /* default vf2 function LCD RESET */
GPIO24 << BT_UART_CTS /* default vf2 function MIPI_LCD_BL */
GPIO25 << BT_UART_RTS /* default vf2 function TP_DET_GPIO25 */
GPIO30 << BT_EN_H_GPIO30 /* default vf2 function TP_INT_L */
GPIO31 << BT_WAKE_GPIO31 /* default vf2 function TP_RST_L */
Of all the above, GPIO21 is defined in jh7110-common.dtsi
&pcie1_pins/wake-pins and may need consideration.
There is a note about "PMIC_PWRON as Key" and so does this have the
meaning of it is used as an input device?
Also noted is that the USB over-current circuit appears to be
implemented, different than being absent in other VF2 designs.
1:
http://www.orangepi.org/html/hardWare/computerAndMicrocontrollers/service-and-support/Orange-Pi-RV.html
2: https://lore.kernel.org/lkml/20251005174450.1949110-1-e@freeshell.de/
With the card detect describing hardware corrected, and clean up the
vendor property sort, then please confirm if you think GPIO21 is
described correctly.
Acked-by: E Shattow <e@...eshell.de>
Powered by blists - more mailing lists