diff options
| author | Devin Heitmueller <dheitmueller@linuxtv.org> | 2009-03-11 02:01:01 -0400 |
|---|---|---|
| committer | Devin Heitmueller <dheitmueller@linuxtv.org> | 2009-03-11 02:01:01 -0400 |
| commit | 3f7bd4e3eef22d1416780db3ae3300c16bc21961 (patch) | |
| tree | 187b6a24a87d379e0aaa84d9a1de894002828b79 /linux/drivers/media/video/au0828/au0828-cards.c | |
| parent | 954299a93e6d076c844770136c359f48049b3d88 (diff) | |
| download | mediapointer-dvb-s2-3f7bd4e3eef22d1416780db3ae3300c16bc21961.tar.gz mediapointer-dvb-s2-3f7bd4e3eef22d1416780db3ae3300c16bc21961.tar.bz2 | |
au0828: properly handle non-existent analog inputs
From: Devin Heitmueller <dheitmueller@linuxtv.org>
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.
Priority: normal
Signed-off-by: Devin Heitmueller <dheitmueller@linuxtv.org>
Signed-off-by: Michael Krufky <mkrufky@linuxtv.org>
Diffstat (limited to 'linux/drivers/media/video/au0828/au0828-cards.c')
| -rw-r--r-- | linux/drivers/media/video/au0828/au0828-cards.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/linux/drivers/media/video/au0828/au0828-cards.c b/linux/drivers/media/video/au0828/au0828-cards.c index e10b1b922..0516c0608 100644 --- a/linux/drivers/media/video/au0828/au0828-cards.c +++ b/linux/drivers/media/video/au0828/au0828-cards.c @@ -195,7 +195,7 @@ void au0828_card_setup(struct au0828_dev *dev) break; } - if (dev->board.input != NULL) { + if (AUVI_INPUT(0).type != AU0828_VMUX_UNDEFINED) { /* Load the analog demodulator driver (note this would need to be abstracted out if we ever need to support a different demod) */ |