| Message ID | 20210626155941.12251-1-ckeepax@opensource.cirrus.com |
|---|---|
| State | Accepted |
| Commit | dd6fb8ff2210f74b056bf9234d0605e8c26a8ac0 |
| Headers | show |
| Series | [1/3] ASoC: wm_adsp: Correct wm_coeff_tlv_get handling | expand |
On Sat, Jun 26, 2021 at 04:59:41PM +0100, Charles Keepax wrote: > The control fw_name is always directly assigned from the wm_adsp_fw_text > array, so it isn't necessary to compare the actual strings just the > pointer values. This feels like it's asking for trouble in the future...
On Sat, 26 Jun 2021 16:59:39 +0100, Charles Keepax wrote: > When wm_coeff_tlv_get was updated it was accidentally switch to the _raw > version of the helper causing it to ignore the current DSP state it > should be checking. Switch the code back to the correct helper so that > users can't read the controls when they arn't available. Applied to https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next Thanks! [1/3] ASoC: wm_adsp: Correct wm_coeff_tlv_get handling commit: dd6fb8ff2210f74b056bf9234d0605e8c26a8ac0 [2/3] ASoC: wm_adsp: Add CCM_CORE_RESET to Halo start core commit: e588332271b9cde6252dac8973b77e580cd639bd All being well this means that it will be integrated into the linux-next tree (usually sometime in the next 24 hours) and sent to Linus during the next merge window (or sooner if it is a bug fix), however if problems are discovered then the patch may be dropped or reverted. You may get further e-mails resulting from automated or manual testing and review of the tree, please engage with people reporting problems and send followup patches addressing any issues that are reported if needed. If any updates are required or you are submitting further changes they should be sent as incremental updates against current git, existing patches will not be replaced. Please add any relevant lists and maintainers to the CCs when replying to this mail. Thanks, Mark
On Sat, 26 Jun 2021 16:59:39 +0100, Charles Keepax wrote: > When wm_coeff_tlv_get was updated it was accidentally switch to the _raw > version of the helper causing it to ignore the current DSP state it > should be checking. Switch the code back to the correct helper so that > users can't read the controls when they arn't available. Applied to https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next Thanks! [3/3] ASoC: wm_adsp: Remove pointless string comparison commit: 2ba907894f9e69b68e5934b57afb744482a72984 All being well this means that it will be integrated into the linux-next tree (usually sometime in the next 24 hours) and sent to Linus during the next merge window (or sooner if it is a bug fix), however if problems are discovered then the patch may be dropped or reverted. You may get further e-mails resulting from automated or manual testing and review of the tree, please engage with people reporting problems and send followup patches addressing any issues that are reported if needed. If any updates are required or you are submitting further changes they should be sent as incremental updates against current git, existing patches will not be replaced. Please add any relevant lists and maintainers to the CCs when replying to this mail. Thanks, Mark
diff --git a/sound/soc/codecs/wm_adsp.c b/sound/soc/codecs/wm_adsp.c index 37aa020f23f63..59d876d36cfd8 100644 --- a/sound/soc/codecs/wm_adsp.c +++ b/sound/soc/codecs/wm_adsp.c @@ -1213,7 +1213,7 @@ static int wm_coeff_tlv_get(struct snd_kcontrol *kctl, mutex_lock(&ctl->dsp->pwr_lock); - ret = wm_coeff_read_ctrl_raw(ctl, ctl->cache, size); + ret = wm_coeff_read_ctrl(ctl, ctl->cache, size); if (!ret && copy_to_user(bytes, ctl->cache, size)) ret = -EFAULT;
When wm_coeff_tlv_get was updated it was accidentally switch to the _raw version of the helper causing it to ignore the current DSP state it should be checking. Switch the code back to the correct helper so that users can't read the controls when they arn't available. Fixes: 73ecf1a673d3 ("ASoC: wm_adsp: Correct cache handling of new kernel control API") Signed-off-by: Charles Keepax <ckeepax@opensource.cirrus.com> --- sound/soc/codecs/wm_adsp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)