Hi Prabhakar, On Fri, Sep 1, 2023 at 1:37 PM Prabhakar <prabhakar.csengg@xxxxxxxxx> wrote: > From: Lad Prabhakar <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx> > > Make MMC_SDHI_INTERNAL_DMAC visible for RISC-V architecture so that we can > select this config option for RZ/Five SoC which is based on RISC-V > architecture. > > Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx> Thanks for your patch! > I was in two minds here whether I should instead add dependency for > ARCH_R9A07G043 instead of RISCV. But adding dependency for RISCV allows > us to get this config option freely for future coming RISC-V Renesas > SoCs. > > Cheers, Prabhakar > --- > drivers/mmc/host/Kconfig | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/mmc/host/Kconfig b/drivers/mmc/host/Kconfig > index 554e67103c1a..5190881cb204 100644 > --- a/drivers/mmc/host/Kconfig > +++ b/drivers/mmc/host/Kconfig > @@ -677,9 +677,9 @@ config MMC_SDHI_SYS_DMAC > > config MMC_SDHI_INTERNAL_DMAC > tristate "DMA for SDHI SD/SDIO controllers using on-chip bus mastering" > - depends on ARM64 || ARCH_R7S9210 || ARCH_R8A77470 || COMPILE_TEST > + depends on ARM64 || ARCH_R7S9210 || ARCH_R8A77470 || RISCV || COMPILE_TEST > depends on MMC_SDHI > - default MMC_SDHI if (ARM64 || ARCH_R7S9210 || ARCH_R8A77470) > + default MMC_SDHI if (ARM64 || ARCH_R7S9210 || ARCH_R8A77470 || RISCV) This would expose this option to a growing number of non-Renesas RISC-V platforms. What about using ARCH_RENESAS || COMPILE_TEST instead? > help > This provides DMA support for SDHI SD/SDIO controllers > using on-chip bus mastering. This supports the controllers Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx 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