From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752412AbdCMLIh (ORCPT ); Mon, 13 Mar 2017 07:08:37 -0400 Received: from lb1-smtp-cloud6.xs4all.net ([194.109.24.24]:39532 "EHLO lb1-smtp-cloud6.xs4all.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751015AbdCMLI3 (ORCPT ); Mon, 13 Mar 2017 07:08:29 -0400 Subject: Re: [PATCH v4 14/36] [media] v4l2-mc: add a function to inherit controls from a pipeline To: Russell King - ARM Linux References: <20170311101408.272a9187@vento.lan> <20170311153229.yrdjmggb3p2suhdw@ihha.localdomain> <20170311184551.GD21222@n2100.armlinux.org.uk> <1f1b350a-5523-34bc-07b7-f3cd2d1fd4c1@gmail.com> <20170311185959.GF21222@n2100.armlinux.org.uk> <4917d7fb-2f48-17cd-aa2f-d54b0f19ed6e@gmail.com> <20170312073745.GI21222@n2100.armlinux.org.uk> <734a1731-3fb6-b8cd-6806-5405bd21bf83@xs4all.nl> <20170313105842.GG21222@n2100.armlinux.org.uk> Cc: Steve Longerbeam , mark.rutland@arm.com, andrew-ct.chen@mediatek.com, minghsiu.tsai@mediatek.com, nick@shmanahar.org, songjun.wu@microchip.com, pavel@ucw.cz, shuah@kernel.org, devel@driverdev.osuosl.org, markus.heiser@darmarIT.de, laurent.pinchart+renesas@ideasonboard.com, robert.jarzmik@free.fr, Mauro Carvalho Chehab , geert@linux-m68k.org, p.zabel@pengutronix.de, linux-media@vger.kernel.org, devicetree@vger.kernel.org, kernel@pengutronix.de, arnd@arndb.de, tiffany.lin@mediatek.com, bparrot@ti.com, robh+dt@kernel.org, horms+renesas@verge.net.au, mchehab@kernel.org, linux-arm-kernel@lists.infradead.org, niklas.soderlund+renesas@ragnatech.se, gregkh@linuxfoundation.org, linux-kernel@vger.kernel.org, Sakari Ailus , jean-christophe.trotin@st.com, sakari.ailus@linux.intel.com, fabio.estevam@nxp.com, shawnguo@kernel.org, sudipm.mukherjee@gmail.com From: Hans Verkuil Message-ID: Date: Mon, 13 Mar 2017 12:08:14 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.6.0 MIME-Version: 1.0 In-Reply-To: <20170313105842.GG21222@n2100.armlinux.org.uk> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 03/13/2017 11:58 AM, Russell King - ARM Linux wrote: > On Mon, Mar 13, 2017 at 11:44:50AM +0100, Hans Verkuil wrote: >> On 03/12/2017 06:56 PM, Steve Longerbeam wrote: >>> In summary, I do like the media framework, it's a good abstraction of >>> hardware pipelines. It does require a lot of system level knowledge to >>> configure, but as I said that is a matter of good documentation. >> >> And the reason we went into this direction is that the end-users that use >> these SoCs with complex pipelines actually *need* this functionality. Which >> is also part of the reason why work on improved userspace support gets >> little attention: they don't need to have a plugin that allows generic V4L2 >> applications to work (at least with simple scenarios). > > If you stop inheriting controls from the capture sensor to the v4l2 > capture device, then this breaks - generic v4l2 applications are not > going to be able to show the controls, because they're not visible at > the v4l2 capture device anymore. They're only visible through the > subdev interfaces, which these generic applications know nothing about. > >> If you want to blame anyone for this, blame Nokia who set fire to >> their linux-based phones and thus to the funding for this work. > > No, I think that's completely unfair to Nokia. If the MC approach is > the way you want to go, you should be thanking Nokia for the amount of > effort that they have put in to it, and recognising that it was rather > unfortunate that the market had changed, which meant that they weren't > able to continue. > > No one has any right to require any of us to finish what we start > coding up in open source, unless there is a contractual obligation in > place. That goes for Nokia too. > > Nokia's decision had ramifications far and wide (resulting in knock on > effects in TI and further afield), so don't think for a moment I wasn't > affected by what happened in Nokia. Even so, it was a decision for > Nokia to make, they had the right to make it, and we have no right to > attribute "blame" to Nokia for having made that decision. > > To even suggest that Nokia should be blamed is absurd. > > Open source gives rights to everyone. It gives rights to contribute > and use, but it also gives rights to walk away without notice (remember > the "as is" and "no warranty" clauses?) Sorry, unfortunate choice of words. While it lasted they did great work. But the reason why MC development stopped for quite some time (esp. the work on userspace software) was because the funding from Nokia dried up. Regards, Hans From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hans Verkuil Subject: Re: [PATCH v4 14/36] [media] v4l2-mc: add a function to inherit controls from a pipeline Date: Mon, 13 Mar 2017 12:08:14 +0100 Message-ID: References: <20170311101408.272a9187@vento.lan> <20170311153229.yrdjmggb3p2suhdw@ihha.localdomain> <20170311184551.GD21222@n2100.armlinux.org.uk> <1f1b350a-5523-34bc-07b7-f3cd2d1fd4c1@gmail.com> <20170311185959.GF21222@n2100.armlinux.org.uk> <4917d7fb-2f48-17cd-aa2f-d54b0f19ed6e@gmail.com> <20170312073745.GI21222@n2100.armlinux.org.uk> <734a1731-3fb6-b8cd-6806-5405bd21bf83@xs4all.nl> <20170313105842.GG21222@n2100.armlinux.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20170313105842.GG21222@n2100.armlinux.org.uk> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: driverdev-devel-bounces@linuxdriverproject.org Sender: "devel" To: Russell King - ARM Linux Cc: mark.rutland@arm.com, andrew-ct.chen@mediatek.com, minghsiu.tsai@mediatek.com, sakari.ailus@linux.intel.com, nick@shmanahar.org, songjun.wu@microchip.com, pavel@ucw.cz, robert.jarzmik@free.fr, devel@driverdev.osuosl.org, markus.heiser@darmarIT.de, laurent.pinchart+renesas@ideasonboard.com, shuah@kernel.org, Mauro Carvalho Chehab , geert@linux-m68k.org, Steve Longerbeam , linux-media@vger.kernel.org, devicetree@vger.kernel.org, kernel@pengutronix.de, arnd@arndb.de, mchehab@kernel.org, bparrot@ti.com, robh+dt@kernel.org, horms+renesas@verge.net.au, tiffany.lin@mediatek.com, linux-arm-kernel@lists.infradead.org, niklas.soderlund+renesas@ragnatech.se, gregkh@linuxfoundation.org, linux-kernel@vger.kernel.org, Sakari Ailus , jean-christophe.trotin@st.com, p.zabel@pengutronix.de, fabio.estevam@nxp.com, shawng List-Id: devicetree@vger.kernel.org On 03/13/2017 11:58 AM, Russell King - ARM Linux wrote: > On Mon, Mar 13, 2017 at 11:44:50AM +0100, Hans Verkuil wrote: >> On 03/12/2017 06:56 PM, Steve Longerbeam wrote: >>> In summary, I do like the media framework, it's a good abstraction of >>> hardware pipelines. It does require a lot of system level knowledge to >>> configure, but as I said that is a matter of good documentation. >> >> And the reason we went into this direction is that the end-users that use >> these SoCs with complex pipelines actually *need* this functionality. Which >> is also part of the reason why work on improved userspace support gets >> little attention: they don't need to have a plugin that allows generic V4L2 >> applications to work (at least with simple scenarios). > > If you stop inheriting controls from the capture sensor to the v4l2 > capture device, then this breaks - generic v4l2 applications are not > going to be able to show the controls, because they're not visible at > the v4l2 capture device anymore. They're only visible through the > subdev interfaces, which these generic applications know nothing about. > >> If you want to blame anyone for this, blame Nokia who set fire to >> their linux-based phones and thus to the funding for this work. > > No, I think that's completely unfair to Nokia. If the MC approach is > the way you want to go, you should be thanking Nokia for the amount of > effort that they have put in to it, and recognising that it was rather > unfortunate that the market had changed, which meant that they weren't > able to continue. > > No one has any right to require any of us to finish what we start > coding up in open source, unless there is a contractual obligation in > place. That goes for Nokia too. > > Nokia's decision had ramifications far and wide (resulting in knock on > effects in TI and further afield), so don't think for a moment I wasn't > affected by what happened in Nokia. Even so, it was a decision for > Nokia to make, they had the right to make it, and we have no right to > attribute "blame" to Nokia for having made that decision. > > To even suggest that Nokia should be blamed is absurd. > > Open source gives rights to everyone. It gives rights to contribute > and use, but it also gives rights to walk away without notice (remember > the "as is" and "no warranty" clauses?) Sorry, unfortunate choice of words. While it lasted they did great work. But the reason why MC development stopped for quite some time (esp. the work on userspace software) was because the funding from Nokia dried up. Regards, Hans From mboxrd@z Thu Jan 1 00:00:00 1970 From: hverkuil@xs4all.nl (Hans Verkuil) Date: Mon, 13 Mar 2017 12:08:14 +0100 Subject: [PATCH v4 14/36] [media] v4l2-mc: add a function to inherit controls from a pipeline In-Reply-To: <20170313105842.GG21222@n2100.armlinux.org.uk> References: <20170311101408.272a9187@vento.lan> <20170311153229.yrdjmggb3p2suhdw@ihha.localdomain> <20170311184551.GD21222@n2100.armlinux.org.uk> <1f1b350a-5523-34bc-07b7-f3cd2d1fd4c1@gmail.com> <20170311185959.GF21222@n2100.armlinux.org.uk> <4917d7fb-2f48-17cd-aa2f-d54b0f19ed6e@gmail.com> <20170312073745.GI21222@n2100.armlinux.org.uk> <734a1731-3fb6-b8cd-6806-5405bd21bf83@xs4all.nl> <20170313105842.GG21222@n2100.armlinux.org.uk> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 03/13/2017 11:58 AM, Russell King - ARM Linux wrote: > On Mon, Mar 13, 2017 at 11:44:50AM +0100, Hans Verkuil wrote: >> On 03/12/2017 06:56 PM, Steve Longerbeam wrote: >>> In summary, I do like the media framework, it's a good abstraction of >>> hardware pipelines. It does require a lot of system level knowledge to >>> configure, but as I said that is a matter of good documentation. >> >> And the reason we went into this direction is that the end-users that use >> these SoCs with complex pipelines actually *need* this functionality. Which >> is also part of the reason why work on improved userspace support gets >> little attention: they don't need to have a plugin that allows generic V4L2 >> applications to work (at least with simple scenarios). > > If you stop inheriting controls from the capture sensor to the v4l2 > capture device, then this breaks - generic v4l2 applications are not > going to be able to show the controls, because they're not visible at > the v4l2 capture device anymore. They're only visible through the > subdev interfaces, which these generic applications know nothing about. > >> If you want to blame anyone for this, blame Nokia who set fire to >> their linux-based phones and thus to the funding for this work. > > No, I think that's completely unfair to Nokia. If the MC approach is > the way you want to go, you should be thanking Nokia for the amount of > effort that they have put in to it, and recognising that it was rather > unfortunate that the market had changed, which meant that they weren't > able to continue. > > No one has any right to require any of us to finish what we start > coding up in open source, unless there is a contractual obligation in > place. That goes for Nokia too. > > Nokia's decision had ramifications far and wide (resulting in knock on > effects in TI and further afield), so don't think for a moment I wasn't > affected by what happened in Nokia. Even so, it was a decision for > Nokia to make, they had the right to make it, and we have no right to > attribute "blame" to Nokia for having made that decision. > > To even suggest that Nokia should be blamed is absurd. > > Open source gives rights to everyone. It gives rights to contribute > and use, but it also gives rights to walk away without notice (remember > the "as is" and "no warranty" clauses?) Sorry, unfortunate choice of words. While it lasted they did great work. But the reason why MC development stopped for quite some time (esp. the work on userspace software) was because the funding from Nokia dried up. Regards, Hans