From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from galahad.ideasonboard.com ([185.26.127.97]:48382 "EHLO galahad.ideasonboard.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752730AbdBMVgN (ORCPT ); Mon, 13 Feb 2017 16:36:13 -0500 From: Laurent Pinchart To: Kieran Bingham Cc: linux-media@vger.kernel.org, linux-renesas-soc@vger.kernel.org, kieran.bingham@ideasonboard.com Subject: Re: [PATCH 2/8] v4l: vsp1: Track the SRU entity in the pipeline Date: Mon, 13 Feb 2017 23:36:38 +0200 Message-ID: <1593556.4hIcOfm2vc@avalon> In-Reply-To: <8ad9d316c05ab254b65ad5230fb9232325e783ec.1486758327.git-series.kieran.bingham+renesas@ideasonboard.com> References: <8ad9d316c05ab254b65ad5230fb9232325e783ec.1486758327.git-series.kieran.bingham+renesas@ideasonboard.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Sender: linux-media-owner@vger.kernel.org List-ID: Hi Kieran, Thank you for the patch. On Friday 10 Feb 2017 20:27:30 Kieran Bingham wrote: > The UDS and other entities are already tracked directly through the > pipeline object. To follow the design pattern, and allow us to reference > the SRU convert the usage of 'sru_found' I propose squashing this into "v4l: vsp1: Implement partition algorithm restrictions" as it hasn't been merged yet. Or, given that that patch requires more work, we can also reorder them to merge this series first. > Signed-off-by: Kieran Bingham > --- > drivers/media/platform/vsp1/vsp1_pipe.h | 2 ++ > drivers/media/platform/vsp1/vsp1_video.c | 11 ++++++++--- > 2 files changed, 10 insertions(+), 3 deletions(-) > > diff --git a/drivers/media/platform/vsp1/vsp1_pipe.h > b/drivers/media/platform/vsp1/vsp1_pipe.h index bc419ef48d8d..5aa31143ce59 > 100644 > --- a/drivers/media/platform/vsp1/vsp1_pipe.h > +++ b/drivers/media/platform/vsp1/vsp1_pipe.h > @@ -76,6 +76,7 @@ enum vsp1_pipeline_state { > * @output: WPF at the output of the pipeline > * @bru: BRU entity, if present > * @lif: LIF entity, if present > + * @sru: SRU entity, if present > * @uds: UDS entity, if present > * @uds_input: entity at the input of the UDS, if the UDS is present > * @entities: list of entities in the pipeline > @@ -104,6 +105,7 @@ struct vsp1_pipeline { > struct vsp1_rwpf *output; > struct vsp1_entity *bru; > struct vsp1_entity *lif; > + struct vsp1_entity *sru; > struct vsp1_entity *uds; > struct vsp1_entity *uds_input; > > diff --git a/drivers/media/platform/vsp1/vsp1_video.c > b/drivers/media/platform/vsp1/vsp1_video.c index e2f242e7f0fa..be9c860b1c04 > 100644 > --- a/drivers/media/platform/vsp1/vsp1_video.c > +++ b/drivers/media/platform/vsp1/vsp1_video.c > @@ -473,7 +473,6 @@ static int vsp1_video_pipeline_build_branch(struct > vsp1_pipeline *pipe, struct vsp1_entity *entity; > struct media_pad *pad; > bool bru_found = false; > - bool sru_found = false; > int ret; > > ret = media_entity_enum_init(&ent_enum, &input->entity.vsp1- >media_dev); > @@ -523,6 +522,12 @@ static int vsp1_video_pipeline_build_branch(struct > vsp1_pipeline *pipe, if (entity->type == VSP1_ENTITY_SRU) { > struct vsp1_sru *sru = to_sru(&entity->subdev); > > + /* SRU can't be chained. */ > + if (pipe->sru) { > + ret = -EPIPE; > + goto out; > + } You can drop this, there's at most one SRU per VSP instance. > + > /* > * Gen3 partition algorithm restricts SRU double- scaled > * resolution if it is connected after a UDS entity > @@ -530,7 +535,7 @@ static int vsp1_video_pipeline_build_branch(struct > vsp1_pipeline *pipe, if (vsp1->info->gen == 3 && pipe->uds) > sru->force_identity_mode = true; > > - sru_found = true; > + pipe->sru = entity; > } > > if (entity->type == VSP1_ENTITY_UDS) { > @@ -546,7 +551,7 @@ static int vsp1_video_pipeline_build_branch(struct > vsp1_pipeline *pipe, * SRU on Gen3 will always engage the partition > * algorithm > */ > - if (vsp1->info->gen == 3 && sru_found) { > + if (vsp1->info->gen == 3 && pipe->sru) { > ret = -EPIPE; > goto out; > } You need to update vsp1_pipeline_reset() as well. -- Regards, Laurent Pinchart