Am 07.05.2010 20:17, schrieb Bee Hock Goh: > Stefan, > > This fix the ugly green line on the top. > > btw, do you notice that there seem to be black gap on the left and > right of the screen? > Have you seen my picture? I corrected that field inverted is, ergo line 1 field 1 is line 0, line 1 field 0 is line 1, line 2 field 1 is line 2, line 2 field 0 is line 3 ... . I have not wrote from the black gap - that is video calibration! And what for ugly green line, what I have is a green code and that is normal! > On Fri, May 7, 2010 at 11:46 PM, Stefan Ringel <stefan.ringel@xxxxxxxx> wrote: > >> Am 07.05.2010 17:29, schrieb stefan.ringel@xxxxxxxx: >> >>> From: Stefan Ringel <stefan.ringel@xxxxxxxx> >>> >>> bugfix incorrect image and line position in videobuffer >>> >>> >>> Signed-off-by: Stefan Ringel <stefan.ringel@xxxxxxxx> >>> --- >>> drivers/staging/tm6000/tm6000-video.c | 4 ++-- >>> 1 files changed, 2 insertions(+), 2 deletions(-) >>> >>> diff --git a/drivers/staging/tm6000/tm6000-video.c b/drivers/staging/tm6000/tm6000-video.c >>> index 9554472..f7248f0 100644 >>> --- a/drivers/staging/tm6000/tm6000-video.c >>> +++ b/drivers/staging/tm6000/tm6000-video.c >>> @@ -223,8 +223,8 @@ static int copy_packet(struct urb *urb, u32 header, u8 **ptr, u8 *endp, >>> * It should, instead, check if the user selected >>> * entrelaced or non-entrelaced mode >>> */ >>> - pos= ((line<<1)+field)*linewidth + >>> - block*TM6000_URB_MSG_LEN; >>> + pos = ((line << 1) - field - 1) * linewidth + >>> + block * TM6000_URB_MSG_LEN; >>> >>> /* Don't allow to write out of the buffer */ >>> if (pos+TM6000_URB_MSG_LEN > (*buf)->vb.size) { >>> >>> >> >> http://www.stefanringel.de/pub/tm6000_image_07_05_2010.jpg >> >> -- >> Stefan Ringel <stefan.ringel@xxxxxxxx> >> >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-media" in >> the body of a message to majordomo@xxxxxxxxxxxxxxx >> More majordomo info at http://vger.kernel.org/majordomo-info.html >> >> > -- > To unsubscribe from this list: send the line "unsubscribe linux-media" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- Stefan Ringel <stefan.ringel@xxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html