public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] s5p-jpeg: fix encoder and decoder video dev names
@ 2013-10-10  8:45 Seung-Woo Kim
  2013-10-12  9:41 ` Sylwester Nawrocki
  0 siblings, 1 reply; 2+ messages in thread
From: Seung-Woo Kim @ 2013-10-10  8:45 UTC (permalink / raw)
  To: linux-media, linux-samsung-soc, m.chehab, s.nawrocki; +Cc: sw0312.kim

It is hard to distinguish between decoder and encoder video device
because their names are same. So this patch fixes the names.

Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>
---
 drivers/media/platform/s5p-jpeg/jpeg-core.c |    8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/media/platform/s5p-jpeg/jpeg-core.c b/drivers/media/platform/s5p-jpeg/jpeg-core.c
index 15d2396..d5b4a0d 100644
--- a/drivers/media/platform/s5p-jpeg/jpeg-core.c
+++ b/drivers/media/platform/s5p-jpeg/jpeg-core.c
@@ -1387,8 +1387,8 @@ static int s5p_jpeg_probe(struct platform_device *pdev)
 		ret = -ENOMEM;
 		goto vb2_allocator_rollback;
 	}
-	strlcpy(jpeg->vfd_encoder->name, S5P_JPEG_M2M_NAME,
-		sizeof(jpeg->vfd_encoder->name));
+	snprintf(jpeg->vfd_encoder->name, sizeof(jpeg->vfd_encoder->name),
+				"%s-enc", S5P_JPEG_M2M_NAME);
 	jpeg->vfd_encoder->fops		= &s5p_jpeg_fops;
 	jpeg->vfd_encoder->ioctl_ops	= &s5p_jpeg_ioctl_ops;
 	jpeg->vfd_encoder->minor	= -1;
@@ -1415,8 +1415,8 @@ static int s5p_jpeg_probe(struct platform_device *pdev)
 		ret = -ENOMEM;
 		goto enc_vdev_register_rollback;
 	}
-	strlcpy(jpeg->vfd_decoder->name, S5P_JPEG_M2M_NAME,
-		sizeof(jpeg->vfd_decoder->name));
+	snprintf(jpeg->vfd_decoder->name, sizeof(jpeg->vfd_decoder->name),
+				"%s-dec", S5P_JPEG_M2M_NAME);
 	jpeg->vfd_decoder->fops		= &s5p_jpeg_fops;
 	jpeg->vfd_decoder->ioctl_ops	= &s5p_jpeg_ioctl_ops;
 	jpeg->vfd_decoder->minor	= -1;
-- 
1.7.4.1


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] s5p-jpeg: fix encoder and decoder video dev names
  2013-10-10  8:45 [PATCH] s5p-jpeg: fix encoder and decoder video dev names Seung-Woo Kim
@ 2013-10-12  9:41 ` Sylwester Nawrocki
  0 siblings, 0 replies; 2+ messages in thread
From: Sylwester Nawrocki @ 2013-10-12  9:41 UTC (permalink / raw)
  To: Seung-Woo Kim; +Cc: linux-media, linux-samsung-soc, m.chehab, s.nawrocki

On 10/10/2013 10:45 AM, Seung-Woo Kim wrote:
> It is hard to distinguish between decoder and encoder video device
> because their names are same. So this patch fixes the names.
>
> Signed-off-by: Seung-Woo Kim<sw0312.kim@samsung.com>

Patch queued for 3.13.

Thanks,
Sylwester

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2013-10-12  9:41 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-10-10  8:45 [PATCH] s5p-jpeg: fix encoder and decoder video dev names Seung-Woo Kim
2013-10-12  9:41 ` Sylwester Nawrocki

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox