8 Jun
2023
8 Jun
'23
9:18 a.m.
On 08/06/2023 07:42, “Ryan wrote:
From: Ryan Lee ryans.lee@analog.com
Add dt-bindings information for Analog Devices MAX98388 I2S Amplifier
Signed-off-by: Ryan Lee ryans.lee@analog.com
Thank you for your patch. There is something to discuss/improve.
+allOf:
- $ref: dai-common.yaml#
+properties:
- compatible:
- enum:
- adi,max98388
Blank line
- reg:
- maxItems: 1
- description: I2C address of the device.
Drop description, it's obvious.
- '#sound-dai-cells':
- const: 0
- adi,vmon-slot-no:
- description: slot number of the voltage feedback monitor
- $ref: "/schemas/types.yaml#/definitions/uint32"
Drop quotes.
- minimum: 0
- maximum: 15
- default: 0
- adi,imon-slot-no:
- description: slot number of the current feedback monitor
- $ref: "/schemas/types.yaml#/definitions/uint32"
Drop quotes.
- minimum: 0
- maximum: 15
- default: 1
- adi,interleave-mode:
- description:
For cases where a single combined channel for the I/V feedback data
is not sufficient, the device can also be configured to share
a single data output channel on alternating frames.
In this configuration, the current and voltage data will be frame
interleaved on a single output channel.
- type: boolean
- reset-gpios:
- maxItems: 1
+required:
- compatible
- reg
- "#sound-dai-cells"
Keep consistent quotes - either ' or "
+unevaluatedProperties: false
Best regards, Krzysztof