[PATCH V2 09/24] usb/gspca/ov519.c: use IS_ENABLED() macro

2013-01-19 Thread Peter Senna Tschudin
replace: #if defined(CONFIG_INPUT) || \ defined(CONFIG_INPUT_MODULE) with: #if IS_ENABLED(CONFIG_INPUT) This change was made for: CONFIG_INPUT Reported-by: Mauro Carvalho Chehab mche...@redhat.com Signed-off-by: Peter Senna Tschudin peter.se...@gmail.com --- Changes from V1: Updated

[PATCH V2 10/24] usb/gspca/pac207.c: use IS_ENABLED() macro

2013-01-19 Thread Peter Senna Tschudin
replace: #if defined(CONFIG_INPUT) || \ defined(CONFIG_INPUT_MODULE) with: #if IS_ENABLED(CONFIG_INPUT) This change was made for: CONFIG_INPUT Reported-by: Mauro Carvalho Chehab mche...@redhat.com Signed-off-by: Peter Senna Tschudin peter.se...@gmail.com --- Changes from V1: Updated

[PATCH V2 11/24] gspca/pac7302.c: use IS_ENABLED() macro

2013-01-19 Thread Peter Senna Tschudin
replace: #if defined(CONFIG_INPUT) || \ defined(CONFIG_INPUT_MODULE) with: #if IS_ENABLED(CONFIG_INPUT) This change was made for: CONFIG_INPUT Reported-by: Mauro Carvalho Chehab mche...@redhat.com Signed-off-by: Peter Senna Tschudin peter.se...@gmail.com --- Changes from V1: Updated

[PATCH V2 12/24] usb/gspca/pac7311.c: use IS_ENABLED() macro

2013-01-19 Thread Peter Senna Tschudin
replace: #if defined(CONFIG_INPUT) || \ defined(CONFIG_INPUT_MODULE) with: #if IS_ENABLED(CONFIG_INPUT) This change was made for: CONFIG_INPUT Reported-by: Mauro Carvalho Chehab mche...@redhat.com Signed-off-by: Peter Senna Tschudin peter.se...@gmail.com --- Changes from V1: Updated

[PATCH V2 13/24] usb/gspca/se401.c: use IS_ENABLED() macro

2013-01-19 Thread Peter Senna Tschudin
replace: #if defined(CONFIG_INPUT) || \ defined(CONFIG_INPUT_MODULE) with: #if IS_ENABLED(CONFIG_INPUT) This change was made for: CONFIG_INPUT Reported-by: Mauro Carvalho Chehab mche...@redhat.com Signed-off-by: Peter Senna Tschudin peter.se...@gmail.com --- Changes from V1: Updated

[PATCH V2 15/24] usb/gspca/sonixb.c: use IS_ENABLED() macro

2013-01-19 Thread Peter Senna Tschudin
Carvalho Chehab mche...@redhat.com Signed-off-by: Peter Senna Tschudin peter.se...@gmail.com --- Changes from V1: Updated subject Fixed commit message drivers/media/usb/gspca/sonixb.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/media/usb/gspca/sonixb.c

[PATCH V2 16/24] usb/gspca/sonixj.c: use IS_ENABLED() macro

2013-01-19 Thread Peter Senna Tschudin
replace: #if defined(CONFIG_INPUT) || \ defined(CONFIG_INPUT_MODULE) with: #if IS_ENABLED(CONFIG_INPUT) This change was made for: CONFIG_INPUT Reported-by: Mauro Carvalho Chehab mche...@redhat.com Signed-off-by: Peter Senna Tschudin peter.se...@gmail.com --- Changes from V1: Updated

[PATCH V2 17/24] usb/gspca/spca561.c: use IS_ENABLED() macro

2013-01-19 Thread Peter Senna Tschudin
replace: #if defined(CONFIG_INPUT) || \ defined(CONFIG_INPUT_MODULE) with: #if IS_ENABLED(CONFIG_INPUT) This change was made for: CONFIG_INPUT Reported-by: Mauro Carvalho Chehab mche...@redhat.com Signed-off-by: Peter Senna Tschudin peter.se...@gmail.com --- Changes from V1: Updated

[PATCH V2 19/24] usb/gspca/t613.c: use IS_ENABLED() macro

2013-01-19 Thread Peter Senna Tschudin
replace: #if defined(CONFIG_INPUT) || \ defined(CONFIG_INPUT_MODULE) with: #if IS_ENABLED(CONFIG_INPUT) This change was made for: CONFIG_INPUT Reported-by: Mauro Carvalho Chehab mche...@redhat.com Signed-off-by: Peter Senna Tschudin peter.se...@gmail.com --- Changes from V1: Updated

[PATCH V2 18/24] usb/gspca/stv06xx/stv06xx.c: use IS_ENABLED() macro

2013-01-19 Thread Peter Senna Tschudin
replace: #if defined(CONFIG_INPUT) || \ defined(CONFIG_INPUT_MODULE) with: #if IS_ENABLED(CONFIG_INPUT) This change was made for: CONFIG_INPUT Reported-by: Mauro Carvalho Chehab mche...@redhat.com Signed-off-by: Peter Senna Tschudin peter.se...@gmail.com --- Changes from V1: Updated

[PATCH V2 20/24] usb/gspca/xirlink_cit.c: use IS_ENABLED() macro

2013-01-19 Thread Peter Senna Tschudin
replace: #if defined(CONFIG_INPUT) || \ defined(CONFIG_INPUT_MODULE) with: #if IS_ENABLED(CONFIG_INPUT) This change was made for: CONFIG_INPUT Reported-by: Mauro Carvalho Chehab mche...@redhat.com Signed-off-by: Peter Senna Tschudin peter.se...@gmail.com --- Changes from V1: Updated

[PATCH V2 21/24] usb/gspca/zc3xx.c: use IS_ENABLED() macro

2013-01-19 Thread Peter Senna Tschudin
replace: #if defined(CONFIG_INPUT) || \ defined(CONFIG_INPUT_MODULE) with: #if IS_ENABLED(CONFIG_INPUT) This change was made for: CONFIG_INPUT Reported-by: Mauro Carvalho Chehab mche...@redhat.com Signed-off-by: Peter Senna Tschudin peter.se...@gmail.com --- Changes from V1: Updated

[PATCH V2 22/24] usb/hdpvr/hdpvr-core.c: use IS_ENABLED() macro

2013-01-19 Thread Peter Senna Tschudin
replace: #if defined(CONFIG_I2C) || \ defined(CONFIG_I2C_MODULE) with: #if IS_ENABLED(CONFIG_I2C) This change was made for: CONFIG_I2C Reported-by: Mauro Carvalho Chehab mche...@redhat.com Signed-off-by: Peter Senna Tschudin peter.se...@gmail.com --- Changes from V1: Updated subject

[PATCH V2 23/24] usb/hdpvr/hdpvr-i2c.c: use IS_ENABLED() macro

2013-01-19 Thread Peter Senna Tschudin
replace: #if defined(CONFIG_I2C) || \ defined(CONFIG_I2C_MODULE) with: #if IS_ENABLED(CONFIG_I2C) This change was made for: CONFIG_I2C Reported-by: Mauro Carvalho Chehab mche...@redhat.com Signed-off-by: Peter Senna Tschudin peter.se...@gmail.com --- Changes from V1: Updated subject

[PATCH V2 24/24] v4l2-core/v4l2-common.c: use IS_ENABLED() macro

2013-01-19 Thread Peter Senna Tschudin
)) with: #if IS_ENABLED(CONFIG_I2C) Reported-by: Mauro Carvalho Chehab mche...@redhat.com Signed-off-by: Peter Senna Tschudin peter.se...@gmail.com --- Changes from V1: Updated subject Fixed commit message drivers/media/v4l2-core/v4l2-common.c | 4 ++-- 1 file changed, 2 insertions(+), 2

[PATCH V2 14/24] usb/gspca/sn9c20x.c: use IS_ENABLED() macro

2013-01-19 Thread Peter Senna Tschudin
replace: #if defined(CONFIG_INPUT) || \ defined(CONFIG_INPUT_MODULE) with: #if IS_ENABLED(CONFIG_INPUT) This change was made for: CONFIG_INPUT Reported-by: Mauro Carvalho Chehab mche...@redhat.com Signed-off-by: Peter Senna Tschudin peter.se...@gmail.com --- Changes from V1: Updated

[PATCH] [media] use IS_ENABLED() macro

2013-01-19 Thread Peter Senna Tschudin
...@redhat.com Signed-off-by: Peter Senna Tschudin peter.se...@gmail.com --- drivers/media/dvb-frontends/bcm3510.h | 2 +- drivers/media/dvb-frontends/cx22700.h | 2 +- drivers/media/dvb-frontends/cx24110.h | 2 +- drivers/media/dvb-frontends/dib0070.h | 2 +- drivers/media/dvb

[PATCH 2/3] pci/ttpci/av7110.c: use IS_ENABLED() macro

2013-01-09 Thread Peter Senna Tschudin
replace: #if defined(CONFIG_INPUT_EVDEV) || defined(CONFIG_INPUT_EVDEV_MODULE) with: #if IS_ENABLED(CONFIG_INPUT_EVDEV) This change was made for: CONFIG_INPUT_EVDEV, CONFIG_DVB_SP8870 Reported-by: Mauro Carvalho Chehab mche...@redhat.com Signed-off-by: Peter Senna Tschudin peter.se

[PATCH 3/3] pci/saa7134: use IS_ENABLED() macro

2013-01-09 Thread Peter Senna Tschudin
replace: #if defined(CONFIG_VIDEO_CX88_DVB) || defined(CONFIG_VIDEO_CX88_DVB_MODULE) with: #if IS_ENABLED(CONFIG_VIDEO_CX88_DVB) This change was made for: CONFIG_VIDEO_SAA7134_DVB Reported-by: Mauro Carvalho Chehab mche...@redhat.com Signed-off-by: Peter Senna Tschudin peter.se...@gmail.com

[PATCH 1/3] pci/cx88: use IS_ENABLED() macro

2013-01-09 Thread Peter Senna Tschudin
-off-by: Peter Senna Tschudin peter.se...@gmail.com --- drivers/media/pci/cx88/cx88.h | 10 -- 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/drivers/media/pci/cx88/cx88.h b/drivers/media/pci/cx88/cx88.h index ba0dba4..feff53c 100644 --- a/drivers/media/pci/cx88/cx88.h +++ b

Re: [PATCH 1/2] Staging/media: fixed spacing coding style in go7007/wis-ov7640.c

2012-11-02 Thread Peter Senna Tschudin
On Fri, Nov 2, 2012 at 1:08 PM, YAMANE Toshiaki yamaneto...@gmail.com wrote: fixed below checkpatch error. - ERROR: that open brace { should be on the previous line Signed-off-by: YAMANE Toshiaki yamaneto...@gmail.com Tested-by: Peter Senna Tschudin peter.se...@gmail.com --- drivers/staging

Re: [PATCH 2/2] Staging/media: Use dev_ printks in go7007/wis-ov7640.c

2012-11-02 Thread Peter Senna Tschudin
(... to printk(KERN_ERR ... Signed-off-by: YAMANE Toshiaki yamaneto...@gmail.com Tested-by: Peter Senna Tschudin peter.se...@gmail.com --- drivers/staging/media/go7007/wis-ov7640.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/media/go7007/wis

Re: [PATCH] staging/media: Use dev_ printks in cxd2099/cxd2099.[ch]

2012-11-02 Thread Peter Senna Tschudin
(... to printk(KERN_INFO ... - WARNING: Prefer netdev_warn(netdev, ... then dev_warn(dev, ... then pr_warn(... to printk(KERN_WARNING ... Signed-off-by: YAMANE Toshiaki yamaneto...@gmail.com Tested-by: Peter Senna Tschudin peter.se...@gmail.com --- drivers/staging/media/cxd2099/cxd2099.c

[PATCH] drivers/media/pci/saa7134/saa7134-dvb.c: Test if videobuf_dvb_get_frontend return NULL

2012-10-30 Thread Peter Senna Tschudin
= videobuf_dvb_get_frontend(...); ... when != if (!i) S * if (i-a.b) S2 // /smpl Signed-off-by: Peter Senna Tschudin peter.se...@gmail.com --- drivers/media/pci/saa7134/saa7134-dvb.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/media/pci/saa7134/saa7134-dvb.c b/drivers/media/pci

[PATCH V2] drivers/media/v4l2-core/videobuf2-core.c: fix error return code

2012-10-18 Thread Peter Senna Tschudin
-off-by: Peter Senna Tschudin peter.se...@gmail.com --- Changes from V1: Updated commit message drivers/media/v4l2-core/videobuf2-core.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/media/v4l2-core/videobuf2-core.c b/drivers/media/v4l2-core/videobuf2

Re: [PATCH V2] drivers/media/v4l2-core/videobuf2-core.c: fix error return code

2012-10-18 Thread Peter Senna Tschudin
On Thu, Oct 18, 2012 at 5:28 PM, Ezequiel Garcia elezegar...@gmail.com wrote: On Thu, Oct 18, 2012 at 11:47 AM, Peter Senna Tschudin peter.se...@gmail.com wrote: This patch fixes a NULL pointer dereference bug at __vb2_init_fileio(). The NULL pointer deference happens at videobuf2-core.c

Re: [PATCH 4/14] drivers/media/v4l2-core/videobuf2-core.c: fix error return code

2012-10-10 Thread Peter Senna Tschudin
On Sat, Oct 6, 2012 at 1:17 PM, Mauro Carvalho Chehab mche...@infradead.org wrote: Em Thu, 6 Sep 2012 17:23:57 +0200 Peter Senna Tschudin peter.se...@gmail.com escreveu: From: Peter Senna Tschudin peter.se...@gmail.com Convert a nonnegative error return code to a negative one, as returned

[PATCH v3] drivers/media: Remove unnecessary semicolon

2012-09-28 Thread Peter Senna Tschudin
; @@ -;@p // /smpl Signed-off-by: Peter Senna Tschudin peter.se...@gmail.com --- Changes from v2: Include semantic patch in the commit message. The full version of the semantic patch can be found at: http://comments.gmane.org/gmane.comp.version-control.coccinelle/2724 drivers/media/dvb-core

[PATCH] drivers/media: Remove unnecessary semicolon

2012-09-27 Thread Peter Senna Tschudin
Remove unnecessary semicolon And: drivers/media/dvb-frontends/stv0900_core.c: remove unnecessary whitespace before a quoted newline Found by http://coccinelle.lip6.fr/ Signed-off-by: Peter Senna Tschudin peter.se...@gmail.com --- drivers/media/dvb-core/dvb_frontend.c |2

[PATCH v2] drivers/media: Remove unnecessary semicolon

2012-09-27 Thread Peter Senna Tschudin
Remove unnecessary semicolon And: drivers/media/dvb-frontends/stv0900_core.c: remove unnecessary whitespace before a quoted newline Found by http://coccinelle.lip6.fr/ Signed-off-by: Peter Senna Tschudin peter.se...@gmail.com --- drivers/media/dvb-core/dvb_frontend.c | 2 +- drivers

Re: [PATCH] drivers/media: Remove unnecessary semicolon

2012-09-27 Thread Peter Senna Tschudin
On Thu, Sep 27, 2012 at 1:55 PM, Prabhakar Lad prabhakar.cse...@gmail.com wrote: Hi Peter, Thanks for the patch. On Thu, Sep 27, 2012 at 5:00 PM, Peter Senna Tschudin peter.se...@gmail.com wrote: Remove unnecessary semicolon And: drivers/media/dvb-frontends/stv0900_core.c: remove

[PATCH v2] drivers/media: Remove unnecessary semicolon

2012-09-27 Thread Peter Senna Tschudin
Remove unnecessary semicolon And: drivers/media/dvb-frontends/stv0900_core.c: remove unnecessary whitespace before a quoted newline Found by http://coccinelle.lip6.fr/ Signed-off-by: Peter Senna Tschudin peter.se...@gmail.com --- drivers/media/dvb-core/dvb_frontend.c | 2 +- drivers

[PATCH V2 2/14] drivers/media/platform/soc_camera/mx2_camera.c: fix error return code

2012-09-24 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com Convert a nonnegative error return code to a negative one, as returned elsewhere in the function. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // smpl ( if@p1 (\(ret 0\|ret != 0

Re: [PATCH 5/5] drivers/media/platform/omap3isp/isp.c: fix error return code

2012-09-23 Thread Peter Senna Tschudin
On Sun, Sep 23, 2012 at 7:39 PM, Mauro Carvalho Chehab mche...@redhat.com wrote: Laurent, Could you please review this patch? Peter, Please, always c/c the driver maintainer/author on patches you submit. You can check it with scripts/get_maintainer.pl. Sorry for that. I'll be more careful

[PATCH 2/3] cx25821: Replace kmemdup for kstrdup and clean up

2012-09-17 Thread Peter Senna Tschudin
Replace kmemdup for kstrdup and cleanup related code. Signed-off-by: Peter Senna Tschudin peter.se...@gmail.com --- drivers/media/pci/cx25821/cx25821-audio-upstream.c | 11 --- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/drivers/media/pci/cx25821/cx25821-audio

[PATCH 3/3] cx25821: Cleanup filename assignment code

2012-09-17 Thread Peter Senna Tschudin
(_defaultAudioName, GFP_KERNEL); if (!dev-_audiofilename) { err = -ENOMEM; goto error; } Signed-off-by: Peter Senna Tschudin peter.se...@gmail.com --- drivers/media/pci/cx25821/cx25821-audio-upstream.c | 21 ++--- 1 file changed, 6

[PATCH 1/3] cx25821: fix error return code and clean up

2012-09-17 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com The function cx25821_sram_channel_setup_upstream_audio always return zero, so the return value is not saved any more. Convert a nonnegative error return code to a negative one, as returned elsewhere in the function. A simplified version

[PATCH] drivers/media/platform/davinci/vpbe.c: Remove unused label and rename remaining labels

2012-09-14 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com Remove unused label and rename remaining labels Signed-off-by: Peter Senna Tschudin peter.se...@gmail.com --- Depends on patch [v2,7/8] drivers/media/platform/davinci/vpbe.c: Removes useless kfree() - http://patchwork.linuxtv.org/patch/14307

Re: [PATCH] drivers/media: Removes useless kfree()

2012-09-12 Thread Peter Senna Tschudin
I'll split this patch in one patch for file and resend. On Wed, Sep 12, 2012 at 10:06 AM, Dan Carpenter dan.carpen...@oracle.com wrote: On Tue, Sep 11, 2012 at 08:00:32PM +0200, Peter Senna Tschudin wrote: diff --git a/drivers/media/dvb-frontends/lg2160.c b/drivers/media/dvb-frontends/lg2160

[PATCH v2 2/8] drivers/media/dvb-frontends/lg2160.c: Removes useless kfree()

2012-09-12 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com Remove useless kfree() and clean up code related to the removal. The semantic patch that finds this problem is as follows: (http://coccinelle.lip6.fr/) // smpl @r exists@ position p1,p2; expression x; @@ if (x@p1 == NULL) { ... kfree@p2(x

[PATCH v2 8/8] drivers/media/tuners/mt2063.c: Removes useless kfree()

2012-09-12 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com Remove useless kfree() and clean up code related to the removal. The semantic patch that finds this problem is as follows: (http://coccinelle.lip6.fr/) // smpl @r exists@ position p1,p2; expression x; @@ if (x@p1 == NULL) { ... kfree@p2(x

[PATCH v2 1/8] drivers/media/dvb-frontends/dvb_dummy_fe.c: Removes useless kfree()

2012-09-12 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com Remove useless kfree() and clean up code related to the removal. The semantic patch that finds this problem is as follows: (http://coccinelle.lip6.fr/) // smpl @r exists@ position p1,p2; expression x; @@ if (x@p1 == NULL) { ... kfree@p2(x

[PATCH v2 3/8] drivers/media/dvb-frontends/s5h1432.c: Removes useless kfree()

2012-09-12 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com Remove useless kfree() and clean up code related to the removal. The semantic patch that finds this problem is as follows: (http://coccinelle.lip6.fr/) // smpl @r exists@ position p1,p2; expression x; @@ if (x@p1 == NULL) { ... kfree@p2(x

[PATCH v2 4/8] drivers/media/dvb-frontends/s921.c: Removes useless kfree()

2012-09-12 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com Remove useless kfree() and clean up code related to the removal. The semantic patch that finds this problem is as follows: (http://coccinelle.lip6.fr/) // smpl @r exists@ position p1,p2; expression x; @@ if (x@p1 == NULL) { ... kfree@p2(x

[PATCH v2 5/8] drivers/media/dvb-frontends/stb6100.c: Removes useless kfree()

2012-09-12 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com Remove useless kfree() and clean up code related to the removal. The semantic patch that finds this problem is as follows: (http://coccinelle.lip6.fr/) // smpl @r exists@ position p1,p2; expression x; @@ if (x@p1 == NULL) { ... kfree@p2(x

[PATCH v2 6/8] drivers/media/dvb-frontends/tda665x.c: Removes useless kfree()

2012-09-12 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com Remove useless kfree() and clean up code related to the removal. The semantic patch that finds this problem is as follows: (http://coccinelle.lip6.fr/) // smpl @r exists@ position p1,p2; expression x; @@ if (x@p1 == NULL) { ... kfree@p2(x

[PATCH v2 7/8] drivers/media/platform/davinci/vpbe.c: Removes useless kfree()

2012-09-12 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com Remove useless kfree() and clean up code related to the removal. The semantic patch that finds this problem is as follows: (http://coccinelle.lip6.fr/) // smpl @r exists@ position p1,p2; expression x; @@ if (x@p1 == NULL) { ... kfree@p2(x

Re: [PATCH v2 7/8] drivers/media/platform/davinci/vpbe.c: Removes useless kfree()

2012-09-12 Thread Peter Senna Tschudin
or err1, err2, err3, or Any suggestions? -- Peter -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

[PATCH] drivers/media: Removes useless kfree()

2012-09-11 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com The semantic patch that finds this problem is as follows: (http://coccinelle.lip6.fr/) // smpl @r exists@ position p1,p2; expression x; @@ if (x@p1 == NULL) { ... kfree@p2(x); ... return ...; } @unchanged exists@ position r.p1,r.p2; expression e

Re: [PATCH v2] drivers/media/pci/cx25821/cx25821-video-upstream-ch2.c: fix error return code

2012-09-10 Thread Peter Senna Tschudin
. Sorry for that too. Thanks (and sorry for the nitpick)... Thanks! Ezequiel. -- Peter -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

[PATCH] drivers/media/pci/cx25821/cx25821-video-upstream-ch2.c: Replace kmemdup for kstrdup

2012-09-10 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com Replace kmemdup for kstrdup and cleaning up the code. Signed-off-by: Peter Senna Tschudin peter.se...@gmail.com --- It depends on the patch http://patchwork.linuxtv.org/patch/14231/ tmp/cx25821-video-upstream-ch2.c | 27

[PATCH v2] drivers/media/pci/cx25821/cx25821-video-upstream-ch2.c: fix error return code

2012-09-08 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com Convert a nonnegative error return code to a negative one, as returned elsewhere in the function. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // smpl ( if@p1 (\(ret 0\|ret != 0

[PATCH] Unregister device and unlock mutex before exit when error

2012-09-07 Thread Peter Senna Tschudin
Signed-off-by: Peter Senna Tschudin peter.se...@gmail.com --- Depends on patch 14167: http://patchwork.linuxtv.org/patch/14167/ [PATCH 1/5] drivers/media/platform/davinci/vpbe.c: fix error return code drivers/media/platform/davinci/vpbe.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion

[PATCH] drivers/media/pci/cx25821/cx25821-video-upstream-ch2.c: fix error return code

2012-09-07 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com Convert a nonnegative error return code to a negative one, as returned elsewhere in the function. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // smpl ( if@p1 (\(ret 0\|ret != 0

[PATCH 6/10] drivers/media/i2c/tea6415c.c: removes unnecessary semicolon

2012-09-07 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com removes unnecessary semicolon Found by Coccinelle: http://coccinelle.lip6.fr/ Signed-off-by: Peter Senna Tschudin peter.se...@gmail.com --- drivers/media/i2c/tea6415c.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff -u -p

[PATCH 7/10] drivers/media/tuners/tda18271-common.c: removes unnecessary semicolon

2012-09-07 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com removes unnecessary semicolon Found by Coccinelle: http://coccinelle.lip6.fr/ Signed-off-by: Peter Senna Tschudin peter.se...@gmail.com --- drivers/media/tuners/tda18271-common.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions

[PATCH 8/10] drivers/media/dvb-core/dvb_demux.c: removes unnecessary semicolon

2012-09-07 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com removes unnecessary semicolon Found by Coccinelle: http://coccinelle.lip6.fr/ Signed-off-by: Peter Senna Tschudin peter.se...@gmail.com --- drivers/media/dvb-core/dvb_demux.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions

[PATCH 9/10] drivers/media/dvb-frontends/tda10071.c: removes unnecessary semicolon

2012-09-07 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com removes unnecessary semicolon Found by Coccinelle: http://coccinelle.lip6.fr/ Signed-off-by: Peter Senna Tschudin peter.se...@gmail.com --- drivers/media/dvb-frontends/tda10071.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions

[PATCH v2] drivers/media/platform/s5p-tv/sdo_drv.c: fix error return code

2012-09-06 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com Convert a nonnegative error return code to a negative one, as returned elsewhere in the function. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // smpl ( if@p1 (\(ret 0\|ret != 0

[PATCH 8/14] drivers/media/pci/ttpci/budget-av.c: fix error return code

2012-09-06 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com Convert a nonnegative error return code to a negative one, as returned elsewhere in the function. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // smpl ( if@p1 (\(ret 0\|ret != 0

[PATCH 4/14] drivers/media/v4l2-core/videobuf2-core.c: fix error return code

2012-09-06 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com Convert a nonnegative error return code to a negative one, as returned elsewhere in the function. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // smpl ( if@p1 (\(ret 0\|ret != 0

[PATCH 14/14] drivers/media/usb/gspca/cpia1.c: fix error return code

2012-09-06 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com Convert a nonnegative error return code to a negative one, as returned elsewhere in the function. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // smpl ( if@p1 (\(ret 0\|ret != 0

[PATCH 2/14] drivers/media/platform/soc_camera/mx2_camera.c: fix error return code

2012-09-06 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com Convert a nonnegative error return code to a negative one, as returned elsewhere in the function. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // smpl ( if@p1 (\(ret 0\|ret != 0

[PATCH 1/14] drivers/media/platform/soc_camera/soc_camera.c: fix error return code

2012-09-06 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com Convert a nonnegative error return code to a negative one, as returned elsewhere in the function. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // smpl ( if@p1 (\(ret 0\|ret != 0

[PATCH 3/14] drivers/media/platform/blackfin/bfin_capture.c: fix error return code

2012-09-06 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com Convert a nonnegative error return code to a negative one, as returned elsewhere in the function. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // smpl ( if@p1 (\(ret 0\|ret != 0

[PATCH 5/14] drivers/media/pci/cx25821/cx25821-video-upstream.c: fix error return code

2012-09-06 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com Convert a nonnegative error return code to a negative one, as returned elsewhere in the function. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // smpl ( if@p1 (\(ret 0\|ret != 0

[PATCH 13/14] drivers/media/usb/hdpvr/hdpvr-core.c: fix error return code

2012-09-06 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com Convert a nonnegative error return code to a negative one, as returned elsewhere in the function. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // smpl ( if@p1 (\(ret 0\|ret != 0

[PATCH 12/14] drivers/media/usb/tm6000/tm6000-video.c: fix error return code

2012-09-06 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com Convert a nonnegative error return code to a negative one, as returned elsewhere in the function. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // smpl ( if@p1 (\(ret 0\|ret != 0

[PATCH 6/14] drivers/media/pci/dm1105/dm1105.c: fix error return code

2012-09-06 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com Convert a nonnegative error return code to a negative one, as returned elsewhere in the function. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // smpl ( if@p1 (\(ret 0\|ret != 0

[PATCH 7/14] drivers/media/pci/ngene/ngene-core.c: fix error return code

2012-09-06 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com Convert a nonnegative error return code to a negative one, as returned elsewhere in the function. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // smpl ( if@p1 (\(ret 0\|ret != 0

[PATCH 9/14] drivers/media/radio/radio-cadet.c: fix error return code

2012-09-06 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com Convert a nonnegative error return code to a negative one, as returned elsewhere in the function. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // smpl ( if@p1 (\(ret 0\|ret != 0

[PATCH 10/14] drivers/media/i2c/mt9m032.c: fix error return code

2012-09-06 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com Convert a nonnegative error return code to a negative one, as returned elsewhere in the function. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // smpl ( if@p1 (\(ret 0\|ret != 0

[PATCH 11/14] drivers/media/usb/stk1160/stk1160-i2c.c: fix error return code

2012-09-06 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com Convert a nonnegative error return code to a negative one, as returned elsewhere in the function. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // smpl ( if@p1 (\(ret 0\|ret != 0

[PATCH 6/10] drivers/media/pci/cx88/cx88-blackbird.c: removes unnecessary semicolon

2012-09-06 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com removes unnecessary semicolon Found by Coccinelle: http://coccinelle.lip6.fr/ Signed-off-by: Peter Senna Tschudin peter.se...@gmail.com --- drivers/media/pci/cx88/cx88-blackbird.c |6 +++--- 1 file changed, 3 insertions(+), 3 deletions

[PATCH 5/10] drivers/media/dvb-frontends/itd1000.c: removes unnecessary semicolon

2012-09-06 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com removes unnecessary semicolon Found by Coccinelle: http://coccinelle.lip6.fr/ Signed-off-by: Peter Senna Tschudin peter.se...@gmail.com --- drivers/media/dvb-frontends/itd1000.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff -u

[PATCH 4/10] drivers/media/dvb-frontends/sp8870.c: removes unnecessary semicolon

2012-09-06 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com removes unnecessary semicolon Found by Coccinelle: http://coccinelle.lip6.fr/ Signed-off-by: Peter Senna Tschudin peter.se...@gmail.com --- drivers/media/dvb-frontends/sp8870.c |6 +++--- 1 file changed, 3 insertions(+), 3 deletions

[PATCH 3/10] drivers/media/radio/si4713-i2c.c: removes unnecessary semicolon

2012-09-06 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com removes unnecessary semicolon Found by Coccinelle: http://coccinelle.lip6.fr/ Signed-off-by: Peter Senna Tschudin peter.se...@gmail.com --- drivers/media/radio/si4713-i2c.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions

[PATCH 2/3] drivers/media/rc/ati_remote.c: fix error return code

2012-09-04 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com Convert a nonnegative error return code to a negative one, as returned elsewhere in the function. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // smpl ( if@p1 (\(ret 0\|ret != 0

[PATCH 1/3] drivers/media/rc/redrat3.c: fix error return code

2012-09-04 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com Convert a nonnegative error return code to a negative one, as returned elsewhere in the function. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // smpl ( if@p1 (\(ret 0\|ret != 0

[PATCH 3/3] drivers/media/platform/davinci/vpfe_capture.c: fix error return code

2012-09-04 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com Convert a nonnegative error return code to a negative one, as returned elsewhere in the function. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // smpl ( if@p1 (\(ret 0\|ret != 0

[PATCH 2/5] drivers/media/platform/s5p-tv/sdo_drv.c: fix error return code

2012-09-04 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com Convert a nonnegative error return code to a negative one, as returned elsewhere in the function. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // smpl ( if@p1 (\(ret 0\|ret != 0

[PATCH 1/5] drivers/media/platform/davinci/vpbe.c: fix error return code

2012-09-04 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com Convert a nonnegative error return code to a negative one, as returned elsewhere in the function. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // smpl ( if@p1 (\(ret 0\|ret != 0

[PATCH 5/5] drivers/media/platform/omap3isp/isp.c: fix error return code

2012-09-04 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com Convert a nonnegative error return code to a negative one, as returned elsewhere in the function. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // smpl ( if@p1 (\(ret 0\|ret != 0

[PATCH 4/5] drivers/media/platform/vino.c: fix error return code

2012-09-04 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com Convert a nonnegative error return code to a negative one, as returned elsewhere in the function. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // smpl ( if@p1 (\(ret 0\|ret != 0

[PATCH 3/5] drivers/media/platform/s5p-tv/mixer_video.c: fix error return code

2012-09-04 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com Convert a nonnegative error return code to a negative one, as returned elsewhere in the function. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // smpl ( if@p1 (\(ret 0\|ret != 0

Re: Preferred setup for development?

2012-08-17 Thread Peter Senna Tschudin
of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html []'s -- Peter Senna Tschudin peter.se...@gmail.com gpg id: 48274C36 -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org

Re: [PATCH] drivers/media/video/mx2_emmaprp.c: use devm_kzalloc and devm_clk_get

2012-08-06 Thread Lars-Peter Clausen
On 08/06/2012 04:26 PM, Dan Carpenter wrote: On Mon, Aug 06, 2012 at 05:23:23PM +0300, Dan Carpenter wrote: On Sat, Aug 04, 2012 at 08:23:27PM +0200, Julia Lawall wrote: @@ -922,12 +920,7 @@ static int emmaprp_probe(struct platform_device *pdev) platform_set_drvdata(pdev, pcdev); -

RE: stk7700d problem

2012-08-01 Thread Peter Tilley
Apologies for resending. Mailing list bounced the email the first time becuase it wasn't plain text format. I have a dual USB DVB-T tuner which is sold under the brand Kaiser Baas KBA 01004 but under the hood looks pretty much the same as the Emtec s830

Re: [Device-drivers-devel] [RFCv1 PATCH 0/7] Add adv7604/ad9389b drivers

2012-07-06 Thread Lars-Peter Clausen
On 07/05/2012 04:26 PM, Hans Verkuil wrote: Hi all, This RFC patch series builds on an earlier RFC patch series (posted only to linux-media) that adds support for DVI/HDMI/DP connectors to the V4L2 API. This earlier patch series is here:

Re: [PATCH] [V2] stv090x: variable 'no_signal' set but not used

2012-06-28 Thread Peter Senna Tschudin
Hey Ezequiel, On Thu, Jun 28, 2012 at 1:02 AM, Ezequiel Garcia elezegar...@gmail.com wrote: Hey Peter, On Wed, Jun 27, 2012 at 7:18 PM, Peter Senna Tschudin peter.se...@gmail.com wrote: -                       no_signal = stv090x_chk_signal(state); +                       (void

Re: [PATCH] [V2] stv090x: variable 'no_signal' set but not used

2012-06-28 Thread Peter Senna Tschudin
On Thu, Jun 28, 2012 at 10:17 AM, Peter Senna Tschudin peter.se...@gmail.com wrote: Hey Ezequiel, On Thu, Jun 28, 2012 at 1:02 AM, Ezequiel Garcia elezegar...@gmail.com wrote: Hey Peter, On Wed, Jun 27, 2012 at 7:18 PM, Peter Senna Tschudin peter.se...@gmail.com wrote

Re: [PATCH] [V2] stv090x: variable 'no_signal' set but not used

2012-06-28 Thread Peter Senna Tschudin
On Thu, Jun 28, 2012 at 10:41 AM, Ezequiel Garcia elezegar...@gmail.com wrote: On Thu, Jun 28, 2012 at 10:17 AM, Peter Senna Tschudin peter.se...@gmail.com wrote: Hey Ezequiel, On Thu, Jun 28, 2012 at 1:02 AM, Ezequiel Garcia elezegar...@gmail.com wrote: Hey Peter, On Wed, Jun 27, 2012

[PATCH] [V3] stv090x: variable 'no_signal' set but not used

2012-06-28 Thread Peter Senna Tschudin
Remove variable and ignore return value of stv090x_chk_signal(). Tested by compilation only. Signed-off-by: Peter Senna Tschudin peter.se...@gmail.com --- drivers/media/dvb/frontends/stv090x.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/media/dvb/frontends

Re: [git:v4l-dvb/for_v3.6] [media] stv090x: variable 'no_signal' set but not used

2012-06-27 Thread Peter Senna Tschudin
for that. I'll send V2 of this patch just ignoring the return value. Can you please send me some reference about R-M-x registers? Please revert this patch. Thanks, Manu Thanks, Peter On Fri, Jun 22, 2012 at 2:28 AM, Mauro Carvalho Chehab mche...@redhat.com wrote: This is an automatic

[PATCH] [V2] stv090x: variable 'no_signal' set but not used

2012-06-27 Thread Peter Senna Tschudin
Remove variable and ignore return value of stv090x_chk_signal(). Tested by compilation only. Signed-off-by: Peter Senna Tschudin peter.se...@gmail.com --- drivers/media/dvb/frontends/stv090x.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/media/dvb/frontends

[PATCH] media: no semicolon after switch

2012-06-21 Thread Peter Meerwald
From: Peter Meerwald p.meerw...@bct-electronic.com Signed-off-by: Peter Meerwald pme...@pmeerw.net --- drivers/media/video/omap3isp/ispccdc.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/media/video/omap3isp/ispccdc.c b/drivers/media/video/omap3isp

[PATCH] media: remove unused element datawidth from struct mt9m111

2012-06-20 Thread Peter Meerwald
Signed-off-by: Peter Meerwald pme...@pmeerw.net --- drivers/media/video/mt9m111.c |1 - 1 file changed, 1 deletion(-) diff --git a/drivers/media/video/mt9m111.c b/drivers/media/video/mt9m111.c index b0c5299..863d722 100644 --- a/drivers/media/video/mt9m111.c +++ b/drivers/media/video/mt9m111

Re: cron job: media_tree daily build: ERRORS: help needed

2012-06-19 Thread Peter Senna Tschudin
Hans, I've: [peter@ace tmp]$ diff linux-2.6.35.13/scripts/mod/file2alias.c linux-3.4.3/scripts/mod/file2alias.c And found: 727a840 ADD_TO_DEVTABLE(i2c, struct i2c_device_id, do_i2c_entry); This line only exists on 3.4.3 version of file2alias.c. Isn't this why it successfully compile with newer

Re: cron job: media_tree daily build: ERRORS: help needed

2012-06-19 Thread Peter Senna Tschudin
Full diff: http://pastebin.com/BJS2EXcH On Tue, Jun 19, 2012 at 4:39 PM, Peter Senna Tschudin peter.se...@gmail.com wrote: Hans, I've: [peter@ace tmp]$ diff linux-2.6.35.13/scripts/mod/file2alias.c linux-3.4.3/scripts/mod/file2alias.c And found: 727a840 ADD_TO_DEVTABLE(i2c, struct

[PATCH 1/8] [RESEND] cx231xx: Paranoic stack memory save

2012-06-14 Thread Peter Senna Tschudin
Saves 255 bytes of stack memory on cx231xx_usb_probe() by removing a char array. Tested by compilation only. Signed-off-by: Peter Senna Tschudin peter.se...@gmail.com --- drivers/media/video/cx231xx/cx231xx-cards.c | 17 +++-- 1 file changed, 3 insertions(+), 14 deletions(-) diff

<    2   3   4   5   6   7   8   9   >