All of lore.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.