[PATCH v2 0/2] arm64: qcom: sa8775p: Add IMEM and PIL info region

[PATCH v2 0/2] arm64: qcom: sa8775p: Add IMEM and PIL info region

[Tengfei Fan]

Add IMEM and PIL info region for the SA8775p platform.

Signed-off-by: Tengfei Fan <quic_tengfan@quicinc.com>
---

v1 -> v2:
  - Adjust the position of the "ranges" property

Tengfei Fan (2):
  dt-bindings: soc: qcom: add qcom,sa8775p-imem compatible
  arm64: dts: qcom: sa8775p: Add IMEM and PIL info region

 .../devicetree/bindings/sram/qcom,imem.yaml        |  1 +
 arch/arm64/boot/dts/qcom/sa8775p.dtsi              | 14 ++++++++++++++
 2 files changed, 15 insertions(+)


base-commit: 0e1980c40b6edfa68b6acf926bab22448a6e40c9
-- 
2.25.1

[PATCH v2 1/2] dt-bindings: soc: qcom: add qcom,sa8775p-imem compatible

[Tengfei Fan]

Add qcom,sa8775p-imem compatible name support.

Signed-off-by: Tengfei Fan <quic_tengfan@quicinc.com>
---
 Documentation/devicetree/bindings/sram/qcom,imem.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/sram/qcom,imem.yaml b/Documentation/devicetree/bindings/sram/qcom,imem.yaml
index 8025a852bc9c..faef3d6e0a94 100644
--- a/Documentation/devicetree/bindings/sram/qcom,imem.yaml
+++ b/Documentation/devicetree/bindings/sram/qcom,imem.yaml
@@ -22,6 +22,7 @@ properties:
           - qcom,msm8974-imem
           - qcom,qcs404-imem
           - qcom,qdu1000-imem
+          - qcom,sa8775p-imem
           - qcom,sc7180-imem
           - qcom,sc7280-imem
           - qcom,sdm630-imem
-- 
2.25.1

[PATCH v2 2/2] arm64: dts: qcom: sa8775p: Add IMEM and PIL info region

[Tengfei Fan]

Add a simple-mfd representing IMEM on SA8775p and define the PIL
relocation info region, so that post mortem tools will be able
to locate the loaded remoteprocs.

Signed-off-by: Tengfei Fan <quic_tengfan@quicinc.com>
---
 arch/arm64/boot/dts/qcom/sa8775p.dtsi | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/arch/arm64/boot/dts/qcom/sa8775p.dtsi b/arch/arm64/boot/dts/qcom/sa8775p.dtsi
index eae0de9720b5..89496728d840 100644
--- a/arch/arm64/boot/dts/qcom/sa8775p.dtsi
+++ b/arch/arm64/boot/dts/qcom/sa8775p.dtsi
@@ -3044,6 +3044,20 @@ tlmm: pinctrl@f000000 {
 			wakeup-parent = <&pdc>;
 		};
 
+		sram: sram@146d8000 {
+			compatible = "qcom,sa8775p-imem", "syscon", "simple-mfd";
+			reg = <0x0 0x146d8000 0x0 0x1000>;
+			ranges = <0x0 0x0 0x146d8000 0x1000>;
+
+			#address-cells = <1>;
+			#size-cells = <1>;
+
+			pil-reloc@94c {
+				compatible = "qcom,pil-reloc-info";
+				reg = <0x94c 0xc8>;
+			};
+		};
+
 		apps_smmu: iommu@15000000 {
 			compatible = "qcom,sa8775p-smmu-500", "qcom,smmu-500", "arm,mmu-500";
 			reg = <0x0 0x15000000 0x0 0x100000>;
-- 
2.25.1

Re: [PATCH v2 1/2] dt-bindings: soc: qcom: add qcom,sa8775p-imem compatible

[Krzysztof Kozlowski]

On 31/05/2024 11:35, Tengfei Fan wrote:
> Add qcom,sa8775p-imem compatible name support.
> 
> Signed-off-by: Tengfei Fan <quic_tengfan@quicinc.com>
> ---
>  Documentation/devicetree/bindings/sram/qcom,imem.yaml | 1 +

Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Best regards,
Krzysztof

Re: [PATCH v2 2/2] arm64: dts: qcom: sa8775p: Add IMEM and PIL info region

[Dmitry Baryshkov]

On Fri, May 31, 2024 at 05:35:31PM +0800, Tengfei Fan wrote:
> Add a simple-mfd representing IMEM on SA8775p and define the PIL
> relocation info region, so that post mortem tools will be able
> to locate the loaded remoteprocs.
> 
> Signed-off-by: Tengfei Fan <quic_tengfan@quicinc.com>
> ---
>  arch/arm64/boot/dts/qcom/sa8775p.dtsi | 14 ++++++++++++++
>  1 file changed, 14 insertions(+)
> 

Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>


-- 
With best wishes
Dmitry

Re: [PATCH v2 2/2] arm64: dts: qcom: sa8775p: Add IMEM and PIL info region

[Konrad Dybcio]

On 31.05.2024 11:35 AM, Tengfei Fan wrote:
> Add a simple-mfd representing IMEM on SA8775p and define the PIL
> relocation info region, so that post mortem tools will be able
> to locate the loaded remoteprocs.
> 
> Signed-off-by: Tengfei Fan <quic_tengfan@quicinc.com>
> ---

Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>

Konrad

Re: [PATCH v2 2/2] arm64: dts: qcom: sa8775p: Add IMEM and PIL info region

[Mukesh Ojha]

On 5/31/2024 3:05 PM, Tengfei Fan wrote:
> Add a simple-mfd representing IMEM on SA8775p and define the PIL
> relocation info region, so that post mortem tools will be able
> to locate the loaded remoteprocs.
> 
> Signed-off-by: Tengfei Fan <quic_tengfan@quicinc.com>
> ---
>   arch/arm64/boot/dts/qcom/sa8775p.dtsi | 14 ++++++++++++++
>   1 file changed, 14 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/qcom/sa8775p.dtsi b/arch/arm64/boot/dts/qcom/sa8775p.dtsi
> index eae0de9720b5..89496728d840 100644
> --- a/arch/arm64/boot/dts/qcom/sa8775p.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sa8775p.dtsi
> @@ -3044,6 +3044,20 @@ tlmm: pinctrl@f000000 {
>   			wakeup-parent = <&pdc>;
>   		};
>   
> +		sram: sram@146d8000 {
> +			compatible = "qcom,sa8775p-imem", "syscon", "simple-mfd";
> +			reg = <0x0 0x146d8000 0x0 0x1000>;
> +			ranges = <0x0 0x0 0x146d8000 0x1000>;
> +
> +			#address-cells = <1>;
> +			#size-cells = <1>;
> +
> +			pil-reloc@94c {
> +				compatible = "qcom,pil-reloc-info";
> +				reg = <0x94c 0xc8>;
> +			};
> +		};
> +

Reviewed-by: Mukesh Ojha <quic_mojha@quicinc.com>

-Mukesh

Re: [PATCH v2 0/2] arm64: qcom: sa8775p: Add IMEM and PIL info region

[Bjorn Andersson]

On Fri, 31 May 2024 17:35:29 +0800, Tengfei Fan wrote:
> Add IMEM and PIL info region for the SA8775p platform.
> 
> 

Applied, thanks!

[1/2] dt-bindings: soc: qcom: add qcom,sa8775p-imem compatible
      commit: 9ca6eaf1337693e4e626359b76016912921dc557
[2/2] arm64: dts: qcom: sa8775p: Add IMEM and PIL info region
      commit: 93f340084d05e7c109c0de20cca429492a377c37

Best regards,
-- 
Bjorn Andersson <andersson@kernel.org>