| Message ID | 20240115182017.1610055-1-krzysztof.kozlowski@linaro.org |
|---|---|
| State | Accepted |
| Commit | 059870e53aa06831f1ccdc5e9cf29f933cdf284e |
| Headers | show |
| Series | ASoC: dt-bindings: Do not override firmware-name $ref | expand |
Hi Krzysztof On Mon, 15 Jan 2024 19:20:17 +0100 Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> wrote: > dtschema package defines firmware-name as string-array, so individual > bindings should not make it a string but instead just narrow the number > of expected firmware file names. > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > --- > Documentation/devicetree/bindings/sound/fsl,easrc.yaml | 4 ++-- > Documentation/devicetree/bindings/sound/infineon,peb2466.yaml | 2 +- > 2 files changed, 3 insertions(+), 3 deletions(-) > [...] At least for the infineon,peb2466.yaml, Acked-by: Herve Codina <herve.codina@bootlin.com> Thanks for this patch, Hervé
On Mon, 15 Jan 2024 19:20:17 +0100, Krzysztof Kozlowski wrote: > dtschema package defines firmware-name as string-array, so individual > bindings should not make it a string but instead just narrow the number > of expected firmware file names. > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > --- > Documentation/devicetree/bindings/sound/fsl,easrc.yaml | 4 ++-- > Documentation/devicetree/bindings/sound/infineon,peb2466.yaml | 2 +- > 2 files changed, 3 insertions(+), 3 deletions(-) > Acked-by: Rob Herring <robh@kernel.org>
diff --git a/Documentation/devicetree/bindings/sound/fsl,easrc.yaml b/Documentation/devicetree/bindings/sound/fsl,easrc.yaml index a680d7aff237..0782f3f9947f 100644 --- a/Documentation/devicetree/bindings/sound/fsl,easrc.yaml +++ b/Documentation/devicetree/bindings/sound/fsl,easrc.yaml @@ -51,8 +51,8 @@ properties: - const: ctx3_tx firmware-name: - $ref: /schemas/types.yaml#/definitions/string - const: imx/easrc/easrc-imx8mn.bin + items: + - const: imx/easrc/easrc-imx8mn.bin description: The coefficient table for the filters fsl,asrc-rate: diff --git a/Documentation/devicetree/bindings/sound/infineon,peb2466.yaml b/Documentation/devicetree/bindings/sound/infineon,peb2466.yaml index 66993d378aaf..5e11ce2c13ac 100644 --- a/Documentation/devicetree/bindings/sound/infineon,peb2466.yaml +++ b/Documentation/devicetree/bindings/sound/infineon,peb2466.yaml @@ -51,7 +51,7 @@ properties: maxItems: 1 firmware-name: - $ref: /schemas/types.yaml#/definitions/string + maxItems: 1 description: Filters coefficients file to load. If this property is omitted, internal filters are disabled.
dtschema package defines firmware-name as string-array, so individual bindings should not make it a string but instead just narrow the number of expected firmware file names. Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> --- Documentation/devicetree/bindings/sound/fsl,easrc.yaml | 4 ++-- Documentation/devicetree/bindings/sound/infineon,peb2466.yaml | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-)