diff options
author | Markus Elfring <elfring@users.sourceforge.net> | 2017-06-04 22:21:20 +0300 |
---|---|---|
committer | Stefano Stabellini <sstabellini@kernel.org> | 2017-06-05 20:38:24 +0300 |
commit | 84a0a967a44b9da6c5effff628aafa7698050574 (patch) | |
tree | 7120dca69d6e0fc0072815dd56c0c49dc6abadc8 /arch/arm/xen/p2m.c | |
parent | d6bb4ec32c832fb0fcc43636327e17a0724e2324 (diff) | |
download | linux-84a0a967a44b9da6c5effff628aafa7698050574.tar.xz |
arm/xen: Adjust one function call together with a variable assignment
The script "checkpatch.pl" pointed information out like the following.
ERROR: do not use assignment in if condition
Thus fix the affected source code place.
Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
Reviewed-by: Stefano Stabellini <sstabellini@kernel.org>
Diffstat (limited to 'arch/arm/xen/p2m.c')
-rw-r--r-- | arch/arm/xen/p2m.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/arch/arm/xen/p2m.c b/arch/arm/xen/p2m.c index f5f74ac637b9..e71eefa2e427 100644 --- a/arch/arm/xen/p2m.c +++ b/arch/arm/xen/p2m.c @@ -153,7 +153,8 @@ bool __set_phys_to_machine_multi(unsigned long pfn, p2m_entry->mfn = mfn; write_lock_irqsave(&p2m_lock, irqflags); - if ((rc = xen_add_phys_to_mach_entry(p2m_entry)) < 0) { + rc = xen_add_phys_to_mach_entry(p2m_entry); + if (rc < 0) { write_unlock_irqrestore(&p2m_lock, irqflags); return false; } |