From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail-lb0-f182.google.com ([209.85.217.182]:43614 "EHLO mail-lb0-f182.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755493Ab3FBS41 (ORCPT ); Sun, 2 Jun 2013 14:56:27 -0400 Received: by mail-lb0-f182.google.com with SMTP id q15so866625lbi.27 for ; Sun, 02 Jun 2013 11:56:25 -0700 (PDT) Message-ID: <51AB9556.2080900@cogentembedded.com> Date: Sun, 02 Jun 2013 22:56:22 +0400 From: Sergei Shtylyov MIME-Version: 1.0 To: g.liakhovetski@gmx.de, mchehab@redhat.com, linux-media@vger.kernel.org CC: magnus.damm@gmail.com, linux-sh@vger.kernel.org, phil.edworthy@renesas.com, matsu@igel.co.jp, vladimir.barinov@cogentembedded.com Subject: Re: [PATCH v6] V4L2: soc_camera: Renesas R-Car VIN driver References: <201305240211.29665.sergei.shtylyov@cogentembedded.com> In-Reply-To: <201305240211.29665.sergei.shtylyov@cogentembedded.com> 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 05/24/2013 02:11 AM, Sergei Shtylyov 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 > > --- > This patch is against the 'media_tree.git' repo. > It requires two following patches from Guennadi Liakhovetski: > > https://patchwork.linuxtv.org/patch/18209/ > https://patchwork.linuxtv.org/patch/18210/ > [...] > Index: media_tree/include/linux/platform_data/camera-rcar.h > =================================================================== > --- /dev/null > +++ media_tree/include/linux/platform_data/camera-rcar.h > @@ -0,0 +1,25 @@ > +/* > + * Platform data for Renesas R-Car VIN soc-camera driver > + * > + * Copyright (C) 2011-2013 Renesas Solutions Corp. > + * Copyright (C) 2013 Cogent Embedded, Inc., > + * > + * This program is free software; you can redistribute it and/or modify it > + * under the terms of the GNU General Public License as published by the > + * Free Software Foundation; either version 2 of the License, or (at your > + * option) any later version. > + */ > + > +#ifndef __CAMERA_RCAR_H_ > +#define __CAMERA_RCAR_H_ > + > +#define RCAR_VIN_HSYNC_ACTIVE_LOW (1 << 0) > +#define RCAR_VIN_VSYNC_ACTIVE_LOW (1 << 1) > +#define RCAR_VIN_BT601 (1 << 2) > +#define RCAR_VIN_BT656 (1 << 3) > + > +struct rcar_vin_platform_data { > + unsigned int flags; > +}; > + > +#endif /* __CAMERA_RCAR_H_ */ I wonder how to deal with a cross tree dependency caused by this file. Perhaps the VIN platform code could be merged thru the media tree (or maybe vice versa, all patches merged thru the Renesas tree)? WBR, Sergei