Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1571943 > unrolled thread

[PATCH 0/2] [media] exynos-gsc: Fix support for NV21 and NV61 formats

Started byJavier Martinez Canillas <javier@osg.samsung.com>
First post2017-02-01 21:10 +0100
Last post2017-02-13 15:40 +0100
Articles 8 — 3 participants

Back to article view | Back to linux.kernel

Contents

  [PATCH 0/2] [media] exynos-gsc: Fix support for NV21 and NV61 formats Javier Martinez Canillas <javier@osg.samsung.com> - 2017-02-01 21:10 +0100
    [PATCH 2/2] [media] exynos-gsc: Add support for NV{16,21,61}M pixel formats Javier Martinez Canillas <javier@osg.samsung.com> - 2017-02-01 21:10 +0100
      Re: [PATCH 2/2] [media] exynos-gsc: Add support for NV{16,21,61}M  pixel formats Sylwester Nawrocki <s.nawrocki@samsung.com> - 2017-02-13 15:40 +0100
    [PATCH 1/2] [media] exynos-gsc: Do not swap cb/cr for semi planar formats Javier Martinez Canillas <javier@osg.samsung.com> - 2017-02-01 21:10 +0100
      Re: [PATCH 1/2] [media] exynos-gsc: Do not swap cb/cr for semi planar  formats Sylwester Nawrocki <s.nawrocki@samsung.com> - 2017-02-13 15:40 +0100
    Re: [PATCH 0/2] [media] exynos-gsc: Fix support for NV21 and NV61 formats Javier Martinez Canillas <javier@dowhile0.org> - 2017-02-13 14:00 +0100
      Re: [PATCH 0/2] [media] exynos-gsc: Fix support for NV21 and NV61  formats Sylwester Nawrocki <s.nawrocki@samsung.com> - 2017-02-13 15:40 +0100
        Re: [PATCH 0/2] [media] exynos-gsc: Fix support for NV21 and NV61  formats Javier Martinez Canillas <javier@osg.samsung.com> - 2017-02-13 15:40 +0100

#1571943 — [PATCH 0/2] [media] exynos-gsc: Fix support for NV21 and NV61 formats

FromJavier Martinez Canillas <javier@osg.samsung.com>
Date2017-02-01 21:10 +0100
Subject[PATCH 0/2] [media] exynos-gsc: Fix support for NV21 and NV61 formats
Message-ID<t69o5-26I-1@gated-at.bofh.it>
Hello,

Commit 652bb68018a5 ("[media] exynos-gsc: do proper bytesperline and
sizeimage calculation") fixed corrupted frames for most exynos-gsc
formats, but even after that patch two issues were still remaining:

1) Frames were still not correct for NV21 and NV61 formats.
2) Y42B format didn't work when used as output (only as input).

This patch series fixes (1).

Best regards,
Javier


Thibault Saunier (2):
  [media] exynos-gsc: Do not swap cb/cr for semi planar formats
  [media] exynos-gsc: Add support for NV{16,21,61}M pixel formats

 drivers/media/platform/exynos-gsc/gsc-core.c | 29 ++++++++++++++++++++++++++--
 1 file changed, 27 insertions(+), 2 deletions(-)

-- 
2.7.4

[toc] | [next] | [standalone]

#1571944 — [PATCH 2/2] [media] exynos-gsc: Add support for NV{16,21,61}M pixel formats

FromJavier Martinez Canillas <javier@osg.samsung.com>
Date2017-02-01 21:10 +0100
Subject[PATCH 2/2] [media] exynos-gsc: Add support for NV{16,21,61}M pixel formats
Message-ID<t69o5-26I-3@gated-at.bofh.it>
In reply to#1571943
From: Thibault Saunier <thibault.saunier@osg.samsung.com>

Those are useful formats that should be handled.

Signed-off-by: Thibault Saunier <thibault.saunier@osg.samsung.com>
Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>

---

 drivers/media/platform/exynos-gsc/gsc-core.c | 27 +++++++++++++++++++++++++++
 1 file changed, 27 insertions(+)

diff --git a/drivers/media/platform/exynos-gsc/gsc-core.c b/drivers/media/platform/exynos-gsc/gsc-core.c
index a846659ae5c1..eff636d4502b 100644
--- a/drivers/media/platform/exynos-gsc/gsc-core.c
+++ b/drivers/media/platform/exynos-gsc/gsc-core.c
@@ -112,6 +112,15 @@ static const struct gsc_fmt gsc_formats[] = {
 		.num_planes	= 1,
 		.num_comp	= 2,
 	}, {
+		.name		= "YUV 4:2:2 non-contig, Y/CbCr",
+		.pixelformat	= V4L2_PIX_FMT_NV16M,
+		.depth		= { 8, 8 },
+		.color		= GSC_YUV422,
+		.yorder		= GSC_LSB_Y,
+		.corder		= GSC_CBCR,
+		.num_planes	= 2,
+		.num_comp	= 2,
+	}, {
 		.name		= "YUV 4:2:2 planar, Y/CrCb",
 		.pixelformat	= V4L2_PIX_FMT_NV61,
 		.depth		= { 16 },
@@ -121,6 +130,15 @@ static const struct gsc_fmt gsc_formats[] = {
 		.num_planes	= 1,
 		.num_comp	= 2,
 	}, {
+		.name		= "YUV 4:2:2 non-contig, Y/CrCb",
+		.pixelformat	= V4L2_PIX_FMT_NV61M,
+		.depth		= { 8, 8 },
+		.color		= GSC_YUV422,
+		.yorder		= GSC_LSB_Y,
+		.corder		= GSC_CRCB,
+		.num_planes	= 2,
+		.num_comp	= 2,
+	}, {
 		.name		= "YUV 4:2:0 planar, YCbCr",
 		.pixelformat	= V4L2_PIX_FMT_YUV420,
 		.depth		= { 12 },
@@ -158,6 +176,15 @@ static const struct gsc_fmt gsc_formats[] = {
 		.num_planes	= 1,
 		.num_comp	= 2,
 	}, {
+		.name		= "YUV 4:2:0 non-contig. 2p, Y/CrCb",
+		.pixelformat	= V4L2_PIX_FMT_NV21M,
+		.depth		= { 8, 4 },
+		.color		= GSC_YUV420,
+		.yorder		= GSC_LSB_Y,
+		.corder		= GSC_CRCB,
+		.num_planes	= 2,
+		.num_comp	= 2,
+	}, {
 		.name		= "YUV 4:2:0 non-contig. 2p, Y/CbCr",
 		.pixelformat	= V4L2_PIX_FMT_NV12M,
 		.depth		= { 8, 4 },
-- 
2.7.4

[toc] | [prev] | [next] | [standalone]

#1579824 — Re: [PATCH 2/2] [media] exynos-gsc: Add support for NV{16,21,61}M pixel formats

FromSylwester Nawrocki <s.nawrocki@samsung.com>
Date2017-02-13 15:40 +0100
SubjectRe: [PATCH 2/2] [media] exynos-gsc: Add support for NV{16,21,61}M pixel formats
Message-ID<tapXk-4TG-35@gated-at.bofh.it>
In reply to#1571944
On 02/01/2017 09:05 PM, Javier Martinez Canillas wrote:
> From: Thibault Saunier <thibault.saunier@osg.samsung.com>
> 
> Those are useful formats that should be handled.
> 
> Signed-off-by: Thibault Saunier <thibault.saunier@osg.samsung.com>
> Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>

Acked-by: Sylwester Nawrocki <s.nawrocki@samsung.com>

[toc] | [prev] | [next] | [standalone]

#1571952 — [PATCH 1/2] [media] exynos-gsc: Do not swap cb/cr for semi planar formats

FromJavier Martinez Canillas <javier@osg.samsung.com>
Date2017-02-01 21:10 +0100
Subject[PATCH 1/2] [media] exynos-gsc: Do not swap cb/cr for semi planar formats
Message-ID<t69o6-26I-27@gated-at.bofh.it>
In reply to#1571943
From: Thibault Saunier <thibault.saunier@osg.samsung.com>

In the case of semi planar formats cb and cr are in the same plane
in memory, meaning that will be set to 'cb' whatever the format is,
and whatever the (packed) order of those components are.

Suggested-by: Nicolas Dufresne <nicolas.dufresne@collabora.com>
Signed-off-by: Thibault Saunier <thibault.saunier@osg.samsung.com>
Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
---

 drivers/media/platform/exynos-gsc/gsc-core.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/drivers/media/platform/exynos-gsc/gsc-core.c b/drivers/media/platform/exynos-gsc/gsc-core.c
index 40aff08dd51d..a846659ae5c1 100644
--- a/drivers/media/platform/exynos-gsc/gsc-core.c
+++ b/drivers/media/platform/exynos-gsc/gsc-core.c
@@ -861,9 +861,7 @@ int gsc_prepare_addr(struct gsc_ctx *ctx, struct vb2_buffer *vb,
 
 	if ((frame->fmt->pixelformat == V4L2_PIX_FMT_VYUY) ||
 		(frame->fmt->pixelformat == V4L2_PIX_FMT_YVYU) ||
-		(frame->fmt->pixelformat == V4L2_PIX_FMT_NV61) ||
 		(frame->fmt->pixelformat == V4L2_PIX_FMT_YVU420) ||
-		(frame->fmt->pixelformat == V4L2_PIX_FMT_NV21) ||
 		(frame->fmt->pixelformat == V4L2_PIX_FMT_YVU420M))
 		swap(addr->cb, addr->cr);
 
-- 
2.7.4

[toc] | [prev] | [next] | [standalone]

#1579821 — Re: [PATCH 1/2] [media] exynos-gsc: Do not swap cb/cr for semi planar formats

FromSylwester Nawrocki <s.nawrocki@samsung.com>
Date2017-02-13 15:40 +0100
SubjectRe: [PATCH 1/2] [media] exynos-gsc: Do not swap cb/cr for semi planar formats
Message-ID<tapXk-4TG-31@gated-at.bofh.it>
In reply to#1571952
On 02/01/2017 09:05 PM, Javier Martinez Canillas wrote:
> From: Thibault Saunier <thibault.saunier@osg.samsung.com>
> 
> In the case of semi planar formats cb and cr are in the same plane
> in memory, meaning that will be set to 'cb' whatever the format is,
> and whatever the (packed) order of those components are.
> 
> Suggested-by: Nicolas Dufresne <nicolas.dufresne@collabora.com>
> Signed-off-by: Thibault Saunier <thibault.saunier@osg.samsung.com>
> Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>

Acked-by: Sylwester Nawrocki <s.nawrocki@samsung.com>

[toc] | [prev] | [next] | [standalone]

#1579690

FromJavier Martinez Canillas <javier@dowhile0.org>
Date2017-02-13 14:00 +0100
Message-ID<taooy-3Oq-19@gated-at.bofh.it>
In reply to#1571943
Hello,

On Wed, Feb 1, 2017 at 5:05 PM, Javier Martinez Canillas
<javier@osg.samsung.com> wrote:
> Hello,
>
> Commit 652bb68018a5 ("[media] exynos-gsc: do proper bytesperline and
> sizeimage calculation") fixed corrupted frames for most exynos-gsc
> formats, but even after that patch two issues were still remaining:
>
> 1) Frames were still not correct for NV21 and NV61 formats.
> 2) Y42B format didn't work when used as output (only as input).
>
> This patch series fixes (1).
>
> Best regards,
> Javier
>
>
> Thibault Saunier (2):
>   [media] exynos-gsc: Do not swap cb/cr for semi planar formats
>   [media] exynos-gsc: Add support for NV{16,21,61}M pixel formats
>
>  drivers/media/platform/exynos-gsc/gsc-core.c | 29 ++++++++++++++++++++++++++--
>  1 file changed, 27 insertions(+), 2 deletions(-)

Any comments on this series?

Best regards,
Javier

[toc] | [prev] | [next] | [standalone]

#1579816 — Re: [PATCH 0/2] [media] exynos-gsc: Fix support for NV21 and NV61 formats

FromSylwester Nawrocki <s.nawrocki@samsung.com>
Date2017-02-13 15:40 +0100
SubjectRe: [PATCH 0/2] [media] exynos-gsc: Fix support for NV21 and NV61 formats
Message-ID<tapXj-4TG-5@gated-at.bofh.it>
In reply to#1579690
Hi Javier,

On 02/13/2017 01:53 PM, Javier Martinez Canillas wrote:
> Any comments on this series?

The patches look good to me, I will Ack the patches in case
Mauro wants to apply them directly.  Alternatively I will
add them to my tree for v4.12 after the merge window.

-- 
Thanks,
Sylwester

[toc] | [prev] | [next] | [standalone]

#1579822 — Re: [PATCH 0/2] [media] exynos-gsc: Fix support for NV21 and NV61 formats

FromJavier Martinez Canillas <javier@osg.samsung.com>
Date2017-02-13 15:40 +0100
SubjectRe: [PATCH 0/2] [media] exynos-gsc: Fix support for NV21 and NV61 formats
Message-ID<tapXj-4TG-25@gated-at.bofh.it>
In reply to#1579816
Hello Sylwester,

On 02/13/2017 11:34 AM, Sylwester Nawrocki wrote:
> Hi Javier,
> 
> On 02/13/2017 01:53 PM, Javier Martinez Canillas wrote:
>> Any comments on this series?
> 
> The patches look good to me, I will Ack the patches in case
> Mauro wants to apply them directly.  Alternatively I will
> add them to my tree for v4.12 after the merge window.
> 

Great, thanks a lot for your help!

Best regards,
-- 
Javier Martinez Canillas
Open Source Group
Samsung Research America

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web