Hi Andy,
On 2/23/25 20:30, Andy Yan wrote:
> From: Andy Yan <andy.yan@rock-chips.com>
>
> Enable the Mini DisplayPort on this board.
> Note that ROCKCHIP_VOP2_EP_DP0 is defined as 10 in dt-binding header,
> but it will trigger a dtc warning like "graph node unit address error,
> expected "a"" if we use it directly after endpoint, so we use "a"
> instead here.
>
> Signed-off-by: Andy Yan <andy.yan@rock-chips.com>
> ---
>
> .../boot/dts/rockchip/rk3588s-coolpi-4b.dts | 37 +++++++++++++++++++
> 1 file changed, 37 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/rockchip/rk3588s-coolpi-4b.dts b/arch/arm64/boot/dts/rockchip/rk3588s-coolpi-4b.dts
> index e892dd7c91aa..0f58eb56557f 100644
> --- a/arch/arm64/boot/dts/rockchip/rk3588s-coolpi-4b.dts
> +++ b/arch/arm64/boot/dts/rockchip/rk3588s-coolpi-4b.dts
> @@ -39,6 +39,18 @@ chosen {
> stdout-path = "serial2:1500000n8";
> };
>
> + dp-con {
> + compatible = "dp-connector";
> + label = "DP OUT";
> + type = "mini";
> +
> + port {
> + dp_con_in: endpoint {
> + remote-endpoint = <&dp0_out_con>;
> + };
> + };
> + };
> +
> hdmi-con {
> compatible = "hdmi-connector";
> type = "d";
> @@ -220,6 +232,24 @@ &gpu {
> status = "okay";
> };
>
> +&dp0 {
> + status = "okay";
> + pinctrl-0 = <&dp0m0_pins>;
> + pinctrl-names = "default";
> +};
> +
> +&dp0_in {
> + dp0_in_vp2: endpoint {
> + remote-endpoint = <&vp2_out_dp0>;
> + };
> +};
> +
> +&dp0_out {
> + dp0_out_con: endpoint {
> + remote-endpoint = <&dp_con_in>;
> + };
> +};
> +
These three nodes should be placed above the &gpu node.
Best regards,
--
FUKAUMI Naoki
Radxa Computer (Shenzhen) Co., Ltd.
> &hdmi0 {
> status = "okay";
> };
> @@ -889,3 +919,10 @@ vp0_out_hdmi0: endpoint@ROCKCHIP_VOP2_EP_HDMI0 {
> remote-endpoint = <&hdmi0_in_vp0>;
> };
> };
> +
> +&vp2 {
> + vp2_out_dp0: endpoint@a {
> + reg = <ROCKCHIP_VOP2_EP_DP0>;
> + remote-endpoint = <&dp0_in_vp2>;
> + };
> +};