| Message ID | 1405455522-20676-2-git-send-email-laurent.pinchart+renesas@ideasonboard.com (mailing list archive) |
|---|---|
| State | Superseded |
| Delegated to: | Vinod Koul |
| Headers | show |
Hello. On 07/16/2014 12:18 AM, Laurent Pinchart wrote: > Separate helpers and drivers in the Kconfig and Makefile to improve > readability and move the CONFIG_OF dependency from the Makefile to > Kconfig. > Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> > --- > drivers/dma/sh/Kconfig | 24 +++++++++++++++++++----- > drivers/dma/sh/Makefile | 14 +++++++++++--- > 2 files changed, 30 insertions(+), 8 deletions(-) > diff --git a/drivers/dma/sh/Kconfig b/drivers/dma/sh/Kconfig > index 0f71981..383b21f 100644 > --- a/drivers/dma/sh/Kconfig > +++ b/drivers/dma/sh/Kconfig > @@ -2,21 +2,39 @@ > # DMA engine configuration for sh > # > > +# > +# DMA Engine Helpers > +# > + > config SH_DMAE_BASE > bool "Renesas SuperH DMA Engine support" > - depends on (SUPERH && SH_DMA) || ARCH_SHMOBILE || COMPILE_TEST > + depends on SUPERH || ARCH_SHMOBILE || COMPILE_TEST > + depends on !SUPERH || SH_DMA s/||/&&/? > depends on !SH_DMA_API > default y > select DMA_ENGINE > help > Enable support for the Renesas SuperH DMA controllers. > > +# > +# DMA Controllers > +# > + > config SH_DMAE > tristate "Renesas SuperH DMAC support" > depends on SH_DMAE_BASE > help > Enable support for the Renesas SuperH DMA controllers. > > +if SH_DMAE > + > +config SH_DMAE_R8A73A4 > + def_bool y > + depends on ARCH_R8A73A4 > + depends on CONFIG_OF s/CONFIG_//. > + > +endif > + > config SUDMAC > tristate "Renesas SUDMAC support" > depends on SH_DMAE_BASE [...] WBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe dmaengine" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hi Sergei, Thank you for the review. On Wednesday 16 July 2014 01:36:21 Sergei Shtylyov wrote: > On 07/16/2014 12:18 AM, Laurent Pinchart wrote: > > Separate helpers and drivers in the Kconfig and Makefile to improve > > readability and move the CONFIG_OF dependency from the Makefile to > > Kconfig. > > > > Signed-off-by: Laurent Pinchart > > <laurent.pinchart+renesas@ideasonboard.com> > > --- > > > > drivers/dma/sh/Kconfig | 24 +++++++++++++++++++----- > > drivers/dma/sh/Makefile | 14 +++++++++++--- > > 2 files changed, 30 insertions(+), 8 deletions(-) > > > > diff --git a/drivers/dma/sh/Kconfig b/drivers/dma/sh/Kconfig > > index 0f71981..383b21f 100644 > > --- a/drivers/dma/sh/Kconfig > > +++ b/drivers/dma/sh/Kconfig > > @@ -2,21 +2,39 @@ > > # DMA engine configuration for sh > > # > > > > +# > > +# DMA Engine Helpers > > +# > > + > > config SH_DMAE_BASE > > bool "Renesas SuperH DMA Engine support" > > - depends on (SUPERH && SH_DMA) || ARCH_SHMOBILE || COMPILE_TEST > > + depends on SUPERH || ARCH_SHMOBILE || COMPILE_TEST > > + depends on !SUPERH || SH_DMA > > s/||/&&/? No. The driver can be used on SUPERH only when SH_DMA is enabled, and can be used on other architectures regardless of SH_DMA. This could have been expressed as (SUPERH && SH_DMA) || !SUPERH as well. > > depends on !SH_DMA_API > > default y > > select DMA_ENGINE > > help > > Enable support for the Renesas SuperH DMA controllers. > > +# > > +# DMA Controllers > > +# > > + > > config SH_DMAE > > tristate "Renesas SuperH DMAC support" > > depends on SH_DMAE_BASE > > help > > Enable support for the Renesas SuperH DMA controllers. > > > > +if SH_DMAE > > + > > +config SH_DMAE_R8A73A4 > > + def_bool y > > + depends on ARCH_R8A73A4 > > + depends on CONFIG_OF > > s/CONFIG_//. My bad, will fix. > > + > > +endif > > + > > config SUDMAC > > tristate "Renesas SUDMAC support" > > depends on SH_DMAE_BASE > > [...]
diff --git a/drivers/dma/sh/Kconfig b/drivers/dma/sh/Kconfig index 0f71981..383b21f 100644 --- a/drivers/dma/sh/Kconfig +++ b/drivers/dma/sh/Kconfig @@ -2,21 +2,39 @@ # DMA engine configuration for sh # +# +# DMA Engine Helpers +# + config SH_DMAE_BASE bool "Renesas SuperH DMA Engine support" - depends on (SUPERH && SH_DMA) || ARCH_SHMOBILE || COMPILE_TEST + depends on SUPERH || ARCH_SHMOBILE || COMPILE_TEST + depends on !SUPERH || SH_DMA depends on !SH_DMA_API default y select DMA_ENGINE help Enable support for the Renesas SuperH DMA controllers. +# +# DMA Controllers +# + config SH_DMAE tristate "Renesas SuperH DMAC support" depends on SH_DMAE_BASE help Enable support for the Renesas SuperH DMA controllers. +if SH_DMAE + +config SH_DMAE_R8A73A4 + def_bool y + depends on ARCH_R8A73A4 + depends on CONFIG_OF + +endif + config SUDMAC tristate "Renesas SUDMAC support" depends on SH_DMAE_BASE @@ -34,7 +52,3 @@ config RCAR_AUDMAC_PP depends on SH_DMAE_BASE help Enable support for the Renesas R-Car Audio DMAC Peripheral Peripheral controllers. - -config SHDMA_R8A73A4 - def_bool y - depends on ARCH_R8A73A4 && SH_DMAE != n diff --git a/drivers/dma/sh/Makefile b/drivers/dma/sh/Makefile index 1ce88b2..0b13b8e 100644 --- a/drivers/dma/sh/Makefile +++ b/drivers/dma/sh/Makefile @@ -1,10 +1,18 @@ +# +# DMA Engine Helpers +# + obj-$(CONFIG_SH_DMAE_BASE) += shdma-base.o shdma-of.o -obj-$(CONFIG_SH_DMAE) += shdma.o + +# +# DMA Controllers +# + shdma-y := shdmac.o -ifeq ($(CONFIG_OF),y) shdma-$(CONFIG_SHDMA_R8A73A4) += shdma-r8a73a4.o -endif shdma-objs := $(shdma-y) +obj-$(CONFIG_SH_DMAE) += shdma.o + obj-$(CONFIG_SUDMAC) += sudmac.o obj-$(CONFIG_RCAR_HPB_DMAE) += rcar-hpbdma.o obj-$(CONFIG_RCAR_AUDMAC_PP) += rcar-audmapp.o
Separate helpers and drivers in the Kconfig and Makefile to improve readability and move the CONFIG_OF dependency from the Makefile to Kconfig. Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> --- drivers/dma/sh/Kconfig | 24 +++++++++++++++++++----- drivers/dma/sh/Makefile | 14 +++++++++++--- 2 files changed, 30 insertions(+), 8 deletions(-)