Commit 4c654ec0 authored by Rob Herring's avatar Rob Herring

dt-bindings: display: nxp,tda998x: Fix 'audio-ports' constraints

The constraints for 'audio-ports' don't match the description. There can
be 1 or 2 DAI entries and each entry is exactly 2 values. Also, the
values' sizes are 32-bits, not 8-bits. Move the size constraints to the
outer dimension (number of DAIs) and add constraints on inner array
values.

Link: https://lore.kernel.org/r/20240122204959.1665970-1-robh@kernel.orgSigned-off-by: default avatarRob Herring <robh@kernel.org>
parent 2d3b3ab8
...@@ -29,19 +29,22 @@ properties: ...@@ -29,19 +29,22 @@ properties:
audio-ports: audio-ports:
description: description:
Array of 8-bit values, 2 values per DAI (Documentation/sound/soc/dai.rst). Array of 2 values per DAI (Documentation/sound/soc/dai.rst).
The implementation allows one or two DAIs. The implementation allows one or two DAIs.
If two DAIs are defined, they must be of different type. If two DAIs are defined, they must be of different type.
$ref: /schemas/types.yaml#/definitions/uint32-matrix $ref: /schemas/types.yaml#/definitions/uint32-matrix
items:
minItems: 1 minItems: 1
maxItems: 2
items:
items: items:
- description: | - description: |
The first value defines the DAI type: TDA998x_SPDIF or TDA998x_I2S The first value defines the DAI type: TDA998x_SPDIF or TDA998x_I2S
(see include/dt-bindings/display/tda998x.h). (see include/dt-bindings/display/tda998x.h).
enum: [ 1, 2 ]
- description: - description:
The second value defines the tda998x AP_ENA reg content when the The second value defines the tda998x AP_ENA reg content when the
DAI in question is used. DAI in question is used.
maximum: 0xff
'#sound-dai-cells': '#sound-dai-cells':
enum: [ 0, 1 ] enum: [ 0, 1 ]
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment