From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from smtp-vbr10.xs4all.nl ([194.109.24.30]:4541 "EHLO smtp-vbr10.xs4all.nl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752831Ab3KKLOs (ORCPT ); Mon, 11 Nov 2013 06:14:48 -0500 Message-ID: <5280BC1A.7010701@xs4all.nl> Date: Mon, 11 Nov 2013 12:14:34 +0100 From: Hans Verkuil MIME-Version: 1.0 To: Andy Walls CC: Rajil Saraswat , linux-media@vger.kernel.org Subject: Re: ivtv 1.4.2/1.4.3 broken in recent kernels? References: <1381371651.1889.21.camel@palomino.walls.org> <1381620192.22245.18.camel@palomino.walls.org> <1381668541.2209.14.camel@palomino.walls.org> <1381707800.1875.63.camel@palomino.walls.org> <1382202581.2405.5.camel@palomino.walls.org> In-Reply-To: <1382202581.2405.5.camel@palomino.walls.org> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Sender: linux-media-owner@vger.kernel.org List-ID: On 10/19/2013 07:09 PM, Andy Walls wrote: > On Wed, 2013-10-16 at 01:10 +0100, Rajil Saraswat wrote: >> I was finally able to carry out a git bisect. Had to do a git pull on >> a fast internet hooked machine and ftp the files over to the remote >> machine. >> >> I started with 'git bisect bad v2.6.36.4' and 'git bisect good v2.6.35.10'. >> >> And the result was: >> >> 5aa9ae5ed5d449a85fbf7aac3d1fdc241c542a79 is the first bad commit >> commit 5aa9ae5ed5d449a85fbf7aac3d1fdc241c542a79 >> Author: Hans Verkuil >> Date: Sat Apr 24 08:23:53 2010 -0300 >> >> V4L/DVB: wm8775: convert to the new control framework >> >> Signed-off-by: Hans Verkuil >> Signed-off-by: Mauro Carvalho Chehab >> >> :040000 040000 37847ffe592f255c6a9d9daedaf7bbfd3cd7b055 >> 2f094df6f65d7fb296657619c1ad6f93fe085a75 M drivers >> >> I then removed the patch from linux-2.6.36-gentoo-r8 which are gentoo >> sources, and confirmed that video/audio now works fine on v4l2-ctl -d >> /dev/video1 --set-input 4 >> >> I wasnt able to remove the patch in 3.10.7 which is gentoo stable >> kernel. Any idea how can i do that? > > Try applying the following (untested) patch that is made against the > bleeding edge Linux kernel. The test on the mute control state in > wm8775_s_routing() appears to have been inverted in the bad commit you > isolated. Can you give me your Signed-off-by? Thanks! Hans > > Along with '--set-input', you may also want to use v4l2-ctl to exercise > the mute control as well, to see if it works as expected, once this > patch is applied. > > Regards, > Andy > > file: wm8775_s_route_mute_test_inverted.patch > > diff --git a/drivers/media/i2c/wm8775.c b/drivers/media/i2c/wm8775.c > index 3f584a7..bee7946 100644 > --- a/drivers/media/i2c/wm8775.c > +++ b/drivers/media/i2c/wm8775.c > @@ -130,12 +130,10 @@ static int wm8775_s_routing(struct v4l2_subdev *sd, > return -EINVAL; > } > state->input = input; > - if (!v4l2_ctrl_g_ctrl(state->mute)) > + if (v4l2_ctrl_g_ctrl(state->mute)) > return 0; > if (!v4l2_ctrl_g_ctrl(state->vol)) > return 0; > - if (!v4l2_ctrl_g_ctrl(state->bal)) > - return 0; > wm8775_set_audio(sd, 1); > return 0; > } > >