2020-12-31 15:40:45

by Prabhakar Mahadev Lad

[permalink] [raw]
Subject: [PATCH] gpio: Kconfig: Update help description for GPIO_RCAR

The gpio-rcar driver supports R-Car Gen{1,2,3} and RZ/G{1,2} SoC's, update
the description to reflect this.

Signed-off-by: Lad Prabhakar <[email protected]>
---
drivers/gpio/Kconfig | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/drivers/gpio/Kconfig b/drivers/gpio/Kconfig
index c70f46e80a3b..47e545d71df1 100644
--- a/drivers/gpio/Kconfig
+++ b/drivers/gpio/Kconfig
@@ -486,11 +486,12 @@ config GPIO_PXA
Say yes here to support the PXA GPIO device

config GPIO_RCAR
- tristate "Renesas R-Car GPIO"
+ tristate "Renesas R-Car Gen{1,2,3} and RZ/G{1,2} GPIO support"
depends on ARCH_RENESAS || COMPILE_TEST
select GPIOLIB_IRQCHIP
help
- Say yes here to support GPIO on Renesas R-Car SoCs.
+ Say yes here to support GPIO on Renesas R-Car Gen{1,2,3} and
+ RZ/G{1,2} SoCs.

config GPIO_RDA
bool "RDA Micro GPIO controller support"
--
2.17.1


2021-01-01 17:10:09

by Biju Das

[permalink] [raw]
Subject: RE: [PATCH] gpio: Kconfig: Update help description for GPIO_RCAR

Hi Prabhakar,

> -----Original Message-----
> From: Lad Prabhakar <[email protected]>
> Sent: 31 December 2020 15:39
> To: Linus Walleij <[email protected]>; Bartosz Golaszewski
> <[email protected]>; Geert Uytterhoeven <[email protected]>
> Cc: [email protected]; [email protected]; linux-
> [email protected]; Prabhakar <[email protected]>;
> Prabhakar Mahadev Lad <[email protected]>
> Subject: [PATCH] gpio: Kconfig: Update help description for GPIO_RCAR
>
> The gpio-rcar driver supports R-Car Gen{1,2,3} and RZ/G{1,2} SoC's, update
> the description to reflect this.

Not sure we need to make this generic by dropping {1,2,3}/{1,2} and use R-Car and RZ/G SoC's instead ???

Regards,
Biju

>
> Signed-off-by: Lad Prabhakar <[email protected]>
> ---
> drivers/gpio/Kconfig | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpio/Kconfig b/drivers/gpio/Kconfig index
> c70f46e80a3b..47e545d71df1 100644
> --- a/drivers/gpio/Kconfig
> +++ b/drivers/gpio/Kconfig
> @@ -486,11 +486,12 @@ config GPIO_PXA
> Say yes here to support the PXA GPIO device
>
> config GPIO_RCAR
> - tristate "Renesas R-Car GPIO"
> + tristate "Renesas R-Car Gen{1,2,3} and RZ/G{1,2} GPIO support"
> depends on ARCH_RENESAS || COMPILE_TEST
> select GPIOLIB_IRQCHIP
> help
> - Say yes here to support GPIO on Renesas R-Car SoCs.
> + Say yes here to support GPIO on Renesas R-Car Gen{1,2,3} and
> + RZ/G{1,2} SoCs.
>
> config GPIO_RDA
> bool "RDA Micro GPIO controller support"
> --
> 2.17.1

2021-01-03 15:43:22

by Bartosz Golaszewski

[permalink] [raw]
Subject: Re: [PATCH] gpio: Kconfig: Update help description for GPIO_RCAR

On Fri, Jan 1, 2021 at 6:07 PM Biju Das <[email protected]> wrote:
>
> Hi Prabhakar,
>
> > -----Original Message-----
> > From: Lad Prabhakar <[email protected]>
> > Sent: 31 December 2020 15:39
> > To: Linus Walleij <[email protected]>; Bartosz Golaszewski
> > <[email protected]>; Geert Uytterhoeven <[email protected]>
> > Cc: [email protected]; [email protected]; linux-
> > [email protected]; Prabhakar <[email protected]>;
> > Prabhakar Mahadev Lad <[email protected]>
> > Subject: [PATCH] gpio: Kconfig: Update help description for GPIO_RCAR
> >
> > The gpio-rcar driver supports R-Car Gen{1,2,3} and RZ/G{1,2} SoC's, update
> > the description to reflect this.
>
> Not sure we need to make this generic by dropping {1,2,3}/{1,2} and use R-Car and RZ/G SoC's instead ???
>

This looks better IMO - if Geert is OK with that, then let's change it.

Bart

> Regards,
> Biju
>
> >
> > Signed-off-by: Lad Prabhakar <[email protected]>
> > ---
> > drivers/gpio/Kconfig | 5 +++--
> > 1 file changed, 3 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/gpio/Kconfig b/drivers/gpio/Kconfig index
> > c70f46e80a3b..47e545d71df1 100644
> > --- a/drivers/gpio/Kconfig
> > +++ b/drivers/gpio/Kconfig
> > @@ -486,11 +486,12 @@ config GPIO_PXA
> > Say yes here to support the PXA GPIO device
> >
> > config GPIO_RCAR
> > - tristate "Renesas R-Car GPIO"
> > + tristate "Renesas R-Car Gen{1,2,3} and RZ/G{1,2} GPIO support"
> > depends on ARCH_RENESAS || COMPILE_TEST
> > select GPIOLIB_IRQCHIP
> > help
> > - Say yes here to support GPIO on Renesas R-Car SoCs.
> > + Say yes here to support GPIO on Renesas R-Car Gen{1,2,3} and
> > + RZ/G{1,2} SoCs.
> >
> > config GPIO_RDA
> > bool "RDA Micro GPIO controller support"
> > --
> > 2.17.1
>

2021-01-04 11:07:23

by Geert Uytterhoeven

[permalink] [raw]
Subject: Re: [PATCH] gpio: Kconfig: Update help description for GPIO_RCAR

Hi Bartosz et al,

On Sun, Jan 3, 2021 at 4:29 PM Bartosz Golaszewski
<[email protected]> wrote:
> On Fri, Jan 1, 2021 at 6:07 PM Biju Das <[email protected]> wrote:
> > > -----Original Message-----
> > > From: Lad Prabhakar <[email protected]>
> > > Sent: 31 December 2020 15:39
> > > To: Linus Walleij <[email protected]>; Bartosz Golaszewski
> > > <[email protected]>; Geert Uytterhoeven <[email protected]>
> > > Cc: [email protected]; [email protected]; linux-
> > > [email protected]; Prabhakar <[email protected]>;
> > > Prabhakar Mahadev Lad <[email protected]>
> > > Subject: [PATCH] gpio: Kconfig: Update help description for GPIO_RCAR
> > >
> > > The gpio-rcar driver supports R-Car Gen{1,2,3} and RZ/G{1,2} SoC's, update
> > > the description to reflect this.
> >
> > Not sure we need to make this generic by dropping {1,2,3}/{1,2} and use R-Car and RZ/G SoC's instead ???
> >
>
> This looks better IMO - if Geert is OK with that, then let's change it.

"R-Car and RZ/G" sounds better to me, as it is present on all known
R-Car and RZ/G SoCs. We can change the help text if that ever changes.

> > > --- a/drivers/gpio/Kconfig
> > > +++ b/drivers/gpio/Kconfig
> > > @@ -486,11 +486,12 @@ config GPIO_PXA
> > > Say yes here to support the PXA GPIO device
> > >
> > > config GPIO_RCAR
> > > - tristate "Renesas R-Car GPIO"
> > > + tristate "Renesas R-Car Gen{1,2,3} and RZ/G{1,2} GPIO support"
> > > depends on ARCH_RENESAS || COMPILE_TEST
> > > select GPIOLIB_IRQCHIP
> > > help
> > > - Say yes here to support GPIO on Renesas R-Car SoCs.
> > > + Say yes here to support GPIO on Renesas R-Car Gen{1,2,3} and
> > > + RZ/G{1,2} SoCs.
> > >
> > > config GPIO_RDA
> > > bool "RDA Micro GPIO controller support"

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]

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