From:   Yihao Han <hanyihao@vivo.com>
To:     Pavel Machek <pavel@ucw.cz>, linux-leds@vger.kernel.org,
        linux-kernel@vger.kernel.org
Cc:     kernel@vivo.com, Yihao Han <hanyihao@vivo.com>
Subject: [PATCH] leds: tca6507: use swap() to make code cleaner
Date:   Sun, 14 Nov 2021 19:24:28 -0800
Message-Id: <20211115032428.4379-1-hanyihao@vivo.com>
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?scZGJu6Q6MSNHDdP17KCA4DnYwulRgB4EAglpQI1yK3GEEUfHfogLcFjuhAs?=
 =?us-ascii?Q?YT8e7uBoQ07f8R748KKfHmVvCBtQd55UQ9cUP6LRf2d/3LP9QqDXNIO4cCPH?=
 =?us-ascii?Q?/RUn5lk3+8AuVKuS4wxoz/Hd6ITDpYeuSNrlD56B/EvXktnEUajStE3NFN1k?=
 =?us-ascii?Q?iPh2LaX9jEOB+USnKa8kCjJa/38IZXvajrQCqVXDzo6i/SWD9OIQnfd1/jtM?=
 =?us-ascii?Q?bonOzK+s5/Tne9DJeS8zVr/SKpJ/e1DS333ovcIQpPkcTqBdOcPB2AIDjOJU?=
 =?us-ascii?Q?ii2+Un6uwVtRQxK9rgV1wOuRysQuME/ySAsY8IzteQptgisiVa7C5bHkcYFF?=
 =?us-ascii?Q?YgepJ7xsdOy6ijsEwr3sufYpP4yT2V49Fj+e1OZ/yvn7N3gkwlojOhdcFaSd?=
 =?us-ascii?Q?IT/NgJKeKYmmdSf0A9ny3UH+gqeSDCA598RV4kzENicawa3vgz3fz0pFwYx9?=
 =?us-ascii?Q?yV0kqLC+gK6blFHc7TCdWYKHbS30zcUfy5r0x0wcHLCOjJEUBOZKSD06Pwyq?=
 =?us-ascii?Q?NMjIpYk0eDASJMqjfXM/EhnJeehsbBonZzdtbhJJC2i0whFHlRKBYFOcrmLk?=
 =?us-ascii?Q?x7oPht76DOG7qJY+j+Qp2jYjzt5to5GMn7MV/A+xtxGevHhnWPd/DNGGELC/?=
 =?us-ascii?Q?kDN711tSjchL4H2vodJyw5iZZ/fhJ8pGYgbLznGmdYgiW5xO2meOZG1ggeis?=
 =?us-ascii?Q?+lSJALUmdu+90Y5UOWXY2UWZit6LIvl9+VahnsstxYM9hLKmC1Lxglv0AbxF?=
 =?us-ascii?Q?M2y/mOjmi/UxWpFeQ2/bFHw10WBRYJH4YTWe3ok95zy+aA1xj5FieIBK1BFH?=
 =?us-ascii?Q?8VuVVgskQS0Qyg5GtiA/ndJ/GB/4SPgudzYzz4fr48HeckHj/AAQLzqoVFLs?=
 =?us-ascii?Q?dpom1X0WrPZZmXmHa0LqN7xCFdl/0Mkr/3UD6KxdYyF06M2BvDjGyb/Nmhtj?=
 =?us-ascii?Q?w++CPYtczP1ti2Zq9vjZRpy3VY9dZHIf9ZfxwCzGP5em6m4c2Bfdykv1XLIw?=
 =?us-ascii?Q?jEeXXFI8Gd8K3UsP4pDNRGEgSUtM7kz8UvAkNreQsJfSqQWX3pT6LvgbWJj2?=
 =?us-ascii?Q?hcQibrSKJ6q2uvKxnMIU+dZ49hkfsKc/q3bbi1K+kUpSvzN9FBwE5RlbKlYY?=
 =?us-ascii?Q?RDZlkvtS4Q2u8a0ieWq6OM5xzFKfojlze5pS3MdfDVzJhqS4X3Mmt/NYh9pj?=
 =?us-ascii?Q?9g52LU0Gy/XhCsCcjBYP+boD2/fKBSHD7PsOYJV0L2o0H8WnOyL1uKANC4l/?=
 =?us-ascii?Q?t/S+BbqRFpHCuQDHhxDhWxWlOrVjFCHHIvo94ML8oFIGb3+MwkP/dAC9lKGN?=
 =?us-ascii?Q?gpRCjZIHlC2odEKxOrAZBhH6eu23rnsJovENyqaTJdTPHCuTKS74TiYJmWxb?=
 =?us-ascii?Q?t3pRfjGsHvhUmcOKIuqOflCuV20BDVuLK+MyGbNMAY8G8+HOuEtfgD2o2Wn4?=
 =?us-ascii?Q?PhLRvIxf9luQOMH+ygk0lrcm+WDZ/y2CcPSGYbrFrIEgQegJcrRFM+pLAAhM?=
 =?us-ascii?Q?aL04XshkmNuCiz2cfLzolNSxfMdL7fo+EPveTV1rzcUqmM/BIVtfsQy/pC87?=
 =?us-ascii?Q?QwYohxleNj4ZkbUJFvbTe5gOwAe1xr+Z7KPD1vpwkaEA94enA3znOSpSywMm?=
 =?us-ascii?Q?8ezwcH9Ok/Ft6KlUH9VAjbY=3D?=
X-OriginatorOrg: vivo.com
X-MS-Exchange-CrossTenant-Network-Message-Id: c84856f3-6320-49a4-d4d7-08d9a7e7788a
X-MS-Exchange-CrossTenant-AuthSource: TYZPR06MB4173.apcprd06.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 15 Nov 2021 03:24:45.2668
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 923e42dc-48d5-4cbe-b582-1a797a6412ed
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: d5PO8x7a+ZgsR+HrP02Atcaw7axjrqSsu3j0+X81VYZ7TWd4dm9KYltpeyQwuuZUz8cxeYpvxZo8g/OF0XFqkw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: TYZPR06MB3902
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

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