Re: [PATCH v4 2/2] arm64: dts: rockchip: Add rk3576 evb2 board

[Shawn Lin]

在 2026/01/21 星期三 9:53, Chaoyi Chen 写道:
> From: Chaoyi Chen <chaoyi.chen@xxxxxxxxxxxxxx>
>
> General features for rk3576 evb2 board:
>      - Rockchip RK3576
>      - LPDDR4/4X
>      - eMMC5.1
>      - RK806-2x2pcs + DiscretePower
>      - 1x HDMI2.1 TX / HDMI2.0 RX
>      - 1x full size DP1.4 TX (Only 2 Lanes)
>      - 2x 10/100/1000M Ethernet
>      - 5x SATA3.0 7Pin Slot
>      - 2x USB3.2 Gen1 Host
>      - 3x USB2.0 Host
>      - WIFI/BT
>      - ...
>
> Tested with eMMC/SDMMC/HDMI/USB/Ethernet/WIFI/BT module.
>
> Signed-off-by: Chaoyi Chen <chaoyi.chen@xxxxxxxxxxxxxx>
> Reviewed-by: Alexey Charkov <alchark@xxxxxxxxx>
> ---

---8<----

> +
> +&sdio {
> +	#address-cells = <1>;
> +	#size-cells = <0>;
> +	bus-width = <4>;
> +	cap-sd-highspeed;
> +	cap-sdio-irq;
> +	disable-wp;

disable-wp is not used for SDIO case, could be removed.


> +	keep-power-in-suspend;
> +	max-frequency = <150000000>;
> +	mmc-pwrseq = <&sdio_pwrseq>;
> +	no-sd;
> +	no-mmc;
> +	non-removable;
> +	pinctrl-names = "default";
> +	pinctrl-0 = <&sdmmc1m0_bus4 &sdmmc1m0_clk &sdmmc1m0_cmd>;
> +	sd-uhs-sdr104;
> +	status = "okay";
> +
> +	brcmf: wifi@1 {
> +		compatible = "brcm,bcm4329-fmac";
> +		reg = <1>;
> +		interrupt-parent = <&gpio0>;
> +		interrupts = <RK_PB0 IRQ_TYPE_LEVEL_HIGH>;
> +		interrupt-names = "host-wake";
> +		pinctrl-names = "default";
> +		pinctrl-0 = <&wifi_host_wake>;
> +	};
> +};
> +
> +&sdhci {
> +	bus-width = <8>;
> +	full-pwr-cycle-in-suspend;
> +	mmc-hs400-1_8v;
> +	mmc-hs400-enhanced-strobe;
> +	no-sdio;
> +	no-sd;
> +	non-removable;
> +	status = "okay";
> +};
> +
> +&sdmmc {
> +	bus-width = <4>;
> +	cap-sd-highspeed;
> +	cd-gpios = <&gpio0 RK_PA7 GPIO_ACTIVE_LOW>;
> +	disable-wp;
> +	no-sdio;
> +	no-mmc;
> +	sd-uhs-sdr104;
> +	vqmmc-supply = <&vccio_sd_s0>;

Should use vmmc-supply to contrl the power, the same as EVB1[1].

[1] 
https://lore.kernel.org/linux-rockchip/1768524932-163929-5-git-send-email-shawn.lin@xxxxxxxxxxxxxx/T/#u

> +	status = "okay";
> +};
> +
> +&u2phy0 {
> +	status = "okay";
> +};
> +