diff options
| author | Takashi Iwai <tiwai@suse.de> | 2024-12-20 12:44:16 +0100 |
|---|---|---|
| committer | Takashi Iwai <tiwai@suse.de> | 2024-12-20 12:45:38 +0100 |
| commit | 66a0a2b0473c39ae85c44628d14e4366fdc0aa0d (patch) | |
| tree | 2d03ff1c1272b63f8811862b1ea575a645814996 /sound | |
| parent | selftests/alsa: Fix circular dependency involving global-timer (diff) | |
| download | linux-66a0a2b0473c39ae85c44628d14e4366fdc0aa0d.tar.xz linux-66a0a2b0473c39ae85c44628d14e4366fdc0aa0d.zip | |
ALSA: sh: Fix wrong argument order for copy_from_iter()
Fix a brown paper bag bug I introduced at converting to the standard
iter helper; the arguments were wrongly passed and have to be
swapped.
Fixes: 9b5f8ee43e48 ("ALSA: sh: Use standard helper for buffer accesses")
Reported-by: kernel test robot <lkp@intel.com>
Closes: https://lore.kernel.org/oe-kbuild-all/202412140019.jat5Dofr-lkp@intel.com/
Link: https://patch.msgid.link/20241220114417.5898-1-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'sound')
| -rw-r--r-- | sound/sh/sh_dac_audio.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sound/sh/sh_dac_audio.c b/sound/sh/sh_dac_audio.c index a4d07438ad64..3f5422145c5e 100644 --- a/sound/sh/sh_dac_audio.c +++ b/sound/sh/sh_dac_audio.c @@ -163,7 +163,7 @@ static int snd_sh_dac_pcm_copy(struct snd_pcm_substream *substream, /* channel is not used (interleaved data) */ struct snd_sh_dac *chip = snd_pcm_substream_chip(substream); - if (copy_from_iter(chip->data_buffer + pos, src, count) != count) + if (copy_from_iter(chip->data_buffer + pos, count, src) != count) return -EFAULT; chip->buffer_end = chip->data_buffer + pos + count; |