From mboxrd@z Thu Jan 1 00:00:00 1970 Date: Tue, 3 Sep 2019 14:07:42 -0700 From: Bjorn Andersson Subject: Re: [PATCH 3/4] remoteproc: don't allow modular build Message-ID: <20190903210739.GW6167@minitux> References: <20190902200746.16185-1-hch@lst.de> <20190902200746.16185-4-hch@lst.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20190902200746.16185-4-hch@lst.de> To: Christoph Hellwig Cc: iommu@lists.linux-foundation.org, Loic Pallardy , Ohad Ben-Cohen , linux-remoteproc@vger.kernel.org, linux-kernel@vger.kernel.org List-ID: On Mon 02 Sep 13:07 PDT 2019, Christoph Hellwig wrote: > Remoteproc started using dma_declare_coherent_memory recently, which is > a bad idea from drivers, and the maintainers agreed to fix that. But > until that is fixed only allow building the driver built in so that we > can remove the dma_declare_coherent_memory export and prevent other > drivers from "accidentally" using it like remoteproc. Note that the > driver would also leak the declared coherent memory on unload if it > actually was built as a module at the moment. > > Signed-off-by: Christoph Hellwig Reviewed-by: Bjorn Andersson Please pick this together with the other patches. Regards, Bjorn > --- > drivers/remoteproc/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/remoteproc/Kconfig b/drivers/remoteproc/Kconfig > index 28ed306982f7..94afdde4bc9f 100644 > --- a/drivers/remoteproc/Kconfig > +++ b/drivers/remoteproc/Kconfig > @@ -2,7 +2,7 @@ > menu "Remoteproc drivers" > > config REMOTEPROC > - tristate "Support for Remote Processor subsystem" > + bool "Support for Remote Processor subsystem" > depends on HAS_DMA > select CRC32 > select FW_LOADER > -- > 2.20.1 >