Linux-Media Archive on lore.kernel.org
 help / color / Atom feed
From: Hans Verkuil <hverkuil@xs4all.nl>
To: Vandana BN <bnvandana@gmail.com>, linux-media@vger.kernel.org
Subject: Re: [PATCH v8] vivid: Add touch support
Date: Mon, 25 Nov 2019 11:02:07 +0100
Message-ID: <c2f25d01-93cc-3678-bae5-58fb13001d8d@xs4all.nl> (raw)
In-Reply-To: <20191125042336.6205-1-bnvandana@gmail.com>

Hi Vandana,

This looks really good. I have a few very small comments (I'd have changed it
myself, but we're in the code freeze period so there is time for a v9):

On 11/25/19 5:23 AM, Vandana BN wrote:
> Support to emulate touch devices in vivid driver.
> It generates touch patterns simulating single tap, double tap, triple
> tap, move from left to right, zoom in, zoom out, palm press simulating
> large area being pressed on screen, and simulating 16 different
> simultaneous touch points.The values generated are based on
> behavior of the rmi_f54 driver.
> 
> Signed-off-by: Vandana BN <bnvandana@gmail.com>
> ---
>  drivers/media/platform/vivid/Makefile         |   3 +-
>  drivers/media/platform/vivid/vivid-core.c     | 164 ++++++++-
>  drivers/media/platform/vivid/vivid-core.h     |  20 ++
>  drivers/media/platform/vivid/vivid-ctrls.c    |  11 +
>  .../platform/vivid/vivid-kthread-touch.c      | 181 ++++++++++
>  .../platform/vivid/vivid-kthread-touch.h      |  13 +
>  .../media/platform/vivid/vivid-touch-cap.c    | 320 ++++++++++++++++++
>  .../media/platform/vivid/vivid-touch-cap.h    |  38 +++
>  8 files changed, 737 insertions(+), 13 deletions(-)
>  create mode 100644 drivers/media/platform/vivid/vivid-kthread-touch.c
>  create mode 100644 drivers/media/platform/vivid/vivid-kthread-touch.h
>  create mode 100644 drivers/media/platform/vivid/vivid-touch-cap.c
>  create mode 100644 drivers/media/platform/vivid/vivid-touch-cap.h
> 

<snip>

> +void vivid_fillbuff_tch(struct vivid_dev *dev, struct vivid_buffer *buf)
> +{
> +	struct v4l2_pix_format *f = &dev->tch_format;
> +	int size = f->width * f->height;
> +	int x, y, xstart, ystart, offset_x, offset_y;
> +	unsigned int test_pattern, test_pat_idx, rand;
> +
> +	__s16 *tch_buf = vb2_plane_vaddr(&buf->vb.vb2_buf, 0);
> +
> +	buf->vb.sequence = dev->touch_cap_seq_count;
> +	test_pattern = (buf->vb.sequence / TCH_SEQ_COUNT) % TEST_CASE_MAX;
> +	test_pat_idx = buf->vb.sequence % TCH_SEQ_COUNT;
> +
> +	vivid_fill_buff_noise(tch_buf, size);
> +
> +	if (test_pat_idx >= TCH_PATTERN_COUNT)
> +		return;
> +
> +	if (test_pat_idx == 0)
> +		dev->tch_pat_random = get_random_int();
> +	rand = dev->tch_pat_random;
> +
> +	switch (test_pattern) {
> +	case SINGLE_TAP:
> +		if (test_pat_idx == 5)

Let's change 5 to TCH_PATTERN_COUNT / 2

> +			vivid_tch_buf_set(f, tch_buf, rand % size);
> +		break;
> +	case DOUBLE_TAP:
> +		if (test_pat_idx == 3 || test_pat_idx == 6)

And 3 and 6 to:

	TCH_PATTERN_COUNT / 3 and 2 * TCH_PATTERN_COUNT / 3

> +			vivid_tch_buf_set(f, tch_buf, rand % size);
> +		break;
> +	case TRIPLE_TAP:
> +		if (test_pat_idx == 3 || test_pat_idx == 6 || test_pat_idx == 9)

TCH_PATTERN_COUNT / 4, 2 * TCH_PATTERN_COUNT / 4, 3 * TCH_PATTERN_COUNT / 4

> +			vivid_tch_buf_set(f, tch_buf, rand % size);
> +		break;
> +	case MOVE_LEFT_TO_RIGHT:
> +		vivid_tch_buf_set(f, tch_buf,
> +				  (rand % f->height) * f->width +
> +				  test_pat_idx * (f->width / 10));

10 -> TCH_PATTERN_COUNT

Regards,

	Hans

> +		break;
> +	case ZOOM_IN:
> +		x = f->width / 2;
> +		y = f->height / 2;
> +		offset_x = ((TCH_PATTERN_COUNT - 1 - test_pat_idx) * x) /
> +				TCH_PATTERN_COUNT;
> +		offset_y = ((TCH_PATTERN_COUNT - 1 - test_pat_idx) * y) /
> +				TCH_PATTERN_COUNT;
> +		vivid_tch_buf_set(f, tch_buf,
> +				  (x - offset_x) + f->width * (y - offset_y));
> +		vivid_tch_buf_set(f, tch_buf,
> +				  (x + offset_x) + f->width * (y + offset_y));
> +		break;
> +	case ZOOM_OUT:
> +		x = f->width / 2;
> +		y = f->height / 2;
> +		offset_x = (test_pat_idx * x) / TCH_PATTERN_COUNT;
> +		offset_y = (test_pat_idx * y) / TCH_PATTERN_COUNT;
> +		vivid_tch_buf_set(f, tch_buf,
> +				  (x - offset_x) + f->width * (y - offset_y));
> +		vivid_tch_buf_set(f, tch_buf,
> +				  (x + offset_x) + f->width * (y + offset_y));
> +		break;
> +	case PALM_PRESS:
> +		for (x = 0; x < f->width; x++)
> +			for (y = f->height / 2; y < f->height; y++)
> +				vivid_tch_buf_set(f, tch_buf, x + f->width * y);
> +		break;
> +	case MULTIPLE_PRESS:
> +		/* 16 pressure points */
> +		for (y = 0; y < 4; y++) {
> +			for (x = 0; x < 4; x++) {
> +				ystart = (y * f->height) / 4 + f->height / 8;
> +				xstart = (x * f->width) / 4 + f->width / 8;
> +				vivid_tch_buf_set(f, tch_buf,
> +						  ystart * f->width + xstart);
> +			}
> +		}
> +		break;
> +	}
> +#ifdef __BIG_ENDIAN__
> +	for (x = 0; x < size; x++)
> +		tch_buf[x] = (__force s16)__cpu_to_le16((u16)tch_buf[x]);
> +#endif
> +}
> diff --git a/drivers/media/platform/vivid/vivid-touch-cap.h b/drivers/media/platform/vivid/vivid-touch-cap.h
> new file mode 100644
> index 000000000000..761050b652eb
> --- /dev/null
> +++ b/drivers/media/platform/vivid/vivid-touch-cap.h
> @@ -0,0 +1,38 @@
> +/* SPDX-License-Identifier: GPL-2.0-only */
> +/*
> + * vivid-touch-cap.h - touch support functions.
> + */
> +#ifndef _VIVID_TOUCH_CAP_H_
> +#define _VIVID_TOUCH_CAP_H_
> +
> +#define VIVID_TCH_HEIGHT	12
> +#define VIVID_TCH_WIDTH		21
> +#define VIVID_MIN_PRESSURE	180
> +#define VIVID_PRESSURE_LIMIT	40
> +#define TCH_SEQ_COUNT		16
> +#define TCH_PATTERN_COUNT	12
> +
> +enum vivid_tch_test {
> +	SINGLE_TAP,
> +	DOUBLE_TAP,
> +	TRIPLE_TAP,
> +	MOVE_LEFT_TO_RIGHT,
> +	ZOOM_IN,
> +	ZOOM_OUT,
> +	PALM_PRESS,
> +	MULTIPLE_PRESS,
> +	TEST_CASE_MAX
> +};
> +
> +extern const struct vb2_ops vivid_touch_cap_qops;
> +
> +int vivid_enum_fmt_tch(struct file *file, void  *priv, struct v4l2_fmtdesc *f);
> +int vivid_g_fmt_tch(struct file *file, void *priv, struct v4l2_format *f);
> +int vivid_enum_input_tch(struct file *file, void *priv, struct v4l2_input *inp);
> +int vivid_g_input_tch(struct file *file, void *priv, unsigned int *i);
> +int vivid_s_input_tch(struct file *file, void *priv, unsigned int i);
> +void vivid_fillbuff_tch(struct vivid_dev *dev, struct vivid_buffer *buf);
> +int vivid_set_touch(struct vivid_dev *dev, unsigned int i);
> +int vivid_g_parm_tch(struct file *file, void *priv,
> +		     struct v4l2_streamparm *parm);
> +#endif
> 


  reply index

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <9719685b-1b71-17e0-f43a-efbb53592e27@xs4all.nl>
2019-11-18 15:53 ` [PATCH v7] " Vandana BN
2019-11-19  9:02   ` Hans Verkuil
2019-11-19  9:23     ` Hans Verkuil
2019-11-19  9:55       ` Hans Verkuil
2019-11-25  4:23         ` [PATCH v8] " Vandana BN
2019-11-25 10:02           ` Hans Verkuil [this message]
2019-11-25 10:13             ` Hans Verkuil
2019-11-25 10:19             ` Hans Verkuil
2019-11-25 12:00             ` Hans Verkuil
2019-11-26 14:16               ` [PATCH v9] " Vandana BN

Reply instructions:

You may reply publically to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=c2f25d01-93cc-3678-bae5-58fb13001d8d@xs4all.nl \
    --to=hverkuil@xs4all.nl \
    --cc=bnvandana@gmail.com \
    --cc=linux-media@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

Linux-Media Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-media/0 linux-media/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-media linux-media/ https://lore.kernel.org/linux-media \
		linux-media@vger.kernel.org
	public-inbox-index linux-media

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.linux-media


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git