Re: Hauppauge WinTV regreession from 2.6.24 to 2.6.25

[Mauro Carvalho Chehab <mchehab@infradead.org>]

On Sun, 27 Apr 2008 00:10:21 +0200
hermann pitton <hermann-pitton@arcor.de> wrote:
> Cool stuff!
> 
> Works immediately for all tuners again. Analog TV, radio and DVB-T on
> that machine is tested.
> 
> Reviewed-by: Hermann Pitton <hermann-pitton@arcor.de>

Thanks. I'll add it to the patch.

> Maybe Hartmut can help too, but I will test also on the triple stuff and
> the FMD1216ME/I MK3 hybrid tomorrow.

Thanks.

It would be helpful if tda9887 conf could also be validated. I didn't touch at
the logic, but I saw some weird things:

For example, SAA7134_BOARD_PHILIPS_EUROPA defines this:
	.tda9887_conf   = TDA9887_PRESENT | TDA9887_PORT1_ACTIVE

And SAA7134_BOARD_PHILIPS_SNAKE keep the default values.

However, there's an autodetection code that changes from EUROPA to SNAKE,
without cleaning tda9887_conf:

        case SAA7134_BOARD_PHILIPS_EUROPA:
                if (dev->autodetected && (dev->eedata[0x41] == 0x1c)) {
                        /* Reconfigure board as Snake reference design */
                        dev->board = SAA7134_BOARD_PHILIPS_SNAKE;
                        dev->tuner_type = saa7134_boards[dev->board].tuner_type;
                        printk(KERN_INFO "%s: Reconfigured board as %s\n",
                                dev->name, saa7134_boards[dev->board].name);
                        break;

I'm not sure if .tda9887_conf is missing at SNAKE board entry, or if the above
code should be doing, instead:

	dev->tda9887_conf = saa7134_boards[dev->board].tda9887_conf;

If the right thing to do is to initialize SNAKE with the same tda9887
parameters as EUROPE, the better would be to add the .tda9887_conf to SNAKE
entry.

Cheers,
Mauro

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