diff options
| author | Yang Guang <yang.guang5@zte.com.cn> | 2021-12-18 09:59:17 +0800 |
|---|---|---|
| committer | Michael Ellerman <mpe@ellerman.id.au> | 2021-12-20 12:01:21 +1100 |
| commit | a605b39e8ef703828b9e26750ea1925a6a5ef848 (patch) | |
| tree | 0a33c4e6ec1641ce9ed69664d3107fdc1c675861 | |
| parent | 2fe4ca6ad7f6a0b98f97c498320051e5066e4b95 (diff) | |
| download | linux-a605b39e8ef703828b9e26750ea1925a6a5ef848.tar.bz2 | |
powerpc: use swap() to make code cleaner
Use the macro 'swap()' defined in 'include/linux/minmax.h' to avoid
opencoding it.
Reported-by: Zeal Robot <zealci@zte.com.cn>
Signed-off-by: David Yang <davidcomponentone@gmail.com>
Signed-off-by: Yang Guang <yang.guang5@zte.com.cn>
[mpe: Add include of linux/minmax.h]
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Link: https://lore.kernel.org/r/71a702c2189b16c152affd8a8cda1d84ce32741c.1639792543.git.yang.guang5@zte.com.cn
| -rw-r--r-- | arch/powerpc/platforms/powermac/pic.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/arch/powerpc/platforms/powermac/pic.c b/arch/powerpc/platforms/powermac/pic.c index 4921bccf0376..bb0566633af5 100644 --- a/arch/powerpc/platforms/powermac/pic.c +++ b/arch/powerpc/platforms/powermac/pic.c @@ -18,6 +18,7 @@ #include <linux/interrupt.h> #include <linux/syscore_ops.h> #include <linux/adb.h> +#include <linux/minmax.h> #include <linux/pmu.h> #include <asm/sections.h> @@ -311,11 +312,8 @@ static void __init pmac_pic_probe_oldstyle(void) /* Check ordering of master & slave */ if (of_device_is_compatible(master, "gatwick")) { - struct device_node *tmp; BUG_ON(slave == NULL); - tmp = master; - master = slave; - slave = tmp; + swap(master, slave); } /* We found a slave */ |