* [PATCH] dt-bindings: nvmem: qfprom: Add Kaanapali compatible
@ 2026-03-06 6:40 Jingyi Wang
2026-03-06 6:55 ` Akhil P Oommen
` (2 more replies)
0 siblings, 3 replies; 8+ messages in thread
From: Jingyi Wang @ 2026-03-06 6:40 UTC (permalink / raw)
To: Srinivas Kandagatla, Rob Herring, Krzysztof Kozlowski,
Conor Dooley
Cc: aiqun.yu, tingwei.zhang, trilok.soni, yijie.yang, linux-arm-msm,
devicetree, linux-kernel, Jingyi Wang
Document compatible string for the QFPROM on Kaanapali platform.
Signed-off-by: Jingyi Wang <jingyi.wang@oss.qualcomm.com>
---
Documentation/devicetree/bindings/nvmem/qcom,qfprom.yaml | 1 +
1 file changed, 1 insertion(+)
diff --git a/Documentation/devicetree/bindings/nvmem/qcom,qfprom.yaml b/Documentation/devicetree/bindings/nvmem/qcom,qfprom.yaml
index 839513d4b499..2ab047f2bb69 100644
--- a/Documentation/devicetree/bindings/nvmem/qcom,qfprom.yaml
+++ b/Documentation/devicetree/bindings/nvmem/qcom,qfprom.yaml
@@ -26,6 +26,7 @@ properties:
- qcom,ipq8064-qfprom
- qcom,ipq8074-qfprom
- qcom,ipq9574-qfprom
+ - qcom,kaanapali-qfprom
- qcom,msm8226-qfprom
- qcom,msm8916-qfprom
- qcom,msm8917-qfprom
---
base-commit: fc7b1a72c6cd5cbbd989c6c32a6486e3e4e3594d
change-id: 20260305-knp-qfprom-binding-efcff6ea9b7c
Best regards,
--
Jingyi Wang <jingyi.wang@oss.qualcomm.com>
^ permalink raw reply related [flat|nested] 8+ messages in thread* Re: [PATCH] dt-bindings: nvmem: qfprom: Add Kaanapali compatible 2026-03-06 6:40 [PATCH] dt-bindings: nvmem: qfprom: Add Kaanapali compatible Jingyi Wang @ 2026-03-06 6:55 ` Akhil P Oommen 2026-03-06 9:10 ` Konrad Dybcio 2026-03-06 15:06 ` Krzysztof Kozlowski 2026-03-06 7:24 ` Krzysztof Kozlowski 2026-03-11 7:35 ` Srinivas Kandagatla 2 siblings, 2 replies; 8+ messages in thread From: Akhil P Oommen @ 2026-03-06 6:55 UTC (permalink / raw) To: Jingyi Wang, Srinivas Kandagatla, Rob Herring, Krzysztof Kozlowski, Conor Dooley, Dmitry Baryshkov Cc: aiqun.yu, tingwei.zhang, trilok.soni, yijie.yang, linux-arm-msm, devicetree, linux-kernel On 3/6/2026 12:10 PM, Jingyi Wang wrote: > Document compatible string for the QFPROM on Kaanapali platform. > > Signed-off-by: Jingyi Wang <jingyi.wang@oss.qualcomm.com> > --- > Documentation/devicetree/bindings/nvmem/qcom,qfprom.yaml | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/Documentation/devicetree/bindings/nvmem/qcom,qfprom.yaml b/Documentation/devicetree/bindings/nvmem/qcom,qfprom.yaml > index 839513d4b499..2ab047f2bb69 100644 > --- a/Documentation/devicetree/bindings/nvmem/qcom,qfprom.yaml > +++ b/Documentation/devicetree/bindings/nvmem/qcom,qfprom.yaml > @@ -26,6 +26,7 @@ properties: > - qcom,ipq8064-qfprom > - qcom,ipq8074-qfprom > - qcom,ipq9574-qfprom > + - qcom,kaanapali-qfprom A question to the maintainers. Do we need a new compatible for every chipset? If there are no KMD facing differences in the HW, can we use an existing compatible string, like sm8750's in this case? The fuse definitions (which map to nvmem cells) will obviously differ between chipsets, but I am not sure if this alone warrants introducing a new compatible string. -Akhil. > - qcom,msm8226-qfprom > - qcom,msm8916-qfprom > - qcom,msm8917-qfprom > > --- > base-commit: fc7b1a72c6cd5cbbd989c6c32a6486e3e4e3594d > change-id: 20260305-knp-qfprom-binding-efcff6ea9b7c > > Best regards, ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] dt-bindings: nvmem: qfprom: Add Kaanapali compatible 2026-03-06 6:55 ` Akhil P Oommen @ 2026-03-06 9:10 ` Konrad Dybcio 2026-03-06 9:54 ` Akhil P Oommen 2026-03-06 15:06 ` Krzysztof Kozlowski 1 sibling, 1 reply; 8+ messages in thread From: Konrad Dybcio @ 2026-03-06 9:10 UTC (permalink / raw) To: Akhil P Oommen, Jingyi Wang, Srinivas Kandagatla, Rob Herring, Krzysztof Kozlowski, Conor Dooley, Dmitry Baryshkov Cc: aiqun.yu, tingwei.zhang, trilok.soni, yijie.yang, linux-arm-msm, devicetree, linux-kernel On 3/6/26 7:55 AM, Akhil P Oommen wrote: > On 3/6/2026 12:10 PM, Jingyi Wang wrote: >> Document compatible string for the QFPROM on Kaanapali platform. >> >> Signed-off-by: Jingyi Wang <jingyi.wang@oss.qualcomm.com> >> --- >> Documentation/devicetree/bindings/nvmem/qcom,qfprom.yaml | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/Documentation/devicetree/bindings/nvmem/qcom,qfprom.yaml b/Documentation/devicetree/bindings/nvmem/qcom,qfprom.yaml >> index 839513d4b499..2ab047f2bb69 100644 >> --- a/Documentation/devicetree/bindings/nvmem/qcom,qfprom.yaml >> +++ b/Documentation/devicetree/bindings/nvmem/qcom,qfprom.yaml >> @@ -26,6 +26,7 @@ properties: >> - qcom,ipq8064-qfprom >> - qcom,ipq8074-qfprom >> - qcom,ipq9574-qfprom >> + - qcom,kaanapali-qfprom > > A question to the maintainers. > > Do we need a new compatible for every chipset? If there are no KMD > facing differences in the HW, can we use an existing compatible string, > like sm8750's in this case? > > The fuse definitions (which map to nvmem cells) will obviously differ > between chipsets, but I am not sure if this alone warrants introducing a > new compatible string. This is to prevent the case where it later turns out that QFPROM on 8750 is deeply flawed under certain conditions and needs to have workarounds applied retroactively (because we're pinky-promise working towards stable DT ABI) Konrad ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] dt-bindings: nvmem: qfprom: Add Kaanapali compatible 2026-03-06 9:10 ` Konrad Dybcio @ 2026-03-06 9:54 ` Akhil P Oommen 2026-03-09 10:50 ` Konrad Dybcio 0 siblings, 1 reply; 8+ messages in thread From: Akhil P Oommen @ 2026-03-06 9:54 UTC (permalink / raw) To: Konrad Dybcio Cc: aiqun.yu, tingwei.zhang, trilok.soni, yijie.yang, linux-arm-msm, devicetree, linux-kernel, Jingyi Wang, Srinivas Kandagatla, Rob Herring, Krzysztof Kozlowski, Conor Dooley, Dmitry Baryshkov On 3/6/2026 2:40 PM, Konrad Dybcio wrote: > On 3/6/26 7:55 AM, Akhil P Oommen wrote: >> On 3/6/2026 12:10 PM, Jingyi Wang wrote: >>> Document compatible string for the QFPROM on Kaanapali platform. >>> >>> Signed-off-by: Jingyi Wang <jingyi.wang@oss.qualcomm.com> >>> --- >>> Documentation/devicetree/bindings/nvmem/qcom,qfprom.yaml | 1 + >>> 1 file changed, 1 insertion(+) >>> >>> diff --git a/Documentation/devicetree/bindings/nvmem/qcom,qfprom.yaml b/Documentation/devicetree/bindings/nvmem/qcom,qfprom.yaml >>> index 839513d4b499..2ab047f2bb69 100644 >>> --- a/Documentation/devicetree/bindings/nvmem/qcom,qfprom.yaml >>> +++ b/Documentation/devicetree/bindings/nvmem/qcom,qfprom.yaml >>> @@ -26,6 +26,7 @@ properties: >>> - qcom,ipq8064-qfprom >>> - qcom,ipq8074-qfprom >>> - qcom,ipq9574-qfprom >>> + - qcom,kaanapali-qfprom >> >> A question to the maintainers. >> >> Do we need a new compatible for every chipset? If there are no KMD >> facing differences in the HW, can we use an existing compatible string, >> like sm8750's in this case? >> >> The fuse definitions (which map to nvmem cells) will obviously differ >> between chipsets, but I am not sure if this alone warrants introducing a >> new compatible string. > > This is to prevent the case where it later turns out that QFPROM on 8750 > is deeply flawed under certain conditions and needs to have workarounds > applied retroactively (because we're pinky-promise working towards stable > DT ABI) But this is a super simple HW IP, so make an exception for this? In the worst case, use a SoC related compatible in the driver for quirks? I am just trying to see if there is a way to avoid this dependency for the DT series. :) -Akhil. > > Konrad ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] dt-bindings: nvmem: qfprom: Add Kaanapali compatible 2026-03-06 9:54 ` Akhil P Oommen @ 2026-03-09 10:50 ` Konrad Dybcio 0 siblings, 0 replies; 8+ messages in thread From: Konrad Dybcio @ 2026-03-09 10:50 UTC (permalink / raw) To: Akhil P Oommen Cc: aiqun.yu, tingwei.zhang, trilok.soni, yijie.yang, linux-arm-msm, devicetree, linux-kernel, Jingyi Wang, Srinivas Kandagatla, Rob Herring, Krzysztof Kozlowski, Conor Dooley, Dmitry Baryshkov On 3/6/26 10:54 AM, Akhil P Oommen wrote: > On 3/6/2026 2:40 PM, Konrad Dybcio wrote: >> On 3/6/26 7:55 AM, Akhil P Oommen wrote: >>> On 3/6/2026 12:10 PM, Jingyi Wang wrote: >>>> Document compatible string for the QFPROM on Kaanapali platform. >>>> >>>> Signed-off-by: Jingyi Wang <jingyi.wang@oss.qualcomm.com> >>>> --- >>>> Documentation/devicetree/bindings/nvmem/qcom,qfprom.yaml | 1 + >>>> 1 file changed, 1 insertion(+) >>>> >>>> diff --git a/Documentation/devicetree/bindings/nvmem/qcom,qfprom.yaml b/Documentation/devicetree/bindings/nvmem/qcom,qfprom.yaml >>>> index 839513d4b499..2ab047f2bb69 100644 >>>> --- a/Documentation/devicetree/bindings/nvmem/qcom,qfprom.yaml >>>> +++ b/Documentation/devicetree/bindings/nvmem/qcom,qfprom.yaml >>>> @@ -26,6 +26,7 @@ properties: >>>> - qcom,ipq8064-qfprom >>>> - qcom,ipq8074-qfprom >>>> - qcom,ipq9574-qfprom >>>> + - qcom,kaanapali-qfprom >>> >>> A question to the maintainers. >>> >>> Do we need a new compatible for every chipset? If there are no KMD >>> facing differences in the HW, can we use an existing compatible string, >>> like sm8750's in this case? >>> >>> The fuse definitions (which map to nvmem cells) will obviously differ >>> between chipsets, but I am not sure if this alone warrants introducing a >>> new compatible string. >> >> This is to prevent the case where it later turns out that QFPROM on 8750 >> is deeply flawed under certain conditions and needs to have workarounds >> applied retroactively (because we're pinky-promise working towards stable >> DT ABI) > > But this is a super simple HW IP, so make an exception for this? In the > worst case, use a SoC related compatible in the driver for quirks? > > I am just trying to see if there is a way to avoid this dependency for > the DT series. :) I think this is the incorrect level of zoom - yes, it's annoying, but we have probably 10-20 more places where we need a 'meaningless' compatible. The quick solution to getting over this, would be to let platform maintainers (qc, mtk, nv..) take such simple patches via the same trees that grab DT changes - and I think there's been some discussion around that Konrad ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] dt-bindings: nvmem: qfprom: Add Kaanapali compatible 2026-03-06 6:55 ` Akhil P Oommen 2026-03-06 9:10 ` Konrad Dybcio @ 2026-03-06 15:06 ` Krzysztof Kozlowski 1 sibling, 0 replies; 8+ messages in thread From: Krzysztof Kozlowski @ 2026-03-06 15:06 UTC (permalink / raw) To: Akhil P Oommen, Jingyi Wang, Srinivas Kandagatla, Rob Herring, Krzysztof Kozlowski, Conor Dooley, Dmitry Baryshkov Cc: aiqun.yu, tingwei.zhang, trilok.soni, yijie.yang, linux-arm-msm, devicetree, linux-kernel On 06/03/2026 07:55, Akhil P Oommen wrote: > On 3/6/2026 12:10 PM, Jingyi Wang wrote: >> Document compatible string for the QFPROM on Kaanapali platform. >> >> Signed-off-by: Jingyi Wang <jingyi.wang@oss.qualcomm.com> >> --- >> Documentation/devicetree/bindings/nvmem/qcom,qfprom.yaml | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/Documentation/devicetree/bindings/nvmem/qcom,qfprom.yaml b/Documentation/devicetree/bindings/nvmem/qcom,qfprom.yaml >> index 839513d4b499..2ab047f2bb69 100644 >> --- a/Documentation/devicetree/bindings/nvmem/qcom,qfprom.yaml >> +++ b/Documentation/devicetree/bindings/nvmem/qcom,qfprom.yaml >> @@ -26,6 +26,7 @@ properties: >> - qcom,ipq8064-qfprom >> - qcom,ipq8074-qfprom >> - qcom,ipq9574-qfprom >> + - qcom,kaanapali-qfprom > > A question to the maintainers. > > Do we need a new compatible for every chipset? If there are no KMD Yes, you need. Best regards, Krzysztof ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] dt-bindings: nvmem: qfprom: Add Kaanapali compatible 2026-03-06 6:40 [PATCH] dt-bindings: nvmem: qfprom: Add Kaanapali compatible Jingyi Wang 2026-03-06 6:55 ` Akhil P Oommen @ 2026-03-06 7:24 ` Krzysztof Kozlowski 2026-03-11 7:35 ` Srinivas Kandagatla 2 siblings, 0 replies; 8+ messages in thread From: Krzysztof Kozlowski @ 2026-03-06 7:24 UTC (permalink / raw) To: Jingyi Wang, Srinivas Kandagatla, Rob Herring, Krzysztof Kozlowski, Conor Dooley Cc: aiqun.yu, tingwei.zhang, trilok.soni, yijie.yang, linux-arm-msm, devicetree, linux-kernel On 06/03/2026 07:40, Jingyi Wang wrote: > Document compatible string for the QFPROM on Kaanapali platform. > > Signed-off-by: Jingyi Wang <jingyi.wang@oss.qualcomm.com> > --- > Documentation/devicetree/bindings/nvmem/qcom,qfprom.yaml | 1 + > 1 file changed, 1 insertion(+) Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com> Best regards, Krzysztof ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] dt-bindings: nvmem: qfprom: Add Kaanapali compatible 2026-03-06 6:40 [PATCH] dt-bindings: nvmem: qfprom: Add Kaanapali compatible Jingyi Wang 2026-03-06 6:55 ` Akhil P Oommen 2026-03-06 7:24 ` Krzysztof Kozlowski @ 2026-03-11 7:35 ` Srinivas Kandagatla 2 siblings, 0 replies; 8+ messages in thread From: Srinivas Kandagatla @ 2026-03-11 7:35 UTC (permalink / raw) To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Jingyi Wang Cc: aiqun.yu, tingwei.zhang, trilok.soni, yijie.yang, linux-arm-msm, devicetree, linux-kernel On Thu, 05 Mar 2026 22:40:41 -0800, Jingyi Wang wrote: > Document compatible string for the QFPROM on Kaanapali platform. > > Applied, thanks! [1/1] dt-bindings: nvmem: qfprom: Add Kaanapali compatible commit: c693038ce48b93f73294f158e8b26b1119d226d4 Best regards, -- Srinivas Kandagatla <srini@kernel.org> ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2026-03-11 7:35 UTC | newest] Thread overview: 8+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2026-03-06 6:40 [PATCH] dt-bindings: nvmem: qfprom: Add Kaanapali compatible Jingyi Wang 2026-03-06 6:55 ` Akhil P Oommen 2026-03-06 9:10 ` Konrad Dybcio 2026-03-06 9:54 ` Akhil P Oommen 2026-03-09 10:50 ` Konrad Dybcio 2026-03-06 15:06 ` Krzysztof Kozlowski 2026-03-06 7:24 ` Krzysztof Kozlowski 2026-03-11 7:35 ` Srinivas Kandagatla
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox