Re: [REVIEW PATCH 3/5] v4l2: pass std by value to the write-only s_std ioctl.

2013-03-16 Thread Guennadi Liakhovetski
Hi Hans On Fri, 15 Mar 2013, Hans Verkuil wrote: From: Hans Verkuil hans.verk...@cisco.com This ioctl is defined as IOW, so pass the argument by value instead of by reference. I could have chosen to add const instead, but this is 1) easier to handle in drivers and 2) consistent with the

Re: [REVIEW PATCH 4/5] v4l2: add const to argument of write-only s_register ioctl.

2013-03-16 Thread Guennadi Liakhovetski
On Fri, 15 Mar 2013, Hans Verkuil wrote: From: Hans Verkuil hans.verk...@cisco.com This ioctl is defined as IOW, so pass the argument as const. Signed-off-by: Hans Verkuil hans.verk...@cisco.com --- [snip] drivers/media/i2c/soc_camera/mt9m001.c |2 +-

Re: [REVIEW PATCH 3/5] v4l2: pass std by value to the write-only s_std ioctl.

2013-03-16 Thread Prabhakar Lad
Hi Hans, Thanks for the patch! On Fri, Mar 15, 2013 at 3:57 PM, Hans Verkuil hverk...@xs4all.nl wrote: From: Hans Verkuil hans.verk...@cisco.com This ioctl is defined as IOW, so pass the argument by value instead of by reference. I could have chosen to add const instead, but this is 1)

Re: [REVIEW PATCH 4/5] v4l2: add const to argument of write-only s_register ioctl.

2013-03-16 Thread Prabhakar Lad
Hi Hans, Thanks for the patch! On Fri, Mar 15, 2013 at 3:57 PM, Hans Verkuil hverk...@xs4all.nl wrote: From: Hans Verkuil hans.verk...@cisco.com This ioctl is defined as IOW, so pass the argument as const. Signed-off-by: Hans Verkuil hans.verk...@cisco.com --- [snip]

Re: [PATCH v3] media: davinci: kconfig: fix incorrect selects

2013-03-16 Thread Prabhakar Lad
Hi Sekhar, Thanks for the patch! On Tue, Mar 12, 2013 at 2:44 PM, Sekhar Nori nsek...@ti.com wrote: drivers/media/platform/davinci/Kconfig uses selects where it should be using 'depends on'. This results in warnings of the following sort when doing randconfig builds. warning:

Re: [REVIEW PATCH 2/5] v4l2: add const to argument of write-only s_tuner ioctl.

2013-03-16 Thread Alexey Klimov
Hi Hans, On Fri, Mar 15, 2013 at 2:27 PM, Hans Verkuil hverk...@xs4all.nl wrote: From: Hans Verkuil hans.verk...@cisco.com This ioctl is defined as IOW, so pass the argument as const. Signed-off-by: Hans Verkuil hans.verk...@cisco.com [snip] drivers/media/radio/dsbr100.c

Re: dvb-apps: Additional channels for Netherlands

2013-03-16 Thread Oliver Schinagl
On 16-03-13 16:13, Dmitry Katsubo wrote: On 15.03.2013 13:07, Oliver Schinagl wrote: On 698 MHz we have NTS1 (Bouquet 2) which is on a code rate of 1/2. Still I cannot make them working. More exactly: This works (722 MHz): xine dvb://Nederland 1 xine dvb://Nederland 2 xine dvb://Nederland 3

cron job: media_tree daily build: WARNINGS

2013-03-16 Thread Hans Verkuil
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 Mar 16 19:01:50 CET 2013 git branch: test git hash: 4d35435d3ffb853b491f5bb21a62529cd925d660 gcc