qcom: Refactor sa8775p/qcs9100 based ride boards

[PATCH 6/8] arm64: dts: qcom: sa8775p: Introduce QAM8775p SOM

Posted by Wasim Nazir 7 months, 1 week ago

qam8775p-som.dtsi specifies SA8775p based SOM having SOC, PMICs & DDR.
sa8775p-ride & sa8775p-ride-r3 boards are based on QAM8775p SOM.

Signed-off-by: Wasim Nazir <quic_wasimn@quicinc.com>
---
 arch/arm64/boot/dts/qcom/qam8775p-som.dtsi   | 9 +++++++++
 arch/arm64/boot/dts/qcom/sa8775p-ride-r3.dts | 5 ++---
 arch/arm64/boot/dts/qcom/sa8775p-ride.dts    | 5 ++---
 3 files changed, 13 insertions(+), 6 deletions(-)
 create mode 100644 arch/arm64/boot/dts/qcom/qam8775p-som.dtsi

diff --git a/arch/arm64/boot/dts/qcom/qam8775p-som.dtsi b/arch/arm64/boot/dts/qcom/qam8775p-som.dtsi
new file mode 100644
index 000000000000..92adebb2e18f
--- /dev/null
+++ b/arch/arm64/boot/dts/qcom/qam8775p-som.dtsi
@@ -0,0 +1,9 @@
+// SPDX-License-Identifier: BSD-3-Clause
+/*
+ * Copyright (c) 2025, Qualcomm Innovation Center, Inc. All rights reserved.
+ */
+
+/dts-v1/;
+
+#include "sa8775p.dtsi"
+#include "sa8775p-pmics.dtsi"
diff --git a/arch/arm64/boot/dts/qcom/sa8775p-ride-r3.dts b/arch/arm64/boot/dts/qcom/sa8775p-ride-r3.dts
index a9014095daba..f75e92e05bcd 100644
--- a/arch/arm64/boot/dts/qcom/sa8775p-ride-r3.dts
+++ b/arch/arm64/boot/dts/qcom/sa8775p-ride-r3.dts
@@ -6,13 +6,12 @@

 /dts-v1/;

-#include "sa8775p.dtsi"
-#include "sa8775p-pmics.dtsi"
+#include "qam8775p-som.dtsi"

 #include "sa8775p-ride-common.dtsi"
 #include "sa8775p-ride-ethernet-aqr115c.dtsi"

 / {
 	model = "Qualcomm SA8775P Ride Rev3";
-	compatible = "qcom,sa8775p-ride-r3", "qcom,sa8775p";
+	compatible = "qcom,sa8775p-ride-r3", "qcom,qam8775p-som", "qcom,sa8775p";
 };
diff --git a/arch/arm64/boot/dts/qcom/sa8775p-ride.dts b/arch/arm64/boot/dts/qcom/sa8775p-ride.dts
index e98554f825d5..568eff8c1999 100644
--- a/arch/arm64/boot/dts/qcom/sa8775p-ride.dts
+++ b/arch/arm64/boot/dts/qcom/sa8775p-ride.dts
@@ -6,13 +6,12 @@

 /dts-v1/;

-#include "sa8775p.dtsi"
-#include "sa8775p-pmics.dtsi"
+#include "qam8775p-som.dtsi"

 #include "sa8775p-ride-common.dtsi"
 #include "sa8775p-ride-ethernet-88ea1512.dtsi"

 / {
 	model = "Qualcomm SA8775P Ride";
-	compatible = "qcom,sa8775p-ride", "qcom,sa8775p";
+	compatible = "qcom,sa8775p-ride", "qcom,qam8775p-som", "qcom,sa8775p";
 };
--
2.49.0

Re: [PATCH 6/8] arm64: dts: qcom: sa8775p: Introduce QAM8775p SOM

Posted by Krzysztof Kozlowski 7 months, 1 week ago

On 07/05/2025 08:51, Wasim Nazir wrote:
> qam8775p-som.dtsi specifies SA8775p based SOM having SOC, PMICs & DDR.
> sa8775p-ride & sa8775p-ride-r3 boards are based on QAM8775p SOM.
> 
> Signed-off-by: Wasim Nazir <quic_wasimn@quicinc.com>
> ---
>  arch/arm64/boot/dts/qcom/qam8775p-som.dtsi   | 9 +++++++++
>  arch/arm64/boot/dts/qcom/sa8775p-ride-r3.dts | 5 ++---
>  arch/arm64/boot/dts/qcom/sa8775p-ride.dts    | 5 ++---
>  3 files changed, 13 insertions(+), 6 deletions(-)
>  create mode 100644 arch/arm64/boot/dts/qcom/qam8775p-som.dtsi
> 
> diff --git a/arch/arm64/boot/dts/qcom/qam8775p-som.dtsi b/arch/arm64/boot/dts/qcom/qam8775p-som.dtsi
> new file mode 100644
> index 000000000000..92adebb2e18f
> --- /dev/null
> +++ b/arch/arm64/boot/dts/qcom/qam8775p-som.dtsi
> @@ -0,0 +1,9 @@
> +// SPDX-License-Identifier: BSD-3-Clause
> +/*
> + * Copyright (c) 2025, Qualcomm Innovation Center, Inc. All rights reserved.
> + */
> +
> +/dts-v1/;
> +
> +#include "sa8775p.dtsi"
> +#include "sa8775p-pmics.dtsi"

That's an empty file. What is the point of it except making everything
more complicated?


Best regards,
Krzysztof