From mboxrd@z Thu Jan 1 00:00:00 1970 From: Geert Uytterhoeven Subject: Re: [PATCH] pinctrl: sh-pfc: r8a7795: Add GP-1-28 port pin support Date: Thu, 23 Nov 2017 11:03:40 +0100 Message-ID: References: <1510802091-30535-1-git-send-email-ykaneko0929@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Return-path: Received: from mail-qt0-f195.google.com ([209.85.216.195]:35755 "EHLO mail-qt0-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752206AbdKWKDm (ORCPT ); Thu, 23 Nov 2017 05:03:42 -0500 In-Reply-To: <1510802091-30535-1-git-send-email-ykaneko0929@gmail.com> Sender: linux-gpio-owner@vger.kernel.org List-Id: linux-gpio@vger.kernel.org To: Yoshihiro Kaneko Cc: "linux-gpio@vger.kernel.org" , Linus Walleij , Laurent Pinchart , Geert Uytterhoeven , Simon Horman , Magnus Damm , Linux-Renesas Hi Kaneko-san, On Thu, Nov 16, 2017 at 4:14 AM, Yoshihiro Kaneko wrote: > From: Takeshi Kihara > > This patch supports GP-1-28 port pin of R8A7795 ES2.0 SoC added in > Rev.0.54E of the R-Car Gen3 Hardware User's Manual or later version. > > Signed-off-by: Takeshi Kihara > Signed-off-by: Yoshihiro Kaneko Thanks for your patch! > --- a/drivers/pinctrl/sh-pfc/pfc-r8a7795.c > +++ b/drivers/pinctrl/sh-pfc/pfc-r8a7795.c > @@ -5246,7 +5248,7 @@ enum { > { RCAR_GP_PIN(1, 19), 0, 3 }, /* A19 */ > } }, > { PINMUX_DRIVE_REG("DRVCTRL8", 0xe6060320) { > - { PIN_NUMBER('F', 1), 28, 3 }, /* CLKOUT */ > + { RCAR_GP_PIN(1, 28), 28, 3 }, /* CLKOUT */ > { RCAR_GP_PIN(1, 20), 24, 3 }, /* CS0 */ > { RCAR_GP_PIN(1, 21), 20, 3 }, /* CS1_A26 */ > { RCAR_GP_PIN(1, 22), 16, 3 }, /* BS */ You should make a similar change to the corresponding entry for "PUEN2", cfr. drivers/pinctrl/sh-pfc/pfc-r8a7796.c. With that fixed: Reviewed-by: Geert Uytterhoeven 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