media: saa7134: simplify if-if to if-else

use if and else instead of if(A) and if (!A).

Signed-off-by: Wan Jiabing <wanjiabing@vivo.com>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>

diff --git a/drivers/media/pci/saa7134/saa7134-video.c b/drivers/media/pci/saa7134/saa7134-video.c
index 98c258a1cd0129b0ba3cbe239236ea79e58c3edb..4d8974c9fcc984d7fb64e6f5facc4c692918453a 100644 (file)
--- a/drivers/media/pci/saa7134/saa7134-video.c
+++ b/drivers/media/pci/saa7134/saa7134-video.c
@@ -1832,8 +1832,7 @@ static int saa7134_overlay(struct file *file, void *priv, unsigned int on)
                spin_lock_irqsave(&dev->slock, flags);
                start_preview(dev);
                spin_unlock_irqrestore(&dev->slock, flags);
-       }
-       if (!on) {
+       } else {
                if (priv != dev->overlay_owner)
                        return -EINVAL;
                spin_lock_irqsave(&dev->slock, flags);