linux-toradex.git/drivers/media/i2c/mt9m032.c, branch v4.2.6

[media] v4l2-subdev: replace v4l2_subdev_fh by v4l2_subdev_pad_config

2015-03-23T18:41:36+00:00

If a subdevice pad op is called from a bridge driver, then there is
no v4l2_subdev_fh struct that can be passed to the subdevice. This
made it hard to use such subdevs from a bridge driver.

This patch replaces the v4l2_subdev_fh pointer by a v4l2_subdev_pad_config
pointer in the pad ops. This allows bridge drivers to use the various
try_ pad ops by creating a v4l2_subdev_pad_config struct and passing it
along to the pad op.

The v4l2_subdev_get_try_* macros had to be changed because of this, so
I also took the opportunity to use the full name of the v4l2_subdev_get_try_*
functions in the __V4L2_SUBDEV_MK_GET_TRY macro arguments: if you now do
'git grep v4l2_subdev_get_try_format' you will actually find the header
where it is defined.

One remark regarding the drivers/staging/media/davinci_vpfe patches: the
*_init_formats() functions assumed that fh could be NULL. However, that's
not true for this driver, it's always set. This is almost certainly a copy
and paste from the omap3isp driver. I've updated the code to reflect the
fact that fh is never NULL.

Signed-off-by: Hans Verkuil 
Acked-by: Lad, Prabhakar 
Tested-by: Lad, Prabhakar 
Acked-by: Laurent Pinchart 
Acked-by: Sakari Ailus 
Acked-by: Sylwester Nawrocki 
Signed-off-by: Mauro Carvalho Chehab

[media] v4l2 subdevs: replace get/set_crop by get/set_selection

2014-12-22T20:11:34+00:00

The crop and selection pad ops are duplicates. Replace all uses of get/set_crop
by get/set_selection. This will make it possible to drop get/set_crop
altogether.

Signed-off-by: Hans Verkuil 
Acked-by: Sylwester Nawrocki 
Acked-by: Laurent Pinchart 
Cc: Prabhakar Lad 
Cc: Philipp Zabel 
Signed-off-by: Mauro Carvalho Chehab

[media] i2c: Make use of media_bus_format enum

2014-11-14T19:52:48+00:00

In order to have subsytem agnostic media bus format definitions we've
moved media bus definitions to include/uapi/linux/media-bus-format.h and
prefixed values with MEDIA_BUS_FMT instead of V4L2_MBUS_FMT.

Replace all references to the old definitions in i2c drivers.

Signed-off-by: Boris Brezillon 
Acked-by: Hans Verkuil 
Acked-by: Sakari Ailus 
Acked-by: Lad, Prabhakar 
Signed-off-by: Hans Verkuil 
Signed-off-by: Mauro Carvalho Chehab

[media] videodev2: Set vb2_rect's width and height as unsigned

2014-01-07T10:02:39+00:00

As discussed on the media summit 2013, there is no reason for the width
and height to be signed.

Therefore this patch is an attempt to convert those fields from __s32 to
__u32.

Signed-off-by: Ricardo Ribalda Delgado 
Acked-by: Sakari Ailus  (documentation and smiapp)
Acked-by: Lad, Prabhakar 
Signed-off-by: Hans Verkuil 
Signed-off-by: Mauro Carvalho Chehab

[media] media/i2c: remove g_chip_ident op

2013-06-17T11:52:10+00:00

This is no longer needed since the core now handles this through DBG_G_CHIP_INFO.

Signed-off-by: Hans Verkuil 
Acked-by: Laurent Pinchart 
Acked-by: Lad, Prabhakar 
Signed-off-by: Mauro Carvalho Chehab

[media] media: i2c: Convert to devm_kzalloc()

2013-05-21T10:52:15+00:00

Using the managed function the kfree() calls can be removed from the
probe error path and the remove handler.

Signed-off-by: Laurent Pinchart 
Acked-by: Sylwester Nawrocki 
Acked-by: Lad, Prabhakar 
Acked-by: Andy Shevchenko 
Reviewed-by: Benoît Thébaudeau 
Signed-off-by: Mauro Carvalho Chehab

[media] mt9m032: Define MT9M032_READ_MODE1 bits

2013-04-14T18:58:45+00:00

Replace hardcoded values with #define's.

Signed-off-by: Laurent Pinchart 
Signed-off-by: Mauro Carvalho Chehab

[media] mt9m032: Fix PLL setup

2013-04-14T18:55:41+00:00

The MT9M032 PLL was assumed to be identical to the MT9P031 PLL but
differs significantly. Update the registers definitions and PLL limits
according to the datasheet.

Signed-off-by: Laurent Pinchart 
Signed-off-by: Mauro Carvalho Chehab

[media] v4l2: add const to argument of write-only s_register ioctl

2013-03-24T11:47:01+00:00

This ioctl is defined as IOW, so pass the argument as const.

Signed-off-by: Hans Verkuil 
Acked-by: Guennadi Liakhovetski 
Acked-by: Lad, Prabhakar 
Signed-off-by: Mauro Carvalho Chehab

[media] mt9m032.c: introduce missing initialization

2012-09-14T14:10:59+00:00

The result of one call to a function is tested, and then at the second call
to the same function, the previous result, and not the current result, is
tested again.

The semantic match that finds this problem is as follows:
(http://coccinelle.lip6.fr/)

// 
@@
expression ret;
identifier f;
statement S1,S2;
@@

*ret = f(...);
if (\(ret != 0\|ret < 0\|ret == NULL\)) S1
... when any
*f(...);
if (\(ret != 0\|ret < 0\|ret == NULL\)) S2
// 

Signed-off-by: Julia Lawall 
Signed-off-by: Mauro Carvalho Chehab