All of lore.kernel.org
 help / color / mirror / Atom feed
From: Geert Uytterhoeven <geert+renesas@glider.be>
To: Linus Walleij <linus.walleij@linaro.org>
Cc: linux-renesas-soc@vger.kernel.org, linux-gpio@vger.kernel.org,
	Geert Uytterhoeven <geert+renesas@glider.be>
Subject: [PATCH 03/60] pinctrl: renesas: Reformat macros defining struct initializers
Date: Thu, 23 Dec 2021 15:41:13 +0100	[thread overview]
Message-ID: <03a1eed3c4f57d7b14ef53ab49e04de10d0e383c.1640269757.git.geert+renesas@glider.be> (raw)
In-Reply-To: <cover.1640269757.git.geert+renesas@glider.be>

Reformat all macros that define structure initializers, to visually
resemble structure definitions:
  - Move the opening curly brace to the previous line,
  - Move the closing curly brace to the first position,
  - Reduce indentation of the block to a single TAB, decreasing the need
    for line breaks,
  - Align backslashes for line continuation to the last TAB block where
    possible,

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 drivers/pinctrl/renesas/sh_pfc.h | 115 +++++++++++++++----------------
 1 file changed, 54 insertions(+), 61 deletions(-)

diff --git a/drivers/pinctrl/renesas/sh_pfc.h b/drivers/pinctrl/renesas/sh_pfc.h
index 2479b4fb9cf957ff..d355e60244c6e5a8 100644
--- a/drivers/pinctrl/renesas/sh_pfc.h
+++ b/drivers/pinctrl/renesas/sh_pfc.h
@@ -49,14 +49,13 @@ struct sh_pfc_pin {
 	u16 enum_id;
 };
 
-#define SH_PFC_PIN_GROUP_ALIAS(alias, n)		\
-	{						\
-		.name = #alias,				\
-		.pins = n##_pins,			\
-		.mux = n##_mux,				\
-		.nr_pins = ARRAY_SIZE(n##_pins) +	\
-		BUILD_BUG_ON_ZERO(sizeof(n##_pins) != sizeof(n##_mux)), \
-	}
+#define SH_PFC_PIN_GROUP_ALIAS(alias, n) {				\
+	.name = #alias,							\
+	.pins = n##_pins,						\
+	.mux = n##_mux,							\
+	.nr_pins = ARRAY_SIZE(n##_pins) +				\
+	BUILD_BUG_ON_ZERO(sizeof(n##_pins) != sizeof(n##_mux)),		\
+}
 #define SH_PFC_PIN_GROUP(n)	SH_PFC_PIN_GROUP_ALIAS(n, n)
 
 struct sh_pfc_pin_group {
@@ -72,13 +71,12 @@ struct sh_pfc_pin_group {
  * in this case. It accepts an optional 'version' argument used when the
  * same group can appear on a different set of pins.
  */
-#define VIN_DATA_PIN_GROUP(n, s, ...)					\
-	{								\
-		.name = #n#s#__VA_ARGS__,				\
-		.pins = n##__VA_ARGS__##_pins.data##s,			\
-		.mux = n##__VA_ARGS__##_mux.data##s,			\
-		.nr_pins = ARRAY_SIZE(n##__VA_ARGS__##_pins.data##s),	\
-	}
+#define VIN_DATA_PIN_GROUP(n, s, ...) {					\
+	.name = #n#s#__VA_ARGS__,					\
+	.pins = n##__VA_ARGS__##_pins.data##s,				\
+	.mux = n##__VA_ARGS__##_mux.data##s,				\
+	.nr_pins = ARRAY_SIZE(n##__VA_ARGS__##_pins.data##s),		\
+}
 
 union vin_data12 {
 	unsigned int data12[12];
@@ -103,12 +101,11 @@ union vin_data {
 	unsigned int data4[4];
 };
 
-#define SH_PFC_FUNCTION(n)				\
-	{						\
-		.name = #n,				\
-		.groups = n##_groups,			\
-		.nr_groups = ARRAY_SIZE(n##_groups),	\
-	}
+#define SH_PFC_FUNCTION(n) {						\
+	.name = #n,							\
+	.groups = n##_groups,						\
+	.nr_groups = ARRAY_SIZE(n##_groups),				\
+}
 
 struct sh_pfc_function {
 	const char *name;
@@ -231,8 +228,9 @@ struct pinmux_irq {
  * Describe the mapping from GPIOs to a single IRQ
  *   - ids...: List of GPIOs that are mapped to the same IRQ
  */
-#define PINMUX_IRQ(ids...)			   \
-	{ .gpios = (const short []) { ids, -1 } }
+#define PINMUX_IRQ(ids...) {						\
+	.gpios = (const short []) { ids, -1 }				\
+}
 
 struct pinmux_range {
 	u16 begin;
@@ -624,13 +622,12 @@ extern const struct sh_pfc_soc_info shx3_pinmux_info;
 #define GP_ALL(str)			CPU_ALL_GP(_GP_ALL, str)
 
 /* PINMUX_GPIO_GP_ALL - Expand to a list of sh_pfc_pin entries */
-#define _GP_GPIO(bank, _pin, _name, sfx, cfg)				\
-	{								\
-		.pin = (bank * 32) + _pin,				\
-		.name = __stringify(_name),				\
-		.enum_id = _name##_DATA,				\
-		.configs = cfg,						\
-	}
+#define _GP_GPIO(bank, _pin, _name, sfx, cfg) {				\
+	.pin = (bank * 32) + _pin,					\
+	.name = __stringify(_name),					\
+	.enum_id = _name##_DATA,					\
+	.configs = cfg,							\
+}
 #define PINMUX_GPIO_GP_ALL()		CPU_ALL_GP(_GP_GPIO, unused)
 
 /* PINMUX_DATA_GP_ALL -  Expand to a list of name_DATA, name_FN marks */
@@ -688,13 +685,12 @@ extern const struct sh_pfc_soc_info shx3_pinmux_info;
 	}
 
 /* SH_PFC_PIN_CFG - Expand to a sh_pfc_pin entry (named PORT#) with config */
-#define SH_PFC_PIN_CFG(_pin, cfgs)					\
-	{								\
-		.pin = _pin,						\
-		.name = __stringify(PORT##_pin),			\
-		.enum_id = PORT##_pin##_DATA,				\
-		.configs = cfgs,					\
-	}
+#define SH_PFC_PIN_CFG(_pin, cfgs) {					\
+	.pin = _pin,							\
+	.name = __stringify(PORT##_pin),				\
+	.enum_id = PORT##_pin##_DATA,					\
+	.configs = cfgs,						\
+}
 
 /* PINMUX_DATA_ALL - Expand to a list of PORT_name_DATA, PORT_name_FN0,
  *		     PORT_name_OUT, PORT_name_IN marks
@@ -743,35 +739,32 @@ extern const struct sh_pfc_soc_info shx3_pinmux_info;
 #define NOGP_ALL()				CPU_ALL_NOGP(_NOGP_ALL)
 
 /* PINMUX_NOGP_ALL - Expand to a list of sh_pfc_pin entries */
-#define _NOGP_PINMUX(_pin, _name, cfg)					\
-	{								\
-		.pin = PIN_##_pin,					\
-		.name = "PIN_" _name,					\
-		.configs = SH_PFC_PIN_CFG_NO_GPIO | cfg,		\
-	}
+#define _NOGP_PINMUX(_pin, _name, cfg) {				\
+	.pin = PIN_##_pin,						\
+	.name = "PIN_" _name,						\
+	.configs = SH_PFC_PIN_CFG_NO_GPIO | cfg,			\
+}
 #define PINMUX_NOGP_ALL()		CPU_ALL_NOGP(_NOGP_PINMUX)
 
 /*
  * PORTnCR helper macro for SH-Mobile/R-Mobile
  */
-#define PORTCR(nr, reg)							\
-	{								\
-		PINMUX_CFG_REG_VAR("PORT" nr "CR", reg, 8,		\
-				   GROUP(2, 2, 1, 3),			\
-				   GROUP(				\
-			/* PULMD[1:0], handled by .set_bias() */	\
-			0, 0, 0, 0,					\
-			/* IE and OE */					\
-			0, PORT##nr##_OUT, PORT##nr##_IN, 0,		\
-			/* SEC, not supported */			\
-			0, 0,						\
-			/* PTMD[2:0] */					\
-			PORT##nr##_FN0, PORT##nr##_FN1,			\
-			PORT##nr##_FN2, PORT##nr##_FN3,			\
-			PORT##nr##_FN4, PORT##nr##_FN5,			\
-			PORT##nr##_FN6, PORT##nr##_FN7			\
-		))							\
-	}
+#define PORTCR(nr, reg) {						\
+	PINMUX_CFG_REG_VAR("PORT" nr "CR", reg, 8, GROUP(2, 2, 1, 3),	\
+			   GROUP(					\
+		/* PULMD[1:0], handled by .set_bias() */		\
+		0, 0, 0, 0,						\
+		/* IE and OE */						\
+		0, PORT##nr##_OUT, PORT##nr##_IN, 0,			\
+		/* SEC, not supported */				\
+		0, 0,							\
+		/* PTMD[2:0] */						\
+		PORT##nr##_FN0, PORT##nr##_FN1,				\
+		PORT##nr##_FN2, PORT##nr##_FN3,				\
+		PORT##nr##_FN4, PORT##nr##_FN5,				\
+		PORT##nr##_FN6, PORT##nr##_FN7				\
+	))								\
+}
 
 /*
  * GPIO number helper macro for R-Car
-- 
2.25.1


  parent reply	other threads:[~2021-12-23 14:42 UTC|newest]

Thread overview: 61+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-12-23 14:41 [PATCH 00/60] pinctrl: renesas: Share more pin group data Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 01/60] pinctrl: renesas: r8a77470: Reduce size for narrow VIN1 channel Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 02/60] pinctrl: renesas: Rename sh_pfc_soc_operations instances Geert Uytterhoeven
2021-12-23 14:41 ` Geert Uytterhoeven [this message]
2021-12-23 14:41 ` [PATCH 04/60] pinctrl: renesas: Rename SH_PFC_PIN_GROUP{,_ALIAS} args Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 05/60] pinctrl: renesas: Add generic support for pin group subsets Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 06/60] pinctrl: renesas: Add generic support for resizable buses Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 07/60] pinctrl: renesas: r8a7740: Share BSC pin group data Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 08/60] pinctrl: renesas: emev2: Share CF " Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 09/60] pinctrl: renesas: r8a7791: Share HSCIF1 " Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 10/60] pinctrl: renesas: sh73a0: Share KEYIN " Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 11/60] pinctrl: renesas: r8a7740: Share LCD " Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 12/60] pinctrl: renesas: sh73a0: " Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 13/60] pinctrl: renesas: r8a73a4: Share MMC " Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 14/60] pinctrl: renesas: r8a7740: " Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 15/60] pinctrl: renesas: r8a77470: " Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 16/60] pinctrl: renesas: r8a7778: " Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 17/60] pinctrl: renesas: r8a7779: " Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 18/60] pinctrl: renesas: r8a7790: " Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 19/60] pinctrl: renesas: r8a7791: " Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 20/60] pinctrl: renesas: r8a7794: " Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 21/60] pinctrl: renesas: r8a77970: " Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 22/60] pinctrl: renesas: r8a77980: " Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 23/60] pinctrl: renesas: r8a77995: " Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 24/60] pinctrl: renesas: r8a779a0: " Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 25/60] pinctrl: renesas: sh73a0: " Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 26/60] pinctrl: renesas: r8a77470: Share QSPI " Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 27/60] pinctrl: renesas: r8a7790: " Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 28/60] pinctrl: renesas: r8a7791: " Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 29/60] pinctrl: renesas: r8a7792: " Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 30/60] pinctrl: renesas: r8a7794: " Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 31/60] pinctrl: renesas: r8a77950: " Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 32/60] pinctrl: renesas: r8a77951: " Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 33/60] pinctrl: renesas: r8a77965: " Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 34/60] pinctrl: renesas: r8a7796: " Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 35/60] pinctrl: renesas: r8a77990: " Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 36/60] pinctrl: renesas: r8a779a0: " Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 37/60] pinctrl: renesas: r8a77970: Share RPC " Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 38/60] pinctrl: renesas: r8a77980: " Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 39/60] pinctrl: renesas: r8a73a4: Share SDHI " Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 40/60] pinctrl: renesas: r8a7740: " Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 41/60] pinctrl: renesas: r8a77470: " Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 42/60] pinctrl: renesas: r8a7778: " Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 43/60] pinctrl: renesas: r8a7779: " Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 44/60] pinctrl: renesas: r8a7790: " Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 45/60] pinctrl: renesas: r8a7791: " Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 46/60] pinctrl: renesas: r8a7792: " Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 47/60] pinctrl: renesas: r8a7794: " Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 48/60] pinctrl: renesas: r8a77950: " Geert Uytterhoeven
2021-12-23 14:41 ` [PATCH 49/60] pinctrl: renesas: r8a77951: " Geert Uytterhoeven
2021-12-23 14:42 ` [PATCH 50/60] pinctrl: renesas: r8a77965: " Geert Uytterhoeven
2021-12-23 14:42 ` [PATCH 51/60] pinctrl: renesas: r8a7796: " Geert Uytterhoeven
2021-12-23 14:42 ` [PATCH 52/60] pinctrl: renesas: r8a77990: " Geert Uytterhoeven
2021-12-23 14:42 ` [PATCH 53/60] pinctrl: renesas: sh73a0: " Geert Uytterhoeven
2021-12-23 14:42 ` [PATCH 54/60] pinctrl: renesas: emev2: Share SDI " Geert Uytterhoeven
2021-12-23 14:42 ` [PATCH 55/60] pinctrl: renesas: r8a7790: Share USB1 " Geert Uytterhoeven
2021-12-23 14:42 ` [PATCH 56/60] pinctrl: renesas: r8a7790: Share more VIN " Geert Uytterhoeven
2021-12-23 14:42 ` [PATCH 57/60] pinctrl: renesas: r8a77951: " Geert Uytterhoeven
2021-12-23 14:42 ` [PATCH 58/60] pinctrl: renesas: r8a7796: " Geert Uytterhoeven
2021-12-23 14:42 ` [PATCH 59/60] pinctrl: renesas: r8a77965: " Geert Uytterhoeven
2021-12-23 14:42 ` [PATCH 60/60] pinctrl: renesas: r8a77990: " Geert Uytterhoeven

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=03a1eed3c4f57d7b14ef53ab49e04de10d0e383c.1640269757.git.geert+renesas@glider.be \
    --to=geert+renesas@glider.be \
    --cc=linus.walleij@linaro.org \
    --cc=linux-gpio@vger.kernel.org \
    --cc=linux-renesas-soc@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.