From patchwork Wed May 25 09:58:58 2011 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Alex Gershgorin X-Patchwork-Id: 815522 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by demeter1.kernel.org (8.14.4/8.14.3) with ESMTP id p4PA0wg2018472 for ; Wed, 25 May 2011 10:00:58 GMT Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932197Ab1EYKAz (ORCPT ); Wed, 25 May 2011 06:00:55 -0400 Received: from mail.meprolight.com ([194.90.149.17]:46521 "EHLO meprolight.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1755719Ab1EYKAz convert rfc822-to-8bit (ORCPT ); Wed, 25 May 2011 06:00:55 -0400 Received: from mail.meprolight.com ([192.168.111.10]) by mail.meprolight.com with ESMTP id p4P9xYYI007643; Wed, 25 May 2011 11:59:34 +0200 Received: from MEP-EXCH.meprolight.com ([192.168.111.10]) by MEP-EXCH.meprolight.com ([192.168.111.10]) with mapi; Wed, 25 May 2011 12:58:58 +0300 From: Alex Gershgorin To: "'Laurent Pinchart'" CC: "'Sakari Ailus'" , Michael Jones , "'linux-media@vger.kernel.org'" , "'agersh@rambler.ru'" Date: Wed, 25 May 2011 12:58:58 +0300 Subject: RE: FW: OMAP 3 ISP Thread-Topic: FW: OMAP 3 ISP Thread-Index: AcwarGarn0FuqICPTO6jsm0pM2AXAQAFd+/w Message-ID: <4875438356E7CA4A8F2145FCD3E61C0B15D3557D42@MEP-EXCH.meprolight.com> In-Reply-To: <201105250922.28496.laurent.pinchart@ideasonboard.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: acceptlanguage: en-US MIME-Version: 1.0 Sender: linux-media-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-media@vger.kernel.org X-Greylist: IP, sender and recipient auto-whitelisted, not delayed by milter-greylist-4.2.6 (demeter1.kernel.org [140.211.167.41]); Wed, 25 May 2011 10:00:58 +0000 (UTC) Hi Laurent, Unfortunately, at this point I have no Hardware platforms, but in the next week we should get Zoom OMAP35 Torpedo evaluation kit and then I can test it. I have already applied this patch on the last main line Kernel version (2.6.39) and continue to work on the platform device for Zoom OMAP35xx Torpedo. Thanks for this patch :-) Regards, Alex Gershgorin -----Original Message----- From: Laurent Pinchart [mailto:laurent.pinchart@ideasonboard.com] Sent: Wednesday, May 25, 2011 10:22 AM To: Alex Gershgorin Cc: 'Sakari Ailus'; Michael Jones; 'linux-media@vger.kernel.org'; 'agersh@rambler.ru' Subject: Re: FW: OMAP 3 ISP Hi Alex, On Tuesday 24 May 2011 16:11:16 Alex Gershgorin wrote: > Hi All, > > I wrote a simple V4L2 subdevs I2C driver which returns a fixed format and > size. I do not understand who reads these parameters, user application > through IOCTL or OMAP3 ISP driver uses them regardless of the user space > application? Both. media-ctl (and other userspace applications) can use them, and the OMAP3 ISP driver retrieves them when starting the video stream to make sure that the formats at the "sensor" output and at the CCDC input match. > Another question, if I need to change polarity of Vertical or Horizontal > synchronization signals, according struct isp_parallel_platform_data, is > it not possible? > > struct isp_parallel_platform_data { > unsigned int data_lane_shift:2; > unsigned int clk_pol:1; > unsigned int bridge:4; > }; Could you please try the following patch ? From 7f8eff25e63880a93bc283cd97840227cd092622 Mon Sep 17 00:00:00 2001 From: Laurent Pinchart Date: Wed, 25 May 2011 09:16:28 +0200 Subject: [PATCH] omap3isp: Support configurable HS/VS polarities Add two fields to the ISP parallel platform data to set the HS and VS signals polarities. Signed-off-by: Laurent Pinchart --- drivers/media/video/omap3isp/isp.h | 6 ++++++ drivers/media/video/omap3isp/ispccdc.c | 4 ++-- 2 files changed, 8 insertions(+), 2 deletions(-) -- 1.7.3.4 -- Regards, Laurent Pinchart __________ Information from ESET NOD32 Antivirus, version of virus signature database 6149 (20110524) __________ The message was checked by ESET NOD32 Antivirus. http://www.eset.com __________ Information from ESET NOD32 Antivirus, version of virus signature database 6149 (20110524) __________ The message was checked by ESET NOD32 Antivirus. http://www.eset.com -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/drivers/media/video/omap3isp/isp.h b/drivers/media/video/omap3isp/isp.h index 2620c40..529e582 100644 --- a/drivers/media/video/omap3isp/isp.h +++ b/drivers/media/video/omap3isp/isp.h @@ -139,6 +139,10 @@ struct isp_reg { * 3 - CAMEXT[13:6] -> CAM[7:0] * @clk_pol: Pixel clock polarity * 0 - Non Inverted, 1 - Inverted + * @hs_pol: Horizontal synchronization polarity + * 0 - Active high, 1 - Active low + * @vs_pol: Vertical synchronization polarity + * 0 - Active high, 1 - Active low * @bridge: CCDC Bridge input control * ISPCTRL_PAR_BRIDGE_DISABLE - Disable * ISPCTRL_PAR_BRIDGE_LENDIAN - Little endian @@ -147,6 +151,8 @@ struct isp_reg { struct isp_parallel_platform_data { unsigned int data_lane_shift:2; unsigned int clk_pol:1; + unsigned int hs_pol:1; + unsigned int vs_pol:1; unsigned int bridge:4; }; diff --git a/drivers/media/video/omap3isp/ispccdc.c b/drivers/media/video/omap3isp/ispccdc.c index 39d501b..5e742b2 100644 --- a/drivers/media/video/omap3isp/ispccdc.c +++ b/drivers/media/video/omap3isp/ispccdc.c @@ -1148,6 +1148,8 @@ static void ccdc_configure(struct isp_ccdc_device *ccdc) omap3isp_configure_bridge(isp, ccdc->input, pdata, shift); ccdc->syncif.datsz = depth_out; + ccdc->syncif.hdpol = pdata ? pdata-> hs_pol : 0; + ccdc->syncif.vdpol = pdata ? pdata-> vs_pol : 0; ccdc_config_sync_if(ccdc, &ccdc->syncif); /* CCDC_PAD_SINK */ @@ -2257,8 +2259,6 @@ int omap3isp_ccdc_init(struct isp_device *isp) ccdc->syncif.fldout = 0; ccdc->syncif.fldpol = 0; ccdc->syncif.fldstat = 0; - ccdc->syncif.hdpol = 0; - ccdc->syncif.vdpol = 0; ccdc->clamp.oblen = 0; ccdc->clamp.dcsubval = 0;