| Message ID | 20230531090141.23546-1-zong.li@sifive.com (mailing list archive) |
|---|---|
| State | Handled Elsewhere |
| Headers | show |
| Series | [v3] dmaengine: xilinx: dma: remove arch dependency | expand |
| Context | Check | Description |
|---|---|---|
| conchuod/cover_letter | success | Single patches do not need cover letters |
| conchuod/tree_selection | success | Guessed tree name to be for-next at HEAD ac9a78681b92 |
| conchuod/fixes_present | success | Fixes tag not required for -next series |
| conchuod/maintainers_pattern | success | MAINTAINERS pattern errors before the patch: 6 and now 6 |
| conchuod/verify_signedoff | success | Signed-off-by tag matches author and committer |
| conchuod/kdoc | success | Errors and warnings before: 0 this patch: 0 |
| conchuod/build_rv64_clang_allmodconfig | success | Errors and warnings before: 14 this patch: 14 |
| conchuod/module_param | success | Was 0 now: 0 |
| conchuod/build_rv64_gcc_allmodconfig | success | Errors and warnings before: 28 this patch: 28 |
| conchuod/build_rv32_defconfig | success | Build OK |
| conchuod/dtb_warn_rv64 | success | Errors and warnings before: 3 this patch: 3 |
| conchuod/header_inline | success | No static functions without inline keyword in header files |
| conchuod/checkpatch | warning | WARNING: Reported-by: should be immediately followed by Closes: with a URL to the report |
| conchuod/build_rv64_nommu_k210_defconfig | success | Build OK |
| conchuod/verify_fixes | success | No Fixes tag |
| conchuod/build_rv64_nommu_virt_defconfig | success | Build OK |
On Wed, May 31, 2023 at 5:01 PM Zong Li <zong.li@sifive.com> wrote: > > As following patches, xilinx dma is also now architecture agnostic, > and it can be compiled for several architectures. We have verified the > CDMA on RISC-V platform, let's remove the ARCH dependency list instead > of adding new ARCH. > > To avoid breaking the s390 build, add a dependency on HAS_IOMEM. > > 'e8b6c54f6d57 ("net: xilinx: temac: Relax Kconfig dependencies")' > 'd7eaf962a90b ("net: axienet: In kconfig remove arch dependency for axi_emac")' > > Signed-off-by: Zong Li <zong.li@sifive.com> > Acked-by: Palmer Dabbelt <palmer@rivosinc.com> > Suggested-by: Radhey Shyam Pandey <radhey.shyam.pandey@amd.com> > Reported-by: kernel test robot <lkp@intel.com> > --- > > Changes in v3: > - Add a dependency on HAS_IOMEM to avoid breaking the s390 build > > Changes in v2: > - Remove ARCH dependency list instead of adding new ARCH > > drivers/dma/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/dma/Kconfig b/drivers/dma/Kconfig > index f5f422f9b850..fd5a94e67163 100644 > --- a/drivers/dma/Kconfig > +++ b/drivers/dma/Kconfig > @@ -696,7 +696,7 @@ config XGENE_DMA > > config XILINX_DMA > tristate "Xilinx AXI DMAS Engine" > - depends on (ARCH_ZYNQ || MICROBLAZE || ARM64) > + depends on HAS_IOMEM > select DMA_ENGINE > help > Enable support for Xilinx AXI VDMA Soft IP. > -- > 2.17.1 > Hi Vinod, Linux 6.5-rc1 is opening, could I know if this patch is good for you? Would it be considered in this merge window? Thanks.
On Mon, Jun 26, 2023 at 10:54 AM Zong Li <zong.li@sifive.com> wrote: > > On Wed, May 31, 2023 at 5:01 PM Zong Li <zong.li@sifive.com> wrote: > > > > As following patches, xilinx dma is also now architecture agnostic, > > and it can be compiled for several architectures. We have verified the > > CDMA on RISC-V platform, let's remove the ARCH dependency list instead > > of adding new ARCH. > > > > To avoid breaking the s390 build, add a dependency on HAS_IOMEM. > > > > 'e8b6c54f6d57 ("net: xilinx: temac: Relax Kconfig dependencies")' > > 'd7eaf962a90b ("net: axienet: In kconfig remove arch dependency for axi_emac")' > > > > Signed-off-by: Zong Li <zong.li@sifive.com> > > Acked-by: Palmer Dabbelt <palmer@rivosinc.com> > > Suggested-by: Radhey Shyam Pandey <radhey.shyam.pandey@amd.com> > > Reported-by: kernel test robot <lkp@intel.com> > > --- > > > > Changes in v3: > > - Add a dependency on HAS_IOMEM to avoid breaking the s390 build > > > > Changes in v2: > > - Remove ARCH dependency list instead of adding new ARCH > > > > drivers/dma/Kconfig | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/dma/Kconfig b/drivers/dma/Kconfig > > index f5f422f9b850..fd5a94e67163 100644 > > --- a/drivers/dma/Kconfig > > +++ b/drivers/dma/Kconfig > > @@ -696,7 +696,7 @@ config XGENE_DMA > > > > config XILINX_DMA > > tristate "Xilinx AXI DMAS Engine" > > - depends on (ARCH_ZYNQ || MICROBLAZE || ARM64) > > + depends on HAS_IOMEM > > select DMA_ENGINE > > help > > Enable support for Xilinx AXI VDMA Soft IP. > > -- > > 2.17.1 > > > > Hi Vinod, > Linux 6.5-rc1 is opening, could I know if this patch is good for you? > Would it be considered in this merge window? > Thanks. Hi Vinod, It seems that we are transitioning to Linux v6.5-rc2. I'm uncertain whether you consider this a new feature or a fix. Is there a possibility of incorporating it during this cycle? Thank you.
On Wed, 31 May 2023 09:01:41 +0000, Zong Li wrote: > As following patches, xilinx dma is also now architecture agnostic, > and it can be compiled for several architectures. We have verified the > CDMA on RISC-V platform, let's remove the ARCH dependency list instead > of adding new ARCH. > > To avoid breaking the s390 build, add a dependency on HAS_IOMEM. > > [...] Applied, thanks! [1/1] dmaengine: xilinx: dma: remove arch dependency commit: 9945a80fe40886f2d0655c0c3cdee95aaecafb87 Best regards,
diff --git a/drivers/dma/Kconfig b/drivers/dma/Kconfig index f5f422f9b850..fd5a94e67163 100644 --- a/drivers/dma/Kconfig +++ b/drivers/dma/Kconfig @@ -696,7 +696,7 @@ config XGENE_DMA config XILINX_DMA tristate "Xilinx AXI DMAS Engine" - depends on (ARCH_ZYNQ || MICROBLAZE || ARM64) + depends on HAS_IOMEM select DMA_ENGINE help Enable support for Xilinx AXI VDMA Soft IP.
As following patches, xilinx dma is also now architecture agnostic, and it can be compiled for several architectures. We have verified the CDMA on RISC-V platform, let's remove the ARCH dependency list instead of adding new ARCH. To avoid breaking the s390 build, add a dependency on HAS_IOMEM. 'e8b6c54f6d57 ("net: xilinx: temac: Relax Kconfig dependencies")' 'd7eaf962a90b ("net: axienet: In kconfig remove arch dependency for axi_emac")' Signed-off-by: Zong Li <zong.li@sifive.com> Acked-by: Palmer Dabbelt <palmer@rivosinc.com> Suggested-by: Radhey Shyam Pandey <radhey.shyam.pandey@amd.com> Reported-by: kernel test robot <lkp@intel.com> --- Changes in v3: - Add a dependency on HAS_IOMEM to avoid breaking the s390 build Changes in v2: - Remove ARCH dependency list instead of adding new ARCH drivers/dma/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)