Thread (50 messages) 50 messages, 16 authors, 2018-04-17

Re: [PATCH v2 06/21] fpga: Remove depends on HAS_DMA in case of platform dependency

From: Geert Uytterhoeven <hidden>
Date: 2018-03-20 10:04:09
Also in: alsa-devel, linux-crypto, linux-fpga, linux-i2c, linux-ide, linux-iio, linux-iommu, linux-mmc, linux-remoteproc, linux-scsi, linux-spi, lkml, netdev

Hi Alan,

On Mon, Mar 19, 2018 at 5:06 PM, Alan Tull [off-list ref] wrote:
On Fri, Mar 16, 2018 at 8:51 AM, Geert Uytterhoeven
[off-list ref] wrote:
This essentially removes this commit

commit 1c8cb409491403036919dd1c6b45013dc8835a44
Author: Sudip Mukherjee [off-list ref]
Date:   Wed Aug 3 13:45:46 2016 -0700

    drivers/fpga/Kconfig: fix build failure

    While building m32r allmodconfig the build is failing with the error:

      ERROR: "bad_dma_ops" [drivers/fpga/zynq-fpga.ko] undefined!

    Xilinx Zynq FPGA is using DMA but there was no dependency while
    building.

    Link: http://lkml.kernel.org/r/1464346526-13913-1-git-send-email-sudipm.mukherjee-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org
    Signed-off-by: Sudip Mukherjee [off-list ref]
    Acked-by: Moritz Fischer [off-list ref]
    Cc: Alan Tull [off-list ref]
    Signed-off-by: Andrew Morton [off-list ref]
    Signed-off-by: Linus Torvalds [off-list ref]
Yes it does. The major change is that the first (core) series introduces
all needed dummies to do successful compile-testing on NO_DMA=y platforms.
quoted
Remove dependencies on HAS_DMA where a Kconfig symbol depends on another
symbol that implies HAS_DMA, and, optionally, on "|| COMPILE_TEST".
In most cases this other symbol is an architecture or platform specific
symbol, or PCI.

Generic symbols and drivers without platform dependencies keep their
dependencies on HAS_DMA, to prevent compiling subsystems or drivers that
cannot work anyway.

This simplifies the dependencies, and allows to improve compile-testing.

Signed-off-by: Geert Uytterhoeven <redacted>
Reviewed-by: Mark Brown <redacted>
Acked-by: Robin Murphy <redacted>
Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert-Td1EMuHUCqxL1ZNQvxDV9g@public.gmane.org

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
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help