linux-kernel - Re: [PATCH v3 12/14] media: platform: pxa

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <c0b25215-b11a-c4bc-fa08-dbd55470daa7@xs4all.nl>
Date:	Sat, 13 Aug 2016 20:46:58 +0200
From:	Hans Verkuil <hverkuil@...all.nl>
To:	Robert Jarzmik <robert.jarzmik@...e.fr>,
	Mauro Carvalho Chehab <mchehab@...nel.org>,
	Guennadi Liakhovetski <g.liakhovetski@....de>,
	Jiri Kosina <trivial@...nel.org>
Cc:	linux-kernel@...r.kernel.org, linux-media@...r.kernel.org
Subject: Re: [PATCH v3 12/14] media: platform: pxa_camera: add debug register
 access

On 08/08/2016 09:30 PM, Robert Jarzmik wrote:
> Add pxa_camera registers access through advanced video debugging.
> 
> Signed-off-by: Robert Jarzmik <robert.jarzmik@...e.fr>
> ---
>  drivers/media/platform/soc_camera/pxa_camera.c | 57 ++++++++++++++++++++++++++
>  1 file changed, 57 insertions(+)
> 
> diff --git a/drivers/media/platform/soc_camera/pxa_camera.c b/drivers/media/platform/soc_camera/pxa_camera.c
> index 1e15734ae287..20340489e07e 100644
> --- a/drivers/media/platform/soc_camera/pxa_camera.c
> +++ b/drivers/media/platform/soc_camera/pxa_camera.c
> @@ -1344,6 +1344,58 @@ static int pxa_camera_check_frame(u32 width, u32 height)
>  		(width & 0x01);
>  }
>  
> +#ifdef CONFIG_VIDEO_ADV_DEBUG
> +static int pxac_vidioc_g_register(struct file *file, void *priv,
> +				  struct v4l2_dbg_register *reg)
> +{
> +	struct pxa_camera_dev *pcdev = video_drvdata(file);
> +
> +	if (reg->reg > CIBR2)
> +		return -ERANGE;
> +
> +	reg->val = __raw_readl(pcdev->base + reg->reg);
> +	reg->size = sizeof(__u32);
> +	return 0;
> +}
> +
> +static int pxac_vidioc_s_register(struct file *file, void *priv,
> +				  const struct v4l2_dbg_register *reg)
> +{
> +	struct pxa_camera_dev *pcdev = video_drvdata(file);
> +
> +	if (reg->reg > CIBR2)
> +		return -ERANGE;
> +	if (reg->size != sizeof(__u32))
> +		return -EINVAL;
> +	__raw_writel(reg->val, pcdev->base + reg->reg);
> +	return 0;
> +}
> +
> +static int pxac_vidioc_g_chip_info(struct file *file, void *fh,
> +				   struct v4l2_dbg_chip_info *chip)
> +{
> +	struct pxa_camera_dev *pcdev = video_drvdata(file);
> +
> +	switch (chip->match.type) {
> +	case V4L2_CHIP_MATCH_BRIDGE:
> +		if (chip->match.addr > 0)
> +			return -EINVAL;
> +
> +		strlcpy(chip->name, "pxa-camera", sizeof(chip->name));
> +		return 0;
> +	case V4L2_CHIP_MATCH_SUBDEV:
> +		if (chip->match.addr > 0)
> +			return -EINVAL;
> +
> +		strlcpy(chip->name, pcdev->sensor->name, sizeof(chip->name));
> +		return 0;
> +	default:
> +		return -EINVAL;
> +	}
> +}

You shouldn't need this g_chip_info function, it should be handled automatically
by the v4l2 core. Just drop it.

> +
> +#endif
> +
>  static int pxac_vidioc_enum_fmt_vid_cap(struct file *filp, void  *priv,
>  					struct v4l2_fmtdesc *f)
>  {
> @@ -1594,6 +1646,11 @@ static const struct v4l2_ioctl_ops pxa_camera_ioctl_ops = {
>  	.vidioc_expbuf			= vb2_ioctl_expbuf,
>  	.vidioc_streamon		= vb2_ioctl_streamon,
>  	.vidioc_streamoff		= vb2_ioctl_streamoff,
> +#ifdef CONFIG_VIDEO_ADV_DEBUG
> +	.vidioc_g_register		= pxac_vidioc_g_register,
> +	.vidioc_s_register		= pxac_vidioc_s_register,
> +	.vidioc_g_chip_info		= pxac_vidioc_g_chip_info,
> +#endif
>  };
>  
>  static struct v4l2_clk_ops pxa_camera_mclk_ops = {
> 

Regards,

	Hans