From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail.kapsi.fi ([217.30.184.167]:37476 "EHLO mail.kapsi.fi" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932876Ab2JEUzI (ORCPT ); Fri, 5 Oct 2012 16:55:08 -0400 Message-ID: <506F4915.1090908@iki.fi> Date: Fri, 05 Oct 2012 23:54:45 +0300 From: Antti Palosaari MIME-Version: 1.0 To: Michael Krufky CC: linux-media@vger.kernel.org Subject: Re: [PATCH] mxl111sf: revert patch: fix error on stream stop in mxl111sf_ep6_streaming_ctrl() References: <1349469857-21396-1-git-send-email-crope@iki.fi> In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-media-owner@vger.kernel.org List-ID: On 10/05/2012 11:49 PM, Michael Krufky wrote: > On Fri, Oct 5, 2012 at 4:44 PM, Antti Palosaari wrote: >> This reverts commits: >> 3fd7e4341e04f80e2605f56bbd8cb1e8b027901a >> [media] mxl111sf: remove an unused variable >> 3be5bb71fbf18f83cb88b54a62a78e03e5a4f30a >> [media] mxl111sf: fix error on stream stop in mxl111sf_ep6_streaming_ctrl() >> >> ...as bug behind these is fixed by the DVB USB v2. >> >> Cc: Michael Krufky >> Signed-off-by: Antti Palosaari >> --- >> drivers/media/usb/dvb-usb-v2/mxl111sf.c | 7 +++++-- >> 1 file changed, 5 insertions(+), 2 deletions(-) >> >> diff --git a/drivers/media/usb/dvb-usb-v2/mxl111sf.c b/drivers/media/usb/dvb-usb-v2/mxl111sf.c >> index efdcb15..fcfe124 100644 >> --- a/drivers/media/usb/dvb-usb-v2/mxl111sf.c >> +++ b/drivers/media/usb/dvb-usb-v2/mxl111sf.c >> @@ -343,6 +343,7 @@ static int mxl111sf_ep6_streaming_ctrl(struct dvb_frontend *fe, int onoff) >> struct mxl111sf_state *state = fe_to_priv(fe); >> struct mxl111sf_adap_state *adap_state = &state->adap_state[fe->id]; >> int ret = 0; >> + u8 tmp; >> >> deb_info("%s(%d)\n", __func__, onoff); >> >> @@ -353,13 +354,15 @@ static int mxl111sf_ep6_streaming_ctrl(struct dvb_frontend *fe, int onoff) >> adap_state->ep6_clockphase, >> 0, 0); >> mxl_fail(ret); >> -#if 0 >> } else { >> ret = mxl111sf_disable_656_port(state); >> mxl_fail(ret); >> -#endif >> } >> >> + mxl111sf_read_reg(state, 0x12, &tmp); >> + tmp &= ~0x04; >> + mxl111sf_write_reg(state, 0x12, tmp); >> + >> return ret; >> } >> > > > I disabled that code on purpose - its redundant. please do not apply > this patch. According to comments you have added patch changelog you disabled it doe to that bug: [media] mxl111sf: fix error on stream stop in mxl111sf_ep6_streaming_ctrl() Remove unnecessary register access in mxl111sf_ep6_streaming_ctrl() This code breaks driver operation in kernel 3.3 and later, although it works properly in 3.2 Disable register access to 0x12 for now. are you saying there is some other reason than mentioned here? I am quite 100% sure I fixed that bug in dvb-usb. regards Antti -- http://palosaari.fi/