From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail-lb0-f180.google.com ([209.85.217.180]:56945 "EHLO mail-lb0-f180.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752237Ab3F2PEr (ORCPT ); Sat, 29 Jun 2013 11:04:47 -0400 Received: by mail-lb0-f180.google.com with SMTP id o10so1539536lbi.39 for ; Sat, 29 Jun 2013 08:04:46 -0700 (PDT) Message-ID: <51CEF78F.1040500@cogentembedded.com> Date: Sat, 29 Jun 2013 19:04:47 +0400 From: Sergei Shtylyov MIME-Version: 1.0 To: Guennadi Liakhovetski CC: Vladimir Barinov , mchehab@redhat.com, linux-media@vger.kernel.org, magnus.damm@gmail.com, linux-sh@vger.kernel.org, phil.edworthy@renesas.com, matsu@igel.co.jp Subject: Re: [PATCH v7] V4L2: soc_camera: Renesas R-Car VIN driver References: <201306220052.30572.sergei.shtylyov@cogentembedded.com> <51CCD1B7.3040009@cogentembedded.com> <51CDC3BE.1040603@cogentembedded.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: Hello. On 29-06-2013 14:57, Guennadi Liakhovetski wrote: >>>>> From: Vladimir Barinov >>>>> Add Renesas R-Car VIN (Video In) V4L2 driver. >>>>> Based on the patch by Phil Edworthy . >>>>> Signed-off-by: Vladimir Barinov >>>>> [Sergei: removed deprecated IRQF_DISABLED flag, reordered/renamed >>>>> 'enum chip_id' >>>>> values, reordered rcar_vin_id_table[] entries, removed senseless >>>>> parens from >>>>> to_buf_list() macro, used ALIGN() macro in rcar_vin_setup(), added {} >>>>> to the >>>>> *if* statement and used 'bool' values instead of 0/1 where >>>>> necessary, removed >>>>> unused macros, done some reformatting and clarified some comments.] >>>>> Signed-off-by: Sergei Shtylyov >>>> Reviewing this iteration of the patch is still on my todo, in the >>>> meantime you might verify whether it works on top of the for-3.11-3 >>>> branch of my >>>> http://git.linuxtv.org/gliakhovetski/v4l-dvb.git >>>> git-tree, or "next" after it's been pulled by Mauro and pushed >>>> upstream. With that branch you shouldn't need any additional patches >>>> andy more. >>> Actually we need to apply/merge more patches here that enables VIN >>> support on separate platform (like pinctrl/clock/setup/) :) >>> Despite of above the rcar_vin driver works fine on Marzen board in >>> v4l-dvb.git after adding soc_camera_host_ops clock_start/clock_stop. >> Guennadi, does that mean that we should rebase the driver to the branch >> that you've named now? > IIUC, your last couple of versions were already developed on top of > v4l2-clk + v4l2-asybc + soc_scale_crop patches, right? No, I think only atop of the latter one. > But those patches > were out of tree, and thus unstable. Whereas now they've hit Mauro's tree > at git.linuxtv.org and are about to be pulled into "next." So, you don't > need anymore to apply any external patches, you will be able to just > develop on top of "next." I presume, this should make your work easier, > not harder. Just please make sure to double-check your stack on top of > "next" to make sure it still works. OK. > And let's try to get your driver ready for 3.12. Sigh, another couple months probably mean nothing for the media tree, but it may be disastrous for Renesas' customers which use the LTSI tree... > Thanks > Guennadi WBR, Sergei