From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail-bw0-f219.google.com ([209.85.218.219]:63023 "EHLO mail-bw0-f219.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754244AbZINVPi convert rfc822-to-8bit (ORCPT ); Mon, 14 Sep 2009 17:15:38 -0400 Received: by bwz19 with SMTP id 19so2390380bwz.37 for ; Mon, 14 Sep 2009 14:15:41 -0700 (PDT) From: Marek Vasut To: Guennadi Liakhovetski Subject: Re: V4L2: Add a v4l2-subdev (soc-camera) driver for OmniVision OV9640 sensor Date: Mon, 14 Sep 2009 23:15:14 +0200 Cc: Linux Media Mailing List References: <200908220850.07435.marek.vasut@gmail.com> <200909142202.46154.marek.vasut@gmail.com> In-Reply-To: MIME-Version: 1.0 Content-Type: Text/Plain; charset="utf-8" Content-Transfer-Encoding: 8BIT Message-Id: <200909142315.14697.marek.vasut@gmail.com> Sender: linux-media-owner@vger.kernel.org List-ID: Dne Po 14. září 2009 22:30:41 Guennadi Liakhovetski napsal(a): > On Mon, 14 Sep 2009, Marek Vasut wrote: > > Dne Po 14. září 2009 21:29:26 Guennadi Liakhovetski napsal(a): > > > From: Marek Vasut > > > > > > Signed-off-by: Marek Vasut > > > Signed-off-by: Guennadi Liakhovetski > > > --- > > > > > > Marek, please confirm, that this version is ok. I'll push it upstream > > > for 2.6.32 then. > > > > No, it's not OK. You removed the RGB part. Either enclose those parts > > into ifdef OV9640_RGB_BUGGY or preserve it in some other way. Someone > > will certainly want to re-add RGB parts later and will have to figure it > > out all over again. > > Ok, make a proposal, how you would like to see it. But - I do not want > commented out code, including "#ifdef MACRO_THAT_DOESNT_GET_DEFINED." I > think, I described it in sufficient detail, so that re-adding that code > should not take longer than 10 minutes for anyone sufficiently familiar > with the code. Referencing another driver also has an advantage, that if > we switch to imagebus or any other API, you don't get stale commented out > code, but you look up updated code in a functional driver. But I am open > to your ideas / but no commented out code, please. The RGB is broken only in a way where it swaps colours, the color matrix coeficients and register configurations are OK (which is what other people who will want to add it will need to figure out again from scratch if you remove the code). I dont want this merged before this is solved in some way where those values are preserved. > > Thanks > Guennadi > --- > Guennadi Liakhovetski, Ph.D. > Freelance Open-Source Software Developer > http://www.open-technology.de/ >