V4L/DVB (5820): Cleanup on cinergyT2: Remove unneeded if(1)

Before kernel 2.6.14, the driver checked for status before stopping the
thread. So, a compatibility test did exist. After 2.6.14, the
        if (state)
were replaced by:
        if (1)
However, it makes no sense to keep the if(1).

Signed-off-by: Mauro Carvalho Chehab <mchehab@infradead.org>

1 files changed, 7 insertions, 10 deletions

diff --git a/drivers/media/dvb/cinergyT2/cinergyT2.c b/drivers/media/dvb/cinergyT2/cinergyT2.c
index bc22064b50ef..5a1449f485cf 100644
--- a/drivers/media/dvb/cinergyT2/cinergyT2.c
+++ b/drivers/media/dvb/cinergyT2/cinergyT2.c
@@ -1000,18 +1000,15 @@ static int cinergyt2_suspend (struct usb_interface *intf, pm_message_t state)
         if (cinergyt2->disconnect_pending || mutex_lock_interruptible(&cinergyt2->wq_sem))
                 return -ERESTARTSYS;
 
-        if (1) {
-                cinergyt2_suspend_rc(cinergyt2);
-                cancel_rearming_delayed_work(&cinergyt2->query_work);
+        cinergyt2_suspend_rc(cinergyt2);
+        cancel_rearming_delayed_work(&cinergyt2->query_work);
 
-                mutex_lock(&cinergyt2->sem);
-                if (cinergyt2->streaming)
-                        cinergyt2_stop_stream_xfer(cinergyt2);
-                cinergyt2_sleep(cinergyt2, 1);
-                mutex_unlock(&cinergyt2->sem);
-        }
+        mutex_lock(&cinergyt2->sem);
+        if (cinergyt2->streaming)
+                cinergyt2_stop_stream_xfer(cinergyt2);
+        cinergyt2_sleep(cinergyt2, 1);
+        mutex_unlock(&cinergyt2->sem);
 
-        mutex_unlock(&cinergyt2->wq_sem);
         return 0;
 }