Skip to content

L
linux
Commit 6c6ba11f authored by Takashi Iwai's avatar Takashi Iwai
Browse files
Options

ALSA: korg1212: Convert to the new PCM ops 

Replace the copy and the silence ops with the new PCM ops.
Although we can refactor this messy code, at this time, the changes
are kept as small as possible.  Let's clean up later.
Reviewed-by: default avatarTakashi Sakamoto <o-takashi@sakamocchi.jp>
Signed-off-by: default avatarTakashi Iwai <tiwai@suse.de>
parent 21de5673
Show whitespace changes
Inline Side-by-side
Showing with 65 additions and 47 deletions
sound/pci/korg1212/korg1212.c
View file @ 6c6ba11f
...@@ -1299,13 +1299,21 @@ static int snd_korg1212_silence(struct snd_korg1212 *korg1212, int pos, int coun ...@@ -1299,13 +1299,21 @@ static int snd_korg1212_silence(struct snd_korg1212 *korg1212, int pos, int coun
return 0; return 0;
} }
static int snd_korg1212_copy_to(struct snd_korg1212 *korg1212, void __user *dst, int pos, int count, int offset, int size) static int snd_korg1212_copy_to(struct snd_pcm_substream *substream,
void __user *dst, int pos, int count,
bool in_kernel)
{ {
struct KorgAudioFrame * src = korg1212->recordDataBufsPtr[0].bufferData + pos; struct snd_pcm_runtime *runtime = substream->runtime;
int i, rc; struct snd_korg1212 *korg1212 = snd_pcm_substream_chip(substream);
struct KorgAudioFrame *src;
K1212_DEBUG_PRINTK_VERBOSE("K1212_DEBUG: snd_korg1212_copy_to pos=%d offset=%d size=%d\n", int i, size;
pos, offset, size);
pos = bytes_to_frames(runtime, pos);
count = bytes_to_frames(runtime, count);
size = korg1212->channels * 2;
src = korg1212->recordDataBufsPtr[0].bufferData + pos;
K1212_DEBUG_PRINTK_VERBOSE("K1212_DEBUG: snd_korg1212_copy_to pos=%d size=%d count=%d\n",
pos, size, count);
if (snd_BUG_ON(pos + count > K1212_MAX_SAMPLES)) if (snd_BUG_ON(pos + count > K1212_MAX_SAMPLES))
return -EINVAL; return -EINVAL;
...@@ -1317,11 +1325,10 @@ static int snd_korg1212_copy_to(struct snd_korg1212 *korg1212, void __user *dst, ...@@ -1317,11 +1325,10 @@ static int snd_korg1212_copy_to(struct snd_korg1212 *korg1212, void __user *dst,
return -EFAULT; return -EFAULT;
} }
#endif #endif
rc = copy_to_user(dst + offset, src, size); if (in_kernel)
if (rc) { memcpy((void *)dst, src, size);
K1212_DEBUG_PRINTK("K1212_DEBUG: snd_korg1212_copy_to USER EFAULT src=%p dst=%p iter=%d\n", src, dst, i); else if (copy_to_user(dst, src, size))
return -EFAULT; return -EFAULT;
}
src++; src++;
dst += size; dst += size;
} }
...@@ -1329,13 +1336,22 @@ static int snd_korg1212_copy_to(struct snd_korg1212 *korg1212, void __user *dst, ...@@ -1329,13 +1336,22 @@ static int snd_korg1212_copy_to(struct snd_korg1212 *korg1212, void __user *dst,
return 0; return 0;
} }
static int snd_korg1212_copy_from(struct snd_korg1212 *korg1212, void __user *src, int pos, int count, int offset, int size) static int snd_korg1212_copy_from(struct snd_pcm_substream *substream,
void __user *src, int pos, int count,
bool in_kernel)
{ {
struct KorgAudioFrame * dst = korg1212->playDataBufsPtr[0].bufferData + pos; struct snd_pcm_runtime *runtime = substream->runtime;
int i, rc; struct snd_korg1212 *korg1212 = snd_pcm_substream_chip(substream);
struct KorgAudioFrame *dst;
int i, size;
K1212_DEBUG_PRINTK_VERBOSE("K1212_DEBUG: snd_korg1212_copy_from pos=%d offset=%d size=%d count=%d\n", pos = bytes_to_frames(runtime, pos);
pos, offset, size, count); count = bytes_to_frames(runtime, count);
size = korg1212->channels * 2;
dst = korg1212->playDataBufsPtr[0].bufferData + pos;
K1212_DEBUG_PRINTK_VERBOSE("K1212_DEBUG: snd_korg1212_copy_from pos=%d size=%d count=%d\n",
pos, size, count);
if (snd_BUG_ON(pos + count > K1212_MAX_SAMPLES)) if (snd_BUG_ON(pos + count > K1212_MAX_SAMPLES))
return -EINVAL; return -EINVAL;
...@@ -1348,11 +1364,10 @@ static int snd_korg1212_copy_from(struct snd_korg1212 *korg1212, void __user *sr ...@@ -1348,11 +1364,10 @@ static int snd_korg1212_copy_from(struct snd_korg1212 *korg1212, void __user *sr
return -EFAULT; return -EFAULT;
} }
#endif #endif
rc = copy_from_user((void*) dst + offset, src, size); if (in_kernel)
if (rc) { memcpy((void *)dst, src, size);
K1212_DEBUG_PRINTK("K1212_DEBUG: snd_korg1212_copy_from USER EFAULT src=%p dst=%p iter=%d\n", src, dst, i); else if (copy_from_user(dst, src, size))
return -EFAULT; return -EFAULT;
}
dst++; dst++;
src += size; src += size;
} }
...@@ -1640,45 +1655,46 @@ static snd_pcm_uframes_t snd_korg1212_capture_pointer(struct snd_pcm_substream * ...@@ -1640,45 +1655,46 @@ static snd_pcm_uframes_t snd_korg1212_capture_pointer(struct snd_pcm_substream *
} }
static int snd_korg1212_playback_copy(struct snd_pcm_substream *substream, static int snd_korg1212_playback_copy(struct snd_pcm_substream *substream,
int channel, /* not used (interleaved data) */ int channel, unsigned long pos,
snd_pcm_uframes_t pos, void __user *src, unsigned long count)
void __user *src,
snd_pcm_uframes_t count)
{ {
struct snd_korg1212 *korg1212 = snd_pcm_substream_chip(substream); return snd_korg1212_copy_from(substream, src, pos, count, false);
}
K1212_DEBUG_PRINTK_VERBOSE("K1212_DEBUG: snd_korg1212_playback_copy [%s] %ld %ld\n",
stateName[korg1212->cardState], pos, count);
return snd_korg1212_copy_from(korg1212, src, pos, count, 0, korg1212->channels * 2);
static int snd_korg1212_playback_copy_kernel(struct snd_pcm_substream *substream,
int channel, unsigned long pos,
void *src, unsigned long count)
{
return snd_korg1212_copy_from(substream, (void __user *)src,
pos, count, true);
} }
static int snd_korg1212_playback_silence(struct snd_pcm_substream *substream, static int snd_korg1212_playback_silence(struct snd_pcm_substream *substream,
int channel, /* not used (interleaved data) */ int channel, /* not used (interleaved data) */
snd_pcm_uframes_t pos, unsigned long pos,
snd_pcm_uframes_t count) unsigned long count)
{ {
struct snd_pcm_runtime *runtime = substream->runtime;
struct snd_korg1212 *korg1212 = snd_pcm_substream_chip(substream); struct snd_korg1212 *korg1212 = snd_pcm_substream_chip(substream);
K1212_DEBUG_PRINTK_VERBOSE("K1212_DEBUG: snd_korg1212_playback_silence [%s]\n", return snd_korg1212_silence(korg1212, bytes_to_frames(runtime, pos),
stateName[korg1212->cardState]); bytes_to_frames(runtime, count),
0, korg1212->channels * 2);
return snd_korg1212_silence(korg1212, pos, count, 0, korg1212->channels * 2);
} }
static int snd_korg1212_capture_copy(struct snd_pcm_substream *substream, static int snd_korg1212_capture_copy(struct snd_pcm_substream *substream,
int channel, /* not used (interleaved data) */ int channel, unsigned long pos,
snd_pcm_uframes_t pos, void __user *dst, unsigned long count)
void __user *dst,
snd_pcm_uframes_t count)
{ {
struct snd_korg1212 *korg1212 = snd_pcm_substream_chip(substream); return snd_korg1212_copy_to(substream, dst, pos, count, false);
}
K1212_DEBUG_PRINTK_VERBOSE("K1212_DEBUG: snd_korg1212_capture_copy [%s] %ld %ld\n",
stateName[korg1212->cardState], pos, count);
return snd_korg1212_copy_to(korg1212, dst, pos, count, 0, korg1212->channels * 2); static int snd_korg1212_capture_copy_kernel(struct snd_pcm_substream *substream,
int channel, unsigned long pos,
void *dst, unsigned long count)
{
return snd_korg1212_copy_to(substream, (void __user *)dst,
pos, count, true);
} }
static const struct snd_pcm_ops snd_korg1212_playback_ops = { static const struct snd_pcm_ops snd_korg1212_playback_ops = {
...@@ -1689,8 +1705,9 @@ static const struct snd_pcm_ops snd_korg1212_playback_ops = { ...@@ -1689,8 +1705,9 @@ static const struct snd_pcm_ops snd_korg1212_playback_ops = {
.prepare = snd_korg1212_prepare, .prepare = snd_korg1212_prepare,
.trigger = snd_korg1212_trigger, .trigger = snd_korg1212_trigger,
.pointer = snd_korg1212_playback_pointer, .pointer = snd_korg1212_playback_pointer,
.copy = snd_korg1212_playback_copy, .copy_user = snd_korg1212_playback_copy,
.silence = snd_korg1212_playback_silence, .copy_kernel = snd_korg1212_playback_copy_kernel,
.fill_silence = snd_korg1212_playback_silence,
}; };
static const struct snd_pcm_ops snd_korg1212_capture_ops = { static const struct snd_pcm_ops snd_korg1212_capture_ops = {
...@@ -1701,7 +1718,8 @@ static const struct snd_pcm_ops snd_korg1212_capture_ops = { ...@@ -1701,7 +1718,8 @@ static const struct snd_pcm_ops snd_korg1212_capture_ops = {
.prepare = snd_korg1212_prepare, .prepare = snd_korg1212_prepare,
.trigger = snd_korg1212_trigger, .trigger = snd_korg1212_trigger,
.pointer = snd_korg1212_capture_pointer, .pointer = snd_korg1212_capture_pointer,
.copy = snd_korg1212_capture_copy, .copy_user = snd_korg1212_capture_copy,
.copy_kernel = snd_korg1212_capture_copy_kernel,
}; };
/* /*
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment
GitLab Nexedi Edition | About GitLab | About Nexedi | 沪ICP备2021021310号-2 | 沪ICP备2021021310号-7