Hi Sakari,
Thank you for the patch.
On Friday 09 Dec 2016 16:53:38 Sakari Ailus wrote:
> Instead of double quotes, use single quotes to quote entity names. Using
> single quotes is consistent with the English language and is also in line
> with the practices across the kernel.
>
> Signed-off-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx>
Reviewed-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx>
> ---
> drivers/media/media-entity.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/media/media-entity.c b/drivers/media/media-entity.c
> index 1de28ce..5064ba0 100644
> --- a/drivers/media/media-entity.c
> +++ b/drivers/media/media-entity.c
> @@ -444,7 +444,7 @@ __must_check int __media_pipeline_start(struct
> media_entity *entity, ret = entity->ops->link_validate(link);
> if (ret < 0 && ret != -ENOIOCTLCMD) {
> dev_dbg(entity->graph_obj.mdev->dev,
> - "link validation failed for \"%s\":%u
-> \"%s\":%u, error %d\n",
> + "link validation failed for '%s':%u ->
'%s':%u, error %d\n",
> link->source->entity->name,
> link->source->index,
> entity->name, link->sink->index, ret);
> @@ -458,7 +458,7 @@ __must_check int __media_pipeline_start(struct
> media_entity *entity, if (!bitmap_full(active, entity->num_pads)) {
> ret = -ENOLINK;
> dev_dbg(entity->graph_obj.mdev->dev,
> - "\"%s\":%u must be connected by an enabled
link\n",
> + "'%s':%u must be connected by an enabled
link\n",
> entity->name,
> (unsigned)find_first_zero_bit(
> active, entity->num_pads));
--
Regards,
Laurent Pinchart
--
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
