diff options
author | Sudip Mukherjee <sudipm.mukherjee@gmail.com> | 2015-10-28 05:11:40 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2016-01-03 19:32:59 -0500 |
commit | e0a7f1f04cd9bb13df7503ba7156ff0a37c9f460 (patch) | |
tree | 79bba5716874a0570db82cd2ca4f2bc8da45a86e | |
parent | e732b93c3276548bfa903d79c2083b2c8dc552af (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.c | 3 |
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; |