aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/media/video/au0828
diff options
context:
space:
mode:
authorDevin Heitmueller <dheitmueller@linuxtv.org>2009-03-11 02:01:01 -0400
committerMauro Carvalho Chehab <mchehab@redhat.com>2009-03-30 11:43:26 -0400
commit220be77c6e4302207c34afc78a98b4994f92f6e5 (patch)
tree718563a237cfe56fb495d195092929446a1e725e /drivers/media/video/au0828
parentfc4ce6cd9855dcd1151a9afc067ea354179089f9 (diff)
V4L/DVB (11078): au0828: properly handle non-existent analog inputs
It is not valid to look for dev->board.input == NULL to detect an undefined analog configuration section, since it is a member of the struct and not a pointer (hence it will *always* be non-NULL). Do the check based on whether the first input is actually a valid input type instead. Thanks to Michael Krufky <mkrufky@linuxtv.org> for providing sample hardware of various configurations to test with. Signed-off-by: Devin Heitmueller <dheitmueller@linuxtv.org> Signed-off-by: Michael Krufky <mkrufky@linuxtv.org> Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'drivers/media/video/au0828')
-rw-r--r--drivers/media/video/au0828/au0828-cards.c2
-rw-r--r--drivers/media/video/au0828/au0828-core.c4
-rw-r--r--drivers/media/video/au0828/au0828.h3
3 files changed, 5 insertions, 4 deletions
diff --git a/drivers/media/video/au0828/au0828-cards.c b/drivers/media/video/au0828/au0828-cards.c
index e10b1b9221e..0516c060810 100644
--- a/drivers/media/video/au0828/au0828-cards.c
+++ b/drivers/media/video/au0828/au0828-cards.c
@@ -195,7 +195,7 @@ void au0828_card_setup(struct au0828_dev *dev)
195 break; 195 break;
196 } 196 }
197 197
198 if (dev->board.input != NULL) { 198 if (AUVI_INPUT(0).type != AU0828_VMUX_UNDEFINED) {
199 /* Load the analog demodulator driver (note this would need to 199 /* Load the analog demodulator driver (note this would need to
200 be abstracted out if we ever need to support a different 200 be abstracted out if we ever need to support a different
201 demod) */ 201 demod) */
diff --git a/drivers/media/video/au0828/au0828-core.c b/drivers/media/video/au0828/au0828-core.c
index 2b5ade626e5..5199c8aa588 100644
--- a/drivers/media/video/au0828/au0828-core.c
+++ b/drivers/media/video/au0828/au0828-core.c
@@ -146,7 +146,7 @@ static void au0828_usb_disconnect(struct usb_interface *interface)
146 /* Digital TV */ 146 /* Digital TV */
147 au0828_dvb_unregister(dev); 147 au0828_dvb_unregister(dev);
148 148
149 if (dev->board.input != NULL) 149 if (AUVI_INPUT(0).type != AU0828_VMUX_UNDEFINED)
150 au0828_analog_unregister(dev); 150 au0828_analog_unregister(dev);
151 151
152 /* I2C */ 152 /* I2C */
@@ -205,7 +205,7 @@ static int au0828_usb_probe(struct usb_interface *interface,
205 au0828_card_setup(dev); 205 au0828_card_setup(dev);
206 206
207 /* Analog TV */ 207 /* Analog TV */
208 if (dev->board.input != NULL) 208 if (AUVI_INPUT(0).type != AU0828_VMUX_UNDEFINED)
209 au0828_analog_register(dev, interface); 209 au0828_analog_register(dev, interface);
210 210
211 /* Digital TV */ 211 /* Digital TV */
diff --git a/drivers/media/video/au0828/au0828.h b/drivers/media/video/au0828/au0828.h
index d2e54c8e18c..590d15e461d 100644
--- a/drivers/media/video/au0828/au0828.h
+++ b/drivers/media/video/au0828/au0828.h
@@ -66,7 +66,8 @@
66#define AU0828_MAX_INPUT 4 66#define AU0828_MAX_INPUT 4
67 67
68enum au0828_itype { 68enum au0828_itype {
69 AU0828_VMUX_COMPOSITE = 1, 69 AU0828_VMUX_UNDEFINED = 0,
70 AU0828_VMUX_COMPOSITE,
70 AU0828_VMUX_SVIDEO, 71 AU0828_VMUX_SVIDEO,
71 AU0828_VMUX_CABLE, 72 AU0828_VMUX_CABLE,
72 AU0828_VMUX_TELEVISION, 73 AU0828_VMUX_TELEVISION,