Re: [PATCH] dt-bindings: soc: qcom,wcnss: Drop redundant $ref of firmware-name property

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Low] Schema validation error: `default` value for `firmware-name` is a scalar string but should be a string array.
--

--- Patch [1]: [PATCH] dt-bindings: soc: qcom,wcnss: Drop redundant $ref of firmware-name property ---
commit ee3e2676da46575844622ab43164871fd3c8d4ab
Author: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>

dt-bindings: soc: qcom,wcnss: Drop redundant $ref of firmware-name property

This commit removes the explicit string reference for the firmware-name
property and replaces it with maxItems: 1, implicitly adopting the
string-array type from the core schema.

> diff --git a/Documentation/devicetree/bindings/soc/qcom/qcom,wcnss.yaml b/Documentation/devicetree/bindings/soc/qcom/qcom,wcnss.yaml
> --- a/Documentation/devicetree/bindings/soc/qcom/qcom,wcnss.yaml
> +++ b/Documentation/devicetree/bindings/soc/qcom/qcom,wcnss.yaml
> @@ -19,7 +19,7 @@ properties:
>      const: qcom,wcnss
>  
>    firmware-name:
> -    $ref: /schemas/types.yaml#/definitions/string
> +    maxItems: 1
>      default: wlan/prima/WCNSS_qcom_wlan_nv.bin

[Severity: Low]
Will this cause dt_binding_check to fail? Since the property type is now
implicitly a string-array due to maxItems, should the default value be
updated to an array format like [ wlan/prima/WCNSS_qcom_wlan_nv.bin ] to
avoid a JSON schema validation error?

>      description:
>        Relative firmware image path for the WLAN NV blob.

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260712145654.126171-2-krzysztof.kozlowski@oss.qualcomm.com?part=1