diff options
| author | Markus Elfring <elfring@users.sourceforge.net> | 2017-06-04 21:21:20 +0200 |
|---|---|---|
| committer | Stefano Stabellini <sstabellini@kernel.org> | 2017-06-05 19:38:24 +0200 |
| commit | 84a0a967a44b9da6c5effff628aafa7698050574 (patch) | |
| tree | 7120dca69d6e0fc0072815dd56c0c49dc6abadc8 /arch/arm/xen | |
| parent | arm/xen: Delete an error message for a failed memory allocation in __set_phys... (diff) | |
| download | linux-84a0a967a44b9da6c5effff628aafa7698050574.tar.xz linux-84a0a967a44b9da6c5effff628aafa7698050574.zip | |
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')
| -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; } |