On Mon, 2021-01-25 at 17:51 -0800, Ranjani Sridharan wrote:
> DMI L1 entry is currently disabled whenever any capture stream is
> opened to prevent xruns during pause/release. But, in
> order to maximise power savings for the wake-on-voice usecase,
> DMI L1 entry should be enabled for D0i3-compatible capture streams.
> Introduce a new field, flags in struct sof_intel_hda_stream
> that stores whether a stream is dmi_l1_compatible. All playback
> streams,
> and D0i3-compatible capture streams are DMI L1 compatible.
>
> Reviewed-by: Pierre-Louis Bossart <
> pierre-louis.bossart@xxxxxxxxxxxxxxx>
> Reviewed-by: Kai Vehmanen <kai.vehmanen@xxxxxxxxxxxxxxx>
> Signed-off-by: Ranjani Sridharan <ranjani.sridharan@xxxxxxxxxxxxxxx>
> ---
> sound/soc/sof/intel/hda-compress.c | 2 +-
> sound/soc/sof/intel/hda-loader.c | 2 +-
> sound/soc/sof/intel/hda-pcm.c | 16 ++++++++++++-
> sound/soc/sof/intel/hda-stream.c | 38 +++++++++++++++++-----------
> --
> sound/soc/sof/intel/hda-trace.c | 3 +--
> sound/soc/sof/intel/hda.h | 6 ++++-
> 6 files changed, 45 insertions(+), 22 deletions(-)
>
> diff --git a/sound/soc/sof/intel/hda-compress.c
> b/sound/soc/sof/intel/hda-compress.c
> index 53c08034fa22..7899004df8d1 100644
> --- a/sound/soc/sof/intel/hda-compress.c
> +++ b/sound/soc/sof/intel/hda-compress.c
> @@ -25,7 +25,7 @@ int hda_probe_compr_assign(struct snd_sof_dev
> *sdev,
> {
> struct hdac_ext_stream *stream;
>
> - stream = hda_dsp_stream_get(sdev, cstream->direction);
> + stream = hda_dsp_stream_get(sdev, cstream->direction, 0)
Hi Mark,
Sorry, this wont build with the probes feature enabled in Kconfig
because of the missing semicolon. I will post v2 soon.
Thanks,
Ranjani
