From: "Niklas Söderlund" <niklas.soderlund@ragnatech.se>
To: Geert Uytterhoeven <geert+renesas@glider.be>
Cc: Linus Walleij <linus.walleij@linaro.org>,
linux-renesas-soc@vger.kernel.org, linux-gpio@vger.kernel.org
Subject: Re: [PATCH 05/12] pinctrl: renesas: r8a779{51,6,65}: Reduce non-functional differences
Date: Sat, 1 May 2021 10:07:05 +0200 [thread overview]
Message-ID: <YI0MKZ9UnzuLBw06@oden.dyn.berto.se> (raw)
In-Reply-To: <527b45ebfc664a80e41cb0136677db7260e11437.1619785375.git.geert+renesas@glider.be>
Hi Geert,
Thanks for your work.
On 2021-04-30 14:31:04 +0200, Geert Uytterhoeven wrote:
> Change whitespace in the pin control drivers for R-Car H3 ES2.0+, R-Car
> M3-W/M3-W+, and R-Car M3-N, to reduce the differences among these very
> similar drivers.
>
> These changes have no functional impact.
>
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Reviewed-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
> ---
> drivers/pinctrl/renesas/pfc-r8a77951.c | 4 +-
> drivers/pinctrl/renesas/pfc-r8a7796.c | 7 ++-
> drivers/pinctrl/renesas/pfc-r8a77965.c | 79 +++++++++++++-------------
> 3 files changed, 46 insertions(+), 44 deletions(-)
>
> diff --git a/drivers/pinctrl/renesas/pfc-r8a77951.c b/drivers/pinctrl/renesas/pfc-r8a77951.c
> index be4eee0708427988..84c0ea5d59c1ac31 100644
> --- a/drivers/pinctrl/renesas/pfc-r8a77951.c
> +++ b/drivers/pinctrl/renesas/pfc-r8a77951.c
> @@ -241,7 +241,7 @@
> #define GPSR6_3 F_(SSI_SDATA1_A, IP15_3_0)
> #define GPSR6_2 F_(SSI_SDATA0, IP14_31_28)
> #define GPSR6_1 F_(SSI_WS01239, IP14_27_24)
> -#define GPSR6_0 F_(SSI_SCK01239, IP14_23_20)
> +#define GPSR6_0 F_(SSI_SCK01239, IP14_23_20)
>
> /* GPSR7 */
> #define GPSR7_3 FM(GP7_03)
> @@ -668,7 +668,7 @@ static const u16 pinmux_data[] = {
> PINMUX_IPSR_PHYS_MSEL(IP0_23_20, AVB_AVTP_CAPTURE_A, I2C_SEL_5_0, SEL_ETHERAVB_0),
> PINMUX_IPSR_PHYS_MSEL(IP0_23_20, MSIOF2_TXD_C, I2C_SEL_5_0, SEL_MSIOF2_2),
> PINMUX_IPSR_PHYS_MSEL(IP0_23_20, RTS4_N_A, I2C_SEL_5_0, SEL_SCIF4_0),
> - PINMUX_IPSR_PHYS(IP0_23_20, SDA5, I2C_SEL_5_1),
> + PINMUX_IPSR_PHYS(IP0_23_20, SDA5, I2C_SEL_5_1),
>
> PINMUX_IPSR_GPSR(IP0_27_24, IRQ0),
> PINMUX_IPSR_GPSR(IP0_27_24, QPOLB),
> diff --git a/drivers/pinctrl/renesas/pfc-r8a7796.c b/drivers/pinctrl/renesas/pfc-r8a7796.c
> index bbb1b436ded3123f..a4d74df3d20105e8 100644
> --- a/drivers/pinctrl/renesas/pfc-r8a7796.c
> +++ b/drivers/pinctrl/renesas/pfc-r8a7796.c
> @@ -1549,7 +1549,7 @@ static const u16 pinmux_data[] = {
> * core will do the right thing and skip trying to mux the pin
> * while still applying configuration to it.
> */
> -#define FM(x) PINMUX_DATA(x##_MARK, 0),
> +#define FM(x) PINMUX_DATA(x##_MARK, 0),
> PINMUX_STATIC
> #undef FM
> };
> @@ -4234,7 +4234,7 @@ static const struct {
> SH_PFC_PIN_GROUP(avb_link),
> SH_PFC_PIN_GROUP(avb_magic),
> SH_PFC_PIN_GROUP(avb_phy_int),
> - SH_PFC_PIN_GROUP_ALIAS(avb_mdc, avb_mdio), /* Deprecated */
> + SH_PFC_PIN_GROUP_ALIAS(avb_mdc, avb_mdio), /* Deprecated */
> SH_PFC_PIN_GROUP(avb_mdio),
> SH_PFC_PIN_GROUP(avb_mii),
> SH_PFC_PIN_GROUP(avb_avtp_pps),
> @@ -5991,7 +5991,8 @@ static const struct pinmux_ioctrl_reg pinmux_ioctrl_regs[] = {
> { /* sentinel */ },
> };
>
> -static int r8a7796_pin_to_pocctrl(struct sh_pfc *pfc, unsigned int pin, u32 *pocctrl)
> +static int r8a7796_pin_to_pocctrl(struct sh_pfc *pfc,
> + unsigned int pin, u32 *pocctrl)
> {
> int bit = -EINVAL;
>
> diff --git a/drivers/pinctrl/renesas/pfc-r8a77965.c b/drivers/pinctrl/renesas/pfc-r8a77965.c
> index e69210cc61486edc..a7607a6798865868 100644
> --- a/drivers/pinctrl/renesas/pfc-r8a77965.c
> +++ b/drivers/pinctrl/renesas/pfc-r8a77965.c
> @@ -666,14 +666,14 @@ static const u16 pinmux_data[] = {
> PINMUX_IPSR_MSEL(IP0_15_12, TX4_A, SEL_SCIF4_0),
> PINMUX_IPSR_GPSR(IP0_19_16, FSCLKST2_N_A),
>
> - PINMUX_IPSR_PHYS_MSEL(IP0_19_16, AVB_AVTP_MATCH_A, I2C_SEL_5_0, SEL_ETHERAVB_0),
> - PINMUX_IPSR_PHYS_MSEL(IP0_19_16, MSIOF2_RXD_C, I2C_SEL_5_0, SEL_MSIOF2_2),
> - PINMUX_IPSR_PHYS_MSEL(IP0_19_16, CTS4_N_A, I2C_SEL_5_0, SEL_SCIF4_0),
> + PINMUX_IPSR_PHYS_MSEL(IP0_19_16, AVB_AVTP_MATCH_A, I2C_SEL_5_0, SEL_ETHERAVB_0),
> + PINMUX_IPSR_PHYS_MSEL(IP0_19_16, MSIOF2_RXD_C, I2C_SEL_5_0, SEL_MSIOF2_2),
> + PINMUX_IPSR_PHYS_MSEL(IP0_19_16, CTS4_N_A, I2C_SEL_5_0, SEL_SCIF4_0),
> PINMUX_IPSR_PHYS(IP0_19_16, SCL5, I2C_SEL_5_1),
>
> - PINMUX_IPSR_PHYS_MSEL(IP0_23_20, AVB_AVTP_CAPTURE_A, I2C_SEL_5_0, SEL_ETHERAVB_0),
> - PINMUX_IPSR_PHYS_MSEL(IP0_23_20, MSIOF2_TXD_C, I2C_SEL_5_0, SEL_MSIOF2_2),
> - PINMUX_IPSR_PHYS_MSEL(IP0_23_20, RTS4_N_A, I2C_SEL_5_0, SEL_SCIF4_0),
> + PINMUX_IPSR_PHYS_MSEL(IP0_23_20, AVB_AVTP_CAPTURE_A, I2C_SEL_5_0, SEL_ETHERAVB_0),
> + PINMUX_IPSR_PHYS_MSEL(IP0_23_20, MSIOF2_TXD_C, I2C_SEL_5_0, SEL_MSIOF2_2),
> + PINMUX_IPSR_PHYS_MSEL(IP0_23_20, RTS4_N_A, I2C_SEL_5_0, SEL_SCIF4_0),
> PINMUX_IPSR_PHYS(IP0_23_20, SDA5, I2C_SEL_5_1),
>
> PINMUX_IPSR_GPSR(IP0_27_24, IRQ0),
> @@ -727,16 +727,16 @@ static const u16 pinmux_data[] = {
> PINMUX_IPSR_MSEL(IP1_19_16, VI4_DATA6_B, SEL_VIN4_1),
> PINMUX_IPSR_MSEL(IP1_19_16, IECLK_B, SEL_IEBUS_1),
>
> - PINMUX_IPSR_PHYS_MSEL(IP1_23_20, PWM1_A, I2C_SEL_3_0, SEL_PWM1_0),
> - PINMUX_IPSR_PHYS_MSEL(IP1_23_20, HRX3_D, I2C_SEL_3_0, SEL_HSCIF3_3),
> - PINMUX_IPSR_PHYS_MSEL(IP1_23_20, VI4_DATA7_B, I2C_SEL_3_0, SEL_VIN4_1),
> - PINMUX_IPSR_PHYS_MSEL(IP1_23_20, IERX_B, I2C_SEL_3_0, SEL_IEBUS_1),
> - PINMUX_IPSR_PHYS(IP1_23_20, SCL3, I2C_SEL_3_1),
> + PINMUX_IPSR_PHYS_MSEL(IP1_23_20, PWM1_A, I2C_SEL_3_0, SEL_PWM1_0),
> + PINMUX_IPSR_PHYS_MSEL(IP1_23_20, HRX3_D, I2C_SEL_3_0, SEL_HSCIF3_3),
> + PINMUX_IPSR_PHYS_MSEL(IP1_23_20, VI4_DATA7_B, I2C_SEL_3_0, SEL_VIN4_1),
> + PINMUX_IPSR_PHYS_MSEL(IP1_23_20, IERX_B, I2C_SEL_3_0, SEL_IEBUS_1),
> + PINMUX_IPSR_PHYS(IP1_23_20, SCL3, I2C_SEL_3_1),
>
> - PINMUX_IPSR_PHYS_MSEL(IP1_27_24, PWM2_A, I2C_SEL_3_0, SEL_PWM2_0),
> - PINMUX_IPSR_PHYS_MSEL(IP1_27_24, HTX3_D, I2C_SEL_3_0, SEL_HSCIF3_3),
> - PINMUX_IPSR_PHYS_MSEL(IP1_27_24, IETX_B, I2C_SEL_3_0, SEL_IEBUS_1),
> - PINMUX_IPSR_PHYS(IP1_27_24, SDA3, I2C_SEL_3_1),
> + PINMUX_IPSR_PHYS_MSEL(IP1_27_24, PWM2_A, I2C_SEL_3_0, SEL_PWM2_0),
> + PINMUX_IPSR_PHYS_MSEL(IP1_27_24, HTX3_D, I2C_SEL_3_0, SEL_HSCIF3_3),
> + PINMUX_IPSR_PHYS_MSEL(IP1_27_24, IETX_B, I2C_SEL_3_0, SEL_IEBUS_1),
> + PINMUX_IPSR_PHYS(IP1_27_24, SDA3, I2C_SEL_3_1),
>
> PINMUX_IPSR_GPSR(IP1_31_28, A0),
> PINMUX_IPSR_GPSR(IP1_31_28, LCDOUT16),
> @@ -1171,13 +1171,13 @@ static const u16 pinmux_data[] = {
> PINMUX_IPSR_MSEL(IP11_15_12, SDA2_B, SEL_I2C2_1),
>
> PINMUX_IPSR_MSEL(IP11_19_16, SD1_CD, I2C_SEL_0_0),
> - PINMUX_IPSR_PHYS_MSEL(IP11_19_16, NFRB_N_A, I2C_SEL_0_0, SEL_NDF_0),
> - PINMUX_IPSR_PHYS_MSEL(IP11_19_16, SIM0_CLK_B, I2C_SEL_0_0, SEL_SIMCARD_1),
> + PINMUX_IPSR_PHYS_MSEL(IP11_19_16, NFRB_N_A, I2C_SEL_0_0, SEL_NDF_0),
> + PINMUX_IPSR_PHYS_MSEL(IP11_19_16, SIM0_CLK_B, I2C_SEL_0_0, SEL_SIMCARD_1),
> PINMUX_IPSR_PHYS(IP11_19_16, SCL0, I2C_SEL_0_1),
>
> PINMUX_IPSR_MSEL(IP11_23_20, SD1_WP, I2C_SEL_0_0),
> - PINMUX_IPSR_PHYS_MSEL(IP11_23_20, NFCE_N_A, I2C_SEL_0_0, SEL_NDF_0),
> - PINMUX_IPSR_PHYS_MSEL(IP11_23_20, SIM0_D_B, I2C_SEL_0_0, SEL_SIMCARD_1),
> + PINMUX_IPSR_PHYS_MSEL(IP11_23_20, NFCE_N_A, I2C_SEL_0_0, SEL_NDF_0),
> + PINMUX_IPSR_PHYS_MSEL(IP11_23_20, SIM0_D_B, I2C_SEL_0_0, SEL_SIMCARD_1),
> PINMUX_IPSR_PHYS(IP11_23_20, SDA0, I2C_SEL_0_1),
>
> PINMUX_IPSR_GPSR(IP11_27_24, SCK0),
> @@ -1553,7 +1553,7 @@ static const u16 pinmux_data[] = {
> * core will do the right thing and skip trying to mux the pin
> * while still applying configuration to it.
> */
> -#define FM(x) PINMUX_DATA(x##_MARK, 0),
> +#define FM(x) PINMUX_DATA(x##_MARK, 0),
> PINMUX_STATIC
> #undef FM
> };
> @@ -4224,24 +4224,24 @@ static const unsigned int vin4_data18_a_pins[] = {
> RCAR_GP_PIN(0, 10), RCAR_GP_PIN(0, 11),
> RCAR_GP_PIN(0, 12), RCAR_GP_PIN(0, 13),
> RCAR_GP_PIN(0, 14), RCAR_GP_PIN(0, 15),
> - RCAR_GP_PIN(1, 2), RCAR_GP_PIN(1, 3),
> - RCAR_GP_PIN(1, 4), RCAR_GP_PIN(1, 5),
> - RCAR_GP_PIN(1, 6), RCAR_GP_PIN(1, 7),
> - RCAR_GP_PIN(0, 2), RCAR_GP_PIN(0, 3),
> - RCAR_GP_PIN(0, 4), RCAR_GP_PIN(0, 5),
> - RCAR_GP_PIN(0, 6), RCAR_GP_PIN(0, 7),
> + RCAR_GP_PIN(1, 2), RCAR_GP_PIN(1, 3),
> + RCAR_GP_PIN(1, 4), RCAR_GP_PIN(1, 5),
> + RCAR_GP_PIN(1, 6), RCAR_GP_PIN(1, 7),
> + RCAR_GP_PIN(0, 2), RCAR_GP_PIN(0, 3),
> + RCAR_GP_PIN(0, 4), RCAR_GP_PIN(0, 5),
> + RCAR_GP_PIN(0, 6), RCAR_GP_PIN(0, 7),
> };
>
> static const unsigned int vin4_data18_a_mux[] = {
> VI4_DATA2_A_MARK, VI4_DATA3_A_MARK,
> VI4_DATA4_A_MARK, VI4_DATA5_A_MARK,
> VI4_DATA6_A_MARK, VI4_DATA7_A_MARK,
> - VI4_DATA10_MARK, VI4_DATA11_MARK,
> - VI4_DATA12_MARK, VI4_DATA13_MARK,
> - VI4_DATA14_MARK, VI4_DATA15_MARK,
> - VI4_DATA18_MARK, VI4_DATA19_MARK,
> - VI4_DATA20_MARK, VI4_DATA21_MARK,
> - VI4_DATA22_MARK, VI4_DATA23_MARK,
> + VI4_DATA10_MARK, VI4_DATA11_MARK,
> + VI4_DATA12_MARK, VI4_DATA13_MARK,
> + VI4_DATA14_MARK, VI4_DATA15_MARK,
> + VI4_DATA18_MARK, VI4_DATA19_MARK,
> + VI4_DATA20_MARK, VI4_DATA21_MARK,
> + VI4_DATA22_MARK, VI4_DATA23_MARK,
> };
>
> static const union vin_data vin4_data_a_pins = {
> @@ -4294,12 +4294,12 @@ static const unsigned int vin4_data18_b_mux[] = {
> VI4_DATA2_B_MARK, VI4_DATA3_B_MARK,
> VI4_DATA4_B_MARK, VI4_DATA5_B_MARK,
> VI4_DATA6_B_MARK, VI4_DATA7_B_MARK,
> - VI4_DATA10_MARK, VI4_DATA11_MARK,
> - VI4_DATA12_MARK, VI4_DATA13_MARK,
> - VI4_DATA14_MARK, VI4_DATA15_MARK,
> - VI4_DATA18_MARK, VI4_DATA19_MARK,
> - VI4_DATA20_MARK, VI4_DATA21_MARK,
> - VI4_DATA22_MARK, VI4_DATA23_MARK,
> + VI4_DATA10_MARK, VI4_DATA11_MARK,
> + VI4_DATA12_MARK, VI4_DATA13_MARK,
> + VI4_DATA14_MARK, VI4_DATA15_MARK,
> + VI4_DATA18_MARK, VI4_DATA19_MARK,
> + VI4_DATA20_MARK, VI4_DATA21_MARK,
> + VI4_DATA22_MARK, VI4_DATA23_MARK,
> };
>
> static const union vin_data vin4_data_b_pins = {
> @@ -6248,7 +6248,8 @@ static const struct pinmux_ioctrl_reg pinmux_ioctrl_regs[] = {
> { /* sentinel */ },
> };
>
> -static int r8a77965_pin_to_pocctrl(struct sh_pfc *pfc, unsigned int pin, u32 *pocctrl)
> +static int r8a77965_pin_to_pocctrl(struct sh_pfc *pfc,
> + unsigned int pin, u32 *pocctrl)
> {
> int bit = -EINVAL;
>
> --
> 2.25.1
>
--
Regards,
Niklas Söderlund
next prev parent reply other threads:[~2021-05-01 8:07 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-04-30 12:30 [PATCH 00/12] pinctrl: renesas: Add more bias pinconf support Geert Uytterhoeven
2021-04-30 12:30 ` Geert Uytterhoeven
2021-04-30 12:31 ` [PATCH 01/12] pinctrl: renesas: r8a7796: Add missing bias for PRESET# pin Geert Uytterhoeven
2021-05-01 7:12 ` Niklas Söderlund
2021-04-30 12:31 ` [PATCH 02/12] pinctrl: renesas: r8a77990: JTAG pins do not have pull-down capabilities Geert Uytterhoeven
2021-05-01 7:19 ` Niklas Söderlund
2021-04-30 12:31 ` [PATCH 03/12] pinctrl: renesas: r8a77990: Drop bogus PUEN_ prefixes in comments Geert Uytterhoeven
2021-05-01 7:31 ` Niklas Söderlund
2021-04-30 12:31 ` [PATCH 04/12] pinctrl: renesas: r8a7778: Remove unused PORT_GP_PUP_1() macro Geert Uytterhoeven
2021-05-01 7:36 ` Niklas Söderlund
2021-04-30 12:31 ` [PATCH 05/12] pinctrl: renesas: r8a779{51,6,65}: Reduce non-functional differences Geert Uytterhoeven
2021-05-01 8:07 ` Niklas Söderlund [this message]
2021-04-30 12:31 ` [PATCH 06/12] pinctrl: renesas: r8a77470: Add bias pinconf support Geert Uytterhoeven
2021-05-01 8:24 ` Niklas Söderlund
2021-04-30 12:31 ` [PATCH 07/12] pinctrl: renesas: r8a7790: " Geert Uytterhoeven
2021-05-01 9:04 ` Niklas Söderlund
2021-05-25 7:23 ` Wolfram Sang
2021-04-30 12:31 ` [PATCH 08/12] pinctrl: renesas: r8a7792: " Geert Uytterhoeven
2021-05-01 9:13 ` Niklas Söderlund
2021-04-30 12:31 ` [PATCH 09/12] pinctrl: renesas: r8a7794: " Geert Uytterhoeven
2021-05-01 9:25 ` Niklas Söderlund
2021-05-25 9:03 ` Wolfram Sang
2021-04-30 12:31 ` [PATCH 10/12] pinctrl: renesas: r8a77970: " Geert Uytterhoeven
2021-05-01 9:33 ` Niklas Söderlund
2021-04-30 12:31 ` [PATCH 11/12] pinctrl: renesas: r8a77980: " Geert Uytterhoeven
2021-05-01 9:42 ` Niklas Söderlund
2021-04-30 12:31 ` [PATCH 12/12] pinctrl: renesas: r8a77995: " Geert Uytterhoeven
2021-05-01 9:54 ` Niklas Söderlund
2021-06-10 8:01 ` 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=YI0MKZ9UnzuLBw06@oden.dyn.berto.se \
--to=niklas.soderlund@ragnatech.se \
--cc=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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).