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

[Sen Wang]

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@xxxxxx>

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@xxxxxx>
> > +
> > +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