diff mbox series

can: rcar: Update help description for CAN_RCAR config

Message ID 20201231155957.31165-2-prabhakar.mahadev-lad.rj@bp.renesas.com (mailing list archive)
State Superseded
Delegated to: Netdev Maintainers
Headers show
Series can: rcar: Update help description for CAN_RCAR config | expand

Checks

Context Check Description
netdev/tree_selection success Series ignored based on subject

Commit Message

Prabhakar Dec. 31, 2020, 3:59 p.m. UTC
The rcar_can 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 <prabhakar.mahadev-lad.rj@bp.renesas.com>
---
 drivers/net/can/rcar/Kconfig | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Biju Das Jan. 1, 2021, 5:06 p.m. UTC | #1
Hi Prabhakar,

Thanks for the patch.

> -----Original Message-----
> From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
> Sent: 31 December 2020 16:00
> To: Geert Uytterhoeven <geert+renesas@glider.be>; Wolfgang Grandegger
> <wg@grandegger.com>; Marc Kleine-Budde <mkl@pengutronix.de>; David S.
> Miller <davem@davemloft.net>; Jakub Kicinski <kuba@kernel.org>; Masahiro
> Yamada <masahiroy@kernel.org>
> Cc: linux-can@vger.kernel.org; netdev@vger.kernel.org; linux-
> kernel@vger.kernel.org; linux-renesas-soc@vger.kernel.org; Prabhakar
> <prabhakar.csengg@gmail.com>; Prabhakar Mahadev Lad <prabhakar.mahadev-
> lad.rj@bp.renesas.com>
> Subject: [PATCH] can: rcar: Update help description for CAN_RCAR config
> 
> The rcar_can 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 it generic like dropping {1,2,3}/{1,2} and  instead use R-Car and RZ/G SoC's??

Cheers,
Biju


> 
> Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
> ---
>  drivers/net/can/rcar/Kconfig | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/net/can/rcar/Kconfig b/drivers/net/can/rcar/Kconfig
> index 8d36101b78e3..6bb0e7c052ad 100644
> --- a/drivers/net/can/rcar/Kconfig
> +++ b/drivers/net/can/rcar/Kconfig
> @@ -1,10 +1,10 @@
>  # SPDX-License-Identifier: GPL-2.0
>  config CAN_RCAR
> -	tristate "Renesas R-Car CAN controller"
> +	tristate "Renesas R-Car Gen{1,2,3} and RZ/G{1,2} CAN controller"
>  	depends on ARCH_RENESAS || ARM
>  	help
>  	  Say Y here if you want to use CAN controller found on Renesas R-
> Car
> -	  SoCs.
> +	  Gen{1,2,3} and RZ/G{1,2} SoCs.
> 
>  	  To compile this driver as a module, choose M here: the module will
>  	  be called rcar_can.
> --
> 2.17.1
Marc Kleine-Budde Jan. 4, 2021, 8:46 a.m. UTC | #2
On 1/1/21 6:06 PM, Biju Das wrote:
>> The rcar_can 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 it generic like dropping {1,2,3}/{1,2} and  instead
> use R-Car and RZ/G SoC's??

Sounds good. Better make the description more generic instead of updating it
each time a new SoC comes along.

regards,
Marc
diff mbox series

Patch

diff --git a/drivers/net/can/rcar/Kconfig b/drivers/net/can/rcar/Kconfig
index 8d36101b78e3..6bb0e7c052ad 100644
--- a/drivers/net/can/rcar/Kconfig
+++ b/drivers/net/can/rcar/Kconfig
@@ -1,10 +1,10 @@ 
 # SPDX-License-Identifier: GPL-2.0
 config CAN_RCAR
-	tristate "Renesas R-Car CAN controller"
+	tristate "Renesas R-Car Gen{1,2,3} and RZ/G{1,2} CAN controller"
 	depends on ARCH_RENESAS || ARM
 	help
 	  Say Y here if you want to use CAN controller found on Renesas R-Car
-	  SoCs.
+	  Gen{1,2,3} and RZ/G{1,2} SoCs.
 
 	  To compile this driver as a module, choose M here: the module will
 	  be called rcar_can.