Re: [PATCH v4 3/7] media: vsp1: Replace leftover occurrence of fragment with body

From: Kieran Bingham <kieran.bingham@ideasonboard.com>
To: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>,
	linux-media@vger.kernel.org, dri-devel@lists.freedesktop.org
Subject: Re: [PATCH v4 3/7] media: vsp1: Replace leftover occurrence of fragment with body
Date: Sun, 17 Feb 2019 20:20:53 +0000	[thread overview]
Message-ID: <5f2885f8-7d45-1447-02ba-608fc7933169@ideasonboard.com> (raw)
In-Reply-To: <20190217024852.23328-4-laurent.pinchart+renesas@ideasonboard.com>

Hi Laurent,

On 17/02/2019 02:48, Laurent Pinchart wrote:
> Display list fragments have been renamed to bodies. Replace one last
> occurrence of the word fragment in the documentation.
> 

Interesting, this must have been a change that was written before the
fragments were renamed, but got in /after/ they were removed or something.

> Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Regardless, it looks correct to fix this up.

Reviewed-by: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com>

> ---
>  drivers/media/platform/vsp1/vsp1_dl.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/media/platform/vsp1/vsp1_dl.c b/drivers/media/platform/vsp1/vsp1_dl.c
> index 26289adaf658..64af449791b0 100644
> --- a/drivers/media/platform/vsp1/vsp1_dl.c
> +++ b/drivers/media/platform/vsp1/vsp1_dl.c
> @@ -699,8 +699,8 @@ struct vsp1_dl_body *vsp1_dl_list_get_body0(struct vsp1_dl_list *dl)
>   * which bodies are added.
>   *
>   * Adding a body to a display list passes ownership of the body to the list. The
> - * caller retains its reference to the fragment when adding it to the display
> - * list, but is not allowed to add new entries to the body.
> + * caller retains its reference to the body when adding it to the display list,
> + * but is not allowed to add new entries to the body.
>   *
>   * The reference must be explicitly released by a call to vsp1_dl_body_put()
>   * when the body isn't needed anymore.
> 

-- 
Regards
--
Kieran