diff options
| author | Keyon Jie <yang.jie@linux.intel.com> | 2020-08-25 16:50:37 -0700 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2020-08-26 13:19:14 +0100 |
| commit | 878694dcbe51794c7a68195b92a3707ed4ff5826 (patch) | |
| tree | 8ce5d6958e079a81d460a26347f4cc2c515504db /sound/soc/sof/topology.c | |
| parent | 3dca35e35b42b3405ddad7ee95c02a2d8cf28592 (diff) | |
| download | lwn-878694dcbe51794c7a68195b92a3707ed4ff5826.tar.gz lwn-878694dcbe51794c7a68195b92a3707ed4ff5826.zip | |
ASoC: SOF: topology: fix the ipc_size calculation for process component
The topology private struct is used for token parsing and its size
should not be included to the ipc_size, fix it here though it didn't
cause any real issue as the Firmware won't use this wrong-added data.
Signed-off-by: Keyon Jie <yang.jie@linux.intel.com>
Reviewed-by: Guennadi Liakhovetski <guennadi.liakhovetski@linux.intel.com>
Reviewed-by: Jaska Uimonen <jaska.uimonen@intel.com>
Signed-off-by: Ranjani Sridharan <ranjani.sridharan@linux.intel.com>
Link: https://lore.kernel.org/r/20200825235040.1586478-5-ranjani.sridharan@linux.intel.com
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'sound/soc/sof/topology.c')
| -rw-r--r-- | sound/soc/sof/topology.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/sound/soc/sof/topology.c b/sound/soc/sof/topology.c index 707fbac3e64f..95e63d138326 100644 --- a/sound/soc/sof/topology.c +++ b/sound/soc/sof/topology.c @@ -2114,9 +2114,7 @@ static int sof_process_load(struct snd_soc_component *scomp, int index, goto out; } - ipc_size = sizeof(struct sof_ipc_comp_process) + - le32_to_cpu(private->size) + - ipc_data_size; + ipc_size = sizeof(struct sof_ipc_comp_process) + ipc_data_size; /* we are exceeding max ipc size, config needs to be sent separately */ if (ipc_size > SOF_IPC_MSG_MAX_SIZE) { |