* [PATCH 1/2] spi: rspi: Relax DMA dependency and increase build coverage
@ 2014-07-09 21:27 Geert Uytterhoeven
[not found] ` <1404941228-9922-1-git-send-email-geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
0 siblings, 1 reply; 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 related [flat|nested] 3+ messages in thread[parent not found: <1404941228-9922-1-git-send-email-geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>]
* [PATCH 2/2] spi: sh-msiof: SPI_SH_MSIOF should depend on HAS_DMA [not found] ` <1404941228-9922-1-git-send-email-geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org> @ 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-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA, 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-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org> --- 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 -- To unsubscribe from this list: send the line "unsubscribe linux-spi" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH 1/2] spi: rspi: Relax DMA dependency and increase build coverage [not found] ` <1404941228-9922-1-git-send-email-geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org> 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-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA [-- 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:40 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
[not found] ` <1404941228-9922-1-git-send-email-geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
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; as well as URLs for NNTP newsgroup(s).