[PATCH 04/14] m5mols: use atomic_dec_not_zero()

2017-01-30 Thread Fabian Frederick
instead of atomic_add_unless(value, -1, 0) Signed-off-by: Fabian Frederick --- drivers/media/i2c/m5mols/m5mols_core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/i2c/m5mols/m5mols_core.c b/drivers/media/i2c/m5mols/m5mols_core.c index acb804b..3aab2ca

[PATCH 05/14] omap3isp: use atomic_dec_not_zero()

2017-01-30 Thread Fabian Frederick
instead of atomic_add_unless(value, -1, 0) Signed-off-by: Fabian Frederick --- drivers/media/platform/omap3isp/ispstat.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/platform/omap3isp/ispstat.c b/drivers/media/platform/omap3isp/ispstat.c index 47cbc7e

[PATCH 00/14] use atomic_dec_not_zero()

2017-01-30 Thread Fabian Frederick
complementary definition to atomic_inc_not_zero() featured in lib/fault-inject.c. This small patchset moves it to include/linux/atomic.h using it instead of atomic_add_unless(value, -1, 0) s390 patches were not compile-tested. Fabian Frederick (14): locking/atomic: import atomic_dec_not_zero

[PATCH 1/1 linux-next] wl128x: use swap() in fm_rdsparse_swapbytes()

2015-06-10 Thread Fabian Frederick
Use kernel.h macro definition. Thanks to Julia Lawall for Coccinelle scripting support. Signed-off-by: Fabian Frederick --- drivers/media/radio/wl128x/fmdrv_common.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/media/radio/wl128x/fmdrv_common.c b/drivers

[PATCH 1/1 linux-next] ttusb-dec: use swap() in swap_bytes()

2015-06-10 Thread Fabian Frederick
Use kernel.h macro definition. Thanks to Julia Lawall for Coccinelle scripting support. Signed-off-by: Fabian Frederick --- drivers/media/usb/ttusb-dec/ttusb_dec.c | 9 ++--- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/drivers/media/usb/ttusb-dec/ttusb_dec.c b/drivers

[PATCH 1/1 linux-next] [media] btcx-risc: use swap() in btcx_sort_clips()

2015-06-10 Thread Fabian Frederick
Use kernel.h macro definition. Thanks to Julia Lawall for Coccinelle scripting support. Signed-off-by: Fabian Frederick --- drivers/media/pci/bt8xx/btcx-risc.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/media/pci/bt8xx/btcx-risc.c b/drivers/media/pci/bt8xx

[PATCH 1/1 linux-next] saa7146: use swap() in sort_and_eliminate()

2015-06-10 Thread Fabian Frederick
Use kernel.h macro definition. Thanks to Julia Lawall for Coccinelle scripting support. Signed-off-by: Fabian Frederick --- drivers/media/common/saa7146/saa7146_hlp.c | 9 +++-- 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/drivers/media/common/saa7146/saa7146_hlp.c b

[PATCH 1/1 linux-next] saa6588: use swap() in saa6588_i2c_poll()

2015-06-10 Thread Fabian Frederick
Use kernel.h macro definition. Thanks to Julia Lawall for Coccinelle scripting support. Signed-off-by: Fabian Frederick --- drivers/media/i2c/saa6588.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/media/i2c/saa6588.c b/drivers/media/i2c/saa6588.c index 2960b5a

[PATCH 1/1 linux-next] v4l2-dv-timings: use swap() in v4l2_calc_aspect_ratio()

2015-06-10 Thread Fabian Frederick
Use kernel.h macro definition. Thanks to Julia Lawall for Coccinelle scripting support. Signed-off-by: Fabian Frederick --- drivers/media/v4l2-core/v4l2-dv-timings.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/media/v4l2-core/v4l2-dv-timings.c b/drivers

Re: [PATCH 1/1 linux-next] omap_vout: use swap() in omapvid_init()

2015-05-18 Thread Fabian Frederick
> On 18 May 2015 at 20:20 Vaibhav Hiremath wrote: > > > > > On Monday 18 May 2015 11:24 PM, Fabian Frederick wrote: > > Use kernel.h macro definition. > > > > Signed-off-by: Fabian Frederick > > --- > >   drivers/media/platform/omap/oma

[PATCH 1/1 linux-next] omap_vout: use swap() in omapvid_init()

2015-05-18 Thread Fabian Frederick
Use kernel.h macro definition. Signed-off-by: Fabian Frederick --- drivers/media/platform/omap/omap_vout.c | 10 +++--- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/drivers/media/platform/omap/omap_vout.c b/drivers/media/platform/omap/omap_vout.c index 17b189a..f09c5f1

[PATCH 1/1 linux-next] [media] siano: define SRVM_MAX_PID_FILTERS only once

2015-05-02 Thread Fabian Frederick
SRVM_MAX_PID_FILTERS was defined in 2 sms_tx_stats structures Signed-off-by: Fabian Frederick --- drivers/media/common/siano/smscoreapi.h | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/media/common/siano/smscoreapi.h b/drivers/media/common/siano/smscoreapi.h

[PATCH 23/35 linux-next] [media] constify of_device_id array

2015-03-16 Thread Fabian Frederick
of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick --- drivers/media/i2c/adv7604.c | 2 +- drivers/media/platform/fsl-viu.c | 2 +- drivers/media/platform/soc_camera/rcar_vin.c | 2

[PATCH 00/35 linux-next] constify of_device_id array

2015-03-16 Thread Fabian Frederick
This small patchset adds const to of_device_id arrays in drivers branch. Fabian Frederick (35): ata: constify of_device_id array regulator: constify of_device_id array thermal: constify of_device_id array tty/hvc_opal: constify of_device_id array tty: constify of_device_id array power

[PATCH 4/7 linux-next] saa7146: replace current->state by set_current_state()

2015-02-20 Thread Fabian Frederick
Use helper functions to access current->state. Direct assignments are prone to races and therefore buggy. current->state = TASK_RUNNING can be replaced by __set_current_state() Thanks to Peter Zijlstra for the exact definition of the problem. Suggested-By: Peter Zijlstra Signed-off-by:

Re: [PATCH 09/11 linux-next] [media] uvcvideo: remove unnecessary version.h inclusion

2014-12-30 Thread Fabian Frederick
> On 30 December 2014 at 00:42 Laurent Pinchart > wrote: > > > Hi Fabian, > > Thank you for the patch. > > On Monday 29 December 2014 15:29:43 Fabian Frederick wrote: > > Based on versioncheck. > > > > Signed-off-by: Fabian Frederick > > Acke

[PATCH 08/11 linux-next] [media] vivid: remove unnecessary version.h inclusion

2014-12-29 Thread Fabian Frederick
Based on versioncheck. Signed-off-by: Fabian Frederick --- drivers/media/platform/vivid/vivid-tpg.h | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/media/platform/vivid/vivid-tpg.h b/drivers/media/platform/vivid/vivid-tpg.h index 9dc463a4..bd8b1c7 100644 --- a/drivers/media/platform

[PATCH 06/11 linux-next] [media] s5p-g2d: remove unnecessary version.h inclusion

2014-12-29 Thread Fabian Frederick
Based on versioncheck. Signed-off-by: Fabian Frederick --- drivers/media/platform/s5p-g2d/g2d.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/media/platform/s5p-g2d/g2d.c b/drivers/media/platform/s5p-g2d/g2d.c index 47ba8fb..ec3e124 100644 --- a/drivers/media/platform/s5p-g2d

[PATCH 09/11 linux-next] [media] uvcvideo: remove unnecessary version.h inclusion

2014-12-29 Thread Fabian Frederick
Based on versioncheck. Signed-off-by: Fabian Frederick --- drivers/media/usb/uvc/uvc_v4l2.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/media/usb/uvc/uvc_v4l2.c b/drivers/media/usb/uvc/uvc_v4l2.c index 9c5cbcf..43e953f 100644 --- a/drivers/media/usb/uvc/uvc_v4l2.c +++ b/drivers

[PATCH 07/11 linux-next] [media] s5p-mfc: remove unnecessary version.h inclusion

2014-12-29 Thread Fabian Frederick
Based on versioncheck. Signed-off-by: Fabian Frederick --- drivers/media/platform/s5p-mfc/s5p_mfc_dec.c | 1 - drivers/media/platform/s5p-mfc/s5p_mfc_enc.c | 1 - 2 files changed, 2 deletions(-) diff --git a/drivers/media/platform/s5p-mfc/s5p_mfc_dec.c b/drivers/media/platform/s5p-mfc

[PATCH 05/11 linux-next] [media] tw68: remove unnecessary version.h inclusion

2014-12-29 Thread Fabian Frederick
Based on versioncheck. Signed-off-by: Fabian Frederick --- drivers/media/pci/tw68/tw68.h | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/media/pci/tw68/tw68.h b/drivers/media/pci/tw68/tw68.h index 7a7501b..93f2335 100644 --- a/drivers/media/pci/tw68/tw68.h +++ b/drivers/media/pci

[PATCH 00/11 linux-next] drivers: remove unnecessary version.h inclusion

2014-12-29 Thread Fabian Frederick
This small patchset removes unnecessary version.h includes detected by versioncheck in drivers branch. Fabian Frederick (11): rsxx: remove unnecessary version.h inclusion skd: remove unnecessary version.h inclusion ipmi: remove unnecessary version.h inclusion input: remove unnecessary

Re: [PATCH 0/7 linux-next] drivers: remove deprecated IRQF_DISABLED

2014-10-06 Thread Fabian Frederick
> On 06 October 2014 at 20:46 Borislav Petkov wrote: > > > On Mon, Oct 06, 2014 at 08:33:00PM +0200, Fabian Frederick wrote: > >    You're right, I guess we can forget this patchset. > > I didn't see it was already done. (nothing in linux-next yet). > >

Re: [PATCH 0/7 linux-next] drivers: remove deprecated IRQF_DISABLED

2014-10-06 Thread Fabian Frederick
> On 06 October 2014 at 19:40 Borislav Petkov wrote: > > > On Mon, Oct 06, 2014 at 05:35:47PM +0200, Fabian Frederick wrote: > > This small patchset removes IRQF_DISABLED from drivers branch. > > > > See include/linux/interrupt.h: > > "This f

[PATCH 3/7 linux-next] tw68: remove deprecated IRQF_DISABLED

2014-10-06 Thread Fabian Frederick
See include/linux/interrupt.h: "This flag is a NOOP and scheduled to be removed" Signed-off-by: Fabian Frederick --- drivers/media/pci/tw68/tw68-core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/pci/tw68/tw68-core.c b/drivers/media/pci/tw68/t

[PATCH 0/7 linux-next] drivers: remove deprecated IRQF_DISABLED

2014-10-06 Thread Fabian Frederick
This small patchset removes IRQF_DISABLED from drivers branch. See include/linux/interrupt.h: "This flag is a NOOP and scheduled to be removed" Note: (cross)compiled but untested. Fabian Frederick (7): mv64x60_edac: remove deprecated IRQF_DISABLED ppc4xx_edac: remove

Re: [PATCH 1/1] drivers/base/dma-buf.c: replace dma_buf_uninit_debugfs by debugfs_remove_recursive

2014-07-12 Thread Fabian Frederick
> On 10 July 2014 at 02:34 Greg Kroah-Hartman > wrote: > > > On Fri, Jun 27, 2014 at 10:32:10PM +0200, Fabian Frederick wrote: > > null test before debugfs_remove_recursive is not needed so one line function > > dma_buf_uninit_debugfs can be remove

Re: [PATCH 1/1] dvb-frontends: remove unnecessary break after goto

2014-07-08 Thread Fabian Frederick
till no automatic detection of those cases. Regards, Fabian > > regards > Atnti > > > On 07/08/2014 08:23 PM, Fabian Frederick wrote: > > Cc: Antti Palosaari > > Cc: Mauro Carvalho Chehab > > Cc: linux-media@vger.kernel.org > > Signed-off-by: Fabian Fr

[PATCH 1/1] xc5000: remove unnecessary break after goto

2014-07-08 Thread Fabian Frederick
Cc: Mauro Carvalho Chehab Cc: linux-media@vger.kernel.org Signed-off-by: Fabian Frederick --- drivers/media/tuners/xc5000.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/media/tuners/xc5000.c b/drivers/media/tuners/xc5000.c index 2b3d514..f059ba2 100644 --- a/drivers/media/tuners

[PATCH 1/1] xc4000: remove unnecessary break after goto

2014-07-08 Thread Fabian Frederick
Cc: Mauro Carvalho Chehab Cc: linux-media@vger.kernel.org Signed-off-by: Fabian Frederick --- drivers/media/tuners/xc4000.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/media/tuners/xc4000.c b/drivers/media/tuners/xc4000.c index 2018bef..d2ee5df 100644 --- a/drivers/media/tuners

[PATCH 1/1] xc2028: remove unnecessary break after goto

2014-07-08 Thread Fabian Frederick
Cc: Mauro Carvalho Chehab Cc: linux-media@vger.kernel.org Signed-off-by: Fabian Frederick --- drivers/media/tuners/tuner-xc2028.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/media/tuners/tuner-xc2028.c b/drivers/media/tuners/tuner-xc2028.c index 6ef93ee..565eeeb 100644 --- a

[PATCH 1/1] dvb-frontends: remove unnecessary break after goto

2014-07-08 Thread Fabian Frederick
Cc: Antti Palosaari Cc: Mauro Carvalho Chehab Cc: linux-media@vger.kernel.org Signed-off-by: Fabian Frederick --- drivers/media/dvb-frontends/af9013.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/media/dvb-frontends/af9013.c b/drivers/media/dvb-frontends/af9013.c index fb504f1

[PATCH 1/1] r820t: remove unnecessary break after goto

2014-07-08 Thread Fabian Frederick
Cc: Mauro Carvalho Chehab Cc: Antti Palosaari Cc: Michael Krufky Cc: linux-media@vger.kernel.org Signed-off-by: Fabian Frederick --- drivers/media/tuners/r820t.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/media/tuners/r820t.c b/drivers/media/tuners/r820t.c index 96ccfeb

[RFC 1/1] em28xx: fix configuration warning

2014-07-07 Thread Fabian Frederick
kernel.org Signed-off-by: Fabian Frederick --- drivers/media/usb/em28xx/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/media/usb/em28xx/Kconfig b/drivers/media/usb/em28xx/Kconfig index f5d7198..d42e1de 100644 --- a/drivers/media/usb/em28xx/Kconfig +++ b/drivers/media/usb/em2

[PATCH 1/1] drivers/media/usb/ttusb-budget/dvb-ttusb-budget.c: remove unnecessary null test before usb_free_urb

2014-06-28 Thread Fabian Frederick
Fix checkpatch warning: WARNING: usb_free_urb(NULL) is safe this check is probably not required Cc: Mauro Carvalho Chehab Cc: linux-media@vger.kernel.org Signed-off-by: Fabian Frederick --- drivers/media/usb/ttusb-budget/dvb-ttusb-budget.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions

[PATCH 1/1] drivers/base/dma-buf.c: replace dma_buf_uninit_debugfs by debugfs_remove_recursive

2014-06-27 Thread Fabian Frederick
null test before debugfs_remove_recursive is not needed so one line function dma_buf_uninit_debugfs can be removed. This patch calls debugfs_remove_recursive under CONFIG_DEBUG_FS Cc: Sumit Semwal Cc: Greg Kroah-Hartman Cc: linux-media@vger.kernel.org Signed-off-by: Fabian Frederick --- This