From mboxrd@z Thu Jan 1 00:00:00 1970 From: Josh Wu Subject: Re: [v2] media: soc-camera: OF cameras Date: Fri, 11 Apr 2014 10:43:25 +0800 Message-ID: <534756CD.90409@atmel.com> References: <1392235552-28134-1-git-send-email-pengw@nvidia.com> <1394794130-13660-1-git-send-email-josh.wu@atmel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: linux-tegra-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Guennadi Liakhovetski , Bryan Wu Cc: "linux-media-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , linux-tegra , Ben Dooks List-Id: linux-tegra@vger.kernel.org Hi, Guennadi On 4/11/2014 5:18 AM, Guennadi Liakhovetski wrote: > Hi Bryan, > > On Tue, 8 Apr 2014, Bryan Wu wrote: > >> Thanks Josh, I think I will take you point and rework my patch again. >> But I need Guennadi's review firstly, Guennadi, could you please help >> to review it? > Ok, let me double check the situation: > > 1. We've got this patch from you, aiming at adding OF probing support to > soc-camra > > 2. We've got an alternative patch from Ben to do the same, his last reply > to a comment to his patch was "Thanks, I will look into this." > > 3. We've got Ben's patches for rcar-vin, that presumably work with his > patch from (2) above > > 4. We've got Josh's patches to add OF / async probing to atmel-isi and > ov2640, that are not known to work with either (1) or (2) above, so, they > don't work at all, right? Right, the atmel-isi dt cannot work in those two patches unless the mclk stuff is added or adjusted. > > So, to summarise, there is a core patch from Ben, that he possibly wants > to adjust, and that works with his rcar-vin OF, there is a patch from you > that isn't known to work with any driver, and there are patches from Josh, > that don't work, because there isn't a suitable patch available for them. > I will have a look at your and Ben's soc-camera OF patches to compare them > and compare them with my early code (hopefully this coming weekend), but > so far it looks like only Ben's solution has a complete working stack. Am > I missing something? > > Thanks > Guennadi > --- > Guennadi Liakhovetski, Ph.D. > Freelance Open-Source Software Developer > http://www.open-technology.de/ Best Regards, Josh Wu From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from eusmtp01.atmel.com ([212.144.249.243]:23131 "EHLO eusmtp01.atmel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754357AbaDKCwh (ORCPT ); Thu, 10 Apr 2014 22:52:37 -0400 Message-ID: <534756CD.90409@atmel.com> Date: Fri, 11 Apr 2014 10:43:25 +0800 From: Josh Wu MIME-Version: 1.0 To: Guennadi Liakhovetski , Bryan Wu CC: "linux-media@vger.kernel.org" , linux-tegra , Ben Dooks Subject: Re: [v2] media: soc-camera: OF cameras References: <1392235552-28134-1-git-send-email-pengw@nvidia.com> <1394794130-13660-1-git-send-email-josh.wu@atmel.com> In-Reply-To: Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-media-owner@vger.kernel.org List-ID: Hi, Guennadi On 4/11/2014 5:18 AM, Guennadi Liakhovetski wrote: > Hi Bryan, > > On Tue, 8 Apr 2014, Bryan Wu wrote: > >> Thanks Josh, I think I will take you point and rework my patch again. >> But I need Guennadi's review firstly, Guennadi, could you please help >> to review it? > Ok, let me double check the situation: > > 1. We've got this patch from you, aiming at adding OF probing support to > soc-camra > > 2. We've got an alternative patch from Ben to do the same, his last reply > to a comment to his patch was "Thanks, I will look into this." > > 3. We've got Ben's patches for rcar-vin, that presumably work with his > patch from (2) above > > 4. We've got Josh's patches to add OF / async probing to atmel-isi and > ov2640, that are not known to work with either (1) or (2) above, so, they > don't work at all, right? Right, the atmel-isi dt cannot work in those two patches unless the mclk stuff is added or adjusted. > > So, to summarise, there is a core patch from Ben, that he possibly wants > to adjust, and that works with his rcar-vin OF, there is a patch from you > that isn't known to work with any driver, and there are patches from Josh, > that don't work, because there isn't a suitable patch available for them. > I will have a look at your and Ben's soc-camera OF patches to compare them > and compare them with my early code (hopefully this coming weekend), but > so far it looks like only Ben's solution has a complete working stack. Am > I missing something? > > Thanks > Guennadi > --- > Guennadi Liakhovetski, Ph.D. > Freelance Open-Source Software Developer > http://www.open-technology.de/ Best Regards, Josh Wu