On 23/02/2022 17:38, Daniel Baluta wrote:
> From: Daniel Baluta <daniel.baluta@xxxxxxx>
>
> Now that sof_compressed_ops initial implementation was merged
> we can enable it in SOF platform component driver.
and fixes the following sparse error:
sound/soc/sof/compress.c:310:25: error: symbol 'sof_compressed_ops' was
not declared. Should it be static?
> This partially reverts commit
> 8a720724589e ("ASoC: SOF: pcm: Remove non existent CONFIG_SND_SOC_SOF_COMPRESS reference")
Back at that time there were _no_ sof_compressed_ops in the tree...
> Reported-by: Peter Ujfalusi <peter.ujfalusi@xxxxxxxxxxxxxxx>
> Signed-off-by: Daniel Baluta <daniel.baluta@xxxxxxx>
> ---
> sound/soc/sof/pcm.c | 4 ++++
> sound/soc/sof/sof-priv.h | 5 +++++
> 2 files changed, 9 insertions(+)
>
> diff --git a/sound/soc/sof/pcm.c b/sound/soc/sof/pcm.c
> index 137f8ed71677..a312ed855f1a 100644
> --- a/sound/soc/sof/pcm.c
> +++ b/sound/soc/sof/pcm.c
> @@ -922,6 +922,10 @@ void snd_sof_new_platform_drv(struct snd_sof_dev *sdev)
> pd->pointer = sof_pcm_pointer;
> pd->ack = sof_pcm_ack;
>
> +#if IS_ENABLED(CONFIG_SND_SOC_SOF_COMPRESS)
> + pd->compress_ops = &sof_compressed_ops;
> +#endif
> +
> pd->pcm_construct = sof_pcm_new;
> pd->ignore_machine = drv_name;
> pd->be_hw_params_fixup = sof_pcm_dai_link_fixup;
> diff --git a/sound/soc/sof/sof-priv.h b/sound/soc/sof/sof-priv.h
> index 2c8e556cd5cc..886787a9997f 100644
> --- a/sound/soc/sof/sof-priv.h
> +++ b/sound/soc/sof/sof-priv.h
> @@ -556,6 +556,11 @@ int snd_sof_debugfs_add_region_item_iomem(struct snd_sof_dev *sdev,
> enum snd_sof_fw_blk_type blk_type, u32 offset, size_t size,
> const char *name, enum sof_debugfs_access_type access_type);
>
> +/*
> + * Platform specific ops.
> + */
> +extern struct snd_compress_ops sof_compressed_ops;
This is not really platform ops?
> +
> /*
> * DSP Architectures.
> */
--
Péter
