pinctrl: sh-pfc: r8a7795: add newlines for better diff view

Message ID 20171006103725.13804-1-wsa+renesas@sang-engineering.com
State New
Headers show
Series
  • pinctrl: sh-pfc: r8a7795: add newlines for better diff view
Related show

Commit Message

Wolfram Sang Oct. 6, 2017, 10:37 a.m.
Be consistent with the newlines after structs. This makes diffs a lot
easier to parse when comparing with other SoCs/versions.

Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
---
 drivers/pinctrl/sh-pfc/pfc-r8a7795.c | 2 ++
 1 file changed, 2 insertions(+)

Comments

Geert Uytterhoeven Oct. 6, 2017, 11:35 a.m. | #1
Hi Wolfram,

On Fri, Oct 6, 2017 at 12:37 PM, Wolfram Sang
<wsa+renesas@sang-engineering.com> wrote:
> Be consistent with the newlines after structs. This makes diffs a lot
> easier to parse when comparing with other SoCs/versions.

I'm all for consistency, but...

> --- a/drivers/pinctrl/sh-pfc/pfc-r8a7795.c
> +++ b/drivers/pinctrl/sh-pfc/pfc-r8a7795.c
> @@ -1678,6 +1678,7 @@ static const unsigned int audio_clkout2_b_pins[] = {
>  static const unsigned int audio_clkout2_b_mux[] = {
>         AUDIO_CLKOUT2_B_MARK,
>  };
> +

That's a stray newline in both drivers/pinctrl/sh-pfc/pfc-r8a7795-es1.c and
drivers/pinctrl/sh-pfc/pfc-r8a7796.c, which should be removed from the latter
instead.

>  static const unsigned int audio_clkout3_a_pins[] = {
>         /* CLKOUT3 */
>         RCAR_GP_PIN(5, 19),
> @@ -2967,6 +2968,7 @@ static const unsigned int msiof3_rxd_d_pins[] = {
>  static const unsigned int msiof3_rxd_d_mux[] = {
>         MSIOF3_RXD_D_MARK,
>  };
> +

drivers/pinctrl/sh-pfc/pfc-r8a7795-es1.c has a newline here because H3 ES1.x
doesn't have the fifth set of MSIOF pins, thus msiof3_rxd_d_mux() is the
last entry of the MSIOF section.

>  static const unsigned int msiof3_clk_e_pins[] = {
>         /* SCK */
>         RCAR_GP_PIN(2, 3),

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds
--
To unsubscribe from this list: send the line "unsubscribe linux-gpio" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Patch

diff --git a/drivers/pinctrl/sh-pfc/pfc-r8a7795.c b/drivers/pinctrl/sh-pfc/pfc-r8a7795.c
index ccfc96c4d28c3d..105b564e39c1b2 100644
--- a/drivers/pinctrl/sh-pfc/pfc-r8a7795.c
+++ b/drivers/pinctrl/sh-pfc/pfc-r8a7795.c
@@ -1678,6 +1678,7 @@  static const unsigned int audio_clkout2_b_pins[] = {
 static const unsigned int audio_clkout2_b_mux[] = {
 	AUDIO_CLKOUT2_B_MARK,
 };
+
 static const unsigned int audio_clkout3_a_pins[] = {
 	/* CLKOUT3 */
 	RCAR_GP_PIN(5, 19),
@@ -2967,6 +2968,7 @@  static const unsigned int msiof3_rxd_d_pins[] = {
 static const unsigned int msiof3_rxd_d_mux[] = {
 	MSIOF3_RXD_D_MARK,
 };
+
 static const unsigned int msiof3_clk_e_pins[] = {
 	/* SCK */
 	RCAR_GP_PIN(2, 3),