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: <CANXhq0q+tkM=_C5gJq1Hv-uMx_ihiLLYyYgvJNAAMA=9SRurAg@mail.gmail.com>
Date:   Mon, 10 Jul 2023 10:31:27 +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 Mon, Jun 26, 2023 at 10:54 AM Zong Li <zong.li@...ive.com> wrote:
>
> 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.

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.