Commit ba3a5a81 authored by Michael Krufky's avatar Michael Krufky Committed by Linus Torvalds

V4L/DVB (3504): Medion 7134: Autodetect second bridge chip

The device, Medion 7134, has two saa7134 chips on it, but only one of them
is functional in the current saa7134 driver.
This patch adds autodetection for the second, unsupported saa7134 chip,
as SAA7134_BOARD_MD7134_BRIDGE_2, and displays a message to the user
(in dmesg) indicating that the second chip isn't yet functional.
This is useful for users, since two instances of the saa7134 driver
will spawn. This patch will prevent confusion by warning the user that
only one of the chips on the board are functional.
There are other versions of the SAA7134_BOARD_MD7134 with only a single
saa7134 bridge/decoder -- those devices will not be affected by this patch.
Only devices containing the second chip will display the warning.
Signed-off-by: default avatarMichael Krufky <mkrufky@linuxtv.org>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@infradead.org>
parent 67ac9a40
...@@ -91,3 +91,4 @@ ...@@ -91,3 +91,4 @@
90 -> Kworld ATSC110 [17de:7350] 90 -> Kworld ATSC110 [17de:7350]
91 -> AVerMedia A169 B [1461:7360] 91 -> AVerMedia A169 B [1461:7360]
92 -> AVerMedia A169 B1 [1461:6360] 92 -> AVerMedia A169 B1 [1461:6360]
93 -> Medion 7134 Bridge #2 [16be:0005]
...@@ -2797,6 +2797,17 @@ struct saa7134_board saa7134_boards[] = { ...@@ -2797,6 +2797,17 @@ struct saa7134_board saa7134_boards[] = {
.amux = LINE1, .amux = LINE1,
}}, }},
}, },
[SAA7134_BOARD_MD7134_BRIDGE_2] = {
/* This card has two saa7134 chips on it,
but only one of them is currently working.
The programming for the primary decoder is
in SAA7134_BOARD_MD7134 */
.name = "Medion 7134 Bridge #2",
.audio_clock = 0x00187de7,
.radio_type = UNSET,
.tuner_addr = ADDR_UNSET,
.radio_addr = ADDR_UNSET,
},
}; };
const unsigned int saa7134_bcount = ARRAY_SIZE(saa7134_boards); const unsigned int saa7134_bcount = ARRAY_SIZE(saa7134_boards);
...@@ -3315,6 +3326,12 @@ struct pci_device_id saa7134_pci_tbl[] = { ...@@ -3315,6 +3326,12 @@ struct pci_device_id saa7134_pci_tbl[] = {
.subvendor = 0x1461, .subvendor = 0x1461,
.subdevice = 0x6360, .subdevice = 0x6360,
.driver_data = SAA7134_BOARD_AVERMEDIA_A169_B1, .driver_data = SAA7134_BOARD_AVERMEDIA_A169_B1,
},{
.vendor = PCI_VENDOR_ID_PHILIPS,
.device = PCI_DEVICE_ID_PHILIPS_SAA7134,
.subvendor = 0x16be,
.subdevice = 0x0005,
.driver_data = SAA7134_BOARD_MD7134_BRIDGE_2,
},{ },{
/* --- boards without eeprom + subsystem ID --- */ /* --- boards without eeprom + subsystem ID --- */
.vendor = PCI_VENDOR_ID_PHILIPS, .vendor = PCI_VENDOR_ID_PHILIPS,
...@@ -3471,6 +3488,7 @@ int saa7134_board_init1(struct saa7134_dev *dev) ...@@ -3471,6 +3488,7 @@ int saa7134_board_init1(struct saa7134_dev *dev)
dev->has_remote = SAA7134_REMOTE_I2C; dev->has_remote = SAA7134_REMOTE_I2C;
break; break;
case SAA7134_BOARD_AVERMEDIA_A169_B: case SAA7134_BOARD_AVERMEDIA_A169_B:
case SAA7134_BOARD_MD7134_BRIDGE_2:
printk("%s: %s: dual saa713x broadcast decoders\n" printk("%s: %s: dual saa713x broadcast decoders\n"
"%s: Sorry, none of the inputs to this chip are supported yet.\n" "%s: Sorry, none of the inputs to this chip are supported yet.\n"
"%s: Dual decoder functionality is disabled for now, use the other chip.\n", "%s: Dual decoder functionality is disabled for now, use the other chip.\n",
......
...@@ -220,6 +220,7 @@ struct saa7134_format { ...@@ -220,6 +220,7 @@ struct saa7134_format {
#define SAA7134_BOARD_KWORLD_ATSC110 90 #define SAA7134_BOARD_KWORLD_ATSC110 90
#define SAA7134_BOARD_AVERMEDIA_A169_B 91 #define SAA7134_BOARD_AVERMEDIA_A169_B 91
#define SAA7134_BOARD_AVERMEDIA_A169_B1 92 #define SAA7134_BOARD_AVERMEDIA_A169_B1 92
#define SAA7134_BOARD_MD7134_BRIDGE_2 93
#define SAA7134_MAXBOARDS 8 #define SAA7134_MAXBOARDS 8
#define SAA7134_INPUT_MAX 8 #define SAA7134_INPUT_MAX 8
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment