Re: [PATCH v2 1/4] dt-bindings: sound: Add ti,tas675x

Sen Wang
2026-04-03T00:40:46Z

On 4/2/26 02:53, Krzysztof Kozlowski wrote:
> On Wed, Apr 01, 2026 at 05:28:42PM -0500, Sen Wang wrote:
>> Add device tree binding for the Texas Instruments TAS675x family
>> of four-channel Class-D audio amplifiers with integrated DSP.
>>
>> Signed-off-by: Sen Wang <sen@ti.com>
>

Hi Krzysztof, Thank you for your review.

> Please use subject prefixes matching the subsystem. You can get them for
> example with 'git log --oneline -- DIRECTORY_OR_FILE' on the directory
> your patch is touching. For bindings, the preferred subjects are
> explained here:
> https://www.kernel.org/doc/html/latest/devicetree/bindings/submitting-patches.html#i-for-patch-submitters

Understood, ASoC: dt-bindings: Add ti,tas67524. Apologies for the oversight.

>
>> ---
>> .../devicetree/bindings/sound/ti,tas675x.yaml | 278 ++++++++++++++++++
>
> Filename matching compatible, one of them.
>
>> 1 file changed, 278 insertions(+)
>> create mode 100644 Documentation/devicetree/bindings/sound/ti,tas675x.yaml
>>
>> diff --git a/Documentation/devicetree/bindings/sound/ti,tas675x.yaml b/Documentation/devicetree/bindings/sound/ti,tas675x.yaml
>> new file mode 100644
>> index 000000000000..23e4cc77b4ae
>> --- /dev/null
>> +++ b/Documentation/devicetree/bindings/sound/ti,tas675x.yaml
>> @@ -0,0 +1,278 @@
>> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
>> +%YAML 1.2
>> +---
>> +$id: http://devicetree.org/schemas/sound/ti,tas675x.yaml#
>> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>> +
>> +title: Texas Instruments TAS675x Audio Amplifier
>> +
>> +maintainers:
>> + - Sen Wang <sen@ti.com>
>> +
>> +description: |
>
> Do not need '|' unless you need to preserve formatting.
>
>> + The TAS675x family (TAS6754, TAS67524) are four-channel, digital-input,
>
> And devices are not compatible?
>

They're compatible, and I didn't have any differentiators for this
initial driver. So I should only keep one.
And also rename ti,tas675x.yaml to ti,tas67524.yaml.

>> + automotive Class-D audio amplifiers with load diagnostics and an integrated
>> + DSP for audio processing.
>> +
>> +allOf:
>> + - $ref: dai-common.yaml#
>> +
>> +properties:
>> + compatible:
>> + enum:
>> + - ti,tas6754
>> + - ti,tas67524
>
> Keep alphanumerical order (not natural).
>
>> +
>> + reg:
>> + maxItems: 1
>> +
>> + '#sound-dai-cells':
>> + const: 1
>> + description: |
>> + The device exposes three DAIs, selected by index.
>> + 0 - Standard Audio Path (Playback)
>> + 1 - Low-Latency Playback Path (Playback)
>> + 2 - Sensory Feedback (Capture - Vpredict and Isense)
>> + By default, all four channels of each DAI are active. Runtime
>> + reconfiguration is available through DAPM widgets.
>> +
>> + interrupts:
>> + maxItems: 1
>> + description:
>> + Active-low falling-edge interrupt from the FAULT pin. When provided,
>> + the driver uses IRQ-driven fault reporting instead of polling.
>> +
>> + pd-gpios:
>
> Use names from gpio-consumer-common
>

Okay

>> + maxItems: 1
>> + description:
>> + GPIO connected to the power-down (PD#) pin, active low. Controls the
>> + internal digital circuitry power state. When asserted the device enters
>> + full power-down mode and all register state is lost. Can be omitted if
>> + PD pin is hardwired or externally controlled.
>> +
>> + stby-gpios:
>> + maxItems: 1
>> + description:
>> + GPIO connected to the standby (STBY#) pin, active low. Controls the
>> + analog power stage. When asserted the device enters Deep Sleep mode
>> + but remains I2C-accessible with registers retained. Can be omitted if
>> + STBY pin is tied to PD or hardwired.
>> +
>> + dvdd-supply:
>> + description:
>> + Digital logic supply (1.62 V to 3.6 V). All three supply rails must
>> + be within their recommended operating ranges before the PD pin is
>> + released.
>> +
>> + pvdd-supply:
>> + description:
>> + Output FET power supply (4.5 V to 19 V). All three supply rails must
>> + be within their recommended operating ranges before the PD pin is
>> + released.
>> +
>> + vbat-supply:
>> + description:
>> + Battery supply for the Class-D output stage (4.5 V to 19 V). Optional
>> + when PVDD and VBAT are connected to the same supply rail. When absent,
>> + VBAT is assumed hardwired to PVDD.
>> +
>> + ti,fast-boot:
>> + type: boolean
>> + description:
>> + Skip DC load diagnostic sweep at power-on to reduce boot latency.
>> + Automatic diagnostics after fault conditions remain enabled. Hardware
>> + overcurrent protection is always active.
>> +
>> + ti,audio-slot-no:
>> + $ref: /schemas/types.yaml#/definitions/uint32
>> + description:
>> + TDM slot offset for the standard audio playback path via SDIN1. A value
>> + of 4 maps to slot 4. If omitted, slot assignment is derived from the
>> + tx_mask provided via set_tdm_slot(). Without either property, no slot
>> + mapping is configured.
>> +
>> + ti,llp-slot-no:
>> + $ref: /schemas/types.yaml#/definitions/uint32
>> + description:
>> + TDM slot offset for the low-latency playback path via SDIN1. If omitted,
>> + slot assignment is derived from the tx_mask provided via set_tdm_slot().
>> + Without either property, no slot mapping is configured. Disabled outside
>> + of LLP mode, and only relevant for TDM formats.
>> +
>> + ti,vpredict-slot-no:
>> + $ref: /schemas/types.yaml#/definitions/uint32
>> + description: |
>> + In TDM mode, enables Vpredict output and assigns its starting slot;
>> + four consecutive slots carry Vpredict Ch1-4 on SDOUT1. May coexist
>> + with ti,isense-slot-no using separate non-overlapping slots.
>> +
>> + In I2S mode, enables Vpredict output on SDOUT1 (Ch1/Ch2) and SDOUT2
>> + (Ch3/Ch4). The slot value is unused. Requires a GPIO configured as
>> + sdout2 for Ch3/Ch4; without it only Ch1/Ch2 are output. Mutually
>> + exclusive with ti,isense-slot-no; if both are set, Vpredict takes
>> + priority.
>> +
>> + Irrelevant in Left-J and Right-J modes.
>> +
>> + ti,isense-slot-no:
>> + $ref: /schemas/types.yaml#/definitions/uint32
>> + description: |
>> + In TDM mode, enables Isense output and assigns its starting slot;
>> + four consecutive slots carry Isense Ch1-4 on SDOUT1. May coexist
>> + with ti,vpredict-slot-no using separate non-overlapping slots.
>> +
>> + In I2S mode, enables Isense output on SDOUT1 (Ch1/Ch2) and SDOUT2
>> + (Ch3/Ch4). The slot value is unused. Requires a GPIO configured as
>> + sdout2 for Ch3/Ch4; without it only Ch1/Ch2 are output. Mutually
>> + exclusive with ti,vpredict-slot-no; Vpredict takes priority if both
>> + are set.
>> +
>> + Irrelevant in Left-J and Right-J modes.
>> +
>> + ti,gpio1-function:
>> + $ref: /schemas/types.yaml#/definitions/string
>> + description: |
>
> Do not need '|' unless you need to preserve formatting.
>

Understood, thanks a lot for your review and the tip. Will post a V3.

>> + Function for the GPIO_1 pin. When omitted, GPIO_1 remains in its
>> + power-on default state.
>
> Best regards,
> Krzysztof
>