Em Sun, 29 Nov 2015 21:20:12 +0200
Sakari Ailus <sakari.ailus@xxxxxx> escreveu:
description is missing. Patch looks ok, except for that.
> Signed-off-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx>
> Cc: Hyun Kwon <hyun.kwon@xxxxxxxxxx>
> ---
> drivers/media/platform/xilinx/xilinx-dma.c | 9 +++++++++
> 1 file changed, 9 insertions(+)
>
> diff --git a/drivers/media/platform/xilinx/xilinx-dma.c b/drivers/media/platform/xilinx/xilinx-dma.c
> index bc244a0..0a19824 100644
> --- a/drivers/media/platform/xilinx/xilinx-dma.c
> +++ b/drivers/media/platform/xilinx/xilinx-dma.c
> @@ -182,10 +182,17 @@ static int xvip_pipeline_validate(struct xvip_pipeline *pipe,
> struct media_device *mdev = entity->graph_obj.mdev;
> unsigned int num_inputs = 0;
> unsigned int num_outputs = 0;
> + int ret;
>
> mutex_lock(&mdev->graph_mutex);
>
> /* Walk the graph to locate the video nodes. */
> + ret = media_entity_graph_walk_init(&graph, entity->graph_obj.mdev);
> + if (ret) {
> + mutex_unlock(&mdev->graph_mutex);
> + return ret;
> + }
> +
> media_entity_graph_walk_start(&graph, entity);
>
> while ((entity = media_entity_graph_walk_next(&graph))) {
> @@ -206,6 +213,8 @@ static int xvip_pipeline_validate(struct xvip_pipeline *pipe,
>
> mutex_unlock(&mdev->graph_mutex);
>
> + media_entity_graph_walk_cleanup(&graph);
> +
> /* We need exactly one output and zero or one input. */
> if (num_outputs != 1 || num_inputs > 1)
> return -EPIPE;
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
