From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mauro Carvalho Chehab Subject: Re: [PATCH RFC v3 07/16] media: get rid of unused "extra_links" param on media_entity_init() Date: Fri, 14 Aug 2015 08:07:52 -0300 Message-ID: <20150814080752.1e2d1311@recife.lan> References: <7b9f2654888f0fd54c290c50f50248a367768da7.1439410053.git.mchehab@osg.samsung.com> <20150814103348.GC19840@valkosipuli.retiisi.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20150814103348.GC19840@valkosipuli.retiisi.org.uk> Sender: linux-doc-owner@vger.kernel.org To: Sakari Ailus Cc: Linux Media Mailing List , Mauro Carvalho Chehab , Jonathan Corbet , Harry Wei , Hans Verkuil , Lars-Peter Clausen , Laurent Pinchart , Kyungmin Park , Heungjun Kim , Prabhakar Lad , Andrzej Hajda , Mats Randgaard , Sylwester Nawrocki , Kukjin Kim , Krzysztof Kozlowski , Hyun Kwon , Michal Simek , =?UTF-8?B?U8O2cmVu?= Brinkmann , Greg Kroah-Hartman , Joe Perches , Boris BREZILLON List-Id: linux-samsung-soc@vger.kernel.org Em Fri, 14 Aug 2015 13:33:48 +0300 Sakari Ailus escreveu: > Hi Mauro, > > On Wed, Aug 12, 2015 at 05:14:51PM -0300, Mauro Carvalho Chehab wrote: > > Currently, media_entity_init() creates an array with the links, > > allocated at init time. It provides a parameter (extra_links) > > that would allocate more links than the current needs, but this > > is not used by any driver. > > > > As we want to be able to do dynamic link allocation/removal, > > we'll need to change the implementation of the links. So, > > before doing that, let's first remove that extra unused > > parameter, in order to cleanup the interface first. > > > > Signed-off-by: Mauro Carvalho Chehab > > > ... > > > > diff --git a/include/media/media-entity.h b/include/media/media-entity.h > > index 738e1d5d25dc..be6885e7c8ed 100644 > > --- a/include/media/media-entity.h > > +++ b/include/media/media-entity.h > > @@ -177,7 +177,7 @@ void graph_obj_init(struct media_device *mdev, > > void graph_obj_remove(struct media_graph_obj *gobj); > > > > int media_entity_init(struct media_entity *entity, u16 num_pads, > > - struct media_pad *pads, u16 extra_links); > > + struct media_pad *pads); > > void media_entity_cleanup(struct media_entity *entity); > > > > int media_entity_create_link(struct media_entity *source, u16 source_pad, > > How about putting this in front of the set? It has no dependencies to the > other patches, does it? Yeah this patch can be the first one ;) It just cleans up something that we never used. It just needs to be before patch 8. > > Acked-by: Sakari Ailus > From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mauro Carvalho Chehab Date: Fri, 14 Aug 2015 11:07:52 +0000 Subject: Re: [PATCH RFC v3 07/16] media: get rid of unused "extra_links" param on media_entity_init() Message-Id: <20150814080752.1e2d1311@recife.lan> List-Id: References: <7b9f2654888f0fd54c290c50f50248a367768da7.1439410053.git.mchehab@osg.samsung.com> <20150814103348.GC19840@valkosipuli.retiisi.org.uk> In-Reply-To: <20150814103348.GC19840@valkosipuli.retiisi.org.uk> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: linux-arm-kernel@lists.infradead.org Em Fri, 14 Aug 2015 13:33:48 +0300 Sakari Ailus escreveu: > Hi Mauro, > > On Wed, Aug 12, 2015 at 05:14:51PM -0300, Mauro Carvalho Chehab wrote: > > Currently, media_entity_init() creates an array with the links, > > allocated at init time. It provides a parameter (extra_links) > > that would allocate more links than the current needs, but this > > is not used by any driver. > > > > As we want to be able to do dynamic link allocation/removal, > > we'll need to change the implementation of the links. So, > > before doing that, let's first remove that extra unused > > parameter, in order to cleanup the interface first. > > > > Signed-off-by: Mauro Carvalho Chehab > > > ... > > > > diff --git a/include/media/media-entity.h b/include/media/media-entity.h > > index 738e1d5d25dc..be6885e7c8ed 100644 > > --- a/include/media/media-entity.h > > +++ b/include/media/media-entity.h > > @@ -177,7 +177,7 @@ void graph_obj_init(struct media_device *mdev, > > void graph_obj_remove(struct media_graph_obj *gobj); > > > > int media_entity_init(struct media_entity *entity, u16 num_pads, > > - struct media_pad *pads, u16 extra_links); > > + struct media_pad *pads); > > void media_entity_cleanup(struct media_entity *entity); > > > > int media_entity_create_link(struct media_entity *source, u16 source_pad, > > How about putting this in front of the set? It has no dependencies to the > other patches, does it? Yeah this patch can be the first one ;) It just cleans up something that we never used. It just needs to be before patch 8. > > Acked-by: Sakari Ailus > From mboxrd@z Thu Jan 1 00:00:00 1970 From: mchehab@osg.samsung.com (Mauro Carvalho Chehab) Date: Fri, 14 Aug 2015 08:07:52 -0300 Subject: [PATCH RFC v3 07/16] media: get rid of unused "extra_links" param on media_entity_init() In-Reply-To: <20150814103348.GC19840@valkosipuli.retiisi.org.uk> References: <7b9f2654888f0fd54c290c50f50248a367768da7.1439410053.git.mchehab@osg.samsung.com> <20150814103348.GC19840@valkosipuli.retiisi.org.uk> Message-ID: <20150814080752.1e2d1311@recife.lan> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Em Fri, 14 Aug 2015 13:33:48 +0300 Sakari Ailus escreveu: > Hi Mauro, > > On Wed, Aug 12, 2015 at 05:14:51PM -0300, Mauro Carvalho Chehab wrote: > > Currently, media_entity_init() creates an array with the links, > > allocated at init time. It provides a parameter (extra_links) > > that would allocate more links than the current needs, but this > > is not used by any driver. > > > > As we want to be able to do dynamic link allocation/removal, > > we'll need to change the implementation of the links. So, > > before doing that, let's first remove that extra unused > > parameter, in order to cleanup the interface first. > > > > Signed-off-by: Mauro Carvalho Chehab > > > ... > > > > diff --git a/include/media/media-entity.h b/include/media/media-entity.h > > index 738e1d5d25dc..be6885e7c8ed 100644 > > --- a/include/media/media-entity.h > > +++ b/include/media/media-entity.h > > @@ -177,7 +177,7 @@ void graph_obj_init(struct media_device *mdev, > > void graph_obj_remove(struct media_graph_obj *gobj); > > > > int media_entity_init(struct media_entity *entity, u16 num_pads, > > - struct media_pad *pads, u16 extra_links); > > + struct media_pad *pads); > > void media_entity_cleanup(struct media_entity *entity); > > > > int media_entity_create_link(struct media_entity *source, u16 source_pad, > > How about putting this in front of the set? It has no dependencies to the > other patches, does it? Yeah this patch can be the first one ;) It just cleans up something that we never used. It just needs to be before patch 8. > > Acked-by: Sakari Ailus > From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from lists.s-osg.org ([54.187.51.154]:57747 "EHLO lists.s-osg.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754059AbbHNLIH (ORCPT ); Fri, 14 Aug 2015 07:08:07 -0400 Date: Fri, 14 Aug 2015 08:07:52 -0300 From: Mauro Carvalho Chehab To: Sakari Ailus Cc: Linux Media Mailing List , Mauro Carvalho Chehab , Jonathan Corbet , Harry Wei , Hans Verkuil , Lars-Peter Clausen , Laurent Pinchart , Kyungmin Park , Heungjun Kim , Prabhakar Lad , Andrzej Hajda , Mats Randgaard , Sylwester Nawrocki , Kukjin Kim , Krzysztof Kozlowski , Hyun Kwon , Michal Simek , =?UTF-8?B?U8O2cmVu?= Brinkmann , Greg Kroah-Hartman , Joe Perches , Boris BREZILLON , Guennadi Liakhovetski , Ricardo Ribalda Delgado , Axel Lin , Bryan Wu , Jacek Anaszewski , Aya Mahfouz , Haneen Mohammed , anuvazhayil , Mahati Chamarthy , Navya Sri Nizamkari , Tapasweni Pathak , linux-doc@vger.kernel.org, linux-kernel@zh-kernel.org, linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, linux-sh@vger.kernel.org, devel@driverdev.osuosl.org Subject: Re: [PATCH RFC v3 07/16] media: get rid of unused "extra_links" param on media_entity_init() Message-ID: <20150814080752.1e2d1311@recife.lan> In-Reply-To: <20150814103348.GC19840@valkosipuli.retiisi.org.uk> References: <7b9f2654888f0fd54c290c50f50248a367768da7.1439410053.git.mchehab@osg.samsung.com> <20150814103348.GC19840@valkosipuli.retiisi.org.uk> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-media-owner@vger.kernel.org List-ID: Em Fri, 14 Aug 2015 13:33:48 +0300 Sakari Ailus escreveu: > Hi Mauro, > > On Wed, Aug 12, 2015 at 05:14:51PM -0300, Mauro Carvalho Chehab wrote: > > Currently, media_entity_init() creates an array with the links, > > allocated at init time. It provides a parameter (extra_links) > > that would allocate more links than the current needs, but this > > is not used by any driver. > > > > As we want to be able to do dynamic link allocation/removal, > > we'll need to change the implementation of the links. So, > > before doing that, let's first remove that extra unused > > parameter, in order to cleanup the interface first. > > > > Signed-off-by: Mauro Carvalho Chehab > > > ... > > > > diff --git a/include/media/media-entity.h b/include/media/media-entity.h > > index 738e1d5d25dc..be6885e7c8ed 100644 > > --- a/include/media/media-entity.h > > +++ b/include/media/media-entity.h > > @@ -177,7 +177,7 @@ void graph_obj_init(struct media_device *mdev, > > void graph_obj_remove(struct media_graph_obj *gobj); > > > > int media_entity_init(struct media_entity *entity, u16 num_pads, > > - struct media_pad *pads, u16 extra_links); > > + struct media_pad *pads); > > void media_entity_cleanup(struct media_entity *entity); > > > > int media_entity_create_link(struct media_entity *source, u16 source_pad, > > How about putting this in front of the set? It has no dependencies to the > other patches, does it? Yeah this patch can be the first one ;) It just cleans up something that we never used. It just needs to be before patch 8. > > Acked-by: Sakari Ailus >