diff options
author | Jean Delvare <khali@linux-fr.org> | 2009-03-10 22:28:17 -0400 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2009-03-30 11:43:16 -0400 |
commit | 4dbf46a0485a5b0704e1c4b55a173128fbaedec9 (patch) | |
tree | b169d15df798fb6f8b1f7d4c92d0e760969cd75d | |
parent | 1159b7f19f324db0c61d1277987374865690ec06 (diff) |
V4L/DVB (10931): zoran: Drop the lock_norm module parameter
The lock_norm module parameter doesn't look terribly useful. If you
don't want to change the norm, just don't change it. As a matter of
fact, no other v4l driver has such a parameter.
Cc: Hans Verkuil <hverkuil@xs4all.nl>
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Trent Piepho <xyzzy@speakeasy.org>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
-rw-r--r-- | Documentation/video4linux/Zoran | 3 | ||||
-rw-r--r-- | drivers/media/video/zoran/zoran_driver.c | 20 |
2 files changed, 1 insertions, 22 deletions
diff --git a/Documentation/video4linux/Zoran b/Documentation/video4linux/Zoran index 295462b2317a..0e89e7676298 100644 --- a/Documentation/video4linux/Zoran +++ b/Documentation/video4linux/Zoran | |||
@@ -401,8 +401,7 @@ Additional notes for software developers: | |||
401 | first set the correct norm. Well, it seems logically correct: TV | 401 | first set the correct norm. Well, it seems logically correct: TV |
402 | standard is "more constant" for current country than geometry | 402 | standard is "more constant" for current country than geometry |
403 | settings of a variety of TV capture cards which may work in ITU or | 403 | settings of a variety of TV capture cards which may work in ITU or |
404 | square pixel format. Remember that users now can lock the norm to | 404 | square pixel format. |
405 | avoid any ambiguity. | ||
406 | -- | 405 | -- |
407 | Please note that lavplay/lavrec are also included in the MJPEG-tools | 406 | Please note that lavplay/lavrec are also included in the MJPEG-tools |
408 | (http://mjpeg.sf.net/). | 407 | (http://mjpeg.sf.net/). |
diff --git a/drivers/media/video/zoran/zoran_driver.c b/drivers/media/video/zoran/zoran_driver.c index 26be1a8908a3..1869d307a59d 100644 --- a/drivers/media/video/zoran/zoran_driver.c +++ b/drivers/media/video/zoran/zoran_driver.c | |||
@@ -162,10 +162,6 @@ const struct zoran_format zoran_formats[] = { | |||
162 | }; | 162 | }; |
163 | #define NUM_FORMATS ARRAY_SIZE(zoran_formats) | 163 | #define NUM_FORMATS ARRAY_SIZE(zoran_formats) |
164 | 164 | ||
165 | static int lock_norm; /* 0 = default 1 = Don't change TV standard (norm) */ | ||
166 | module_param(lock_norm, int, 0644); | ||
167 | MODULE_PARM_DESC(lock_norm, "Prevent norm changes (1 = ignore, >1 = fail)"); | ||
168 | |||
169 | /* small helper function for calculating buffersizes for v4l2 | 165 | /* small helper function for calculating buffersizes for v4l2 |
170 | * we calculate the nearest higher power-of-two, which | 166 | * we calculate the nearest higher power-of-two, which |
171 | * will be the recommended buffersize */ | 167 | * will be the recommended buffersize */ |
@@ -1483,22 +1479,6 @@ zoran_set_norm (struct zoran *zr, | |||
1483 | return -EBUSY; | 1479 | return -EBUSY; |
1484 | } | 1480 | } |
1485 | 1481 | ||
1486 | if (lock_norm && norm != zr->norm) { | ||
1487 | if (lock_norm > 1) { | ||
1488 | dprintk(1, | ||
1489 | KERN_WARNING | ||
1490 | "%s: set_norm() - TV standard is locked, can not switch norm\n", | ||
1491 | ZR_DEVNAME(zr)); | ||
1492 | return -EPERM; | ||
1493 | } else { | ||
1494 | dprintk(1, | ||
1495 | KERN_WARNING | ||
1496 | "%s: set_norm() - TV standard is locked, norm was not changed\n", | ||
1497 | ZR_DEVNAME(zr)); | ||
1498 | norm = zr->norm; | ||
1499 | } | ||
1500 | } | ||
1501 | |||
1502 | if (!(norm & zr->card.norms)) { | 1482 | if (!(norm & zr->card.norms)) { |
1503 | dprintk(1, | 1483 | dprintk(1, |
1504 | KERN_ERR "%s: set_norm() - unsupported norm %llx\n", | 1484 | KERN_ERR "%s: set_norm() - unsupported norm %llx\n", |