Re: linux-next: Tree for Apr 10 (media/i2c/adp1653)

Hi Randy and others, On Fri, Apr 10, 2015 at 10:25:23AM -0700, Randy Dunlap wrote: ... ../drivers/media/i2c/adp1653.c:433:6: warning: unused variable 'gpio' [-Wunused-variable] int gpio; A preliminary patch for adp1653 DT support was accidentally merged to media-tree. It's now reverted

[PATCH] [media] omap4iss: avoid broken OMAP4 dependency

The omap4iss driver uses an interface that used to be provided by OMAP4 but has now been removed and replaced with a WARN_ON(1) statement, which likely broke the iss_csiphy code at runtime. It also broke compiling the driver when CONFIG_ARCH_OMAP2PLUS is set, which is implied by OMAP4:

[PATCH] [media] R820T tuner needs CONFIG_BITREVERSE

In a rarely hit randconfig case, the r820t tuner driver can get built when CONFIG_BITREVERSE is not selected by any other driver, resulting in this error: drivers/built-in.o: In function `r820t_read.constprop.3': :(.text+0xa0594): undefined reference to `byte_rev_table' For consistency, this

[PATCH] [media] vb2: remove unused variable

A recent bug fix removed all uses of the 'fileio' variable in vb2_thread_stop(), which now causes warnings in a lot of ARM defconfig builds: drivers/media/v4l2-core/videobuf2-core.c:3228:26: warning: unused variable 'fileio' [-Wunused-variable] This removes the variable as well. The commit that

[PATCH] [media] dvb-usb/dvb-usb-v2: use IS_REACHABLE

Tha ARM randconfig builds came up with another rare build failure for the dib3000mc driver, when dibusb is built-in and dib3000mc is a loadable module: ERROR: dibusb_dib3000mc_frontend_attach [drivers/media/usb/dvb-usb/dvb-usb-nova-t-usb2.ko] undefined! ERROR: dibusb_dib3000mc_tuner_attach

[PATCH] [media] exynos4_is: exynos4-fimc requires i2c

Without i2c, we can get a build error: drivers/media/platform/exynos4-is/fimc-is-i2c.c: In function 'fimc_is_i2c_probe': drivers/media/platform/exynos4-is/fimc-is-i2c.c:58:8: error: implicit declaration of function 'i2c_add_adapter' [-Werror=implicit-function-declaration] The dependency

dual-duty capture driver bridge and sub-device

Hi, I am wondering if there already example of V4L2 capture drivers which can be used as either stand-alone bridge driver or as a sub-device to another bridge driver based on configuration. I am looking for some example I could use as a starting point. Given the following entities: - S ::

[PATCH] media: i2c: ov2659: add VIDEO_V4L2_SUBDEV_API dependency

From: Lad, Prabhakar prabhakar.cse...@gmail.com this patch adds dependency of VIDEO_V4L2_SUBDEV_API for VIDEO_OV2659 so that it doesn't complain for random config builds. Reported-by: Randy Dunlap rdun...@infradead.org Signed-off-by: Lad, Prabhakar prabhakar.cse...@gmail.com ---

Re: [PATCH v4 4/4] smiapp: Use v4l2_of_alloc_parse_endpoint()

Hi Sakari, Thanks for the patch. On Thu, Apr 9, 2015 at 10:25 PM, Sakari Ailus sakari.ai...@iki.fi wrote: Instead of parsing the link-frequencies property in the driver, let v4l2_of_alloc_parse_endpoint() do it. Signed-off-by: Sakari Ailus sakari.ai...@iki.fi Acked-by: Laurent Pinchart

Re: [PATCH v4 4/4] smiapp: Use v4l2_of_alloc_parse_endpoint()

Hi Prabhakar, Thank you for the review. On Fri, Apr 10, 2015 at 10:54:08PM +0100, Lad, Prabhakar wrote: Hi Sakari, Thanks for the patch. On Thu, Apr 9, 2015 at 10:25 PM, Sakari Ailus sakari.ai...@iki.fi wrote: Instead of parsing the link-frequencies property in the driver, let

[PATCH] media: i2c: ov2659: Use v4l2_of_alloc_parse_endpoint()

From: Lad, Prabhakar prabhakar.cse...@gmail.com Instead of parsing the link-frequencies property in the driver, let v4l2_of_alloc_parse_endpoint() do it. Signed-off-by: Lad, Prabhakar prabhakar.cse...@gmail.com --- This patch depends on https://patchwork.kernel.org/patch/6190901/

Re: [PATCH v4 3/4] v4l: of: Parse variable length properties --- link-frequencies

Hi Sakari, Thanks for the patch. On Thu, Apr 9, 2015 at 10:25 PM, Sakari Ailus sakari.ai...@iki.fi wrote: The link-frequencies property is a variable length array of link frequencies in an endpoint. The array is needed by an increasing number of drivers, so it makes sense to add it to struct

Re: [PATCH v4 2/4] v4l: of: Instead of zeroing bus_type and bus field separately, unify this

Hi Sakari, Thanks for the patch. On Thu, Apr 9, 2015 at 10:25 PM, Sakari Ailus sakari.ai...@iki.fi wrote: Zero the entire struct starting from bus_type. As more fields are added, no changes will be needed in the function to reset their value explicitly. Signed-off-by: Sakari Ailus

[PATCH v4.1 4/4] smiapp: Use v4l2_of_alloc_parse_endpoint()

Instead of parsing the link-frequencies property in the driver, let v4l2_of_alloc_parse_endpoint() do it. Signed-off-by: Sakari Ailus sakari.ai...@iki.fi Acked-by: Laurent Pinchart laurent.pinch...@ideasonboard.com Reviewed-by: Sylwester Nawrocki s.nawro...@samsung.com Reviewed-by: Lad, Prabhakar

Re: [PATCH v4 1/4] v4l: of: Remove the head field in struct v4l2_of_endpoint

Hi Sylwester, On Fri, Apr 10, 2015 at 11:47:20AM +0200, Sylwester Nawrocki wrote: Hi Sakari, On 09/04/15 23:25, Sakari Ailus wrote: The field is unused. Remove it. Signed-off-by: Sakari Ailus sakari.ai...@iki.fi --- include/media/v4l2-of.h |2 -- 1 file changed, 2

[PATCH 1/1] smiapp: Remove useless rval assignment in smiapp_get_pdata()

The value is not used after the assignment. Signed-off-by: Sakari Ailus sakari.ai...@iki.fi --- drivers/media/i2c/smiapp/smiapp-core.c |4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/media/i2c/smiapp/smiapp-core.c b/drivers/media/i2c/smiapp/smiapp-core.c index

cron job: media_tree daily build: ERRORS

This message is generated daily by a cron job that builds media_tree for the kernels and architectures in the list below. Results of the daily build of media_tree: date: Sat Apr 11 04:00:21 CEST 2015 git branch: test git hash: e183201b9e917daf2530b637b2f34f1d5afb934d gcc