Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
MIME-Version: 1.0
References: <20230531090141.23546-1-zong.li@sifive.com> <CANXhq0rVred1YMcV5449yYt+MhOu2NUx9Oa-1xJqbS=o_1Qb6g@mail.gmail.com>
In-Reply-To: <CANXhq0rVred1YMcV5449yYt+MhOu2NUx9Oa-1xJqbS=o_1Qb6g@mail.gmail.com>
From:   Zong Li <zong.li@sifive.com>
Date:   Mon, 10 Jul 2023 10:31:27 +0800
Message-ID: <CANXhq0q+tkM=_C5gJq1Hv-uMx_ihiLLYyYgvJNAAMA=9SRurAg@mail.gmail.com>
Subject: Re: [PATCH v3] dmaengine: xilinx: dma: remove arch dependency
To:     vkoul@kernel.org, palmer@dabbelt.com, paul.walmsley@sifive.com,
        radhey.shyam.pandey@amd.com, dmaengine@vger.kernel.org,
        linux-riscv@lists.infradead.org, linux-kernel@vger.kernel.org
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

On Mon, Jun 26, 2023 at 10:54=E2=80=AFAM Zong Li <zong.li@sifive.com> wrote=
:
>
> On Wed, May 31, 2023 at 5:01=E2=80=AFPM Zong Li <zong.li@sifive.com> wrot=
e:
> >
> > 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.