From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 4E5A2C10F1A for ; Tue, 7 May 2024 15:23:10 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 04C7E10ED48; Tue, 7 May 2024 15:23:10 +0000 (UTC) Authentication-Results: gabe.freedesktop.org; dkim=pass (2048-bit key; unprotected) header.d=intel.com header.i=@intel.com header.b="TA1lt7o2"; dkim-atps=neutral Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.17]) by gabe.freedesktop.org (Postfix) with ESMTPS id A22B010E526 for ; Tue, 7 May 2024 15:23:08 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1715095388; x=1746631388; h=from:to:cc:subject:date:message-id:mime-version: content-transfer-encoding; bh=ZRUn/2uPJ3dxr8B6c0QYZFUMFgov+vTxKBbIm2/61wQ=; b=TA1lt7o2R+ceDtbiZx9y/O0PzJ+ROgpYPMeT1Zopn4sNp7cE1vZREPVi rzKBXCdXrYGmYYCBp5ycV6iQRDEskKmbdJyO4B+KLUddqlyupI0C2NX4M ltgR79vi6z8gtVrmyEZ9U8HSQg8aLIZhKTshNDspSN47EUgYiA+h/QgUm WUcytbYHrluGaVxBCvb45MAVHKQ0r0rlKjKgZOuMsjZzFnL7X8g36yqJ1 hps6tTV9ogqNplUuixEkOVBqiwvcjix1b9Ro73oqn0kqGIlhCNy1HTBn4 fKLNEaXJBa+JF8uAxvE66Jpf451KbvmGlL7YMnn3dzXrNufSbQst8OC/w w==; X-CSE-ConnectionGUID: vE+cMCHmQey0wWvBeFbJMg== X-CSE-MsgGUID: BKcYrA8NT3q8Q/8HglWf5w== X-IronPort-AV: E=McAfee;i="6600,9927,11066"; a="10766951" X-IronPort-AV: E=Sophos;i="6.08,142,1712646000"; d="scan'208";a="10766951" Received: from fmviesa008.fm.intel.com ([10.60.135.148]) by fmvoesa111.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 07 May 2024 08:23:06 -0700 X-CSE-ConnectionGUID: wUGL/LnKSUWYUupDbcSCCA== X-CSE-MsgGUID: 9X4/ykg3R1mZciD/rXt0YA== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.08,142,1712646000"; d="scan'208";a="28531952" Received: from mwajdecz-mobl.ger.corp.intel.com ([10.245.96.165]) by fmviesa008-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 07 May 2024 08:23:05 -0700 From: Michal Wajdeczko To: intel-xe@lists.freedesktop.org Cc: Michal Wajdeczko , Andy Shevchenko , Lucas De Marchi Subject: [PATCH v2] drm/xe: Rename few xe_args.h macros Date: Tue, 7 May 2024 17:22:46 +0200 Message-Id: <20240507152246.2791-1-michal.wajdeczko@intel.com> X-Mailer: git-send-email 2.21.0 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-BeenThere: intel-xe@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Intel Xe graphics driver List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-xe-bounces@lists.freedesktop.org Sender: "Intel-xe" To minimize the risk of future name collisions, rename macros to always include the ARG or ARGS tag: DROP_FIRST to DROP_FIRST_ARG PICK_FIRST to FIRST_ARG PICK_LAST to LAST_ARG Suggested-by: Andy Shevchenko Signed-off-by: Michal Wajdeczko Cc: Lucas De Marchi --- v2: s/ARGS_TAIL/DROP_FIRST_ARG (Lucas) --- drivers/gpu/drm/xe/tests/xe_args_test.c | 82 ++++++++++++++----------- drivers/gpu/drm/xe/xe_args.h | 50 +++++++-------- drivers/gpu/drm/xe/xe_rtp_helpers.h | 12 ++-- 3 files changed, 78 insertions(+), 66 deletions(-) diff --git a/drivers/gpu/drm/xe/tests/xe_args_test.c b/drivers/gpu/drm/xe/tests/xe_args_test.c index 9b44c1ab6364..a30d439713b7 100644 --- a/drivers/gpu/drm/xe/tests/xe_args_test.c +++ b/drivers/gpu/drm/xe/tests/xe_args_test.c @@ -21,10 +21,10 @@ static void call_args_example(struct kunit *test) #undef buz } -static void drop_first_example(struct kunit *test) +static void drop_first_arg_example(struct kunit *test) { #define foo X, Y, Z, Q -#define bar CALL_ARGS(COUNT_ARGS, DROP_FIRST(foo)) +#define bar CALL_ARGS(COUNT_ARGS, DROP_FIRST_ARG(foo)) KUNIT_EXPECT_EQ(test, bar, 3); @@ -32,12 +32,12 @@ static void drop_first_example(struct kunit *test) #undef bar } -static void pick_first_example(struct kunit *test) +static void first_arg_example(struct kunit *test) { int X = 1; #define foo X, Y, Z, Q -#define bar PICK_FIRST(foo) +#define bar FIRST_ARG(foo) KUNIT_EXPECT_EQ(test, bar, X); KUNIT_EXPECT_STREQ(test, __stringify(bar), "X"); @@ -46,12 +46,12 @@ static void pick_first_example(struct kunit *test) #undef bar } -static void pick_last_example(struct kunit *test) +static void last_arg_example(struct kunit *test) { int Q = 1; #define foo X, Y, Z, Q -#define bar PICK_LAST(foo) +#define bar LAST_ARG(foo) KUNIT_EXPECT_EQ(test, bar, Q); KUNIT_EXPECT_STREQ(test, __stringify(bar), "Q"); @@ -63,8 +63,8 @@ static void pick_last_example(struct kunit *test) static void sep_comma_example(struct kunit *test) { #define foo(f) f(X) f(Y) f(Z) f(Q) -#define bar DROP_FIRST(foo(ARGS_SEP_COMMA __stringify)) -#define buz CALL_ARGS(COUNT_ARGS, DROP_FIRST(foo(ARGS_SEP_COMMA))) +#define bar DROP_FIRST_ARG(foo(ARGS_SEP_COMMA __stringify)) +#define buz CALL_ARGS(COUNT_ARGS, DROP_FIRST_ARG(foo(ARGS_SEP_COMMA))) static const char * const a[] = { bar }; @@ -123,61 +123,73 @@ static void call_args_test(struct kunit *test) KUNIT_EXPECT_EQ(test, CALL_ARGS(COUNT_ARGS, MAX_ARGS), 12); } -static void drop_first_test(struct kunit *test) +static void drop_first_arg_test(struct kunit *test) { int Y = -2, Z = -3, Q = -4; - int a[] = { DROP_FIRST(FOO_ARGS) }; + int a[] = { DROP_FIRST_ARG(FOO_ARGS) }; - KUNIT_EXPECT_EQ(test, DROP_FIRST(0, -1), -1); - KUNIT_EXPECT_EQ(test, DROP_FIRST(DROP_FIRST(0, -1, -2)), -2); + KUNIT_EXPECT_EQ(test, DROP_FIRST_ARG(0, -1), -1); + KUNIT_EXPECT_EQ(test, DROP_FIRST_ARG(DROP_FIRST_ARG(0, -1, -2)), -2); - KUNIT_EXPECT_EQ(test, CALL_ARGS(COUNT_ARGS, DROP_FIRST(FOO_ARGS)), 3); - KUNIT_EXPECT_EQ(test, DROP_FIRST(DROP_FIRST(DROP_FIRST(FOO_ARGS))), -4); + KUNIT_EXPECT_EQ(test, CALL_ARGS(COUNT_ARGS, DROP_FIRST_ARG(FOO_ARGS)), 3); + KUNIT_EXPECT_EQ(test, DROP_FIRST_ARG(DROP_FIRST_ARG(DROP_FIRST_ARG(FOO_ARGS))), -4); KUNIT_EXPECT_EQ(test, a[0], -2); KUNIT_EXPECT_EQ(test, a[1], -3); KUNIT_EXPECT_EQ(test, a[2], -4); - KUNIT_EXPECT_STREQ(test, __stringify(DROP_FIRST(DROP_FIRST(DROP_FIRST(FOO_ARGS)))), "Q"); + +#define foo DROP_FIRST_ARG(FOO_ARGS) +#define bar DROP_FIRST_ARG(DROP_FIRST_ARG(FOO_ARGS)) +#define buz DROP_FIRST_ARG(DROP_FIRST_ARG(DROP_FIRST_ARG(FOO_ARGS))) + + KUNIT_EXPECT_EQ(test, CALL_ARGS(COUNT_ARGS, foo), 3); + KUNIT_EXPECT_EQ(test, CALL_ARGS(COUNT_ARGS, bar), 2); + KUNIT_EXPECT_EQ(test, CALL_ARGS(COUNT_ARGS, buz), 1); + KUNIT_EXPECT_STREQ(test, __stringify(buz), "Q"); + +#undef foo +#undef bar +#undef buz } -static void pick_first_test(struct kunit *test) +static void first_arg_test(struct kunit *test) { int X = -1; - int a[] = { PICK_FIRST(FOO_ARGS) }; + int a[] = { FIRST_ARG(FOO_ARGS) }; - KUNIT_EXPECT_EQ(test, PICK_FIRST(-1, -2), -1); + KUNIT_EXPECT_EQ(test, FIRST_ARG(-1, -2), -1); - KUNIT_EXPECT_EQ(test, CALL_ARGS(COUNT_ARGS, PICK_FIRST(FOO_ARGS)), 1); - KUNIT_EXPECT_EQ(test, PICK_FIRST(FOO_ARGS), -1); + KUNIT_EXPECT_EQ(test, CALL_ARGS(COUNT_ARGS, FIRST_ARG(FOO_ARGS)), 1); + KUNIT_EXPECT_EQ(test, FIRST_ARG(FOO_ARGS), -1); KUNIT_EXPECT_EQ(test, a[0], -1); - KUNIT_EXPECT_STREQ(test, __stringify(PICK_FIRST(FOO_ARGS)), "X"); + KUNIT_EXPECT_STREQ(test, __stringify(FIRST_ARG(FOO_ARGS)), "X"); } -static void pick_last_test(struct kunit *test) +static void last_arg_test(struct kunit *test) { int Q = -4; - int a[] = { PICK_LAST(FOO_ARGS) }; + int a[] = { LAST_ARG(FOO_ARGS) }; - KUNIT_EXPECT_EQ(test, PICK_LAST(-1, -2), -2); + KUNIT_EXPECT_EQ(test, LAST_ARG(-1, -2), -2); - KUNIT_EXPECT_EQ(test, CALL_ARGS(COUNT_ARGS, PICK_LAST(FOO_ARGS)), 1); - KUNIT_EXPECT_EQ(test, PICK_LAST(FOO_ARGS), -4); + KUNIT_EXPECT_EQ(test, CALL_ARGS(COUNT_ARGS, LAST_ARG(FOO_ARGS)), 1); + KUNIT_EXPECT_EQ(test, LAST_ARG(FOO_ARGS), -4); KUNIT_EXPECT_EQ(test, a[0], -4); - KUNIT_EXPECT_STREQ(test, __stringify(PICK_LAST(FOO_ARGS)), "Q"); + KUNIT_EXPECT_STREQ(test, __stringify(LAST_ARG(FOO_ARGS)), "Q"); - KUNIT_EXPECT_EQ(test, PICK_LAST(MAX_ARGS), -12); - KUNIT_EXPECT_STREQ(test, __stringify(PICK_LAST(MAX_ARGS)), "-12"); + KUNIT_EXPECT_EQ(test, LAST_ARG(MAX_ARGS), -12); + KUNIT_EXPECT_STREQ(test, __stringify(LAST_ARG(MAX_ARGS)), "-12"); } static struct kunit_case args_tests[] = { KUNIT_CASE(count_args_test), KUNIT_CASE(call_args_example), KUNIT_CASE(call_args_test), - KUNIT_CASE(drop_first_example), - KUNIT_CASE(drop_first_test), - KUNIT_CASE(pick_first_example), - KUNIT_CASE(pick_first_test), - KUNIT_CASE(pick_last_example), - KUNIT_CASE(pick_last_test), + KUNIT_CASE(drop_first_arg_example), + KUNIT_CASE(drop_first_arg_test), + KUNIT_CASE(first_arg_example), + KUNIT_CASE(first_arg_test), + KUNIT_CASE(last_arg_example), + KUNIT_CASE(last_arg_test), KUNIT_CASE(sep_comma_example), {} }; diff --git a/drivers/gpu/drm/xe/xe_args.h b/drivers/gpu/drm/xe/xe_args.h index 40b9eb4151d8..5efd3f9ea669 100644 --- a/drivers/gpu/drm/xe/xe_args.h +++ b/drivers/gpu/drm/xe/xe_args.h @@ -35,7 +35,7 @@ #define __CALL_ARGS(f, args...) f(args) /** - * DROP_FIRST - Returns all arguments except the first one. + * DROP_FIRST_ARG - Returns all arguments except the first one. * @args: arguments * * This helper macro allows manipulation the argument list before passing it @@ -44,15 +44,15 @@ * Example: * * #define foo X,Y,Z,Q - * #define bar CALL_ARGS(COUNT_ARGS, DROP_FIRST(foo)) + * #define bar CALL_ARGS(COUNT_ARGS, DROP_FIRST_ARG(foo)) * * With above definitions bar expands to 3. */ -#define DROP_FIRST(args...) __DROP_FIRST(args) -#define __DROP_FIRST(a, b...) b +#define DROP_FIRST_ARG(args...) __DROP_FIRST_ARG(args) +#define __DROP_FIRST_ARG(a, b...) b /** - * PICK_FIRST - Returns the first argument. + * FIRST_ARG - Returns the first argument. * @args: arguments * * This helper macro allows manipulation the argument list before passing it @@ -61,15 +61,15 @@ * Example: * * #define foo X,Y,Z,Q - * #define bar PICK_FIRST(foo) + * #define bar FIRST_ARG(foo) * * With above definitions bar expands to X. */ -#define PICK_FIRST(args...) __PICK_FIRST(args) -#define __PICK_FIRST(a, b...) a +#define FIRST_ARG(args...) __FIRST_ARG(args) +#define __FIRST_ARG(a, b...) a /** - * PICK_LAST - Returns the last argument. + * LAST_ARG - Returns the last argument. * @args: arguments * * This helper macro allows manipulation the argument list before passing it @@ -80,24 +80,24 @@ * Example: * * #define foo X,Y,Z,Q - * #define bar PICK_LAST(foo) + * #define bar LAST_ARG(foo) * * With above definitions bar expands to Q. */ -#define PICK_LAST(args...) __PICK_ARG(COUNT_ARGS(args), args) +#define LAST_ARG(args...) __PICK_ARG(COUNT_ARGS(args), args) #define __PICK_ARG(n, args...) CALL_ARGS(CONCATENATE(PICK_ARG, n), args) -#define PICK_ARG1(args...) PICK_FIRST(args) -#define PICK_ARG2(args...) PICK_ARG1(DROP_FIRST(args)) -#define PICK_ARG3(args...) PICK_ARG2(DROP_FIRST(args)) -#define PICK_ARG4(args...) PICK_ARG3(DROP_FIRST(args)) -#define PICK_ARG5(args...) PICK_ARG4(DROP_FIRST(args)) -#define PICK_ARG6(args...) PICK_ARG5(DROP_FIRST(args)) -#define PICK_ARG7(args...) PICK_ARG6(DROP_FIRST(args)) -#define PICK_ARG8(args...) PICK_ARG7(DROP_FIRST(args)) -#define PICK_ARG9(args...) PICK_ARG8(DROP_FIRST(args)) -#define PICK_ARG10(args...) PICK_ARG9(DROP_FIRST(args)) -#define PICK_ARG11(args...) PICK_ARG10(DROP_FIRST(args)) -#define PICK_ARG12(args...) PICK_ARG11(DROP_FIRST(args)) +#define PICK_ARG1(args...) FIRST_ARG(args) +#define PICK_ARG2(args...) PICK_ARG1(DROP_FIRST_ARG(args)) +#define PICK_ARG3(args...) PICK_ARG2(DROP_FIRST_ARG(args)) +#define PICK_ARG4(args...) PICK_ARG3(DROP_FIRST_ARG(args)) +#define PICK_ARG5(args...) PICK_ARG4(DROP_FIRST_ARG(args)) +#define PICK_ARG6(args...) PICK_ARG5(DROP_FIRST_ARG(args)) +#define PICK_ARG7(args...) PICK_ARG6(DROP_FIRST_ARG(args)) +#define PICK_ARG8(args...) PICK_ARG7(DROP_FIRST_ARG(args)) +#define PICK_ARG9(args...) PICK_ARG8(DROP_FIRST_ARG(args)) +#define PICK_ARG10(args...) PICK_ARG9(DROP_FIRST_ARG(args)) +#define PICK_ARG11(args...) PICK_ARG10(DROP_FIRST_ARG(args)) +#define PICK_ARG12(args...) PICK_ARG11(DROP_FIRST_ARG(args)) /** * ARGS_SEP_COMMA - Definition of a comma character. @@ -109,8 +109,8 @@ * Example: * * #define foo(f) f(X) f(Y) f(Z) f(Q) - * #define bar DROP_FIRST(foo(ARGS_SEP_COMMA __stringify)) - * #define buz CALL_ARGS(COUNT_ARGS, DROP_FIRST(foo(ARGS_SEP_COMMA))) + * #define bar DROP_FIRST_ARG(foo(ARGS_SEP_COMMA __stringify)) + * #define buz CALL_ARGS(COUNT_ARGS, DROP_FIRST_ARG(foo(ARGS_SEP_COMMA))) * * With above definitions bar expands to * "X", "Y", "Z", "Q" diff --git a/drivers/gpu/drm/xe/xe_rtp_helpers.h b/drivers/gpu/drm/xe/xe_rtp_helpers.h index 8129d6d9ef37..7735f217ba71 100644 --- a/drivers/gpu/drm/xe/xe_rtp_helpers.h +++ b/drivers/gpu/drm/xe/xe_rtp_helpers.h @@ -17,7 +17,7 @@ */ #define _XE_ESC(...) __VA_ARGS__ -#define _XE_TUPLE_TAIL(...) (DROP_FIRST(__VA_ARGS__)) +#define _XE_TUPLE_TAIL(...) (DROP_FIRST_ARG(__VA_ARGS__)) #define _XE_RTP_CONCAT(a, b) CONCATENATE(XE_RTP_, CONCATENATE(a, b)) @@ -54,10 +54,10 @@ * XE_RTP_TEST_FOO BANANA XE_RTP_TEST_BAR */ #define XE_RTP_PASTE_FOREACH(prefix_, sep_, args_) _XE_RTP_CONCAT(PASTE_, COUNT_ARGS args_)(prefix_, sep_, args_) -#define XE_RTP_PASTE_1(prefix_, sep_, args_) _XE_RTP_CONCAT(prefix_, PICK_FIRST args_) -#define XE_RTP_PASTE_2(prefix_, sep_, args_) _XE_RTP_CONCAT(prefix_, PICK_FIRST args_) __XE_RTP_PASTE_SEP_ ## sep_ XE_RTP_PASTE_1(prefix_, sep_, _XE_TUPLE_TAIL args_) -#define XE_RTP_PASTE_3(prefix_, sep_, args_) _XE_RTP_CONCAT(prefix_, PICK_FIRST args_) __XE_RTP_PASTE_SEP_ ## sep_ XE_RTP_PASTE_2(prefix_, sep_, _XE_TUPLE_TAIL args_) -#define XE_RTP_PASTE_4(prefix_, sep_, args_) _XE_RTP_CONCAT(prefix_, PICK_FIRST args_) __XE_RTP_PASTE_SEP_ ## sep_ XE_RTP_PASTE_3(prefix_, sep_, _XE_TUPLE_TAIL args_) +#define XE_RTP_PASTE_1(prefix_, sep_, args_) _XE_RTP_CONCAT(prefix_, FIRST_ARG args_) +#define XE_RTP_PASTE_2(prefix_, sep_, args_) _XE_RTP_CONCAT(prefix_, FIRST_ARG args_) __XE_RTP_PASTE_SEP_ ## sep_ XE_RTP_PASTE_1(prefix_, sep_, _XE_TUPLE_TAIL args_) +#define XE_RTP_PASTE_3(prefix_, sep_, args_) _XE_RTP_CONCAT(prefix_, FIRST_ARG args_) __XE_RTP_PASTE_SEP_ ## sep_ XE_RTP_PASTE_2(prefix_, sep_, _XE_TUPLE_TAIL args_) +#define XE_RTP_PASTE_4(prefix_, sep_, args_) _XE_RTP_CONCAT(prefix_, FIRST_ARG args_) __XE_RTP_PASTE_SEP_ ## sep_ XE_RTP_PASTE_3(prefix_, sep_, _XE_TUPLE_TAIL args_) /* * XE_RTP_DROP_CAST - Drop cast to convert a compound statement to a initializer @@ -70,6 +70,6 @@ * * { .a = 10 } */ -#define XE_RTP_DROP_CAST(...) _XE_ESC(DROP_FIRST _XE_ESC __VA_ARGS__) +#define XE_RTP_DROP_CAST(...) _XE_ESC(DROP_FIRST_ARG _XE_ESC __VA_ARGS__) #endif -- 2.43.0