From:   Yihao Han <hanyihao@vivo.com>
To:     Mathieu Poirier <mathieu.poirier@linaro.org>,
        Suzuki K Poulose <suzuki.poulose@arm.com>,
        Mike Leach <mike.leach@linaro.org>,
        Leo Yan <leo.yan@linaro.org>,
        John Garry <john.garry@huawei.com>,
        Will Deacon <will@kernel.org>,
        Peter Zijlstra <peterz@infradead.org>,
        Ingo Molnar <mingo@redhat.com>,
        Arnaldo Carvalho de Melo <acme@kernel.org>,
        Mark Rutland <mark.rutland@arm.com>,
        Alexander Shishkin <alexander.shishkin@linux.intel.com>,
        Jiri Olsa <jolsa@redhat.com>,
        Namhyung Kim <namhyung@kernel.org>, coresight@lists.linaro.org,
        linux-arm-kernel@lists.infradead.org,
        linux-perf-users@vger.kernel.org, linux-kernel@vger.kernel.org
Cc:     kernel@vivo.com, Yihao Han <hanyihao@vivo.com>
Subject: [PATCH] perf cs-etm: use swap() to make code cleaner
Date:   Thu, 11 Nov 2021 18:33:44 -0800
Message-Id: <20211112023344.3642-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?c2FRVk7HKgb8lhTQCWGA2drguo0dDeBFkqTmz7BXCKzJ+sZ4hDSl9Um31mKw?=
 =?us-ascii?Q?gXNlWx8CePwA4zxekuPm0V6O4NniY30SMMa3V5Z9BUXr8/ql1ULx+Tgqt/Xn?=
 =?us-ascii?Q?4yIVJNHOEAUrkqa68H6/KTqGKS2zRiQJg8L75rM9AkobuuNCPxac9YDDfxDD?=
 =?us-ascii?Q?2zLqBL9fW7HJ+qO38qErnrYZ9Adfr6G2e9uoErQd/GOPIY0RMtj1vDDyVKna?=
 =?us-ascii?Q?kgpFpn78KCpxRjokDCLWSLV/27Ci4/KHcabUwpEr+UIa5FqmR3eE4/UcIugU?=
 =?us-ascii?Q?Z7bcpZEAJpgjaPCaxqFT1zeYiTnx0pDOPaOUV6Ex9FCpIYVHZii4Q2vQNw9t?=
 =?us-ascii?Q?shHQCR2KdPtRARmRrnJLEc5zdkU9hqHaGuGnYKbltMKC7erGU2Yn+ttBsN6I?=
 =?us-ascii?Q?1shTALRcOaPNMNjpnnSHBtF70bdA7xFsbGmUOWxnTUSjc9cDzRkuJOsQbpbn?=
 =?us-ascii?Q?2mQyAc21t4Q4YS2CNt07jAgI3GXHquOBqFLEg3voFVi8cz+cmzmw/AwmQTjD?=
 =?us-ascii?Q?ugXqJrM1cuchk9nokROres5UjevMzUjwpWUegd0vFB50woYOiC9QPAQwi0+2?=
 =?us-ascii?Q?rakkxlb5s6YADTzJweYPNbRfJAHzyndrbW1JFzK4Lg8fuiGCkGofS/aRNZuW?=
 =?us-ascii?Q?Zn0usRVsqYKVQ1fFXgk4hP16LFbnNtU/ocrWXqbNBmTxjqoUliCmhONgubZn?=
 =?us-ascii?Q?8jk8oj91xAtrELkHpxNH5mWNE3F89Gh8Fntsv/wRZ1Hy5Jyn2Ei3WhxD8WLl?=
 =?us-ascii?Q?OBNGH1wVg0A1bJJUTwLrbeTgzwWIpOGkb8AiagIUKtSnJXIiOhaHoINC3Pa4?=
 =?us-ascii?Q?4UHKKsTkeZrFKkrNjS1bHtpzsREf8/+p2VZYPC2wq+41STnlMxguUqqGHz/g?=
 =?us-ascii?Q?C/7NACSSj8ZEXOCrhwow+eqrRsz8y4F89wCy0OLDVvnXBwWabqKzhtlsSlyz?=
 =?us-ascii?Q?ZiFDDc6T2vV0IoGq/6l1x1MG8tw5q/YhKe7wAaCL3RYj8z5t9Pu2Io2n7c83?=
 =?us-ascii?Q?BcwSQxvJD5jaQi47x5GdWW3Y/fDJYgeut1/pQx4kpgHDeP3t80iR3Ujri6b6?=
 =?us-ascii?Q?b5a3VlaZvfKTpiRYnKE6eUHtcMEl3C7/cwVJVyuXfS0+0q1X4TorXAcbsgpn?=
 =?us-ascii?Q?MB209W+PCzN/th4WQXE6h95Dmee3VQR/VNh+QmQ3Un55fXCfNrKARVw+/MXl?=
 =?us-ascii?Q?2LEqLPJoQEo5EDZvEw5bzqCxrPOlAK83OA+oWkYMBKNV37bDqJcOofnOUoKq?=
 =?us-ascii?Q?CoQCK49r8SrfoPBawj5/FklUOzJGFxib6OSjrp1r/6t/oo3fqP1pEGL0A83k?=
 =?us-ascii?Q?X1MKyg7ixN3Sx6OBopMELUWsG2c6iHBhPj5yVl8KaEduJulA1s08LhyhCg1K?=
 =?us-ascii?Q?mM9kU18bpGQPXCLl6drlrsui4wCbmeB4tMwGg6HTG/hcz5R3T/eVGrL0wMR1?=
 =?us-ascii?Q?cl04Sv9INeR5oa3nYyox7gx5+PDHovRY6kJyAc7gcIjY+yf4556+M26h/ASa?=
 =?us-ascii?Q?2v86ZT2JRkXFYkxoGOV2g+EDVMA2EOFlrjdP+oZh2D9NKnoWbCAzkAguI8fU?=
 =?us-ascii?Q?zdIciOD75lwF0/O/9ZyoH08R6d97mcQkDWirvIi4AOw7xfmo0KeBGBqY5Dub?=
 =?us-ascii?Q?lVvrDdxQIR6gPgERlG6ermA=3D?=
X-OriginatorOrg: vivo.com
X-MS-Exchange-CrossTenant-Network-Message-Id: f504231a-1c6b-4bd8-e21b-08d9a584e1bf
X-MS-Exchange-CrossTenant-AuthSource: TYZPR06MB4173.apcprd06.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 12 Nov 2021 02:33:59.2734
 (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: Spqgck8ZeJZrH+lbTGaXk8+zsOnFd3X/as89qRm9qFu9KI4+Amh5GlIImpwodwsG2hGb3+tp5jd2cfGtL0ShUA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: TYAPR06MB2191
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>
---
 tools/perf/util/cs-etm.c | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/tools/perf/util/cs-etm.c b/tools/perf/util/cs-etm.c
index f323adb1af85..aaa3ec725002 100644
--- a/tools/perf/util/cs-etm.c
+++ b/tools/perf/util/cs-etm.c
@@ -408,17 +408,13 @@ struct cs_etm_packet_queue
 static void cs_etm__packet_swap(struct cs_etm_auxtrace *etm,
 				struct cs_etm_traceid_queue *tidq)
 {
-	struct cs_etm_packet *tmp;
-
 	if (etm->sample_branches || etm->synth_opts.last_branch ||
 	    etm->sample_instructions) {
 		/*
 		 * Swap PACKET with PREV_PACKET: PACKET becomes PREV_PACKET for
 		 * the next incoming packet.
 		 */
-		tmp = tidq->packet;
-		tidq->packet = tidq->prev_packet;
-		tidq->prev_packet = tmp;
+		swap(tidq->packet, tidq->prev_packet);
 	}
 }
 
-- 
2.17.1