From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail-wi0-f174.google.com ([209.85.212.174]:38339 "EHLO mail-wi0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752642AbbGUVdT (ORCPT ); Tue, 21 Jul 2015 17:33:19 -0400 Received: by wibxm9 with SMTP id xm9so74020830wib.1 for ; Tue, 21 Jul 2015 14:33:18 -0700 (PDT) Message-ID: <55AEBA9C.6010800@gmail.com> Date: Tue, 21 Jul 2015 23:33:16 +0200 From: =?UTF-8?B?VHljaG8gTMO8cnNlbg==?= MIME-Version: 1.0 To: Steven Toth CC: tonyc@wincomm.com.tw, Antti Palosaari , Linux-Media Subject: Re: Adding support for three new Hauppauge HVR-1275 variants - testers reqd. References: <55AB5320.8030100@gmail.com> <55AD1C77.6030208@gmail.com> <55AE6F31.3050308@gmail.com> <55AE8A73.9070802@gmail.com> In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit Sender: linux-media-owner@vger.kernel.org List-ID: Op 21-07-15 om 21:00 schreef Steven Toth: > On Tue, Jul 21, 2015 at 2:07 PM, Tycho Lürsen wrote: >> >> Op 21-07-15 om 18:19 schreef Steven Toth: >>> On Tue, Jul 21, 2015 at 12:11 PM, Tycho Lürsen >>> wrote: >>>> Hi Steven, >>>> I was too curious to wait for you and Antti to settle your differences, >>>> so I >>>> tested again against a 4.2-RC2 >>>> I did not disable DVB-T/T2, instead I reordered the lot. MythTV just sees >>>> the first system in the .delsys line in si2168.c, >>>> so when it looks like this: >>>> SYS_DVBC_ANNEX_A, SYS_DVBT, SYS_DVBT2 >>>> I'm good. >>> We have no differences, its Antti's si2168 driver. If Antti doesn't >>> like the approach for tri-stating, he's free to suggest and >>> alternative. I suggested two alternatives yesterday. >>> >>>> Result: >>>> With your patch both MythTV and Tvheadend still can't tune. Without it, >>>> everything is ok. >>>> >>>> I'm not very interested in czap results, only in real use cases. For me >>>> that's MythTV, but just to be sure I also tested with TVheadend. >>> That's pretty bizarre results, although thank you for testing. :) >>> >>> When you say it can't tune, do you mean the signal does not lock, or >>> that no video appears? >>> >> No lock, or partial lock. > Thanks. > > That's even worse than expected, given that the patch adjusts the TS > interface, and has nothing to do with tuning, lock, rf or signal > status. It still feels like something else is going on, some other > unexpected race for example. > > I can't reproduce that behavior, but given that you can.... Can you > please try this? in si2168.c, change: > > /* Tri-state the TS bus */ > si2168_set_ts_mode(fe, 1); > > to > > /* Tri-state the TS bus */ > si2168_set_ts_mode(fe, 0); > > ... recompile and retest? > > Thx. > I recompiled and tested with this change (still without saa716x or dvbloopback) with TVheadend and all is well. Going to test this with dvbloopback, and if all goes well with saa716x too. But not today. Regards, Tycho.