aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSudip Mukherjee <sudipm.mukherjee@gmail.com>2015-10-28 05:11:40 -0400
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2016-01-03 19:32:59 -0500
commite0a7f1f04cd9bb13df7503ba7156ff0a37c9f460 (patch)
tree79bba5716874a0570db82cd2ca4f2bc8da45a86e
parente732b93c3276548bfa903d79c2083b2c8dc552af (diff)
parport: avoid assignment in if
It is not an usual practise to assign some value to a variable in the if test condition. Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r--drivers/parport/share.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/parport/share.c b/drivers/parport/share.c
index b68f19480dcf..3308427ed9f7 100644
--- a/drivers/parport/share.c
+++ b/drivers/parport/share.c
@@ -1120,7 +1120,8 @@ int parport_claim(struct pardevice *dev)
1120 1120
1121 /* Preempt any current device */ 1121 /* Preempt any current device */
1122 write_lock_irqsave(&port->cad_lock, flags); 1122 write_lock_irqsave(&port->cad_lock, flags);
1123 if ((oldcad = port->cad) != NULL) { 1123 oldcad = port->cad;
1124 if (oldcad) {
1124 if (oldcad->preempt) { 1125 if (oldcad->preempt) {
1125 if (oldcad->preempt(oldcad->private)) 1126 if (oldcad->preempt(oldcad->private))
1126 goto blocked; 1127 goto blocked;