From mboxrd@z Thu Jan 1 00:00:00 1970 From: Laurent Pinchart Subject: Re: [PATCH v10 1/2] s5k5baf: add camera sensor driver Date: Thu, 19 Dec 2013 19:33:44 +0100 Message-ID: <2100889.mqgrGv2fsV@avalon> References: <1386243520-17117-1-git-send-email-a.hajda@samsung.com> <1386243520-17117-2-git-send-email-a.hajda@samsung.com> <52B2FAA4.2070904@samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Return-path: In-Reply-To: <52B2FAA4.2070904@samsung.com> Sender: linux-media-owner@vger.kernel.org To: Sylwester Nawrocki Cc: Andrzej Hajda , linux-media@vger.kernel.org, linux-samsung-soc@vger.kernel.org, devicetree@vger.kernel.org, Kyungmin Park List-Id: devicetree@vger.kernel.org Hi Sylwester, On Thursday 19 December 2013 14:54:44 Sylwester Nawrocki wrote: > On 05/12/13 12:38, Andrzej Hajda wrote: > > Driver for Samsung S5K5BAF UXGA 1/5" 2M CMOS Image Sensor > > with embedded SoC ISP. > > The driver exposes the sensor as two V4L2 subdevices: > > - S5K5BAF-CIS - pure CMOS Image Sensor, fixed 1600x1200 format, > > no controls. > > - S5K5BAF-ISP - Image Signal Processor, formats up to 1600x1200, > > pre/post ISP cropping, downscaling via selection API, controls. > > > > Signed-off-by: Sylwester Nawrocki > > Signed-off-by: Andrzej Hajda > > Signed-off-by: Kyungmin Park > > Hi Laurent, > > Does this driver look sane to you, at its 10'th version? :) > If so I could send a pull request including it this week. Yes, it does. Sorry for the delay. Please send the pull request. -- Regards, Laurent Pinchart