* [PATCH libdrm] modetest: add C8 support to generate SMPTE pattern
@ 2017-11-18 4:56 Ilia Mirkin
2017-11-18 5:19 ` Ilia Mirkin
0 siblings, 1 reply; 2+ messages in thread
From: Ilia Mirkin @ 2017-11-18 4:56 UTC (permalink / raw)
To: dri-devel
Tested on nouveau with the CRTC only.
Signed-off-by: Ilia Mirkin <imirkin@alum.mit.edu>
---
Please note that I have no clue what the proper way to operate the gamma
interface is. This seemed OK, but the 256 seems awefully hardcoded. Perhaps
that won't work on other HW?
tests/modetest/buffers.c | 2 ++
tests/modetest/modetest.c | 13 +++++++++
tests/util/format.c | 2 ++
tests/util/pattern.c | 73 +++++++++++++++++++++++++++++++++++++++++++++++
tests/util/pattern.h | 2 ++
5 files changed, 92 insertions(+)
diff --git a/tests/modetest/buffers.c b/tests/modetest/buffers.c
index 4fd310b9..58f88e85 100644
--- a/tests/modetest/buffers.c
+++ b/tests/modetest/buffers.c
@@ -139,6 +139,7 @@ bo_create(int fd, unsigned int format,
int ret;
switch (format) {
+ case DRM_FORMAT_C8:
case DRM_FORMAT_NV12:
case DRM_FORMAT_NV21:
case DRM_FORMAT_NV16:
@@ -279,6 +280,7 @@ bo_create(int fd, unsigned int format,
planes[2] = virtual + offsets[2];
break;
+ case DRM_FORMAT_C8:
case DRM_FORMAT_ARGB4444:
case DRM_FORMAT_XRGB4444:
case DRM_FORMAT_ABGR4444:
diff --git a/tests/modetest/modetest.c b/tests/modetest/modetest.c
index 62d93327..049bdd5e 100644
--- a/tests/modetest/modetest.c
+++ b/tests/modetest/modetest.c
@@ -1228,6 +1228,19 @@ static void set_mode(struct device *dev, struct pipe_arg *pipes, unsigned int co
fprintf(stderr, "failed to set mode: %s\n", strerror(errno));
return;
}
+
+ if (pipes[0].fourcc == DRM_FORMAT_C8) {
+ uint16_t r[256], g[256], b[256];
+
+ util_smpte_c8_gamma(256, r, g, b);
+ ret = drmModeCrtcSetGamma(
+ dev->fd, pipe->crtc->crtc->crtc_id,
+ 256, r, g, b);
+ if (ret) {
+ fprintf(stderr, "failed to set gamma: %s\n", strerror(errno));
+ return;
+ }
+ }
}
}
diff --git a/tests/util/format.c b/tests/util/format.c
index 043cfe7f..3eefe224 100644
--- a/tests/util/format.c
+++ b/tests/util/format.c
@@ -43,6 +43,8 @@
.yuv = { (order), (xsub), (ysub), (chroma_stride) }
static const struct util_format_info format_info[] = {
+ /* Indexed */
+ { DRM_FORMAT_C8, "C8" },
/* YUV packed */
{ DRM_FORMAT_UYVY, "UYVY", MAKE_YUV_INFO(YUV_YCbCr | YUV_CY, 2, 2, 2) },
{ DRM_FORMAT_VYUY, "VYUY", MAKE_YUV_INFO(YUV_YCrCb | YUV_CY, 2, 2, 2) },
diff --git a/tests/util/pattern.c b/tests/util/pattern.c
index 00b08a8c..41fb541b 100644
--- a/tests/util/pattern.c
+++ b/tests/util/pattern.c
@@ -461,6 +461,77 @@ static void fill_smpte_rgb32(const struct util_rgb_info *rgb, void *mem,
}
}
+static void fill_smpte_c8(void *mem, unsigned int width, unsigned int height,
+ unsigned int stride)
+{
+ unsigned int x;
+ unsigned int y;
+
+ for (y = 0; y < height * 6 / 9; ++y) {
+ for (x = 0; x < width; ++x)
+ ((uint8_t *)mem)[x] = x * 7 / width;
+ mem += stride;
+ }
+
+ for (; y < height * 7 / 9; ++y) {
+ for (x = 0; x < width; ++x)
+ ((uint8_t *)mem)[x] = 7 + (x * 7 / width);
+ mem += stride;
+ }
+
+ for (; y < height; ++y) {
+ for (x = 0; x < width * 5 / 7; ++x)
+ ((uint8_t *)mem)[x] =
+ 14 + (x * 4 / (width * 5 / 7));
+ for (; x < width * 6 / 7; ++x)
+ ((uint8_t *)mem)[x] =
+ 14 + ((x - width * 5 / 7) * 3
+ / (width / 7) + 4);
+ for (; x < width; ++x)
+ ((uint8_t *)mem)[x] = 14 + 7;
+ mem += stride;
+ }
+}
+
+void util_smpte_c8_gamma(unsigned size, uint16_t *r, uint16_t *g, uint16_t *b)
+{
+ if (size < 7 + 7 + 8) {
+ printf("Error: gamma too small: %d < %d\n", size, 7 + 7 + 8);
+ return;
+ }
+#define FILL_COLOR(idx, red, green, blue) \
+ r[idx] = red << 8; \
+ g[idx] = green << 8; \
+ b[idx] = blue << 8
+
+ FILL_COLOR( 0, 192, 192, 192); /* grey */
+ FILL_COLOR( 1, 192, 192, 0 ); /* yellow */
+ FILL_COLOR( 2, 0, 192, 192); /* cyan */
+ FILL_COLOR( 3, 0, 192, 0 ); /* green */
+ FILL_COLOR( 4, 192, 0, 192); /* magenta */
+ FILL_COLOR( 5, 192, 0, 0 ); /* red */
+ FILL_COLOR( 6, 0, 0, 192); /* blue */
+
+ FILL_COLOR( 7, 0, 0, 192); /* blue */
+ FILL_COLOR( 8, 19, 19, 19 ); /* black */
+ FILL_COLOR( 9, 192, 0, 192); /* magenta */
+ FILL_COLOR(10, 19, 19, 19 ); /* black */
+ FILL_COLOR(11, 0, 192, 192); /* cyan */
+ FILL_COLOR(12, 19, 19, 19 ); /* black */
+ FILL_COLOR(13, 192, 192, 192); /* grey */
+
+ FILL_COLOR(14, 0, 33, 76); /* in-phase */
+ FILL_COLOR(15, 255, 255, 255); /* super white */
+ FILL_COLOR(16, 50, 0, 106); /* quadrature */
+ FILL_COLOR(17, 19, 19, 19); /* black */
+ FILL_COLOR(18, 9, 9, 9); /* 3.5% */
+ FILL_COLOR(19, 19, 19, 19); /* 7.5% */
+ FILL_COLOR(20, 29, 29, 29); /* 11.5% */
+ FILL_COLOR(21, 19, 19, 19); /* black */
+
+#undef FILL_COLOR
+}
+
static void fill_smpte(const struct util_format_info *info, void *planes[3],
unsigned int width, unsigned int height,
unsigned int stride)
@@ -468,6 +539,8 @@ static void fill_smpte(const struct util_format_info *info, void *planes[3],
unsigned char *u, *v;
switch (info->format) {
+ case DRM_FORMAT_C8:
+ return fill_smpte_c8(planes[0], width, height, stride);
case DRM_FORMAT_UYVY:
case DRM_FORMAT_VYUY:
case DRM_FORMAT_YUYV:
diff --git a/tests/util/pattern.h b/tests/util/pattern.h
index d5c4260c..302d523e 100644
--- a/tests/util/pattern.h
+++ b/tests/util/pattern.h
@@ -36,4 +36,6 @@ void util_fill_pattern(uint32_t format, enum util_fill_pattern pattern,
void *planes[3], unsigned int width,
unsigned int height, unsigned int stride);
+void util_smpte_c8_gamma(unsigned size, uint16_t *r, uint16_t *g, uint16_t *b);
+
#endif /* UTIL_PATTERN_H */
--
2.13.6
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH libdrm] modetest: add C8 support to generate SMPTE pattern
2017-11-18 4:56 [PATCH libdrm] modetest: add C8 support to generate SMPTE pattern Ilia Mirkin
@ 2017-11-18 5:19 ` Ilia Mirkin
0 siblings, 0 replies; 2+ messages in thread
From: Ilia Mirkin @ 2017-11-18 5:19 UTC (permalink / raw)
To: dri-devel
On Fri, Nov 17, 2017 at 11:56 PM, Ilia Mirkin <imirkin@alum.mit.edu> wrote:
> Tested on nouveau with the CRTC only.
>
> Signed-off-by: Ilia Mirkin <imirkin@alum.mit.edu>
> ---
>
> Please note that I have no clue what the proper way to operate the gamma
> interface is. This seemed OK, but the 256 seems awefully hardcoded. Perhaps
> that won't work on other HW?
>
> tests/modetest/buffers.c | 2 ++
> tests/modetest/modetest.c | 13 +++++++++
> tests/util/format.c | 2 ++
> tests/util/pattern.c | 73 +++++++++++++++++++++++++++++++++++++++++++++++
> tests/util/pattern.h | 2 ++
> 5 files changed, 92 insertions(+)
>
> diff --git a/tests/modetest/buffers.c b/tests/modetest/buffers.c
> index 4fd310b9..58f88e85 100644
> --- a/tests/modetest/buffers.c
> +++ b/tests/modetest/buffers.c
> @@ -139,6 +139,7 @@ bo_create(int fd, unsigned int format,
> int ret;
>
> switch (format) {
> + case DRM_FORMAT_C8:
> case DRM_FORMAT_NV12:
> case DRM_FORMAT_NV21:
> case DRM_FORMAT_NV16:
> @@ -279,6 +280,7 @@ bo_create(int fd, unsigned int format,
> planes[2] = virtual + offsets[2];
> break;
>
> + case DRM_FORMAT_C8:
> case DRM_FORMAT_ARGB4444:
> case DRM_FORMAT_XRGB4444:
> case DRM_FORMAT_ABGR4444:
> diff --git a/tests/modetest/modetest.c b/tests/modetest/modetest.c
> index 62d93327..049bdd5e 100644
> --- a/tests/modetest/modetest.c
> +++ b/tests/modetest/modetest.c
> @@ -1228,6 +1228,19 @@ static void set_mode(struct device *dev, struct pipe_arg *pipes, unsigned int co
> fprintf(stderr, "failed to set mode: %s\n", strerror(errno));
> return;
> }
> +
> + if (pipes[0].fourcc == DRM_FORMAT_C8) {
> + uint16_t r[256], g[256], b[256];
> +
> + util_smpte_c8_gamma(256, r, g, b);
> + ret = drmModeCrtcSetGamma(
> + dev->fd, pipe->crtc->crtc->crtc_id,
> + 256, r, g, b);
> + if (ret) {
> + fprintf(stderr, "failed to set gamma: %s\n", strerror(errno));
> + return;
> + }
Of course once you start messing around with gamma, something has to
undo that. I've added
} else {
uint16_t r[256], g[256], b[256];
for (j = 0; j < 256; j++)
r[j] = g[j] = b[j] = j << 8;
drmModeCrtcSetGamma(
dev->fd, pipe->crtc->crtc->crtc_id,
256, r, g, b);
But that seems like a hack here. Where should it go? clear_mode?
> + }
> }
> }
>
> diff --git a/tests/util/format.c b/tests/util/format.c
> index 043cfe7f..3eefe224 100644
> --- a/tests/util/format.c
> +++ b/tests/util/format.c
> @@ -43,6 +43,8 @@
> .yuv = { (order), (xsub), (ysub), (chroma_stride) }
>
> static const struct util_format_info format_info[] = {
> + /* Indexed */
> + { DRM_FORMAT_C8, "C8" },
> /* YUV packed */
> { DRM_FORMAT_UYVY, "UYVY", MAKE_YUV_INFO(YUV_YCbCr | YUV_CY, 2, 2, 2) },
> { DRM_FORMAT_VYUY, "VYUY", MAKE_YUV_INFO(YUV_YCrCb | YUV_CY, 2, 2, 2) },
> diff --git a/tests/util/pattern.c b/tests/util/pattern.c
> index 00b08a8c..41fb541b 100644
> --- a/tests/util/pattern.c
> +++ b/tests/util/pattern.c
> @@ -461,6 +461,77 @@ static void fill_smpte_rgb32(const struct util_rgb_info *rgb, void *mem,
> }
> }
>
> +static void fill_smpte_c8(void *mem, unsigned int width, unsigned int height,
> + unsigned int stride)
> +{
> + unsigned int x;
> + unsigned int y;
> +
> + for (y = 0; y < height * 6 / 9; ++y) {
> + for (x = 0; x < width; ++x)
> + ((uint8_t *)mem)[x] = x * 7 / width;
> + mem += stride;
> + }
> +
> + for (; y < height * 7 / 9; ++y) {
> + for (x = 0; x < width; ++x)
> + ((uint8_t *)mem)[x] = 7 + (x * 7 / width);
> + mem += stride;
> + }
> +
> + for (; y < height; ++y) {
> + for (x = 0; x < width * 5 / 7; ++x)
> + ((uint8_t *)mem)[x] =
> + 14 + (x * 4 / (width * 5 / 7));
> + for (; x < width * 6 / 7; ++x)
> + ((uint8_t *)mem)[x] =
> + 14 + ((x - width * 5 / 7) * 3
> + / (width / 7) + 4);
> + for (; x < width; ++x)
> + ((uint8_t *)mem)[x] = 14 + 7;
> + mem += stride;
> + }
> +}
> +
> +void util_smpte_c8_gamma(unsigned size, uint16_t *r, uint16_t *g, uint16_t *b)
> +{
> + if (size < 7 + 7 + 8) {
> + printf("Error: gamma too small: %d < %d\n", size, 7 + 7 + 8);
> + return;
> + }
> +#define FILL_COLOR(idx, red, green, blue) \
> + r[idx] = red << 8; \
> + g[idx] = green << 8; \
> + b[idx] = blue << 8
> +
> + FILL_COLOR( 0, 192, 192, 192); /* grey */
> + FILL_COLOR( 1, 192, 192, 0 ); /* yellow */
> + FILL_COLOR( 2, 0, 192, 192); /* cyan */
> + FILL_COLOR( 3, 0, 192, 0 ); /* green */
> + FILL_COLOR( 4, 192, 0, 192); /* magenta */
> + FILL_COLOR( 5, 192, 0, 0 ); /* red */
> + FILL_COLOR( 6, 0, 0, 192); /* blue */
> +
> + FILL_COLOR( 7, 0, 0, 192); /* blue */
> + FILL_COLOR( 8, 19, 19, 19 ); /* black */
> + FILL_COLOR( 9, 192, 0, 192); /* magenta */
> + FILL_COLOR(10, 19, 19, 19 ); /* black */
> + FILL_COLOR(11, 0, 192, 192); /* cyan */
> + FILL_COLOR(12, 19, 19, 19 ); /* black */
> + FILL_COLOR(13, 192, 192, 192); /* grey */
> +
> + FILL_COLOR(14, 0, 33, 76); /* in-phase */
> + FILL_COLOR(15, 255, 255, 255); /* super white */
> + FILL_COLOR(16, 50, 0, 106); /* quadrature */
> + FILL_COLOR(17, 19, 19, 19); /* black */
> + FILL_COLOR(18, 9, 9, 9); /* 3.5% */
> + FILL_COLOR(19, 19, 19, 19); /* 7.5% */
> + FILL_COLOR(20, 29, 29, 29); /* 11.5% */
> + FILL_COLOR(21, 19, 19, 19); /* black */
> +
> +#undef FILL_COLOR
> +}
> +
> static void fill_smpte(const struct util_format_info *info, void *planes[3],
> unsigned int width, unsigned int height,
> unsigned int stride)
> @@ -468,6 +539,8 @@ static void fill_smpte(const struct util_format_info *info, void *planes[3],
> unsigned char *u, *v;
>
> switch (info->format) {
> + case DRM_FORMAT_C8:
> + return fill_smpte_c8(planes[0], width, height, stride);
> case DRM_FORMAT_UYVY:
> case DRM_FORMAT_VYUY:
> case DRM_FORMAT_YUYV:
> diff --git a/tests/util/pattern.h b/tests/util/pattern.h
> index d5c4260c..302d523e 100644
> --- a/tests/util/pattern.h
> +++ b/tests/util/pattern.h
> @@ -36,4 +36,6 @@ void util_fill_pattern(uint32_t format, enum util_fill_pattern pattern,
> void *planes[3], unsigned int width,
> unsigned int height, unsigned int stride);
>
> +void util_smpte_c8_gamma(unsigned size, uint16_t *r, uint16_t *g, uint16_t *b);
> +
> #endif /* UTIL_PATTERN_H */
> --
> 2.13.6
>
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2017-11-18 5:19 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-11-18 4:56 [PATCH libdrm] modetest: add C8 support to generate SMPTE pattern Ilia Mirkin
2017-11-18 5:19 ` Ilia Mirkin
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.