| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-id: <051501cd4869$3f51bde0$bdf539a0$%szyprowski@samsung.com>
Date: Tue, 12 Jun 2012 09:01:47 +0200
From: Marek Szyprowski <m.szyprowski@...sung.com>
To: 'Tomasz Moń' <desowin@...il.com>,
'Mauro Carvalho Chehab' <mchehab@...radead.org>,
'Guennadi Liakhovetski' <g.liakhovetski@....de>,
'Hans Verkuil' <hans.verkuil@...co.com>,
'Hans de Goede' <hdegoede@...hat.com>,
linux-media@...r.kernel.org
Cc: linux-kernel@...r.kernel.org
Subject: RE: [PATCH 1/1] v4l: mem2mem_testdev: Add horizontal and vertical flip.
Hi Tomasz,
On Sunday, June 10, 2012 9:18 PM Tomasz Moń wrote:
> Signed-off-by: Tomasz Moń <desowin@...il.com>
Please add some commit description, even if it is as simple as repeating the subject with one
additional line of comment.
> ---
> drivers/media/video/mem2mem_testdev.c | 135 ++++++++++++++++++++++++++++++---
> 1 file changed, 124 insertions(+), 11 deletions(-)
>
> diff --git a/drivers/media/video/mem2mem_testdev.c b/drivers/media/video/mem2mem_testdev.c
> index d2dec58..e1b66e9 100644
> --- a/drivers/media/video/mem2mem_testdev.c
> +++ b/drivers/media/video/mem2mem_testdev.c
> @@ -60,6 +60,10 @@ MODULE_VERSION("0.1.1");
> #define MEM2MEM_COLOR_STEP (0xff >> 4)
> #define MEM2MEM_NUM_TILES 8
>
> +/* Flags that indicate processing mode */
> +#define MEM2MEM_HFLIP (1 << 0)
> +#define MEM2MEM_VFLIP (1 << 1)
> +
> #define dprintk(dev, fmt, arg...) \
> v4l2_dbg(1, 1, &dev->v4l2_dev, "%s: " fmt, __func__, ## arg)
>
> @@ -131,6 +135,24 @@ static struct m2mtest_q_data *get_q_data(enum v4l2_buf_type type)
>
> static struct v4l2_queryctrl m2mtest_ctrls[] = {
> {
> + .id = V4L2_CID_HFLIP,
> + .type = V4L2_CTRL_TYPE_BOOLEAN,
> + .name = "Mirror",
> + .minimum = 0,
> + .maximum = 1,
> + .step = 1,
> + .default_value = 0,
> + .flags = 0,
> + }, {
> + .id = V4L2_CID_VFLIP,
> + .type = V4L2_CTRL_TYPE_BOOLEAN,
> + .name = "Vertical Mirror",
> + .minimum = 0,
> + .maximum = 1,
> + .step = 1,
> + .default_value = 0,
> + .flags = 0,
> + }, {
> .id = V4L2_CID_TRANS_TIME_MSEC,
> .type = V4L2_CTRL_TYPE_INTEGER,
> .name = "Transaction time (msec)",
> @@ -197,6 +219,9 @@ struct m2mtest_ctx {
> /* Abort requested by m2m */
> int aborting;
>
> + /* Processing mode */
> + int mode;
> +
> struct v4l2_m2m_ctx *m2m_ctx;
> };
>
> @@ -247,19 +272,84 @@ static int device_process(struct m2mtest_ctx *ctx,
> bytes_left = bytesperline - tile_w * MEM2MEM_NUM_TILES;
> w = 0;
>
> - for (y = 0; y < height; ++y) {
> - for (t = 0; t < MEM2MEM_NUM_TILES; ++t) {
> - if (w & 0x1) {
> - for (x = 0; x < tile_w; ++x)
> - *p_out++ = *p_in++ + MEM2MEM_COLOR_STEP;
> - } else {
> - for (x = 0; x < tile_w; ++x)
> - *p_out++ = *p_in++ - MEM2MEM_COLOR_STEP;
> + switch (ctx->mode) {
> + case MEM2MEM_HFLIP | MEM2MEM_VFLIP:
> + p_out += bytesperline * height - bytes_left;
> + for (y = 0; y < height; ++y) {
> + for (t = 0; t < MEM2MEM_NUM_TILES; ++t) {
> + if (w & 0x1) {
> + for (x = 0; x < tile_w; ++x)
> + *--p_out = *p_in++ +
> + MEM2MEM_COLOR_STEP;
> + } else {
> + for (x = 0; x < tile_w; ++x)
> + *--p_out = *p_in++ -
> + MEM2MEM_COLOR_STEP;
> + }
> + ++w;
> }
> - ++w;
> + p_in += bytes_left;
> + p_out -= bytes_left;
> + }
> + break;
> +
> + case MEM2MEM_HFLIP:
> + for (y = 0; y < height; ++y) {
> + p_out += MEM2MEM_NUM_TILES * tile_w;
> + for (t = 0; t < MEM2MEM_NUM_TILES; ++t) {
> + if (w & 0x01) {
> + for (x = 0; x < tile_w; ++x)
> + *--p_out = *p_in++ +
> + MEM2MEM_COLOR_STEP;
> + } else {
> + for (x = 0; x < tile_w; ++x)
> + *--p_out = *p_in++ -
> + MEM2MEM_COLOR_STEP;
> + }
> + ++w;
> + }
> + p_in += bytes_left;
> + p_out += bytesperline;
> + }
> + break;
> +
> + case MEM2MEM_VFLIP:
> + p_out += bytesperline * (height - 1);
> + for (y = 0; y < height; ++y) {
> + for (t = 0; t < MEM2MEM_NUM_TILES; ++t) {
> + if (w & 0x1) {
> + for (x = 0; x < tile_w; ++x)
> + *p_out++ = *p_in++ +
> + MEM2MEM_COLOR_STEP;
> + } else {
> + for (x = 0; x < tile_w; ++x)
> + *p_out++ = *p_in++ -
> + MEM2MEM_COLOR_STEP;
> + }
> + ++w;
> + }
> + p_in += bytes_left;
> + p_out += bytes_left - 2 * bytesperline;
> + }
> + break;
> +
> + default:
> + for (y = 0; y < height; ++y) {
> + for (t = 0; t < MEM2MEM_NUM_TILES; ++t) {
> + if (w & 0x1) {
> + for (x = 0; x < tile_w; ++x)
> + *p_out++ = *p_in++ +
> + MEM2MEM_COLOR_STEP;
> + } else {
> + for (x = 0; x < tile_w; ++x)
> + *p_out++ = *p_in++ -
> + MEM2MEM_COLOR_STEP;
> + }
> + ++w;
> + }
> + p_in += bytes_left;
> + p_out += bytes_left;
> }
> - p_in += bytes_left;
> - p_out += bytes_left;
> }
>
> return 0;
> @@ -646,6 +736,14 @@ static int vidioc_g_ctrl(struct file *file, void *priv,
> struct m2mtest_ctx *ctx = priv;
>
> switch (ctrl->id) {
> + case V4L2_CID_HFLIP:
> + ctrl->value = (ctx->mode & MEM2MEM_HFLIP) ? 1 : 0;
> + break;
> +
> + case V4L2_CID_VFLIP:
> + ctrl->value = (ctx->mode & MEM2MEM_VFLIP) ? 1 : 0;
> + break;
> +
> case V4L2_CID_TRANS_TIME_MSEC:
> ctrl->value = ctx->transtime;
> break;
> @@ -689,6 +787,20 @@ static int vidioc_s_ctrl(struct file *file, void *priv,
> return ret;
>
> switch (ctrl->id) {
> + case V4L2_CID_HFLIP:
> + if (ctrl->value)
> + ctx->mode |= MEM2MEM_HFLIP;
> + else
> + ctx->mode &= ~MEM2MEM_HFLIP;
> + break;
> +
> + case V4L2_CID_VFLIP:
> + if (ctrl->value)
> + ctx->mode |= MEM2MEM_VFLIP;
> + else
> + ctx->mode &= ~MEM2MEM_VFLIP;
> + break;
> +
> case V4L2_CID_TRANS_TIME_MSEC:
> ctx->transtime = ctrl->value;
> break;
> @@ -859,6 +971,7 @@ static int m2mtest_open(struct file *file)
> ctx->translen = MEM2MEM_DEF_TRANSLEN;
> ctx->transtime = MEM2MEM_DEF_TRANSTIME;
> ctx->num_processed = 0;
> + ctx->mode = 0;
>
> ctx->m2m_ctx = v4l2_m2m_ctx_init(dev->m2m_dev, ctx, &queue_init);
>
> --
> 1.7.10
Best regards
--
Marek Szyprowski
Samsung Poland R&D Center
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists