RE: [PATCH 2/9] v4l: add new v4l2-subdev sensor operations, use g_skip_top_lines in soc-camera

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Guennadi,

Murali Karicheri
Software Design Engineer
Texas Instruments Inc.
Germantown, MD 20874
phone: 301-407-9583
email: m-karicheri2@xxxxxx

>-----Original Message-----
>From: linux-media-owner@xxxxxxxxxxxxxxx [mailto:linux-media-
>owner@xxxxxxxxxxxxxxx] On Behalf Of Guennadi Liakhovetski
>Sent: Friday, October 30, 2009 4:32 PM
>To: Karicheri, Muralidharan
>Cc: Linux Media Mailing List; Hans Verkuil; Laurent Pinchart; Sakari Ailus
>Subject: RE: [PATCH 2/9] v4l: add new v4l2-subdev sensor operations, use
>g_skip_top_lines in soc-camera
>
>On Fri, 30 Oct 2009, Karicheri, Muralidharan wrote:
>
>> Guennadi,
>>
>>
>> > 	mt9m111->rect.left	= MT9M111_MIN_DARK_COLS;
>> > 	mt9m111->rect.top	= MT9M111_MIN_DARK_ROWS;
>> >diff --git a/drivers/media/video/mt9t031.c
>b/drivers/media/video/mt9t031.c
>> >index 6966f64..57e04e9 100644
>> >--- a/drivers/media/video/mt9t031.c
>> >+++ b/drivers/media/video/mt9t031.c
>> >@@ -301,9 +301,9 @@ static int mt9t031_set_params(struct
>soc_camera_device
>> >*icd,
>> > 		ret = reg_write(client, MT9T031_WINDOW_WIDTH, rect->width - 1);
>> > 	if (ret >= 0)
>> > 		ret = reg_write(client, MT9T031_WINDOW_HEIGHT,
>> >-				rect->height + icd->y_skip_top - 1);
>> >+				rect->height - 1);
>
>> Why y_skip_top is removed?
>
>Because noone ever said they needed it?
>
I suggest you keep it. It can have default 0. I have not viewed the resulting image for the top line to see if it is corrupted. I just
use it to display it to my display device and I am not seeing any
corruption. I need to view the image at some point to check if it has
any corruption.
>> When I connect the sensor output to our SOC
>> input and do format conversion and resize on the fly (frame by frame
>> conversion before writing to SDRAM) I have found that the frame
>> completion interrupt fails to get generated with zero value for
>> y_skip_top. I have used a value
>> of 10 and it worked fine for me. So I would like to have a
>> s_skip_top_lines() in the sensor operations which can be called to
>> update this value from the host/bridge driver.
>
>Hm, strange, that's actually not the purpose of this parameter. Wouldn't
>it work for you just as well, if you just request 10 more lines when
>sending s_fmt from your bridge driver?
Ok. It might work by asking some additional lines from the bridge driver.
I will try this out.
>
>Thanks
>Guennadi
>---
>Guennadi Liakhovetski, Ph.D.
>Freelance Open-Source Software Developer
>http://www.open-technology.de/
>--
>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

[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux