[PATCH] dt-bindings: nvmem: consumer: Make 'nvmem' an array of one-item entries

[PATCH] dt-bindings: nvmem: consumer: Make 'nvmem' an array of one-item entries

[Konrad Dybcio]

From: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>

'nvmem' unlike 'nvmem-cells', consumes references to just a single
phandle with no arguments (i.e. with 0 cells).

Constrain the schema to enforce that, so that the number of such
single-item entries can then be regulated by IP block-specific YAMLs.

Suggested-by: Rob Herring <robh@kernel.org>
Signed-off-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
---
qcom/qcs6490-rb3gen2.dtb: pmic@2 (qcom,pm8350c): pwm:nvmem: [[397, 398]] is too short
        from schema $id: http://devicetree.org/schemas/mfd/qcom,spmi-pmic.yaml
qcom/qcs6490-rb3gen2.dtb: pwm (qcom,pm8350c-pwm): nvmem: [[397, 398]] is too short
        from schema $id: http://devicetree.org/schemas/leds/leds-qcom-lpg.yaml
---
 Documentation/devicetree/bindings/nvmem/nvmem-consumer.yaml | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/Documentation/devicetree/bindings/nvmem/nvmem-consumer.yaml b/Documentation/devicetree/bindings/nvmem/nvmem-consumer.yaml
index a26633bf52db..45836308c724 100644
--- a/Documentation/devicetree/bindings/nvmem/nvmem-consumer.yaml
+++ b/Documentation/devicetree/bindings/nvmem/nvmem-consumer.yaml
@@ -16,6 +16,8 @@ properties:
     $ref: /schemas/types.yaml#/definitions/phandle-array
     description:
       List of phandle to the nvmem providers.
+    items:
+      maxItems: 1
 
   nvmem-cells:
     $ref: /schemas/types.yaml#/definitions/phandle-array

---
base-commit: 49e02880ec0a8c378e811bc9d85da188d7c6204c
change-id: 20260610-topic-nvmem_schema_warning_fix-1e8d8d39e51c

Best regards,
--  
Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>

Re: [PATCH] dt-bindings: nvmem: consumer: Make 'nvmem' an array of one-item entries

[Rob Herring (Arm)]

On Wed, 10 Jun 2026 14:52:42 +0200, Konrad Dybcio wrote:
> From: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
> 
> 'nvmem' unlike 'nvmem-cells', consumes references to just a single
> phandle with no arguments (i.e. with 0 cells).
> 
> Constrain the schema to enforce that, so that the number of such
> single-item entries can then be regulated by IP block-specific YAMLs.
> 
> Suggested-by: Rob Herring <robh@kernel.org>
> Signed-off-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
> ---
> qcom/qcs6490-rb3gen2.dtb: pmic@2 (qcom,pm8350c): pwm:nvmem: [[397, 398]] is too short
>         from schema $id: http://devicetree.org/schemas/mfd/qcom,spmi-pmic.yaml
> qcom/qcs6490-rb3gen2.dtb: pwm (qcom,pm8350c-pwm): nvmem: [[397, 398]] is too short
>         from schema $id: http://devicetree.org/schemas/leds/leds-qcom-lpg.yaml
> ---
>  Documentation/devicetree/bindings/nvmem/nvmem-consumer.yaml | 2 ++
>  1 file changed, 2 insertions(+)
> 

Given we're close to the merge window I applied so it goes into 7.2.

Rob

Re: [PATCH] dt-bindings: nvmem: consumer: Make 'nvmem' an array of one-item entries

[Konrad Dybcio]

On 6/10/26 9:28 PM, Rob Herring (Arm) wrote:
> 
> On Wed, 10 Jun 2026 14:52:42 +0200, Konrad Dybcio wrote:
>> From: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
>>
>> 'nvmem' unlike 'nvmem-cells', consumes references to just a single
>> phandle with no arguments (i.e. with 0 cells).
>>
>> Constrain the schema to enforce that, so that the number of such
>> single-item entries can then be regulated by IP block-specific YAMLs.
>>
>> Suggested-by: Rob Herring <robh@kernel.org>
>> Signed-off-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
>> ---
>> qcom/qcs6490-rb3gen2.dtb: pmic@2 (qcom,pm8350c): pwm:nvmem: [[397, 398]] is too short
>>         from schema $id: http://devicetree.org/schemas/mfd/qcom,spmi-pmic.yaml
>> qcom/qcs6490-rb3gen2.dtb: pwm (qcom,pm8350c-pwm): nvmem: [[397, 398]] is too short
>>         from schema $id: http://devicetree.org/schemas/leds/leds-qcom-lpg.yaml
>> ---
>>  Documentation/devicetree/bindings/nvmem/nvmem-consumer.yaml | 2 ++
>>  1 file changed, 2 insertions(+)
>>
> 
> Given we're close to the merge window I applied so it goes into 7.2.

Thanks!

Konrad