[PATCH] [media] pxa_camera: constify v4l2_clk_ops structure

2017-08-13 Thread Julia Lawall
This v4l2_clk_ops structure is only passed as the first argument of v4l2_clk_register, which is const, so the v4l2_clk_ops structure can also be const. Done with the help of Coccinelle. Signed-off-by: Julia Lawall --- drivers/media/platform/pxa_camera.c |2 +- 1 file changed, 1 insertion(+

[PATCH] [media] v4l2: av7110_v4l: constify v4l2_audio structure

2017-08-13 Thread Julia Lawall
This v4l2_audio structure is only copied into other structures, so it can be const. Done with the help of Coccinelle. Signed-off-by: Julia Lawall --- drivers/media/pci/ttpci/av7110_v4l.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/pci/ttpci/av7110_v4l.c

cron job: media_tree daily build: ERRORS

2017-08-13 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: Mon Aug 14 05:00:16 CEST 2017 media-tree git hash:ec0c3ec497cabbf3bfa03a9eb5edcc252190a4e0 media_build gi

Re: [PATCH RFC] media: isl6421: add checks for current overflow

2017-08-13 Thread Mauro Carvalho Chehab
Em Sun, 13 Aug 2017 19:47:37 +0100 Jemma Denson escreveu: > Hi Mauro, > > Just tried this on my card - unfortunately the patch as is doesn't work. > > On 13/08/17 13:10, Mauro Carvalho Chehab wrote: > > This Kaffeine's BZ: > > https://bugs.kde.org/show_bug.cgi?id=374693 > > > > affects SkyS

[PATCH v2] media: isl6421: add checks for current overflow

2017-08-13 Thread Mauro Carvalho Chehab
This Kaffeine's BZ: https://bugs.kde.org/show_bug.cgi?id=374693 affects SkyStar S2 PCI DVB-S/S2 rev 3.3 device. It could be due to a Kernel bug. While checking the Isil 6421, comparing with its manual, available at: http://www.intersil.com/content/dam/Intersil/documents/isl6/isl6

Re: [PATCH RFC] media: isl6421: add checks for current overflow

2017-08-13 Thread Jemma Denson
Hi Mauro, Just tried this on my card - unfortunately the patch as is doesn't work. On 13/08/17 13:10, Mauro Carvalho Chehab wrote: This Kaffeine's BZ: https://bugs.kde.org/show_bug.cgi?id=374693 affects SkyStar S2 PCI DVB-S/S2 rev 3.3 device. It could be due to a Kernel bug. While che

[PATCH] [media] media: imx: use setup_timer

2017-08-13 Thread Cihangir Akturk
Use setup_timer function instead of initializing timer with the function and data fields. Generated by: scripts/coccinelle/api/setup_timer.cocci. Signed-off-by: Cihangir Akturk --- drivers/staging/media/imx/imx-ic-prpencvf.c | 5 ++--- drivers/staging/media/imx/imx-media-csi.c | 5 ++--- 2 fi

[PATCH 2/6] [media] pci: make snd_pcm_hardware const

2017-08-13 Thread Bhumika Goyal
Make these const as they are only used during a copy operation. Done using Coccinelle. Signed-off-by: Bhumika Goyal --- drivers/media/pci/cobalt/cobalt-alsa-pcm.c | 4 ++-- drivers/media/pci/cx18/cx18-alsa-pcm.c | 2 +- drivers/media/pci/cx23885/cx23885-alsa.c | 2 +- drivers/media/pci/cx2

[PATCH 4/6] usb: gadget: make snd_pcm_hardware const

2017-08-13 Thread Bhumika Goyal
Make this const as it is only used during a copy operation. Done using Coccinelle. Signed-off-by: Bhumika Goyal --- drivers/usb/gadget/function/u_audio.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/gadget/function/u_audio.c b/drivers/usb/gadget/function/u_aud

[PATCH 1/6] [media] usb: make snd_pcm_hardware const

2017-08-13 Thread Bhumika Goyal
Make these const as they are only used during a copy operation. Done using Coccinelle. Signed-off-by: Bhumika Goyal --- drivers/media/usb/cx231xx/cx231xx-audio.c | 2 +- drivers/media/usb/em28xx/em28xx-audio.c | 2 +- drivers/media/usb/go7007/snd-go7007.c | 2 +- drivers/media/usb/tm6000/t

[PATCH 6/6] [media] tuners: make snd_pcm_hardware const

2017-08-13 Thread Bhumika Goyal
Make these const as they are only used during a copy operation. Done using Coccinelle. Signed-off-by: Bhumika Goyal --- drivers/media/tuners/tda18271-maps.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/media/tuners/tda18271-maps.c b/drivers/media/tuners/tda182

[PATCH 5/6] staging: bcm2835-audio: make snd_pcm_hardware const

2017-08-13 Thread Bhumika Goyal
Make these const as they are only used during a copy operation. Done using Coccinelle. Signed-off-by: Bhumika Goyal --- drivers/staging/vc04_services/bcm2835-audio/bcm2835-pcm.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/vc04_services/bcm2835-audio/bc

[PATCH 3/6] drm: bridge: dw-hdmi: make snd_pcm_hardware const

2017-08-13 Thread Bhumika Goyal
Make this const as it is only used during a copy operation. Done using Coccinelle. Signed-off-by: Bhumika Goyal --- drivers/gpu/drm/bridge/synopsys/dw-hdmi-ahb-audio.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/bridge/synopsys/dw-hdmi-ahb-audio.c b/drive

[PATCH 0/6] drivers: make snd_pcm_hardware const

2017-08-13 Thread Bhumika Goyal
Make snd_pcm_hardware structures const. Bhumika Goyal (6): [media] usb: make snd_pcm_hardware const [media] pci: make snd_pcm_hardware const drm: bridge: dw-hdmi: make snd_pcm_hardware const usb: gadget: make snd_pcm_hardware const staging: bcm2835-audio: make snd_pcm_hardware const [

Re: [PATCH] [media] mxl111sf: Fix potential null pointer dereference

2017-08-13 Thread Michael Ira Krufky
On Sun, Aug 13, 2017 at 6:06 AM, Eugeniu Rosca wrote: > From: Eugeniu Rosca > > Reviewing the delta between cppcheck output of v4.9.39 and v4.9.40 > stable updates, I stumbled on the new warning: > > mxl111sf.c:80: (warning) Possible null pointer dereference: rbuf > > Since copying state->rcvbuf

Re: [PATCH 1/3] [media] usb: constify usb_device_id

2017-08-13 Thread Laurent Pinchart
Hi Arvind, Thank you for the patch. On Sunday 13 Aug 2017 14:24:43 Arvind Yadav wrote: > usb_device_id are not supposed to change at runtime. All functions > working with usb_device_id provided by work with > const usb_device_id. So mark the non-const structs as const. > > 'drivers/media/usb/b2

[PATCH RFC] media: isl6421: add checks for current overflow

2017-08-13 Thread Mauro Carvalho Chehab
This Kaffeine's BZ: https://bugs.kde.org/show_bug.cgi?id=374693 affects SkyStar S2 PCI DVB-S/S2 rev 3.3 device. It could be due to a Kernel bug. While checking the Isil 6421, comparing with its manual, available at: http://www.intersil.com/content/dam/Intersil/documents/isl6/isl6

[PATCH] [media] mxl111sf: Fix potential null pointer dereference

2017-08-13 Thread Eugeniu Rosca
From: Eugeniu Rosca Reviewing the delta between cppcheck output of v4.9.39 and v4.9.40 stable updates, I stumbled on the new warning: mxl111sf.c:80: (warning) Possible null pointer dereference: rbuf Since copying state->rcvbuf into rbuf is not needed in the 'write-only' scenario (i.e. calling m

[PATCH 0/3] constify media usb_device_id

2017-08-13 Thread Arvind Yadav
usb_device_id are not supposed to change at runtime. All functions working with usb_device_id provided by work with const usb_device_id. So mark the non-const structs as const. Arvind Yadav (3): [PATCH 1/3] [media] usb: constify usb_device_id [PATCH 2/3] [media] rc: constify usb_device_id [

[PATCH 3/3] [media] radio: constify usb_device_id

2017-08-13 Thread Arvind Yadav
usb_device_id are not supposed to change at runtime. All functions working with usb_device_id provided by work with const usb_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav --- drivers/media/radio/dsbr100.c | 2 +- drivers/media/radio/radio-keene.

[PATCH 2/3] [media] rc: constify usb_device_id

2017-08-13 Thread Arvind Yadav
usb_device_id are not supposed to change at runtime. All functions working with usb_device_id provided by work with const usb_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav --- drivers/media/rc/ati_remote.c | 2 +- drivers/media/rc/igorplugusb.c | 2 +- drivers/

[PATCH 1/3] [media] usb: constify usb_device_id

2017-08-13 Thread Arvind Yadav
usb_device_id are not supposed to change at runtime. All functions working with usb_device_id provided by work with const usb_device_id. So mark the non-const structs as const. 'drivers/media/usb/b2c2/flexcop-usb.c' Fix checkpatch.pl error: ERROR: space prohibited before open square bracket '['.