* [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: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: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 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 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: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