[media-ctl PATCH 1/1] media-ctl: Compose print fixes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



The compose rectangles were printed incorrectly in my recent patch "Compose
rectangle support for libv4l2subdev" without parenthesis. Fix this.

Signed-off-by: Sakari Ailus <sakari.ailus@xxxxxx>
---
Hi Laurent,

Could you apply this simple fix to your tree? Currently the compose
rectangles are printed differently than the crop rectangles which certainly
isn't the intention.

 src/main.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/main.c b/src/main.c
index af16818..d10094b 100644
--- a/src/main.c
+++ b/src/main.c
@@ -81,14 +81,14 @@ static void v4l2_subdev_print_format(struct media_entity *entity,
 					V4L2_SUBDEV_SEL_TGT_COMPOSE_BOUNDS,
 					which);
 	if (ret == 0)
-		printf("\n\t\t compose.bounds:%u,%u/%ux%u",
+		printf("\n\t\t compose.bounds:(%u,%u)/%ux%u",
 		       rect.left, rect.top, rect.width, rect.height);
 
 	ret = v4l2_subdev_get_selection(entity, &rect, pad,
 					V4L2_SUBDEV_SEL_TGT_COMPOSE_ACTUAL,
 					which);
 	if (ret == 0)
-		printf("\n\t\t compose:%u,%u/%ux%u",
+		printf("\n\t\t compose:(%u,%u)/%ux%u",
 		       rect.left, rect.top, rect.width, rect.height);
 
 	printf("]\n");
-- 
1.7.2.5

--
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


[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux