This patch applies 2.6.37 commit 0310871d8f71da4ad8643687fbc40f219a0dac4d to the 2.6.36 stable series. It was just too late to be included in 2.6.36 at the time, but it should be added to 2.6.36 since it fixes broken audio on this fairly popular audio receiver chip. Regards, Hans >From 0310871d8f71da4ad8643687fbc40f219a0dac4d Mon Sep 17 00:00:00 2001 Message-Id: <0310871d8f71da4ad8643687fbc40f219a0dac4d.1290686265.git.hverkuil@xxxxxxxxx> From: Hans Verkuil <hverkuil@xxxxxxxxx> Date: Sun, 17 Oct 2010 07:24:20 -0300 Subject: [PATCH] [media] msp3400: fix mute audio regression To: linux-media@xxxxxxxxxxxxxxx The switch to the new control framework caused a regression where the audio was no longer unmuted after the carrier scan finished. The original code attempted to set the volume control to its current value in order to have the set-volume control code to be called that handles the volume and muting. However, the framework will not call that code unless the new volume value is different from the old. Instead we now call msp_s_ctrl directly. It is a bit of a hack: we really need a v4l2_ctrl_refresh_ctrl function for this (or something along those lines). Thanks to Andy Walls for bisecting this and to Shane Shrybman for reporting it! Reported-by: Shane Shrybman <shrybman@xxxxxxxxxxxx> Thanks-to: Andy Walls <awalls@xxxxxxxxxxxxxxxx> Signed-off-by: Hans Verkuil <hverkuil@xxxxxxxxx> Signed-off-by: Mauro Carvalho Chehab <mchehab@xxxxxxxxxx> diff --git a/drivers/media/video/msp3400-driver.c b/drivers/media/video/msp3400-driver.c index fe18a0a..b1763ac 100644 --- a/drivers/media/video/msp3400-driver.c +++ b/drivers/media/video/msp3400-driver.c @@ -381,7 +381,12 @@ static int msp_s_ctrl(struct v4l2_ctrl *ctrl) void msp_update_volume(struct msp_state *state) { - v4l2_ctrl_s_ctrl(state->volume, v4l2_ctrl_g_ctrl(state->volume)); + /* Force an update of the volume/mute cluster */ + v4l2_ctrl_lock(state->volume); + state->volume->val = state->volume->cur.val; + state->muted->val = state->muted->cur.val; + msp_s_ctrl(state->volume); + v4l2_ctrl_unlock(state->volume); } /* --- v4l2 ioctls --- */ -- 1.7.0.4 -- Hans Verkuil - video4linux developer - sponsored by Cisco -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html