diff mbox series

[v2,3/3] dmaengine: sh: rz-dmac: add r7s72100 support

Message ID 20241001124310.2336-4-wsa+renesas@sang-engineering.com (mailing list archive)
State Superseded
Headers show
Series dmaengine: sh: rz-dmac: add r7s72100 support | expand

Commit Message

Wolfram Sang Oct. 1, 2024, 12:43 p.m. UTC 
This SoC needs to make getting resets optional. Descriptions are
reworded to be more generic.

Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
Reviewed-by: Philipp Zabel <p.zabel@pengutronix.de>
---
 drivers/dma/sh/Kconfig   | 8 ++++----
 drivers/dma/sh/rz-dmac.c | 2 +-
 2 files changed, 5 insertions(+), 5 deletions(-)

Comments

Geert Uytterhoeven Oct. 4, 2024, 2:47 p.m. UTC | #1 
On Tue, Oct 1, 2024 at 2:53 PM Wolfram Sang
<wsa+renesas@sang-engineering.com> wrote:
> This SoC needs to make getting resets optional. Descriptions are
> reworded to be more generic.
>
> Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
> Reviewed-by: Philipp Zabel <p.zabel@pengutronix.de>

Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

> index c0b2997ab7fd..6ea5a880b433 100644
> --- a/drivers/dma/sh/Kconfig
> +++ b/drivers/dma/sh/Kconfig
> @@ -49,10 +49,10 @@ config RENESAS_USB_DMAC
>           SoCs.
>
>  config RZ_DMAC
> -       tristate "Renesas RZ/{G2L,V2L} DMA Controller"
> -       depends on ARCH_RZG2L || COMPILE_TEST
> +       tristate "Renesas RZ DMA Controller"
> +       depends on ARCH_R7S72100 || ARCH_RZG2L || COMPILE_TEST

And soon this will have to be extended to ARCH_R7S9210...

>         select RENESAS_DMA
>         select DMA_VIRTUAL_CHANNELS
>         help
> -         This driver supports the general purpose DMA controller found in the
> -         Renesas RZ/{G2L,V2L} SoC variants.
> +         This driver supports the general purpose DMA controller typically
> +         found in the Renesas RZ SoC variants.

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
diff mbox series

Patch

diff --git a/drivers/dma/sh/Kconfig b/drivers/dma/sh/Kconfig
index c0b2997ab7fd..6ea5a880b433 100644
--- a/drivers/dma/sh/Kconfig
+++ b/drivers/dma/sh/Kconfig
@@ -49,10 +49,10 @@  config RENESAS_USB_DMAC
 	  SoCs.
 
 config RZ_DMAC
-	tristate "Renesas RZ/{G2L,V2L} DMA Controller"
-	depends on ARCH_RZG2L || COMPILE_TEST
+	tristate "Renesas RZ DMA Controller"
+	depends on ARCH_R7S72100 || ARCH_RZG2L || COMPILE_TEST
 	select RENESAS_DMA
 	select DMA_VIRTUAL_CHANNELS
 	help
-	  This driver supports the general purpose DMA controller found in the
-	  Renesas RZ/{G2L,V2L} SoC variants.
+	  This driver supports the general purpose DMA controller typically
+	  found in the Renesas RZ SoC variants.
diff --git a/drivers/dma/sh/rz-dmac.c b/drivers/dma/sh/rz-dmac.c
index 811389fc9cb8..03f3f99f0f4a 100644
--- a/drivers/dma/sh/rz-dmac.c
+++ b/drivers/dma/sh/rz-dmac.c
@@ -893,7 +893,7 @@  static int rz_dmac_probe(struct platform_device *pdev)
 	/* Initialize the channels. */
 	INIT_LIST_HEAD(&dmac->engine.channels);
 
-	dmac->rstc = devm_reset_control_array_get_exclusive(&pdev->dev);
+	dmac->rstc = devm_reset_control_array_get_optional_exclusive(&pdev->dev);
 	if (IS_ERR(dmac->rstc))
 		return dev_err_probe(&pdev->dev, PTR_ERR(dmac->rstc),
 				     "failed to get resets\n");