public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
From: Gert Vervoort <gert.vervoort@hccnet.nl>
To: Mauro Carvalho Chehab <mchehab@infradead.org>
Cc: DVB ML <linux-dvb@linuxtv.org>,
	video4linux-list@redhat.com, Michael Krufky <mkrufky@linuxtv.org>
Subject: Re: Hauppauge WinTV regreession from 2.6.24 to 2.6.25
Date: Fri, 25 Apr 2008 18:55:49 +0200	[thread overview]
Message-ID: <48120D15.3010109@hccnet.nl> (raw)
In-Reply-To: <20080425120307.69a71e17@gaivota>

Mauro Carvalho Chehab wrote:
> On Fri, 25 Apr 2008 10:40:14 -0400
> "Michael Krufky" <mkrufky@linuxtv.org> wrote:
>
>   
>> On Fri, Apr 25, 2008 at 9:56 AM, Mauro Carvalho Chehab
>> <mchehab@infradead.org> wrote:
>>     
>>> On Thu, 24 Apr 2008 05:55:28 +0200
>>>  hermann pitton <hermann-pitton@arcor.de> wrote:
>>>
>>>  > > > >>>> I am testing a kernel upgrade from 2.6.24.to 2.6.25, and the drivers
>>>  > > > >>>> for   the Hauppauge WinTV appear to have suffered some regression
>>>  > > > >>>> between the two kernel versions.
>>>
>>>
>>>       
>>>> do you see the auto detection issue?
>>>>         
>>>  >
>>>  > Either tell it is just nothing, what I very seriously doubt, or please
>>>  > comment.
>>>  >
>>>  > I don't like to end up on LKML again getting told that written rules
>>>  > don't exist ;)
>>>
>>>  Sorry for now answer earlier. Too busy here, due to the merge window.
>>>
>>>  This seems to be an old bug. On several cases, tuner_type information came from
>>>  some sort of autodetection schema, but the proper setup is not sent to tuner.
>>>
>>>  Please test the enclosed patch. It warrants that TUNER_SET_TYPE_ADDR is called
>>>  at saa7134_board_init2() for all those boards:
>>>
>>>  SAA7134_BOARD_ADS_DUO_CARDBUS_PTV331
>>>  SAA7134_BOARD_ASUS_EUROPA2_HYBRID
>>>  SAA7134_BOARD_ASUSTeK_P7131_DUAL
>>>  SAA7134_BOARD_ASUSTeK_P7131_HYBRID_LNA
>>>  SAA7134_BOARD_AVERMEDIA_SUPER_007
>>>  SAA7134_BOARD_BEHOLD_COLUMBUS_TVFM
>>>  SAA7134_BOARD_BMK_MPEX_NOTUNER
>>>  SAA7134_BOARD_BMK_MPEX_TUNER
>>>  SAA7134_BOARD_CINERGY_HT_PCI
>>>  SAA7134_BOARD_CINERGY_HT_PCMCIA
>>>  SAA7134_BOARD_CREATIX_CTX953
>>>  SAA7134_BOARD_FLYDVBT_HYBRID_CARDBUS
>>>  SAA7134_BOARD_FLYDVB_TRIO
>>>  SAA7134_BOARD_HAUPPAUGE_HVR1110
>>>  SAA7134_BOARD_KWORLD_ATSC110
>>>  SAA7134_BOARD_KWORLD_DVBT_210
>>>  SAA7134_BOARD_MD7134
>>>  SAA7134_BOARD_MEDION_MD8800_QUADRO
>>>  SAA7134_BOARD_PHILIPS_EUROPA
>>>  SAA7134_BOARD_PHILIPS_TIGER
>>>  SAA7134_BOARD_PHILIPS_TIGER_S
>>>  SAA7134_BOARD_PINNACLE_PCTV_310i
>>>  SAA7134_BOARD_TEVION_DVBT_220RF
>>>  SAA7134_BOARD_TWINHAN_DTV_DVB_3056
>>>  SAA7134_BOARD_VIDEOMATE_DVBT_200
>>>  SAA7134_BOARD_VIDEOMATE_DVBT_200A
>>>  SAA7134_BOARD_VIDEOMATE_DVBT_300
>>>
>>>  It is important to test the above boards, to be sure that no regression is
>>>  caused.
>>>
>>>  Signed-off-by: Mauro Carvalho Chehab <mchehab@infradead.org>
>>>
>>>  diff -r 60110897e86a linux/drivers/media/video/saa7134/saa7134-cards.c
>>>  --- a/linux/drivers/media/video/saa7134/saa7134-cards.c Fri Apr 25 08:04:54 2008 -0300
>>>  +++ b/linux/drivers/media/video/saa7134/saa7134-cards.c Fri Apr 25 10:44:16 2008 -0300
>>>       
>> Mauro,
>>
>> I didn't review your patch yet, and it needs to be tested, however,
>> the bug reported in this thread deals with the same regression that
>> you are attempting to repair, but on the cx88 driver -- not the
>> saa7134 driver.
>>
>> Both drivers need to be tested to make sure that this regression has been fixed.
>>     
>
> Ok, this is a cx88 version. Of course, needs testing.
>
> Signed-off-by: Mauro Carvalho Chehab <mchehab@infradead.org>
>
> diff -r 5c9a4decb57b linux/drivers/media/video/cx88/cx88-cards.c
> --- a/linux/drivers/media/video/cx88/cx88-cards.c	Fri Apr 25 11:02:29 2008 -0300
> +++ b/linux/drivers/media/video/cx88/cx88-cards.c	Fri Apr 25 12:01:48 2008 -0300
> @@ -2495,26 +2495,27 @@
>  
>   
This does not make a difference for me:

tuner' 1-0043: chip found @ 0x86 (cx88[0])
tda9887 1-0043: tda988[5/6/7] found
All bytes are equal. It is not a TEA5767
tuner' 1-0060: chip found @ 0xc0 (cx88[0])
tuner-simple 1-0060: type set to 44 (Philips 4 in 1 (ATI TV Wonder 
Pro/Conexant))
cx88[0]: Leadtek Winfast 2000XP Expert config: tuner=38, eeprom[0]=0x01
input: cx88 IR (Leadtek Winfast 2000XP as /class/input/input6
cx88[0]/0: found at 0000:00:0a.0, rev: 5, irq: 18, latency: 32, mmio: 
0xe2000000
cx88[0]/0: registered device video0 [v4l2]
cx88[0]/0: registered device vbi0
cx88[0]/0: registered device radio0


The wrong tuner type has already been set, before the eeprom with the 
correct tuner type is read.

   Gert


--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list

  reply	other threads:[~2008-04-25 16:56 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-04-19 20:57 Hauppauge WinTV regreession from 2.6.24 to 2.6.25 Ian Pickworth
2008-04-20  0:47 ` hermann pitton
2008-04-20  9:15   ` Ian Pickworth
2008-04-20 11:20 ` Gert Vervoort
2008-04-20 12:26   ` Ian Pickworth
2008-04-20 13:06     ` hermann pitton
2008-04-20 16:18       ` Gert Vervoort
2008-04-20 21:16         ` hermann pitton
2008-04-24  3:55           ` hermann pitton
2008-04-25 13:56             ` Mauro Carvalho Chehab
2008-04-25 14:40               ` Michael Krufky
2008-04-25 14:45                 ` Mauro Carvalho Chehab
2008-04-25 15:06                   ` mkrufky
2008-04-25 21:48                     ` hermann pitton
2008-04-25 23:41                       ` hermann pitton
2008-04-26 11:59                     ` Mauro Carvalho Chehab
2008-04-26 12:58                       ` Ian Pickworth
2008-04-26 14:06                         ` Mauro Carvalho Chehab
2008-04-26 22:10                           ` hermann pitton
2008-04-26 23:19                             ` Mauro Carvalho Chehab
2008-04-27 20:15                               ` hermann pitton
2008-04-27 21:18                                 ` [linux-dvb] " Hartmut Hackmann
2008-04-28  1:01                                   ` hermann pitton
2008-04-28 14:21                                   ` Mauro Carvalho Chehab
2008-04-28 14:14                                 ` Mauro Carvalho Chehab
2008-04-25 15:03                 ` Mauro Carvalho Chehab
2008-04-25 16:55                   ` Gert Vervoort [this message]
2008-04-25 17:02                     ` Gert Vervoort
     [not found]                     ` <20080426090725.4a0fdcd4@gaivota>
2008-04-26 14:23                       ` Gert Vervoort
2008-04-26 15:38                         ` Mauro Carvalho Chehab
2008-04-26 16:59                           ` Gert Vervoort

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=48120D15.3010109@hccnet.nl \
    --to=gert.vervoort@hccnet.nl \
    --cc=linux-dvb@linuxtv.org \
    --cc=mchehab@infradead.org \
    --cc=mkrufky@linuxtv.org \
    --cc=video4linux-list@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox