diff options
author | Hans Verkuil <hverkuil@xs4all.nl> | 2009-02-21 16:47:24 -0500 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2009-03-30 11:42:54 -0400 |
commit | 10afbef15e7bba5e1008f583852077743d28c395 (patch) | |
tree | 73dbfe2a3ccde6ef75c3403fb189274247740003 /drivers/media/video/cx18 | |
parent | 1df795370c1392a026c63816368108187aec2ec1 (diff) |
V4L/DVB (10698): v4l2-common: remove v4l2_ctrl_query_fill_std
The v4l2_ctrl_query_fill_std() function wasn't one the best idea I ever had.
It doesn't add anything valuable that cannot be expressed equally well with
v4l2_ctrl_query_fill and only adds overhead.
Replace it with v4l2_ctrl_query_fill() everywhere it is used and remove it
from v4l2_common.c.
Signed-off-by: Hans Verkuil <hverkuil@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'drivers/media/video/cx18')
-rw-r--r-- | drivers/media/video/cx18/cx18-av-core.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/drivers/media/video/cx18/cx18-av-core.c b/drivers/media/video/cx18/cx18-av-core.c index a3bd2c95f582..fc576cf1a8b5 100644 --- a/drivers/media/video/cx18/cx18-av-core.c +++ b/drivers/media/video/cx18/cx18-av-core.c | |||
@@ -788,10 +788,12 @@ int cx18_av_cmd(struct cx18 *cx, unsigned int cmd, void *arg) | |||
788 | 788 | ||
789 | switch (qc->id) { | 789 | switch (qc->id) { |
790 | case V4L2_CID_BRIGHTNESS: | 790 | case V4L2_CID_BRIGHTNESS: |
791 | return v4l2_ctrl_query_fill(qc, 0, 255, 1, 128); | ||
791 | case V4L2_CID_CONTRAST: | 792 | case V4L2_CID_CONTRAST: |
792 | case V4L2_CID_SATURATION: | 793 | case V4L2_CID_SATURATION: |
794 | return v4l2_ctrl_query_fill(qc, 0, 127, 1, 64); | ||
793 | case V4L2_CID_HUE: | 795 | case V4L2_CID_HUE: |
794 | return v4l2_ctrl_query_fill_std(qc); | 796 | return v4l2_ctrl_query_fill(qc, -128, 127, 1, 0); |
795 | default: | 797 | default: |
796 | break; | 798 | break; |
797 | } | 799 | } |
@@ -801,10 +803,11 @@ int cx18_av_cmd(struct cx18 *cx, unsigned int cmd, void *arg) | |||
801 | return v4l2_ctrl_query_fill(qc, 0, 65535, | 803 | return v4l2_ctrl_query_fill(qc, 0, 65535, |
802 | 65535 / 100, state->default_volume); | 804 | 65535 / 100, state->default_volume); |
803 | case V4L2_CID_AUDIO_MUTE: | 805 | case V4L2_CID_AUDIO_MUTE: |
806 | return v4l2_ctrl_query_fill(qc, 0, 1, 1, 0); | ||
804 | case V4L2_CID_AUDIO_BALANCE: | 807 | case V4L2_CID_AUDIO_BALANCE: |
805 | case V4L2_CID_AUDIO_BASS: | 808 | case V4L2_CID_AUDIO_BASS: |
806 | case V4L2_CID_AUDIO_TREBLE: | 809 | case V4L2_CID_AUDIO_TREBLE: |
807 | return v4l2_ctrl_query_fill_std(qc); | 810 | return v4l2_ctrl_query_fill(qc, 0, 65535, 65535 / 100, 32768); |
808 | default: | 811 | default: |
809 | return -EINVAL; | 812 | return -EINVAL; |
810 | } | 813 | } |