Re: [PATCH] warm boot for Compro VideoMate DVB-T300

[Hartmut Hackmann <hartmut.hackmann@xxxxxxxxxxx>]

Hi,

James Lawrence schrieb:
> Hi all,
> 
> in my experience, the Compro VideoMate DVB-T300 shows a different vendor
> ID on warm boot compared to cold boot. The following patch allows the
> board to be correctly recognized on warm boot. I have tested this with
> two different TV cards, on 3 motherboards, with multiple distributions
> and on many kernel revisions from 2.6.15 to 2.6.23.
> 
> [James@mythtv v4l-dvb]$ hg diff
> diff -r 27b2c6a80826 linux/drivers/media/video/saa7134/saa7134-cards.c
> --- a/linux/drivers/media/video/saa7134/saa7134-cards.c    Fri Nov 30
> 18:27:26 2007 +0200
> +++ b/linux/drivers/media/video/saa7134/saa7134-cards.c    Sun Dec 02
> 22:58:36 2007 +1100
> @@ -4027,6 +4027,13 @@ struct pci_device_id saa7134_pci_tbl[] =
>          .device       = PCI_DEVICE_ID_PHILIPS_SAA7134,
>          .subvendor    = 0x185b,
>          .subdevice    = 0xc900,
> +        .driver_data  = SAA7134_BOARD_VIDEOMATE_DVBT_300,
> +    },{
> +        /* deal with warm boot */
> +        .vendor       = PCI_VENDOR_ID_PHILIPS,
> +        .device       = PCI_DEVICE_ID_PHILIPS_SAA7134,
> +        .subvendor    = 0x1850,
> +        .subdevice    = 0x0000,
>          .driver_data  = SAA7134_BOARD_VIDEOMATE_DVBT_300,
>      },{
>          .vendor       = PCI_VENDOR_ID_PHILIPS,
> 
> Signed-off-by: James Lawrence <james.lawrence@xxxxxxxxx>
> 
> -- 
> video4linux-list mailing list
> Unsubscribe mailto:video4linux-list-request@xxxxxxxxxx?subject=unsubscribe
> https://www.redhat.com/mailman/listinfo/video4linux-list
> 
I just read this. This works around a side effect: When i added support
for this card, there was no means to control the I2C bridge of the tda10046
in analog mode, so i left it transparent. Unfortunately, this causes a collision
on the I2C bus during reboot (the 10046 is not reset!) If this patch does not
cause a collision, we can apply it, but please let's mark it as a "hack".
IMHO, its cleaner to force this card type.

Best regards
  Hartmut

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