mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH 1/2] spi: rspi: Relax DMA dependency and increase build coverage
@ 2014-07-09 21:27 Geert Uytterhoeven
  2014-07-09 21:27 ` [PATCH 2/2] spi: sh-msiof: SPI_SH_MSIOF should depend on HAS_DMA Geert Uytterhoeven
  2014-07-10 10:40 ` [PATCH 1/2] spi: rspi: Relax DMA dependency and increase build coverage Mark Brown
  0 siblings, 2 replies; 3+ messages in thread
From: Geert Uytterhoeven @ 2014-07-09 21:27 UTC (permalink / raw)
  To: Mark Brown; +Cc: linux-spi, linux-kernel, Geert Uytterhoeven

As of commit ab116a4df4942c78c189d9b0744dd940ab9e00b9 ("dmaengine:
shdma: fix a build failure on platforms with no DMA support"), the DMA
filter function shdma_chan_filter() is sufficiently abstracted to allow
building without DMA support. Hence drop the SH_DMAE_BASE dependency on
SUPERH.
Also increase build coverage by allowing the driver to be enabled if
COMPILE_TEST=y.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 drivers/spi/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/spi/Kconfig b/drivers/spi/Kconfig
index d6be5e6e93e9..d46b4b9e73b2 100644
--- a/drivers/spi/Kconfig
+++ b/drivers/spi/Kconfig
@@ -396,7 +396,7 @@ config SPI_ROCKCHIP
 
 config SPI_RSPI
 	tristate "Renesas RSPI/QSPI controller"
-	depends on (SUPERH && SH_DMAE_BASE) || ARCH_SHMOBILE
+	depends on SUPERH || ARCH_SHMOBILE || COMPILE_TEST
 	help
 	  SPI driver for Renesas RSPI and QSPI blocks.
 
-- 
1.9.1


^ permalink raw reply	[flat|nested] 3+ messages in thread

* [PATCH 2/2] spi: sh-msiof: SPI_SH_MSIOF should depend on HAS_DMA
  2014-07-09 21:27 [PATCH 1/2] spi: rspi: Relax DMA dependency and increase build coverage Geert Uytterhoeven
@ 2014-07-09 21:27 ` Geert Uytterhoeven
  2014-07-10 10:40 ` [PATCH 1/2] spi: rspi: Relax DMA dependency and increase build coverage Mark Brown
  1 sibling, 0 replies; 3+ messages in thread
From: Geert Uytterhoeven @ 2014-07-09 21:27 UTC (permalink / raw)
  To: Mark Brown; +Cc: linux-spi, linux-kernel, Geert Uytterhoeven

If NO_DMA=y:

    drivers/built-in.o: In function `sh_msiof_release_dma':
    spi-sh-msiof.c:(.text+0x23cbfe): undefined reference to `dma_unmap_single'
    drivers/built-in.o: In function `sh_msiof_request_dma':
    spi-sh-msiof.c:(.text+0x23cd5e): undefined reference to `dma_map_single'
    spi-sh-msiof.c:(.text+0x23cd70): undefined reference to `dma_mapping_error'
    spi-sh-msiof.c:(.text+0x23cdca): undefined reference to `dma_unmap_single'
    drivers/built-in.o: In function `sh_msiof_dma_once':
    spi-sh-msiof.c:(.text+0x23d450): undefined reference to `dma_sync_single_for_cpu'
    spi-sh-msiof.c:(.text+0x23d5ea): undefined reference to `dma_sync_single_for_cpu'

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 drivers/spi/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/spi/Kconfig b/drivers/spi/Kconfig
index d46b4b9e73b2..ab70bca8c7e1 100644
--- a/drivers/spi/Kconfig
+++ b/drivers/spi/Kconfig
@@ -446,7 +446,7 @@ config SPI_SC18IS602
 
 config SPI_SH_MSIOF
 	tristate "SuperH MSIOF SPI controller"
-	depends on HAVE_CLK
+	depends on HAVE_CLK && HAS_DMA
 	depends on SUPERH || ARCH_SHMOBILE || COMPILE_TEST
 	help
 	  SPI driver for SuperH and SH Mobile MSIOF blocks.
-- 
1.9.1


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH 1/2] spi: rspi: Relax DMA dependency and increase build coverage
  2014-07-09 21:27 [PATCH 1/2] spi: rspi: Relax DMA dependency and increase build coverage Geert Uytterhoeven
  2014-07-09 21:27 ` [PATCH 2/2] spi: sh-msiof: SPI_SH_MSIOF should depend on HAS_DMA Geert Uytterhoeven
@ 2014-07-10 10:40 ` Mark Brown
  1 sibling, 0 replies; 3+ messages in thread
From: Mark Brown @ 2014-07-10 10:40 UTC (permalink / raw)
  To: Geert Uytterhoeven; +Cc: linux-spi, linux-kernel

[-- Attachment #1: Type: text/plain, Size: 391 bytes --]

On Wed, Jul 09, 2014 at 11:27:07PM +0200, Geert Uytterhoeven wrote:
> As of commit ab116a4df4942c78c189d9b0744dd940ab9e00b9 ("dmaengine:
> shdma: fix a build failure on platforms with no DMA support"), the DMA
> filter function shdma_chan_filter() is sufficiently abstracted to allow
> building without DMA support. Hence drop the SH_DMAE_BASE dependency on
> SUPERH.

Applied both, thanks.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2014-07-10 10:41 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-07-09 21:27 [PATCH 1/2] spi: rspi: Relax DMA dependency and increase build coverage Geert Uytterhoeven
2014-07-09 21:27 ` [PATCH 2/2] spi: sh-msiof: SPI_SH_MSIOF should depend on HAS_DMA Geert Uytterhoeven
2014-07-10 10:40 ` [PATCH 1/2] spi: rspi: Relax DMA dependency and increase build coverage Mark Brown

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

Powered by JetHome