From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail-lf0-f49.google.com ([209.85.215.49]:36029 "EHLO mail-lf0-f49.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752293AbcCBVpG (ORCPT ); Wed, 2 Mar 2016 16:45:06 -0500 Received: by mail-lf0-f49.google.com with SMTP id l83so1432329lfd.3 for ; Wed, 02 Mar 2016 13:45:04 -0800 (PST) Date: Wed, 2 Mar 2016 22:45:01 +0100 From: Niklas =?iso-8859-1?Q?S=F6derlund?= To: Yoshihiro Kaneko Cc: Hans Verkuil , Linux Media Mailing List , Guennadi Liakhovetski , Simon Horman , Magnus Damm , linux-renesas-soc@vger.kernel.org Subject: Re: [PATCH/RFC 0/4] media: soc_camera: rcar_vin: Add UDS and NV16 scaling support Message-ID: <20160302214501.GE7079@bigcity.dyn.berto.se> References: <1456751563-21246-1-git-send-email-ykaneko0929@gmail.com> <56D4475A.1070203@xs4all.nl> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: Sender: linux-media-owner@vger.kernel.org List-ID: Hi Kaneko-san, On 2016-03-03 02:26:41 +0900, Yoshihiro Kaneko wrote: > Hi Hans, > > 2016-02-29 22:27 GMT+09:00 Hans Verkuil : > > Huh, you must have missed Niklas's work the rcar-vin driver: > > > > http://www.spinics.net/lists/linux-media/msg97816.html > > > > I expect that the old soc-camera driver will be retired soon in favor of > > the new driver, so I don't want to accept patches for that one. > > > > I recommend that you check the new driver and see what (if anything) is needed > > to get this functionality in there and work with Niklas on this. > > > > This is all quite recent work, so it is not surprising that you missed it. > > Thank you for informing me! > I will check it. My plan is to look at VIN for Gen3 once Gen2 support is done. I have somewhat tried to keep the new driver prepared for Gen3. I have separating the Gen2 scaler och clipper out in its own corner since this will be different on Gen3. My understanding is however that Gen3 don't provide UDS blocks (scaler+clipper) to all VIN instances. And the VIN instances that have access to a UDS block have to share it with one other VIN instance (only one user at a time). How to describe this in DT in a good way I do not yet know. If you have any ideas here or know more I would be glad to hear it, I have not yet started any work for Gen3. My initial plan for Gen3 enablement is to ignore the UDS blocks all together. I feel there is enough to adapt VIN driver and get both the CSI2 and sensor driver to work to be able to test the whole chain without worrying about UDS too. > > > > > Regards, > > > > Hans > > Regards, > kaneko > > > > > On 02/29/2016 02:12 PM, Yoshihiro Kaneko wrote: > >> This series adds UDS support, NV16 scaling support and callback functions > >> to be required by a clipping process. > >> > >> This series is against the master branch of linuxtv.org/media_tree.git. > >> > >> Koji Matsuoka (3): > >> media: soc_camera: rcar_vin: Add get_selection callback function > >> media: soc_camera: rcar_vin: Add cropcap callback function > >> media: soc_camera: rcar_vin: Add NV16 scaling support > >> > >> Yoshihiko Mori (1): > >> media: soc_camera: rcar_vin: Add UDS support > >> > >> drivers/media/platform/soc_camera/rcar_vin.c | 220 ++++++++++++++++++++++----- > >> 1 file changed, 184 insertions(+), 36 deletions(-) > >> > > -- Regards, Niklas Söderlund