From nobody Wed Oct 8 14:20:33 2025 Received: from out-184.mta0.migadu.com (out-184.mta0.migadu.com [91.218.175.184]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 6345D2E973E for ; Fri, 27 Jun 2025 15:27:11 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=91.218.175.184 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1751038033; cv=none; b=kdSnaFQOA6oB+23P7zll9Gn8Iev4eYp704iknlQEXxcTnwiyntPU4PLB7f+MlWCZOauelsfALUINBx9Vw01QddHXEGZFtUZikpogjH4E/E+m2SEiAej7aBgmaUSJ8m6bCyp6LAEVjoiuY+e6p7wYP0vyuLzvpTCtCeqD1RdfEzM= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1751038033; c=relaxed/simple; bh=K0JRTKvwPvkqpe00zFUs8iPdwgVDsWTrE+mT7gQdcGc=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=RM7iQDaaj0UZM42Jfxa9NDB89GXQ2RvADfhsEQC3gdAMq3rjHQYC03HUO1pqQkH0E4g4boaH8a5Tg1Ezl3ePsOYNODCtaGbdExjhmsf7qZl17Gl52+oLysNr/CjVUTfL69FaGsVQSLERy7J2bq5he+3wpSZNpzavdgqzZ6gEb80= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=cknow.org; spf=pass smtp.mailfrom=cknow.org; dkim=pass (2048-bit key) header.d=cknow.org header.i=@cknow.org header.b=AIoHMfXx; arc=none smtp.client-ip=91.218.175.184 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=cknow.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=cknow.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=cknow.org header.i=@cknow.org header.b="AIoHMfXx" X-Report-Abuse: Please report any abuse attempt to abuse@migadu.com and include these headers. DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cknow.org; s=key1; t=1751038029; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=IvcxkrvBiBUkVTTSTfgCGQwPzHmN5OZ3q178klwKd4g=; b=AIoHMfXxYuQewBtQ1iiyF+ABi0Xj3b9khQOMIergAB2mRQZ/+1xwoBr8y5pemFTMgZR39Z +JA+u4WUUS4/4S6n9Lo+1EjwWN5yQYHUJ20ds82aGtgZ2Te0G3bGFYW0z63svQ1T1axVFC kOXtYGOMTbXsypYd/vY8bNePXOpDnyYeym4SMabwxbLD2H+4SUsAj+4Ld31ErEPl54QWBB pWqk+hw+Sq/VSddKLaNMEzb8UByOtsY3pXuXd1gIf1yGP5OEU+aVgbuQRlTJjbvWIgYFS2 siUelEdBXyjSQLac4PKcUeL0KBZ7YIB6dfhVrLYp8eEUFksCXYSEHiJmmlycnA== From: Diederik de Haas To: Rob Herring , Krzysztof Kozlowski , Conor Dooley , Heiko Stuebner Cc: Dragan Simic , Quentin Schulz , Johan Jonker , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-rockchip@lists.infradead.org, linux-kernel@vger.kernel.org, Diederik de Haas Subject: [PATCH 2/8] arm64: dts: rockchip: Refactor DSI nodes on rk3399 boards Date: Fri, 27 Jun 2025 17:16:21 +0200 Message-ID: <20250627152645.740981-3-didi.debian@cknow.org> In-Reply-To: <20250627152645.740981-1-didi.debian@cknow.org> References: <20250627152645.740981-1-didi.debian@cknow.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-Migadu-Flow: FLOW_OUT Content-Type: text/plain; charset="utf-8" The #address-cells and #size-cells properties are not useful on the DSI controller nodes; they are only useful/required on ports and panel(s). So remove them from the controller node and add them where actually needed on the various rk3399 based boards. Next to that, there were several (exact) redefinitions of nodes which are already present in rk3399-base.dtsi to add a mipi_out endpoint. Simplify that by referencing the mipi_out phandle and add the endpoint to that, which allows the removeal of the ports redefinition. And fix 1 instance where the mipi_out referenced node was not sorted correctly. This fixes the following DTB validation warnings: unnecessary #address-cells/#size-cells without "ranges", "dma-ranges" or child "reg" property Signed-off-by: Diederik de Haas --- arch/arm64/boot/dts/rockchip/rk3399-base.dtsi | 4 --- .../boot/dts/rockchip/rk3399-gru-scarlet.dtsi | 30 ++++++++----------- .../dts/rockchip/rk3399-pinephone-pro.dts | 2 ++ .../rk3399-puma-haikou-video-demo.dtso | 12 ++++---- .../dts/rockchip/rk3399-rockpro64-screen.dtso | 21 ++++--------- 5 files changed, 26 insertions(+), 43 deletions(-) diff --git a/arch/arm64/boot/dts/rockchip/rk3399-base.dtsi b/arch/arm64/boo= t/dts/rockchip/rk3399-base.dtsi index 9d5f5b083e3c..4dcceb9136b7 100644 --- a/arch/arm64/boot/dts/rockchip/rk3399-base.dtsi +++ b/arch/arm64/boot/dts/rockchip/rk3399-base.dtsi @@ -2071,8 +2071,6 @@ mipi_dsi: dsi@ff960000 { resets =3D <&cru SRST_P_MIPI_DSI0>; reset-names =3D "apb"; rockchip,grf =3D <&grf>; - #address-cells =3D <1>; - #size-cells =3D <0>; status =3D "disabled"; =20 ports { @@ -2112,8 +2110,6 @@ mipi_dsi1: dsi@ff968000 { resets =3D <&cru SRST_P_MIPI_DSI1>; reset-names =3D "apb"; rockchip,grf =3D <&grf>; - #address-cells =3D <1>; - #size-cells =3D <0>; #phy-cells =3D <0>; status =3D "disabled"; =20 diff --git a/arch/arm64/boot/dts/rockchip/rk3399-gru-scarlet.dtsi b/arch/ar= m64/boot/dts/rockchip/rk3399-gru-scarlet.dtsi index 5e068377a0a2..100c22af7265 100644 --- a/arch/arm64/boot/dts/rockchip/rk3399-gru-scarlet.dtsi +++ b/arch/arm64/boot/dts/rockchip/rk3399-gru-scarlet.dtsi @@ -627,18 +627,10 @@ &mipi_dphy_rx0 { }; =20 &mipi_dsi { - status =3D "okay"; clock-master; - - ports { - mipi_out: port@1 { - reg =3D <1>; - - mipi_out_panel: endpoint { - remote-endpoint =3D <&mipi_in_panel>; - }; - }; - }; + #address-cells =3D <1>; + #size-cells =3D <0>; + status =3D "okay"; =20 mipi_panel: panel@0 { /* 2 different panels are used, compatibles are in dts files */ @@ -673,15 +665,17 @@ mipi1_in_panel: endpoint { =20 &mipi_dsi1 { status =3D "okay"; +}; =20 - ports { - mipi1_out: port@1 { - reg =3D <1>; +&mipi_out { + mipi_out_panel: endpoint { + remote-endpoint =3D <&mipi_in_panel>; + }; +}; =20 - mipi1_out_panel: endpoint { - remote-endpoint =3D <&mipi1_in_panel>; - }; - }; +&mipi1_out { + mipi1_out_panel: endpoint { + remote-endpoint =3D <&mipi1_in_panel>; }; }; =20 diff --git a/arch/arm64/boot/dts/rockchip/rk3399-pinephone-pro.dts b/arch/a= rm64/boot/dts/rockchip/rk3399-pinephone-pro.dts index 909ed14035f7..fe32937a2d16 100644 --- a/arch/arm64/boot/dts/rockchip/rk3399-pinephone-pro.dts +++ b/arch/arm64/boot/dts/rockchip/rk3399-pinephone-pro.dts @@ -464,6 +464,8 @@ &io_domains { =20 &mipi_dsi { clock-master; + #address-cells =3D <1>; + #size-cells =3D <0>; status =3D "okay"; =20 panel@0 { diff --git a/arch/arm64/boot/dts/rockchip/rk3399-puma-haikou-video-demo.dts= o b/arch/arm64/boot/dts/rockchip/rk3399-puma-haikou-video-demo.dtso index 0377ec860d35..d28880b8dd44 100644 --- a/arch/arm64/boot/dts/rockchip/rk3399-puma-haikou-video-demo.dtso +++ b/arch/arm64/boot/dts/rockchip/rk3399-puma-haikou-video-demo.dtso @@ -124,12 +124,6 @@ pca9670: gpio@27 { }; }; =20 -&mipi_out { - mipi_out_panel: endpoint { - remote-endpoint =3D <&mipi_in_panel>; - }; -}; - &mipi_dsi { #address-cells =3D <1>; #size-cells =3D <0>; @@ -151,6 +145,12 @@ mipi_in_panel: endpoint { }; }; =20 +&mipi_out { + mipi_out_panel: endpoint { + remote-endpoint =3D <&mipi_in_panel>; + }; +}; + &pinctrl { pca9670 { pca9670_resetn: pca9670-resetn { diff --git a/arch/arm64/boot/dts/rockchip/rk3399-rockpro64-screen.dtso b/ar= ch/arm64/boot/dts/rockchip/rk3399-rockpro64-screen.dtso index b1f4ab22b99c..3a68c5f7c5ff 100644 --- a/arch/arm64/boot/dts/rockchip/rk3399-rockpro64-screen.dtso +++ b/arch/arm64/boot/dts/rockchip/rk3399-rockpro64-screen.dtso @@ -47,25 +47,11 @@ touch: touchscreen@5d { }; =20 &mipi_dsi { + clock-master; #address-cells =3D <1>; #size-cells =3D <0>; - - clock-master; status =3D "okay"; =20 - ports { - #address-cells =3D <1>; - #size-cells =3D <0>; - - mipi_out: port@1 { - reg =3D <1>; - - mipi_out_panel: endpoint { - remote-endpoint =3D <&mipi_in_panel>; - }; - }; - }; - mipi_panel: panel@0 { compatible =3D "feiyang,fy07024di26a30d"; reg =3D <0>; @@ -81,6 +67,11 @@ mipi_in_panel: endpoint { }; }; =20 +&mipi_out { + mipi_out_panel: endpoint { + remote-endpoint =3D <&mipi_in_panel>; + }; +} &pwm0 { status =3D "okay"; }; --=20 2.50.0