diff mbox

[02/30] dmaengine: jz4740: enable COMPILE_TEST

Message ID 1473960849-23024-3-git-send-email-vinod.koul@intel.com (mailing list archive)
State Superseded
Headers show

Commit Message

Vinod Koul Sept. 15, 2016, 5:33 p.m. UTC
To get more coverage, enable COMPILE_TEST for this driver.

Cc: Lars-Peter Clausen <lars@metafoo.de>
Signed-off-by: Vinod Koul <vinod.koul@intel.com>
---
 drivers/dma/Kconfig | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Lars-Peter Clausen Sept. 15, 2016, 6:13 p.m. UTC | #1
On 09/15/2016 07:33 PM, Vinod Koul wrote:
> To get more coverage, enable COMPILE_TEST for this driver.

The patch does bit more. I unconditionally enables the driver on
MACH_JZ4740, do we want this?

>  config DMA_JZ4740
> -	tristate "JZ4740 DMA support"
> -	depends on MACH_JZ4740
> +	tristate "JZ4740 DMA support" if COMPILE_TEST && !MACH_JZ4740
> +	default MACH_JZ4740
--
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
Vinod Koul Sept. 16, 2016, 2:45 p.m. UTC | #2
On Thu, Sep 15, 2016 at 08:13:43PM +0200, Lars-Peter Clausen wrote:
> On 09/15/2016 07:33 PM, Vinod Koul wrote:
> > To get more coverage, enable COMPILE_TEST for this driver.
> 
> The patch does bit more. I unconditionally enables the driver on
> MACH_JZ4740, do we want this?

My bad, I should have _explictly_ mentioned this. Yes that was indeed the
idea..

Thanks
diff mbox

Patch

diff --git a/drivers/dma/Kconfig b/drivers/dma/Kconfig
index 739f797b40d9..0e612edb3583 100644
--- a/drivers/dma/Kconfig
+++ b/drivers/dma/Kconfig
@@ -113,8 +113,8 @@  config DMA_BCM2835
 	select DMA_VIRTUAL_CHANNELS
 
 config DMA_JZ4740
-	tristate "JZ4740 DMA support"
-	depends on MACH_JZ4740
+	tristate "JZ4740 DMA support" if COMPILE_TEST && !MACH_JZ4740
+	default MACH_JZ4740
 	select DMA_ENGINE
 	select DMA_VIRTUAL_CHANNELS