diff mbox series

[7/9] arm64: dts: qcom: sc7280: Add nodes to boot modem

Message ID 1624564058-24095-8-git-send-email-sibis@codeaurora.org
State Superseded
Headers show
Series Add Modem support on SC7280 SoCs | expand

Commit Message

Sibi Sankar June 24, 2021, 7:47 p.m. UTC
Add miscellaneous nodes to boot the modem and support post-mortem debug
on SC7280 SoCs.

Signed-off-by: Sibi Sankar <sibis@codeaurora.org>
---
 arch/arm64/boot/dts/qcom/sc7280.dtsi | 20 ++++++++++++++++++++
 1 file changed, 20 insertions(+)

Comments

Bjorn Andersson July 30, 2021, 6 p.m. UTC | #1
On Thu 24 Jun 14:47 CDT 2021, Sibi Sankar wrote:

> Add miscellaneous nodes to boot the modem and support post-mortem debug

> on SC7280 SoCs.

> 

> Signed-off-by: Sibi Sankar <sibis@codeaurora.org>

> ---

>  arch/arm64/boot/dts/qcom/sc7280.dtsi | 20 ++++++++++++++++++++

>  1 file changed, 20 insertions(+)

> 

> diff --git a/arch/arm64/boot/dts/qcom/sc7280.dtsi b/arch/arm64/boot/dts/qcom/sc7280.dtsi

> index 5ed7a511bfc9..3fb6a6ef39f8 100644

> --- a/arch/arm64/boot/dts/qcom/sc7280.dtsi

> +++ b/arch/arm64/boot/dts/qcom/sc7280.dtsi

> @@ -547,6 +547,11 @@

>  			#hwlock-cells = <1>;

>  		};

>  

> +		tcsr_regs: syscon@1fc0000 {


Is there a different "tcsr"? Does the "_regs" suffix add any value?

> +			compatible = "syscon";


Rob has pointed out a few times that a lone "syscon" isn't going to be
accepted going forward. Could you also add a qualifying
"qcom,sc7280-tcsr" or something like that?

> +			reg = <0 0x01fc0000 0 0x30000>;

> +		};

> +

>  		lpasscc: lpasscc@3000000 {

>  			compatible = "qcom,sc7280-lpasscc";

>  			reg = <0 0x03000000 0 0x40>,

> @@ -1219,6 +1224,21 @@

>  			};

>  		};

>  

> +		imem@146aa000 {

> +			compatible = "syscon", "simple-mfd";


As above "qcom,sc7280-imem"?

I presume we need some new binding documents for these two though,
perhaps you can add the specific compatibles and we agree that one of us
will write these two bindings soon?

Regards,
Bjorn

> +			reg = <0 0x146aa000 0 0x2000>;

> +

> +			#address-cells = <2>;

> +			#size-cells = <2>;

> +

> +			ranges = <0 0x0 0 0x146aa000 0 0x2000>;

> +

> +			pil-reloc@94c {

> +				compatible = "qcom,pil-reloc-info";

> +				reg = <0 0x94c 0 0xc8>;

> +			};

> +		};

> +

>  		apps_smmu: iommu@15000000 {

>  			compatible = "qcom,sc7280-smmu-500", "arm,mmu-500";

>  			reg = <0 0x15000000 0 0x100000>;

> -- 

> The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,

> a Linux Foundation Collaborative Project

>
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/qcom/sc7280.dtsi b/arch/arm64/boot/dts/qcom/sc7280.dtsi
index 5ed7a511bfc9..3fb6a6ef39f8 100644
--- a/arch/arm64/boot/dts/qcom/sc7280.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc7280.dtsi
@@ -547,6 +547,11 @@ 
 			#hwlock-cells = <1>;
 		};
 
+		tcsr_regs: syscon@1fc0000 {
+			compatible = "syscon";
+			reg = <0 0x01fc0000 0 0x30000>;
+		};
+
 		lpasscc: lpasscc@3000000 {
 			compatible = "qcom,sc7280-lpasscc";
 			reg = <0 0x03000000 0 0x40>,
@@ -1219,6 +1224,21 @@ 
 			};
 		};
 
+		imem@146aa000 {
+			compatible = "syscon", "simple-mfd";
+			reg = <0 0x146aa000 0 0x2000>;
+
+			#address-cells = <2>;
+			#size-cells = <2>;
+
+			ranges = <0 0x0 0 0x146aa000 0 0x2000>;
+
+			pil-reloc@94c {
+				compatible = "qcom,pil-reloc-info";
+				reg = <0 0x94c 0 0xc8>;
+			};
+		};
+
 		apps_smmu: iommu@15000000 {
 			compatible = "qcom,sc7280-smmu-500", "arm,mmu-500";
 			reg = <0 0x15000000 0 0x100000>;