Commit f246a817 authored by Michael Schimek's avatar Michael Schimek Committed by Linus Torvalds

[PATCH] v4l: saa7134 ntsc vbi fix

This patch fixes NTSC VBI capturing in the saa7134 driver.
Signed-off-by: default avatarMichael H. Schimek <mschimek@gmx.at>
Cc: <video4linux-list@redhat.com>
Signed-off-by: default avatarAndrew Morton <akpm@osdl.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent 93b43f13
...@@ -60,10 +60,10 @@ static void task_init(struct saa7134_dev *dev, struct saa7134_buf *buf, ...@@ -60,10 +60,10 @@ static void task_init(struct saa7134_dev *dev, struct saa7134_buf *buf,
saa_writeb(SAA7134_VBI_H_START2(task), norm->h_start >> 8); saa_writeb(SAA7134_VBI_H_START2(task), norm->h_start >> 8);
saa_writeb(SAA7134_VBI_H_STOP1(task), norm->h_stop & 0xff); saa_writeb(SAA7134_VBI_H_STOP1(task), norm->h_stop & 0xff);
saa_writeb(SAA7134_VBI_H_STOP2(task), norm->h_stop >> 8); saa_writeb(SAA7134_VBI_H_STOP2(task), norm->h_stop >> 8);
saa_writeb(SAA7134_VBI_V_START1(task), norm->vbi_v_start & 0xff); saa_writeb(SAA7134_VBI_V_START1(task), norm->vbi_v_start_0 & 0xff);
saa_writeb(SAA7134_VBI_V_START2(task), norm->vbi_v_start >> 8); saa_writeb(SAA7134_VBI_V_START2(task), norm->vbi_v_start_0 >> 8);
saa_writeb(SAA7134_VBI_V_STOP1(task), norm->vbi_v_stop & 0xff); saa_writeb(SAA7134_VBI_V_STOP1(task), norm->vbi_v_stop_0 & 0xff);
saa_writeb(SAA7134_VBI_V_STOP2(task), norm->vbi_v_stop >> 8); saa_writeb(SAA7134_VBI_V_STOP2(task), norm->vbi_v_stop_0 >> 8);
saa_writeb(SAA7134_VBI_H_SCALE_INC1(task), VBI_SCALE & 0xff); saa_writeb(SAA7134_VBI_H_SCALE_INC1(task), VBI_SCALE & 0xff);
saa_writeb(SAA7134_VBI_H_SCALE_INC2(task), VBI_SCALE >> 8); saa_writeb(SAA7134_VBI_H_SCALE_INC2(task), VBI_SCALE >> 8);
...@@ -127,7 +127,7 @@ static int buffer_prepare(struct videobuf_queue *q, ...@@ -127,7 +127,7 @@ static int buffer_prepare(struct videobuf_queue *q,
unsigned int lines, llength, size; unsigned int lines, llength, size;
int err; int err;
lines = norm->vbi_v_stop - norm->vbi_v_start +1; lines = norm->vbi_v_stop_0 - norm->vbi_v_start_0 +1;
if (lines > VBI_LINE_COUNT) if (lines > VBI_LINE_COUNT)
lines = VBI_LINE_COUNT; lines = VBI_LINE_COUNT;
#if 1 #if 1
...@@ -177,7 +177,7 @@ buffer_setup(struct videobuf_queue *q, unsigned int *count, unsigned int *size) ...@@ -177,7 +177,7 @@ buffer_setup(struct videobuf_queue *q, unsigned int *count, unsigned int *size)
struct saa7134_dev *dev = fh->dev; struct saa7134_dev *dev = fh->dev;
int llength,lines; int llength,lines;
lines = dev->tvnorm->vbi_v_stop - dev->tvnorm->vbi_v_start +1; lines = dev->tvnorm->vbi_v_stop_0 - dev->tvnorm->vbi_v_start_0 +1;
#if 1 #if 1
llength = VBI_LINE_LENGTH; llength = VBI_LINE_LENGTH;
#else #else
......
...@@ -158,18 +158,20 @@ static struct saa7134_format formats[] = { ...@@ -158,18 +158,20 @@ static struct saa7134_format formats[] = {
.h_stop = 719, \ .h_stop = 719, \
.video_v_start = 24, \ .video_v_start = 24, \
.video_v_stop = 311, \ .video_v_stop = 311, \
.vbi_v_start = 7, \ .vbi_v_start_0 = 7, \
.vbi_v_stop = 22, \ .vbi_v_stop_0 = 22, \
.vbi_v_start_1 = 319, \
.src_timing = 4 .src_timing = 4
#define NORM_525_60 \ #define NORM_525_60 \
.h_start = 0, \ .h_start = 0, \
.h_stop = 703, \ .h_stop = 703, \
.video_v_start = 22, \ .video_v_start = 23, \
.video_v_stop = 22+239, \ .video_v_stop = 262, \
.vbi_v_start = 10, /* FIXME */ \ .vbi_v_start_0 = 10, \
.vbi_v_stop = 21, /* FIXME */ \ .vbi_v_stop_0 = 21, \
.src_timing = 1 .vbi_v_start_1 = 273, \
.src_timing = 7
static struct saa7134_tvnorm tvnorms[] = { static struct saa7134_tvnorm tvnorms[] = {
{ {
...@@ -274,11 +276,12 @@ static struct saa7134_tvnorm tvnorms[] = { ...@@ -274,11 +276,12 @@ static struct saa7134_tvnorm tvnorms[] = {
.h_start = 0, .h_start = 0,
.h_stop = 719, .h_stop = 719,
.video_v_start = 22, .video_v_start = 23,
.video_v_stop = 22+239, .video_v_stop = 262,
.vbi_v_start = 10, /* FIXME */ .vbi_v_start_0 = 10,
.vbi_v_stop = 21, /* FIXME */ .vbi_v_stop_0 = 21,
.src_timing = 1, .vbi_v_start_1 = 273,
.src_timing = 7,
.sync_control = 0x18, .sync_control = 0x18,
.luma_control = 0x40, .luma_control = 0x40,
...@@ -335,8 +338,8 @@ static const struct v4l2_queryctrl video_ctrls[] = { ...@@ -335,8 +338,8 @@ static const struct v4l2_queryctrl video_ctrls[] = {
.default_value = 0, .default_value = 0,
.type = V4L2_CTRL_TYPE_INTEGER, .type = V4L2_CTRL_TYPE_INTEGER,
},{ },{
.id = V4L2_CID_VFLIP, .id = V4L2_CID_HFLIP,
.name = "vertical flip", .name = "Mirror",
.minimum = 0, .minimum = 0,
.maximum = 1, .maximum = 1,
.type = V4L2_CTRL_TYPE_BOOLEAN, .type = V4L2_CTRL_TYPE_BOOLEAN,
...@@ -482,7 +485,7 @@ static void set_tvnorm(struct saa7134_dev *dev, struct saa7134_tvnorm *norm) ...@@ -482,7 +485,7 @@ static void set_tvnorm(struct saa7134_dev *dev, struct saa7134_tvnorm *norm)
dev->crop_bounds.width = norm->h_stop - norm->h_start +1; dev->crop_bounds.width = norm->h_stop - norm->h_start +1;
dev->crop_defrect.width = norm->h_stop - norm->h_start +1; dev->crop_defrect.width = norm->h_stop - norm->h_start +1;
dev->crop_bounds.top = (norm->vbi_v_stop+1)*2; dev->crop_bounds.top = (norm->vbi_v_stop_0+1)*2;
dev->crop_defrect.top = norm->video_v_start*2; dev->crop_defrect.top = norm->video_v_start*2;
dev->crop_bounds.height = ((norm->id & V4L2_STD_525_60) ? 524 : 624) dev->crop_bounds.height = ((norm->id & V4L2_STD_525_60) ? 524 : 624)
- dev->crop_bounds.top; - dev->crop_bounds.top;
...@@ -1064,7 +1067,7 @@ static int get_control(struct saa7134_dev *dev, struct v4l2_control *c) ...@@ -1064,7 +1067,7 @@ static int get_control(struct saa7134_dev *dev, struct v4l2_control *c)
case V4L2_CID_PRIVATE_INVERT: case V4L2_CID_PRIVATE_INVERT:
c->value = dev->ctl_invert; c->value = dev->ctl_invert;
break; break;
case V4L2_CID_VFLIP: case V4L2_CID_HFLIP:
c->value = dev->ctl_mirror; c->value = dev->ctl_mirror;
break; break;
case V4L2_CID_PRIVATE_Y_EVEN: case V4L2_CID_PRIVATE_Y_EVEN:
...@@ -1139,7 +1142,7 @@ static int set_control(struct saa7134_dev *dev, struct saa7134_fh *fh, ...@@ -1139,7 +1142,7 @@ static int set_control(struct saa7134_dev *dev, struct saa7134_fh *fh,
saa_writeb(SAA7134_DEC_CHROMA_SATURATION, saa_writeb(SAA7134_DEC_CHROMA_SATURATION,
dev->ctl_invert ? -dev->ctl_saturation : dev->ctl_saturation); dev->ctl_invert ? -dev->ctl_saturation : dev->ctl_saturation);
break; break;
case V4L2_CID_VFLIP: case V4L2_CID_HFLIP:
dev->ctl_mirror = c->value; dev->ctl_mirror = c->value;
restart_overlay = 1; restart_overlay = 1;
break; break;
...@@ -1407,9 +1410,9 @@ static void saa7134_vbi_fmt(struct saa7134_dev *dev, struct v4l2_format *f) ...@@ -1407,9 +1410,9 @@ static void saa7134_vbi_fmt(struct saa7134_dev *dev, struct v4l2_format *f)
f->fmt.vbi.samples_per_line = 2048 /* VBI_LINE_LENGTH */; f->fmt.vbi.samples_per_line = 2048 /* VBI_LINE_LENGTH */;
f->fmt.vbi.sample_format = V4L2_PIX_FMT_GREY; f->fmt.vbi.sample_format = V4L2_PIX_FMT_GREY;
f->fmt.vbi.offset = 64 * 4; f->fmt.vbi.offset = 64 * 4;
f->fmt.vbi.start[0] = norm->vbi_v_start; f->fmt.vbi.start[0] = norm->vbi_v_start_0;
f->fmt.vbi.count[0] = norm->vbi_v_stop - norm->vbi_v_start +1; f->fmt.vbi.count[0] = norm->vbi_v_stop_0 - norm->vbi_v_start_0 +1;
f->fmt.vbi.start[1] = norm->video_v_stop + norm->vbi_v_start +1; f->fmt.vbi.start[1] = norm->vbi_v_start_1;
f->fmt.vbi.count[1] = f->fmt.vbi.count[0]; f->fmt.vbi.count[1] = f->fmt.vbi.count[0];
f->fmt.vbi.flags = 0; /* VBI_UNSYNC VBI_INTERLACED */ f->fmt.vbi.flags = 0; /* VBI_UNSYNC VBI_INTERLACED */
......
...@@ -21,7 +21,7 @@ ...@@ -21,7 +21,7 @@
*/ */
#include <linux/version.h> #include <linux/version.h>
#define SAA7134_VERSION_CODE KERNEL_VERSION(0,2,12) #define SAA7134_VERSION_CODE KERNEL_VERSION(0,2,13)
#include <linux/pci.h> #include <linux/pci.h>
#include <linux/i2c.h> #include <linux/i2c.h>
...@@ -91,9 +91,10 @@ struct saa7134_tvnorm { ...@@ -91,9 +91,10 @@ struct saa7134_tvnorm {
unsigned int h_stop; unsigned int h_stop;
unsigned int video_v_start; unsigned int video_v_start;
unsigned int video_v_stop; unsigned int video_v_stop;
unsigned int vbi_v_start; unsigned int vbi_v_start_0;
unsigned int vbi_v_stop; unsigned int vbi_v_stop_0;
unsigned int src_timing; unsigned int src_timing;
unsigned int vbi_v_start_1;
}; };
struct saa7134_tvaudio { struct saa7134_tvaudio {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment