2024-05-31 09:36:30

by Tengfei Fan

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

Add IMEM and PIL info region for the SA8775p platform.

Signed-off-by: Tengfei Fan <[email protected]>
---

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



2024-05-31 09:36:58

by Tengfei Fan

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

Add qcom,sa8775p-imem compatible name support.

Signed-off-by: Tengfei Fan <[email protected]>
---
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


2024-05-31 09:55:11

by Tengfei Fan

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

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 <[email protected]>
---
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


2024-05-31 10:00:16

by Krzysztof Kozlowski

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

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

Reviewed-by: Krzysztof Kozlowski <[email protected]>

Best regards,
Krzysztof


2024-05-31 10:11:19

by Dmitry Baryshkov

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

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 <[email protected]>
> ---
> arch/arm64/boot/dts/qcom/sa8775p.dtsi | 14 ++++++++++++++
> 1 file changed, 14 insertions(+)
>

Reviewed-by: Dmitry Baryshkov <[email protected]>


--
With best wishes
Dmitry

2024-05-31 12:39:26

by Konrad Dybcio

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

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 <[email protected]>
> ---

Reviewed-by: Konrad Dybcio <[email protected]>

Konrad

2024-05-31 14:41:27

by Mukesh Ojha

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



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 <[email protected]>
> ---
> 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 <[email protected]>

-Mukesh


2024-06-02 03:55:31

by Bjorn Andersson

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


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 <[email protected]>