diff options
author | Mike Isely <isely@pobox.com> | 2006-09-02 14:56:33 -0400 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@infradead.org> | 2006-09-26 11:30:35 -0400 |
commit | 7f19d0278c86c7f27df287fd0bd5170fe75dc7d2 (patch) | |
tree | a3ff18cd443d79615fb23b748e4563a91c666907 | |
parent | 4d31256f378e1a5fca7f4f3fb8b0a0aa1fff4312 (diff) |
V4L/DVB (4595): Remove CONFIG_VIDEO_PVRUSB2_24XXX from pvrusb2 driver
Support for 24xxx devices was previously explicitly bracketed with
CONFIG_VIDEO_PVRUSB2_24XXX inside the code because we didn't trust the
stability of these changes. We trust it now; so there's no reason to
leave this out of the driver anymore.
Signed-off-by: Mike Isely <isely@pobox.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@infradead.org>
-rw-r--r-- | drivers/media/video/pvrusb2/pvrusb2-hdw-internal.h | 2 | ||||
-rw-r--r-- | drivers/media/video/pvrusb2/pvrusb2-hdw.c | 16 | ||||
-rw-r--r-- | drivers/media/video/pvrusb2/pvrusb2-i2c-chips-v4l2.c | 4 | ||||
-rw-r--r-- | drivers/media/video/pvrusb2/pvrusb2-i2c-core.c | 6 |
4 files changed, 0 insertions, 28 deletions
diff --git a/drivers/media/video/pvrusb2/pvrusb2-hdw-internal.h b/drivers/media/video/pvrusb2/pvrusb2-hdw-internal.h index 74c125ad4071..34b08fbcc6ea 100644 --- a/drivers/media/video/pvrusb2/pvrusb2-hdw-internal.h +++ b/drivers/media/video/pvrusb2/pvrusb2-hdw-internal.h | |||
@@ -168,9 +168,7 @@ struct pvr2_decoder_ctrl { | |||
168 | 168 | ||
169 | /* Known major hardware variants, keyed from device ID */ | 169 | /* Known major hardware variants, keyed from device ID */ |
170 | #define PVR2_HDW_TYPE_29XXX 0 | 170 | #define PVR2_HDW_TYPE_29XXX 0 |
171 | #ifdef CONFIG_VIDEO_PVRUSB2_24XXX | ||
172 | #define PVR2_HDW_TYPE_24XXX 1 | 171 | #define PVR2_HDW_TYPE_24XXX 1 |
173 | #endif | ||
174 | 172 | ||
175 | typedef int (*pvr2_i2c_func)(struct pvr2_hdw *,u8,u8 *,u16,u8 *, u16); | 173 | typedef int (*pvr2_i2c_func)(struct pvr2_hdw *,u8,u8 *,u16,u8 *, u16); |
176 | #define PVR2_I2C_FUNC_CNT 128 | 174 | #define PVR2_I2C_FUNC_CNT 128 |
diff --git a/drivers/media/video/pvrusb2/pvrusb2-hdw.c b/drivers/media/video/pvrusb2/pvrusb2-hdw.c index 199b5e2e5c43..6ab6fd187340 100644 --- a/drivers/media/video/pvrusb2/pvrusb2-hdw.c +++ b/drivers/media/video/pvrusb2/pvrusb2-hdw.c | |||
@@ -38,9 +38,7 @@ | |||
38 | 38 | ||
39 | struct usb_device_id pvr2_device_table[] = { | 39 | struct usb_device_id pvr2_device_table[] = { |
40 | [PVR2_HDW_TYPE_29XXX] = { USB_DEVICE(0x2040, 0x2900) }, | 40 | [PVR2_HDW_TYPE_29XXX] = { USB_DEVICE(0x2040, 0x2900) }, |
41 | #ifdef CONFIG_VIDEO_PVRUSB2_24XXX | ||
42 | [PVR2_HDW_TYPE_24XXX] = { USB_DEVICE(0x2040, 0x2400) }, | 41 | [PVR2_HDW_TYPE_24XXX] = { USB_DEVICE(0x2040, 0x2400) }, |
43 | #endif | ||
44 | { } | 42 | { } |
45 | }; | 43 | }; |
46 | 44 | ||
@@ -48,9 +46,7 @@ MODULE_DEVICE_TABLE(usb, pvr2_device_table); | |||
48 | 46 | ||
49 | static const char *pvr2_device_names[] = { | 47 | static const char *pvr2_device_names[] = { |
50 | [PVR2_HDW_TYPE_29XXX] = "WinTV PVR USB2 Model Category 29xxxx", | 48 | [PVR2_HDW_TYPE_29XXX] = "WinTV PVR USB2 Model Category 29xxxx", |
51 | #ifdef CONFIG_VIDEO_PVRUSB2_24XXX | ||
52 | [PVR2_HDW_TYPE_24XXX] = "WinTV PVR USB2 Model Category 24xxxx", | 49 | [PVR2_HDW_TYPE_24XXX] = "WinTV PVR USB2 Model Category 24xxxx", |
53 | #endif | ||
54 | }; | 50 | }; |
55 | 51 | ||
56 | struct pvr2_string_table { | 52 | struct pvr2_string_table { |
@@ -58,14 +54,12 @@ struct pvr2_string_table { | |||
58 | unsigned int cnt; | 54 | unsigned int cnt; |
59 | }; | 55 | }; |
60 | 56 | ||
61 | #ifdef CONFIG_VIDEO_PVRUSB2_24XXX | ||
62 | // Names of other client modules to request for 24xxx model hardware | 57 | // Names of other client modules to request for 24xxx model hardware |
63 | static const char *pvr2_client_24xxx[] = { | 58 | static const char *pvr2_client_24xxx[] = { |
64 | "cx25840", | 59 | "cx25840", |
65 | "tuner", | 60 | "tuner", |
66 | "wm8775", | 61 | "wm8775", |
67 | }; | 62 | }; |
68 | #endif | ||
69 | 63 | ||
70 | // Names of other client modules to request for 29xxx model hardware | 64 | // Names of other client modules to request for 29xxx model hardware |
71 | static const char *pvr2_client_29xxx[] = { | 65 | static const char *pvr2_client_29xxx[] = { |
@@ -79,12 +73,10 @@ static struct pvr2_string_table pvr2_client_lists[] = { | |||
79 | pvr2_client_29xxx, | 73 | pvr2_client_29xxx, |
80 | sizeof(pvr2_client_29xxx)/sizeof(pvr2_client_29xxx[0]), | 74 | sizeof(pvr2_client_29xxx)/sizeof(pvr2_client_29xxx[0]), |
81 | }, | 75 | }, |
82 | #ifdef CONFIG_VIDEO_PVRUSB2_24XXX | ||
83 | [PVR2_HDW_TYPE_24XXX] = { | 76 | [PVR2_HDW_TYPE_24XXX] = { |
84 | pvr2_client_24xxx, | 77 | pvr2_client_24xxx, |
85 | sizeof(pvr2_client_24xxx)/sizeof(pvr2_client_24xxx[0]), | 78 | sizeof(pvr2_client_24xxx)/sizeof(pvr2_client_24xxx[0]), |
86 | }, | 79 | }, |
87 | #endif | ||
88 | }; | 80 | }; |
89 | 81 | ||
90 | static struct pvr2_hdw *unit_pointers[PVR_NUM] = {[ 0 ... PVR_NUM-1 ] = NULL}; | 82 | static struct pvr2_hdw *unit_pointers[PVR_NUM] = {[ 0 ... PVR_NUM-1 ] = NULL}; |
@@ -363,7 +355,6 @@ static int ctrl_freq_set(struct pvr2_ctrl *cptr,int m,int v) | |||
363 | return 0; | 355 | return 0; |
364 | } | 356 | } |
365 | 357 | ||
366 | #ifdef CONFIG_VIDEO_PVRUSB2_24XXX | ||
367 | static int ctrl_hres_max_get(struct pvr2_ctrl *cptr,int *vp) | 358 | static int ctrl_hres_max_get(struct pvr2_ctrl *cptr,int *vp) |
368 | { | 359 | { |
369 | /* If we're dealing with a 24xxx device, force the horizontal | 360 | /* If we're dealing with a 24xxx device, force the horizontal |
@@ -385,7 +376,6 @@ static int ctrl_hres_min_get(struct pvr2_ctrl *cptr,int *vp) | |||
385 | if (cptr->hdw->hdw_type == PVR2_HDW_TYPE_24XXX) *vp = 720; | 376 | if (cptr->hdw->hdw_type == PVR2_HDW_TYPE_24XXX) *vp = 720; |
386 | return 0; | 377 | return 0; |
387 | } | 378 | } |
388 | #endif | ||
389 | 379 | ||
390 | static int ctrl_cx2341x_is_dirty(struct pvr2_ctrl *cptr) | 380 | static int ctrl_cx2341x_is_dirty(struct pvr2_ctrl *cptr) |
391 | { | 381 | { |
@@ -745,12 +735,10 @@ static const struct pvr2_ctl_info control_defs[] = { | |||
745 | .default_value = 720, | 735 | .default_value = 720, |
746 | DEFREF(res_hor), | 736 | DEFREF(res_hor), |
747 | DEFINT(320,720), | 737 | DEFINT(320,720), |
748 | #ifdef CONFIG_VIDEO_PVRUSB2_24XXX | ||
749 | /* Hook in check for clamp on horizontal resolution in | 738 | /* Hook in check for clamp on horizontal resolution in |
750 | order to avoid unsolved problem involving cx25840. */ | 739 | order to avoid unsolved problem involving cx25840. */ |
751 | .get_max_value = ctrl_hres_max_get, | 740 | .get_max_value = ctrl_hres_max_get, |
752 | .get_min_value = ctrl_hres_min_get, | 741 | .get_min_value = ctrl_hres_min_get, |
753 | #endif | ||
754 | },{ | 742 | },{ |
755 | .desc = "Vertical capture resolution", | 743 | .desc = "Vertical capture resolution", |
756 | .name = "resolution_ver", | 744 | .name = "resolution_ver", |
@@ -966,22 +954,18 @@ static int pvr2_upload_firmware1(struct pvr2_hdw *hdw) | |||
966 | static const char *fw_files_29xxx[] = { | 954 | static const char *fw_files_29xxx[] = { |
967 | "v4l-pvrusb2-29xxx-01.fw", | 955 | "v4l-pvrusb2-29xxx-01.fw", |
968 | }; | 956 | }; |
969 | #ifdef CONFIG_VIDEO_PVRUSB2_24XXX | ||
970 | static const char *fw_files_24xxx[] = { | 957 | static const char *fw_files_24xxx[] = { |
971 | "v4l-pvrusb2-24xxx-01.fw", | 958 | "v4l-pvrusb2-24xxx-01.fw", |
972 | }; | 959 | }; |
973 | #endif | ||
974 | static const struct pvr2_string_table fw_file_defs[] = { | 960 | static const struct pvr2_string_table fw_file_defs[] = { |
975 | [PVR2_HDW_TYPE_29XXX] = { | 961 | [PVR2_HDW_TYPE_29XXX] = { |
976 | fw_files_29xxx, | 962 | fw_files_29xxx, |
977 | sizeof(fw_files_29xxx)/sizeof(fw_files_29xxx[0]), | 963 | sizeof(fw_files_29xxx)/sizeof(fw_files_29xxx[0]), |
978 | }, | 964 | }, |
979 | #ifdef CONFIG_VIDEO_PVRUSB2_24XXX | ||
980 | [PVR2_HDW_TYPE_24XXX] = { | 965 | [PVR2_HDW_TYPE_24XXX] = { |
981 | fw_files_24xxx, | 966 | fw_files_24xxx, |
982 | sizeof(fw_files_24xxx)/sizeof(fw_files_24xxx[0]), | 967 | sizeof(fw_files_24xxx)/sizeof(fw_files_24xxx[0]), |
983 | }, | 968 | }, |
984 | #endif | ||
985 | }; | 969 | }; |
986 | hdw->fw1_state = FW1_STATE_FAILED; // default result | 970 | hdw->fw1_state = FW1_STATE_FAILED; // default result |
987 | 971 | ||
diff --git a/drivers/media/video/pvrusb2/pvrusb2-i2c-chips-v4l2.c b/drivers/media/video/pvrusb2/pvrusb2-i2c-chips-v4l2.c index fbe6039aeb6a..ed3e8105292a 100644 --- a/drivers/media/video/pvrusb2/pvrusb2-i2c-chips-v4l2.c +++ b/drivers/media/video/pvrusb2/pvrusb2-i2c-chips-v4l2.c | |||
@@ -26,10 +26,8 @@ | |||
26 | #include "pvrusb2-audio.h" | 26 | #include "pvrusb2-audio.h" |
27 | #include "pvrusb2-tuner.h" | 27 | #include "pvrusb2-tuner.h" |
28 | #include "pvrusb2-video-v4l.h" | 28 | #include "pvrusb2-video-v4l.h" |
29 | #ifdef CONFIG_VIDEO_PVRUSB2_24XXX | ||
30 | #include "pvrusb2-cx2584x-v4l.h" | 29 | #include "pvrusb2-cx2584x-v4l.h" |
31 | #include "pvrusb2-wm8775.h" | 30 | #include "pvrusb2-wm8775.h" |
32 | #endif | ||
33 | 31 | ||
34 | #define trace_i2c(...) pvr2_trace(PVR2_TRACE_I2C,__VA_ARGS__) | 32 | #define trace_i2c(...) pvr2_trace(PVR2_TRACE_I2C,__VA_ARGS__) |
35 | 33 | ||
@@ -71,7 +69,6 @@ void pvr2_i2c_probe(struct pvr2_hdw *hdw,struct pvr2_i2c_client *cp) | |||
71 | return; | 69 | return; |
72 | } | 70 | } |
73 | } | 71 | } |
74 | #ifdef CONFIG_VIDEO_PVRUSB2_24XXX | ||
75 | if (id == I2C_DRIVERID_CX25840) { | 72 | if (id == I2C_DRIVERID_CX25840) { |
76 | if (pvr2_i2c_cx2584x_v4l_setup(hdw,cp)) { | 73 | if (pvr2_i2c_cx2584x_v4l_setup(hdw,cp)) { |
77 | return; | 74 | return; |
@@ -82,7 +79,6 @@ void pvr2_i2c_probe(struct pvr2_hdw *hdw,struct pvr2_i2c_client *cp) | |||
82 | return; | 79 | return; |
83 | } | 80 | } |
84 | } | 81 | } |
85 | #endif | ||
86 | if (id == I2C_DRIVERID_SAA711X) { | 82 | if (id == I2C_DRIVERID_SAA711X) { |
87 | if (pvr2_i2c_decoder_v4l_setup(hdw,cp)) { | 83 | if (pvr2_i2c_decoder_v4l_setup(hdw,cp)) { |
88 | return; | 84 | return; |
diff --git a/drivers/media/video/pvrusb2/pvrusb2-i2c-core.c b/drivers/media/video/pvrusb2/pvrusb2-i2c-core.c index 7fca47982277..3b9012f8e380 100644 --- a/drivers/media/video/pvrusb2/pvrusb2-i2c-core.c +++ b/drivers/media/video/pvrusb2/pvrusb2-i2c-core.c | |||
@@ -185,8 +185,6 @@ static int pvr2_i2c_basic_op(struct pvr2_hdw *hdw, | |||
185 | } | 185 | } |
186 | } | 186 | } |
187 | 187 | ||
188 | #ifdef CONFIG_VIDEO_PVRUSB2_24XXX | ||
189 | |||
190 | /* This is a special entry point that is entered if an I2C operation is | 188 | /* This is a special entry point that is entered if an I2C operation is |
191 | attempted to a wm8775 chip on model 24xxx hardware. Autodetect of this | 189 | attempted to a wm8775 chip on model 24xxx hardware. Autodetect of this |
192 | part doesn't work, but we know it is really there. So let's look for | 190 | part doesn't work, but we know it is really there. So let's look for |
@@ -289,8 +287,6 @@ static int i2c_hack_cx25840(struct pvr2_hdw *hdw, | |||
289 | return -EIO; | 287 | return -EIO; |
290 | } | 288 | } |
291 | 289 | ||
292 | #endif /* CONFIG_VIDEO_PVRUSB2_24XXX */ | ||
293 | |||
294 | /* This is a very, very limited I2C adapter implementation. We can only | 290 | /* This is a very, very limited I2C adapter implementation. We can only |
295 | support what we actually know will work on the device... */ | 291 | support what we actually know will work on the device... */ |
296 | static int pvr2_i2c_xfer(struct i2c_adapter *i2c_adap, | 292 | static int pvr2_i2c_xfer(struct i2c_adapter *i2c_adap, |
@@ -897,14 +893,12 @@ void pvr2_i2c_core_init(struct pvr2_hdw *hdw) | |||
897 | hdw->i2c_func[idx] = pvr2_i2c_basic_op; | 893 | hdw->i2c_func[idx] = pvr2_i2c_basic_op; |
898 | } | 894 | } |
899 | 895 | ||
900 | #ifdef CONFIG_VIDEO_PVRUSB2_24XXX | ||
901 | // If however we're dealing with new hardware, insert some hacks in | 896 | // If however we're dealing with new hardware, insert some hacks in |
902 | // the I2C transfer stack to let things work better. | 897 | // the I2C transfer stack to let things work better. |
903 | if (hdw->hdw_type == PVR2_HDW_TYPE_24XXX) { | 898 | if (hdw->hdw_type == PVR2_HDW_TYPE_24XXX) { |
904 | hdw->i2c_func[0x1b] = i2c_hack_wm8775; | 899 | hdw->i2c_func[0x1b] = i2c_hack_wm8775; |
905 | hdw->i2c_func[0x44] = i2c_hack_cx25840; | 900 | hdw->i2c_func[0x44] = i2c_hack_cx25840; |
906 | } | 901 | } |
907 | #endif | ||
908 | 902 | ||
909 | // Configure the adapter and set up everything else related to it. | 903 | // Configure the adapter and set up everything else related to it. |
910 | memcpy(&hdw->i2c_adap,&pvr2_i2c_adap_template,sizeof(hdw->i2c_adap)); | 904 | memcpy(&hdw->i2c_adap,&pvr2_i2c_adap_template,sizeof(hdw->i2c_adap)); |