linux-kernel - Re: [PATCH v3] dmaengine: xilinx: dma: remove arch dependency

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CANXhq0rVred1YMcV5449yYt+MhOu2NUx9Oa-1xJqbS=o_1Qb6g@mail.gmail.com>
Date:   Mon, 26 Jun 2023 10:54:38 +0800
From:   Zong Li <zong.li@...ive.com>
To:     vkoul@...nel.org, palmer@...belt.com, paul.walmsley@...ive.com,
        radhey.shyam.pandey@....com, dmaengine@...r.kernel.org,
        linux-riscv@...ts.infradead.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v3] dmaengine: xilinx: dma: remove arch dependency

On Wed, May 31, 2023 at 5:01 PM Zong Li <zong.li@...ive.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@...ive.com>
> Acked-by: Palmer Dabbelt <palmer@...osinc.com>
> Suggested-by: Radhey Shyam Pandey <radhey.shyam.pandey@....com>
> Reported-by: kernel test robot <lkp@...el.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.