Skip to content

Commit

Permalink
V4L/DVB: cx18: Rename snd_cx18_mixer_lock to snd_cx18_lock and increa…
Browse files Browse the repository at this point in the history
…se visibility

Rename snd_cx18_mixer_lock() to snd_cx18_lock() in anticpation of using it
in the cx18-alsa-pcm.c file routines.

Signed-off-by: Andy Walls <awalls@radix.net>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
  • Loading branch information
Andy Walls authored and Mauro Carvalho Chehab committed Feb 26, 2010
1 parent 10e43d9 commit 8369500
Show file tree
Hide file tree
Showing 2 changed files with 20 additions and 20 deletions.
24 changes: 4 additions & 20 deletions drivers/media/video/cx18/cx18-alsa-mixer.c
Original file line number Diff line number Diff line change
Expand Up @@ -35,22 +35,6 @@
#include "cx18-alsa.h"
#include "cx18-driver.h"

/*
* Mixer manipulations are like v4l2 ioctl() calls to manipulate controls,
* just use the same lock we use for ioctl()s for now
*/
static inline void snd_cx18_mixer_lock(struct snd_cx18_card *cxsc)
{
struct cx18 *cx = to_cx18(cxsc->v4l2_dev);
mutex_lock(&cx->serialize_lock);
}

static inline void snd_cx18_mixer_unlock(struct snd_cx18_card *cxsc)
{
struct cx18 *cx = to_cx18(cxsc->v4l2_dev);
mutex_unlock(&cx->serialize_lock);
}

/*
* Note the cx18-av-core volume scale is funny, due to the alignment of the
* scale with another chip's range:
Expand Down Expand Up @@ -108,9 +92,9 @@ static int snd_cx18_mixer_tv_vol_get(struct snd_kcontrol *kctl,
vctrl.id = V4L2_CID_AUDIO_VOLUME;
vctrl.value = dB_to_cx18_av_vol(uctl->value.integer.value[0]);

snd_cx18_mixer_lock(cxsc);
snd_cx18_lock(cxsc);
ret = v4l2_subdev_call(cx->sd_av, core, g_ctrl, &vctrl);
snd_cx18_mixer_unlock(cxsc);
snd_cx18_unlock(cxsc);

if (!ret)
uctl->value.integer.value[0] = cx18_av_vol_to_dB(vctrl.value);
Expand All @@ -128,7 +112,7 @@ static int snd_cx18_mixer_tv_vol_put(struct snd_kcontrol *kctl,
vctrl.id = V4L2_CID_AUDIO_VOLUME;
vctrl.value = dB_to_cx18_av_vol(uctl->value.integer.value[0]);

snd_cx18_mixer_lock(cxsc);
snd_cx18_lock(cxsc);

/* Fetch current state */
ret = v4l2_subdev_call(cx->sd_av, core, g_ctrl, &vctrl);
Expand All @@ -142,7 +126,7 @@ static int snd_cx18_mixer_tv_vol_put(struct snd_kcontrol *kctl,
if (!ret)
ret = 1; /* Indicate control was changed w/o error */
}
snd_cx18_mixer_unlock(cxsc);
snd_cx18_unlock(cxsc);

return ret;
}
Expand Down
16 changes: 16 additions & 0 deletions drivers/media/video/cx18/cx18-alsa.h
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,22 @@ struct snd_cx18_card {

extern int cx18_alsa_debug;

/*
* File operations that manipulate the encoder or video or audio subdevices
* need to be serialized. Use the same lock we use for v4l2 file ops.
*/
static inline void snd_cx18_lock(struct snd_cx18_card *cxsc)
{
struct cx18 *cx = to_cx18(cxsc->v4l2_dev);
mutex_lock(&cx->serialize_lock);
}

static inline void snd_cx18_unlock(struct snd_cx18_card *cxsc)
{
struct cx18 *cx = to_cx18(cxsc->v4l2_dev);
mutex_unlock(&cx->serialize_lock);
}

#define CX18_ALSA_DBGFLG_WARN (1 << 0)
#define CX18_ALSA_DBGFLG_WARN (1 << 0)
#define CX18_ALSA_DBGFLG_INFO (1 << 1)
Expand Down

0 comments on commit 8369500

Please sign in to comment.