[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <dwfvko3hszsoh4ihnz3qdpsugmocbkrbhosijdw5q3bxh64kuo@o74as2li74px>
Date: Wed, 22 Oct 2025 12:34:58 +0300
From: Dmitry Baryshkov <dmitry.baryshkov@....qualcomm.com>
To: Jingyi Wang <jingyi.wang@....qualcomm.com>
Cc: Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>,
Bjorn Andersson <andersson@...nel.org>,
Konrad Dybcio <konradybcio@...nel.org>,
Robert Marko <robimarko@...il.com>,
Das Srinagesh <quic_gurus@...cinc.com>, aiqun.yu@....qualcomm.com,
tingwei.zhang@....qualcomm.com, trilok.soni@....qualcomm.com,
yijie.yang@....qualcomm.com, linux-arm-msm@...r.kernel.org,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
Eugen Hristev <eugen.hristev@...aro.org>
Subject: Re: [PATCH v2 1/4] dt-bindings: soc: qcom: Add qcom,kaanapali-imem
compatible
On Wed, Oct 22, 2025 at 05:05:30PM +0800, Jingyi Wang wrote:
>
>
> On 10/22/2025 4:49 PM, Dmitry Baryshkov wrote:
> > On Wed, Oct 22, 2025 at 12:28:41AM -0700, Jingyi Wang wrote:
> >> Document qcom,kaanapali-imem compatible.
> >>
> >> Reviewed-by: Eugen Hristev <eugen.hristev@...aro.org>
> >> Signed-off-by: Jingyi Wang <jingyi.wang@....qualcomm.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 6a627c57ae2f..1e29a8ff287f 100644
> >> --- a/Documentation/devicetree/bindings/sram/qcom,imem.yaml
> >> +++ b/Documentation/devicetree/bindings/sram/qcom,imem.yaml
> >> @@ -19,6 +19,7 @@ properties:
> >> - enum:
> >> - qcom,apq8064-imem
> >> - qcom,ipq5424-imem
> >> + - qcom,kaanapali-imem
> >
> > Can you use mmio-sram instead?
> >
>
> Here is the node:
>
> sram@...80000 {
> compatible = "qcom,kaanapali-imem", "syscon", "simple-mfd";
> reg = <0x0 0x14680000 0x0 0x1000>;
> ranges = <0 0 0x14680000 0x1000>;
>
> #address-cells = <1>;
> #size-cells = <1>;
>
> pil-reloc@94c {
> compatible = "qcom,pil-reloc-info";
> reg = <0x94c 0xc8>;
> };
> };
>
> other qualcomm are also using imem, could you please give more details on why
> we should use mmio-sram here?
https://lore.kernel.org/linux-arm-msm/e4c5ecc3-fd97-4b13-a057-bb1a3b7f9207@kernel.org/
>
> Thanks,
> Jingyi
>
> >> - qcom,msm8226-imem
> >> - qcom,msm8974-imem
> >> - qcom,msm8976-imem
> >>
> >> --
> >> 2.25.1
> >>
> >
>
--
With best wishes
Dmitry
Powered by blists - more mailing lists