[PATCH] dvb-core: Release semaphore on error path dvb_register_device().

2012-06-23 Thread santosh nayak
From: Santosh Nayak There is a missing "up_write()" here. Semaphore should be released before returning error value. Signed-off-by: Santosh Nayak --- Destination tree "linux-next" drivers/media/dvb/dvb-core/dvbdev.c |1 + 1 files changed, 1 insertions(+), 0 deleti

[PATCH] [media] dib0700: Return -EINTR and unlock mutex if locking attempts fails.

2012-03-19 Thread santosh nayak
From: Santosh Nayak In 'dib0700_i2c_xfer_new()' and 'dib0700_i2c_xfer_legacy()' we are taking two locks: 1. i2c_mutex 2. usb_mutex If attempt to take 'usb_mutex' lock fails then the previously taken lock 'i2c_mutex' should b

[PATCH] [media] staging: use mutex_lock() in s2250_probe().

2012-03-17 Thread santosh nayak
From: Santosh Nayak Use uninterruptable sleep lock 'mutex_lock()' in place of mutex_lock_interruptible() because there is no userspace for s2250_probe(). Return -ENOMEM if kzalloc() fails to allocate and initialize. Signed-off-by: Santosh Nayak --- drivers/staging/media/go

[PATCH] [media] staging: Return -EINTR in s2250_probe() if fails to get lock.

2012-03-16 Thread santosh nayak
From: Santosh Nayak In s2250_probe(), If locking attempt is interrupted by a signal then it should return -EINTR after unregistering audio device and making free the allocated memory. At present, if locking is interrupted by signal it will display message "initialized successfully"

[PATCH] isdn: Return -EINTR in gigaset_start() if locking attempts fails.

2012-03-16 Thread santosh nayak
From: Santosh Nayak If locking attempt was interrupted by a signal then we should return -EINTR so that caller can take appropriate action. We have 3 callers: gigaset_probe(), gigaset_tty_open() and gigaset_probe(). Each caller tries to free allocated memory if lock fails. This is possible if

[PATCH] [media] dib0700: Return -EINTR and unlock mutex if locking attempts fails.

2012-03-16 Thread santosh nayak
From: Santosh Nayak In 'dib0700_i2c_xfer_new()' and 'dib0700_i2c_xfer_legacy()' we are taking two locks: 1. i2c_mutex 2. usb_mutex If attempt to take 'usb_mutex' lock fails then the previously taken lock 'i2c_mutex' should

[PATCH] [media] dvb: negative value assigned to unsigned int in CDRXD().

2012-03-06 Thread santosh nayak
From: Santosh Nayak In CDRXD(), Negative number is assigned to unsigned variable 'state->noise_cal.tdCal2. Members of 'SNoiseCal' should be 'signed short'. Signed-off-by: Santosh Nayak --- drivers/media/dvb/frontends/drxd_hard.c |6 +++--- 1 files changed,

[PATCH 2/2] Driver: video: Use the macro DMA_BIT_MASK().

2012-02-21 Thread santosh nayak
From: Santosh Nayak Use the macro DMA_BIT_MASK instead of the constant 0x. Signed-off-by: Santosh Nayak --- drivers/media/video/cx18/cx18-driver.c |4 ++-- drivers/media/video/ivtv/ivtv-driver.c |4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers