Messages in this thread Patch in this message | | | From | Yihao Han <> | Subject | [PATCH] leds: tca6507: use swap() to make code cleaner | Date | Sun, 14 Nov 2021 19:24:28 -0800 |
| |
Use the macro 'swap()' defined in 'include/linux/minmax.h' to avoid opencoding it.
Signed-off-by: Yihao Han <hanyihao@vivo.com> --- drivers/leds/leds-tca6507.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/leds/leds-tca6507.c b/drivers/leds/leds-tca6507.c index 225b765830bd..de8eed9b667d 100644 --- a/drivers/leds/leds-tca6507.c +++ b/drivers/leds/leds-tca6507.c @@ -242,9 +242,7 @@ static int choose_times(int msec, int *c1p, int *c2p) if (diff < 65536) { int actual; if (msec & 1) { - c1 = *c2p; - *c2p = *c1p; - *c1p = c1; + swap(*c2p, *c1p); } actual = time_codes[*c1p] + time_codes[*c2p]; if (*c1p < *c2p) -- 2.17.1
| |