diff options
author | Devin Heitmueller <dheitmueller@linuxtv.org> | 2009-03-11 02:01:01 -0400 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2009-03-30 11:43:26 -0400 |
commit | 220be77c6e4302207c34afc78a98b4994f92f6e5 (patch) | |
tree | 718563a237cfe56fb495d195092929446a1e725e /drivers/media | |
parent | fc4ce6cd9855dcd1151a9afc067ea354179089f9 (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')
-rw-r--r-- | drivers/media/video/au0828/au0828-cards.c | 2 | ||||
-rw-r--r-- | drivers/media/video/au0828/au0828-core.c | 4 | ||||
-rw-r--r-- | drivers/media/video/au0828/au0828.h | 3 |
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 e10b1b9221e0..0516c060810e 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 2b5ade626e57..5199c8aa5888 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 d2e54c8e18c4..590d15e461dc 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 | ||
68 | enum au0828_itype { | 68 | enum 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, |