Re: [PATCH v7 04/10] ASoC: qcom: Add QAIF CPU DAI ops, regmap, DT parsing and platform init

[Harendra Gautam]

On Mon, Sep 28, 2026 at 3:08 PM Harendra Gautam
<harendra.gautam@xxxxxxxxxxxxxxxx> wrote:
>
> >
> > > +     /*
> > > +      * The sync, lane and frame configuration and the MI2S bit clock are
> > > +      * shared by both directions of an AIF. Concurrent playback and
> > > +      * capture on the same interface are therefore expected to use a
> > > +      * compatible format, rate and slot configuration.
> > > +      */
> >
> > What enforces this?
> Both directions share the bit clock and slot width, so rate and sample
> bits must match on concurrent streams. Would adding symmetric_rate and
> symmetric_sample_bits to the AIF DAI entries work?
>
Please ignore my earlier symmetric_rate/symmetric_sample_bits
suggestion, Playback and capture are separate DAIs on separate
dai_links so it won't work.
The comment text in hw_param() is misleading, hw_params() doesn't set
rate or bit clock (the machine driver does via set_sysclk()). It only
writes sync/lane/frame config from aif_intf_cfg[idx], fixed at probe
and via set_fmt()/set_tdm_slot(), so both directions write the same
values. Will reword the comment.
-Harendra