Skip to content

Commit 44f6804

Browse files
ranj063ujfalusi
ranj063
authored and
ujfalusi
committed
ASoC: SOF: ops: Add new platform-specific ops for compress
Add new ops in the struct snd_sof_ops for platform-specific ops for compresssed streams. Also, define and set them for the HDA platforms. Signed-off-by: Ranjani Sridharan <ranjani.sridharan@linux.intel.com> Co-developed-by: Peter Ujfalusi <peter.ujfalusi@linux.intel.com> Signed-off-by: Peter Ujfalusi <peter.ujfalusi@linux.intel.com>
1 parent 8858114 commit 44f6804

6 files changed

Lines changed: 293 additions & 11 deletions

File tree

‎sound/soc/sof/intel/hda-common-ops.c‎

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -57,6 +57,14 @@ const struct snd_sof_dsp_ops sof_hda_common_ops = {
5757
.pcm_pointer = hda_dsp_pcm_pointer,
5858
.pcm_ack = hda_dsp_pcm_ack,
5959

60+
.compr_open = hda_dsp_compr_open,
61+
.compr_hw_params = hda_dsp_compr_hw_params,
62+
.compr_hw_free = hda_dsp_stream_compr_hw_free,
63+
.compr_close = hda_dsp_compr_close,
64+
.compr_trigger = hda_dsp_compr_trigger,
65+
.compr_pointer = hda_dsp_compr_pointer,
66+
.compr_get_dai_frame_counter = hda_dsp_compr_get_stream_llp,
67+
6068
.get_dai_frame_counter = hda_dsp_get_stream_llp,
6169
.get_host_byte_counter = hda_dsp_get_stream_ldp,
6270

‎sound/soc/sof/intel/hda-pcm.c‎

Lines changed: 142 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -151,6 +151,71 @@ int hda_dsp_pcm_hw_params(struct snd_sof_dev *sdev,
151151
}
152152
EXPORT_SYMBOL_NS(hda_dsp_pcm_hw_params, "SND_SOC_SOF_INTEL_HDA_COMMON");
153153

154+
int hda_dsp_compr_hw_params(struct snd_sof_dev *sdev,
155+
struct snd_compr_stream *cstream,
156+
struct snd_compr_params *params,
157+
struct snd_sof_platform_stream_params *platform_params)
158+
{
159+
struct hdac_stream *hstream = cstream->runtime->private_data;
160+
struct hdac_ext_stream *hext_stream = stream_to_hdac_ext_stream(hstream);
161+
struct sof_intel_hda_dev *hda = sdev->pdata->hw_pdata;
162+
struct snd_dma_buffer *dmab;
163+
u32 bits, rate;
164+
int bps;
165+
int ret;
166+
167+
hstream->cstream = cstream;
168+
dmab = cstream->runtime->dma_buffer_p;
169+
170+
/* Use correct format based on the used codec */
171+
switch (params->codec.id) {
172+
case SND_AUDIOCODEC_PCM:
173+
bps = snd_pcm_format_physical_width(params->codec.format);
174+
break;
175+
case SND_AUDIOCODEC_VORBIS:
176+
bps = snd_pcm_format_physical_width(SNDRV_PCM_FORMAT_S16_LE);
177+
break;
178+
case SND_AUDIOCODEC_FLAC:
179+
{
180+
struct snd_dec_flac *dec_flac = &params->codec.options.flac_d;
181+
182+
if (dec_flac->sample_size == 16)
183+
bps = snd_pcm_format_physical_width(SNDRV_PCM_FORMAT_S16_LE);
184+
else
185+
bps = snd_pcm_format_physical_width(SNDRV_PCM_FORMAT_S32_LE);
186+
break;
187+
}
188+
default:
189+
bps = snd_pcm_format_physical_width(SNDRV_PCM_FORMAT_S32_LE);
190+
}
191+
192+
if (bps < 0)
193+
return bps;
194+
bits = hda_dsp_get_bits(sdev, bps);
195+
rate = hda_dsp_get_mult_div(sdev, params->codec.sample_rate);
196+
197+
hstream->format_val = rate | bits | (params->codec.ch_out - 1);
198+
hstream->bufsize = cstream->runtime->buffer_size;
199+
hstream->period_bytes = cstream->runtime->fragment_size;
200+
hstream->no_period_wakeup = false;
201+
202+
/* params is not used so pass NULL */
203+
dmab = cstream->runtime->dma_buffer_p;
204+
ret = hda_dsp_stream_hw_params(sdev, hext_stream, dmab, NULL);
205+
if (ret < 0) {
206+
dev_err(sdev->dev, "%s: hdac prepare failed: %d\n", __func__, ret);
207+
return ret;
208+
}
209+
210+
if (hda)
211+
platform_params->no_ipc_position = hda->no_ipc_position;
212+
213+
platform_params->stream_tag = hstream->stream_tag;
214+
215+
return 0;
216+
}
217+
EXPORT_SYMBOL_NS(hda_dsp_compr_hw_params, "SND_SOC_SOF_INTEL_HDA_COMMON");
218+
154219
/* update SPIB register with appl position */
155220
int hda_dsp_pcm_ack(struct snd_sof_dev *sdev, struct snd_pcm_substream *substream)
156221
{
@@ -184,6 +249,16 @@ int hda_dsp_pcm_trigger(struct snd_sof_dev *sdev,
184249
}
185250
EXPORT_SYMBOL_NS(hda_dsp_pcm_trigger, "SND_SOC_SOF_INTEL_HDA_COMMON");
186251

252+
int hda_dsp_compr_trigger(struct snd_sof_dev *sdev,
253+
struct snd_compr_stream *cstream, int cmd)
254+
{
255+
struct hdac_stream *hstream = cstream->runtime->private_data;
256+
struct hdac_ext_stream *hext_stream = stream_to_hdac_ext_stream(hstream);
257+
258+
return hda_dsp_stream_trigger(sdev, hext_stream, cmd);
259+
}
260+
EXPORT_SYMBOL_NS(hda_dsp_compr_trigger, "SND_SOC_SOF_INTEL_HDA_COMMON");
261+
187262
snd_pcm_uframes_t hda_dsp_pcm_pointer(struct snd_sof_dev *sdev,
188263
struct snd_pcm_substream *substream)
189264
{
@@ -216,6 +291,20 @@ snd_pcm_uframes_t hda_dsp_pcm_pointer(struct snd_sof_dev *sdev,
216291
}
217292
EXPORT_SYMBOL_NS(hda_dsp_pcm_pointer, "SND_SOC_SOF_INTEL_HDA_COMMON");
218293

294+
int hda_dsp_compr_pointer(struct snd_sof_dev *sdev, struct snd_compr_stream *cstream,
295+
struct snd_compr_tstamp64 *tstamp)
296+
{
297+
struct hdac_stream *hstream = cstream->runtime->private_data;
298+
299+
/* hstream->curr_pos is updated when we receive the ioc */
300+
tstamp->copied_total = hstream->curr_pos;
301+
302+
tstamp->byte_offset = hda_dsp_stream_get_position(hstream, cstream->direction, true);
303+
304+
return 0;
305+
}
306+
EXPORT_SYMBOL_NS(hda_dsp_compr_pointer, "SND_SOC_SOF_INTEL_HDA_COMMON");
307+
219308
int hda_dsp_pcm_open(struct snd_sof_dev *sdev,
220309
struct snd_pcm_substream *substream)
221310
{
@@ -342,6 +431,41 @@ int hda_dsp_pcm_open(struct snd_sof_dev *sdev,
342431
}
343432
EXPORT_SYMBOL_NS(hda_dsp_pcm_open, "SND_SOC_SOF_INTEL_HDA_COMMON");
344433

434+
int hda_dsp_compr_open(struct snd_sof_dev *sdev, struct snd_compr_stream *cstream)
435+
{
436+
struct snd_soc_pcm_runtime *rtd = cstream->private_data;
437+
struct snd_soc_component *scomp = sdev->component;
438+
struct hdac_ext_stream *dsp_stream;
439+
struct snd_sof_pcm *spcm;
440+
int direction = cstream->direction;
441+
442+
spcm = snd_sof_find_spcm_dai(scomp, rtd);
443+
if (!spcm) {
444+
dev_err(sdev->dev, "%s: can't find PCM with DAI ID %d\n",
445+
__func__, rtd->dai_link->id);
446+
return -EINVAL;
447+
}
448+
449+
dsp_stream = hda_dsp_stream_get(sdev, direction, 0);
450+
if (!dsp_stream) {
451+
dev_err(sdev->dev, "%s: no stream available\n", __func__);
452+
return -ENODEV;
453+
}
454+
455+
/* binding compr stream to hda stream */
456+
cstream->runtime->private_data = &dsp_stream->hstream;
457+
458+
/*
459+
* Reset the llp cache values (they are used for LLP compensation in
460+
* case the counter is not reset)
461+
*/
462+
dsp_stream->pplcllpl = 0;
463+
dsp_stream->pplcllpu = 0;
464+
465+
return 0;
466+
}
467+
EXPORT_SYMBOL_NS(hda_dsp_compr_open, "SND_SOC_SOF_INTEL_HDA_COMMON");
468+
345469
int hda_dsp_pcm_close(struct snd_sof_dev *sdev,
346470
struct snd_pcm_substream *substream)
347471
{
@@ -361,3 +485,21 @@ int hda_dsp_pcm_close(struct snd_sof_dev *sdev,
361485
return 0;
362486
}
363487
EXPORT_SYMBOL_NS(hda_dsp_pcm_close, "SND_SOC_SOF_INTEL_HDA_COMMON");
488+
489+
int hda_dsp_compr_close(struct snd_sof_dev *sdev, struct snd_compr_stream *cstream)
490+
{
491+
struct hdac_stream *hstream = cstream->runtime->private_data;
492+
int direction = cstream->direction;
493+
int ret;
494+
495+
ret = hda_dsp_stream_put(sdev, direction, hstream->stream_tag);
496+
if (ret)
497+
return -ENODEV;
498+
499+
/* unbinding compress stream to hda stream */
500+
hstream->cstream = NULL;
501+
cstream->runtime->private_data = NULL;
502+
503+
return 0;
504+
}
505+
EXPORT_SYMBOL_NS(hda_dsp_compr_close, "SND_SOC_SOF_INTEL_HDA_COMMON");

‎sound/soc/sof/intel/hda-stream.c‎

Lines changed: 42 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -747,13 +747,12 @@ int hda_dsp_stream_hw_params(struct snd_sof_dev *sdev,
747747
return ret;
748748
}
749749

750-
int hda_dsp_stream_hw_free(struct snd_sof_dev *sdev,
751-
struct snd_pcm_substream *substream)
750+
static int _hda_dsp_stream_hw_free(struct snd_sof_dev *sdev,
751+
struct hdac_stream *hstream)
752752
{
753-
struct hdac_stream *hstream = substream->runtime->private_data;
754753
struct hdac_ext_stream *hext_stream = container_of(hstream,
755-
struct hdac_ext_stream,
756-
hstream);
754+
struct hdac_ext_stream,
755+
hstream);
757756
int ret;
758757

759758
ret = hda_dsp_stream_reset(sdev, hstream);
@@ -778,8 +777,21 @@ int hda_dsp_stream_hw_free(struct snd_sof_dev *sdev,
778777

779778
return 0;
780779
}
780+
781+
int hda_dsp_stream_hw_free(struct snd_sof_dev *sdev,
782+
struct snd_pcm_substream *substream)
783+
{
784+
return _hda_dsp_stream_hw_free(sdev, substream->runtime->private_data);
785+
}
781786
EXPORT_SYMBOL_NS(hda_dsp_stream_hw_free, "SND_SOC_SOF_INTEL_HDA_COMMON");
782787

788+
int hda_dsp_stream_compr_hw_free(struct snd_sof_dev *sdev,
789+
struct snd_compr_stream *cstream)
790+
{
791+
return _hda_dsp_stream_hw_free(sdev, cstream->runtime->private_data);
792+
}
793+
EXPORT_SYMBOL_NS(hda_dsp_stream_compr_hw_free, "SND_SOC_SOF_INTEL_HDA_COMMON");
794+
783795
bool hda_dsp_check_stream_irq(struct snd_sof_dev *sdev)
784796
{
785797
struct hdac_bus *bus = sof_to_bus(sdev);
@@ -1162,11 +1174,9 @@ EXPORT_SYMBOL_NS(hda_dsp_stream_get_position, "SND_SOC_SOF_INTEL_HDA_COMMON");
11621174
*
11631175
* Returns the raw Linear Link Position value
11641176
*/
1165-
u64 hda_dsp_get_stream_llp(struct snd_sof_dev *sdev,
1166-
struct snd_soc_component *component,
1167-
struct snd_pcm_substream *substream)
1177+
static u64 hda_dsp_get_llp(struct snd_sof_dev *sdev,
1178+
struct snd_soc_pcm_runtime *rtd, int dir)
11681179
{
1169-
struct snd_soc_pcm_runtime *rtd = snd_soc_substream_to_rtd(substream);
11701180
struct snd_soc_pcm_runtime *be_rtd = NULL;
11711181
struct hdac_ext_stream *hext_stream;
11721182
struct snd_soc_dai *cpu_dai;
@@ -1177,7 +1187,7 @@ u64 hda_dsp_get_stream_llp(struct snd_sof_dev *sdev,
11771187
* The LLP needs to be read from the Link DMA used for this FE as it is
11781188
* allowed to use any combination of Link and Host channels
11791189
*/
1180-
for_each_dpcm_be(rtd, substream->stream, dpcm) {
1190+
for_each_dpcm_be(rtd, dir, dpcm) {
11811191
if (dpcm->fe != rtd)
11821192
continue;
11831193

@@ -1191,7 +1201,7 @@ u64 hda_dsp_get_stream_llp(struct snd_sof_dev *sdev,
11911201
if (!cpu_dai)
11921202
return 0;
11931203

1194-
hext_stream = snd_soc_dai_get_dma_data(cpu_dai, substream);
1204+
hext_stream = snd_soc_dai_dma_data_get(cpu_dai, dir);
11951205
if (!hext_stream)
11961206
return 0;
11971207

@@ -1215,8 +1225,29 @@ u64 hda_dsp_get_stream_llp(struct snd_sof_dev *sdev,
12151225

12161226
return merge_u64(llp_u, llp_l);
12171227
}
1228+
1229+
u64 hda_dsp_get_stream_llp(struct snd_sof_dev *sdev,
1230+
struct snd_soc_component *component,
1231+
struct snd_pcm_substream *substream)
1232+
{
1233+
return hda_dsp_get_llp(sdev, snd_soc_substream_to_rtd(substream),
1234+
substream->stream);
1235+
}
12181236
EXPORT_SYMBOL_NS(hda_dsp_get_stream_llp, "SND_SOC_SOF_INTEL_HDA_COMMON");
12191237

1238+
/**
1239+
* hda_dsp_compr_get_stream_llp - Retrieve the LLP (Linear Link Position) of the stream
1240+
* @sdev: SOF device
1241+
* @cstream: Compress stream
1242+
*
1243+
* Returns the raw Linear Link Position value
1244+
*/
1245+
u64 hda_dsp_compr_get_stream_llp(struct snd_sof_dev *sdev, struct snd_compr_stream *cstream)
1246+
{
1247+
return hda_dsp_get_llp(sdev, cstream->private_data, cstream->direction);
1248+
}
1249+
EXPORT_SYMBOL_NS(hda_dsp_compr_get_stream_llp, "SND_SOC_SOF_INTEL_HDA_COMMON");
1250+
12201251
/**
12211252
* hda_dsp_get_stream_ldp - Retrieve the LDP (Linear DMA Position) of the stream
12221253
* @sdev: SOF device

‎sound/soc/sof/intel/hda.h‎

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -660,6 +660,21 @@ snd_pcm_uframes_t hda_dsp_pcm_pointer(struct snd_sof_dev *sdev,
660660
struct snd_pcm_substream *substream);
661661
int hda_dsp_pcm_ack(struct snd_sof_dev *sdev, struct snd_pcm_substream *substream);
662662

663+
int hda_dsp_compr_open(struct snd_sof_dev *sdev, struct snd_compr_stream *cstream);
664+
int hda_dsp_compr_close(struct snd_sof_dev *sdev, struct snd_compr_stream *cstream);
665+
int hda_dsp_compr_hw_params(struct snd_sof_dev *sdev,
666+
struct snd_compr_stream *cstream,
667+
struct snd_compr_params *params,
668+
struct snd_sof_platform_stream_params *platform_params);
669+
int hda_dsp_stream_compr_hw_free(struct snd_sof_dev *sdev,
670+
struct snd_compr_stream *cstream);
671+
int hda_dsp_compr_trigger(struct snd_sof_dev *sdev,
672+
struct snd_compr_stream *cstream, int cmd);
673+
int hda_dsp_compr_pointer(struct snd_sof_dev *sdev, struct snd_compr_stream *cstream,
674+
struct snd_compr_tstamp64 *tstamp);
675+
u64 hda_dsp_compr_get_stream_llp(struct snd_sof_dev *sdev,
676+
struct snd_compr_stream *cstream);
677+
663678
/*
664679
* DSP Stream Operations.
665680
*/

0 commit comments

Comments
 (0)