Hi Akinobu,
Thanks for the patch.
On Sun, 2018-11-04 at 05:11:10 -0800, Akinobu Mita wrote:
> The fwnode_graph_get_next_endpoint() returns an 'endpoint' node pointer
> with refcount incremented, and refcount of the passed as a previous
> 'endpoint' node is decremented.
>
> So when iterating over all nodes using fwnode_graph_get_next_endpoint(),
> we don't need to call fwnode_handle_put() for each node except for error
> exit paths. Otherwise we get "OF: ERROR: Bad of_node_put() on ..."
> messages.
>
> Fixes: d079f94c9046 ("media: platform: Switch to v4l2_async_notifier_add_subdev")
> Cc: Steve Longerbeam <slongerbeam@xxxxxxxxx>
> Cc: Hyun Kwon <hyun.kwon@xxxxxxxxxx>
> Cc: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx>
> Cc: Mauro Carvalho Chehab <mchehab@xxxxxxxxxx>
> Signed-off-by: Akinobu Mita <akinobu.mita@xxxxxxxxx>
This looks good to me,
Reviewed-by: Hyun Kwon <hyun.kwon@xxxxxxxxxx>
Thanks,
-hyun
> ---
> drivers/media/platform/xilinx/xilinx-vipp.c | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/drivers/media/platform/xilinx/xilinx-vipp.c b/drivers/media/platform/xilinx/xilinx-vipp.c
> index 574614d..26b13fd 100644
> --- a/drivers/media/platform/xilinx/xilinx-vipp.c
> +++ b/drivers/media/platform/xilinx/xilinx-vipp.c
> @@ -377,8 +377,6 @@ static int xvip_graph_parse_one(struct xvip_composite_device *xdev,
> goto err_notifier_cleanup;
> }
>
> - fwnode_handle_put(ep);
> -
> /* Skip entities that we have already processed. */
> if (remote == of_fwnode_handle(xdev->dev->of_node) ||
> xvip_graph_find_entity(xdev, remote)) {
> --
> 2.7.4
>
