linux-kernel - Re: [PATCH v6 0/2] Add support for Loongson1 DMA

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: <CAAhV-H5TR=y_AmbF6QMJmoS0BhfB=K7forMg0-b2YWm7trktjA@mail.gmail.com>
Date: Mon, 18 Mar 2024 15:31:59 +0800
From: Huacai Chen <chenhuacai@...nel.org>
To: Keguang Zhang <keguang.zhang@...il.com>
Cc: Vinod Koul <vkoul@...nel.org>, Rob Herring <robh@...nel.org>, 
	Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>, Conor Dooley <conor+dt@...nel.org>, 
	linux-mips@...r.kernel.org, dmaengine@...r.kernel.org, 
	devicetree@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v6 0/2] Add support for Loongson1 DMA

On Mon, Mar 18, 2024 at 10:08 AM Keguang Zhang <keguang.zhang@...ilcom> wrote:
>
> Hi Huacai,
>
> > Hi, Keguang,
> >
> > Sorry for the late reply, there is already a ls2x-apb-dma driver, I'm
> > not sure but can they share the same code base? If not, can rename
> > this driver to ls1x-apb-dma for consistency?
>
> There are some differences between ls1x DMA and ls2x DMA, such as
> registers and DMA descriptors.
> I will rename it to ls1x-apb-dma.
OK, please also rename the yaml file to keep consistency.

Huacai

> Thanks!
>
> >
> > Huacai
> >
> > On Sat, Mar 16, 2024 at 7:34 PM Keguang Zhang via B4 Relay
> > <devnull+keguang.zhang.gmail.com@...nel.org> wrote:
> > >
> > > Add the driver and dt-binding document for Loongson1 DMA.
> > >
> > > Changelog
> > > V5 -> V6:
> > >    Change the compatible to the fallback
> > >    Implement .device_prep_dma_cyclic for Loongson1 sound driver,
> > >    as well as .device_pause and .device_resume.
> > >    Set the limitation LS1X_DMA_MAX_DESC and put all descriptors
> > >    into one page to save memory
> > >    Move dma_pool_zalloc() into ls1x_dma_alloc_desc()
> > >    Drop dma_slave_config structure
> > >    Use .remove_new instead of .remove
> > >    Use KBUILD_MODNAME for the driver name
> > >    Improve the debug information
> > >    Some minor fixes
> > > V4 -> V5:
> > >    Add the dt-binding document
> > >    Add DT support
> > >    Use DT information instead of platform data
> > >    Use chan_id of struct dma_chan instead of own id
> > >    Use of_dma_xlate_by_chan_id() instead of ls1x_dma_filter()
> > >    Update the author information to my official name
> > > V3 -> V4:
> > >    Use dma_slave_map to find the proper channel.
> > >    Explicitly call devm_request_irq() and tasklet_kill().
> > >    Fix namespace issue.
> > >    Some minor fixes and cleanups.
> > > V2 -> V3:
> > >    Rename ls1x_dma_filter_fn to ls1x_dma_filter.
> > > V1 -> V2:
> > >    Change the config from 'DMA_LOONGSON1' to 'LOONGSON1_DMA',
> > >    and rearrange it in alphabetical order in Kconfig and Makefile.
> > >    Fix comment style.
> > >
> > > Keguang Zhang (2):
> > >   dt-bindings: dma: Add Loongson-1 DMA
> > >   dmaengine: Loongson1: Add Loongson1 dmaengine driver
> > >
> > >  .../bindings/dma/loongson,ls1x-dma.yaml       |  64 +++
> > >  drivers/dma/Kconfig                           |   9 +
> > >  drivers/dma/Makefile                          |   1 +
> > >  drivers/dma/loongson1-dma.c                   | 492 ++++++++++++++++++
> > >  4 files changed, 566 insertions(+)
> > >  create mode 100644 Documentation/devicetree/bindings/dma/loongson,ls1x-dma.yaml
> > >  create mode 100644 drivers/dma/loongson1-dma.c
> > >
> > > --
> > > 2.39.2
> > >
> > > base-commit: 719136e5c24768ebdf80b9daa53facebbdd377c3
> > > ---
> > > Keguang Zhang (2):
> > >       dt-bindings: dma: Add Loongson-1 DMA
> > >       dmaengine: Loongson1: Add Loongson1 dmaengine driver
> > >
> > >  .../devicetree/bindings/dma/loongson,ls1x-dma.yaml |  66 ++
> > >  drivers/dma/Kconfig                                |   9 +
> > >  drivers/dma/Makefile                               |   1 +
> > >  drivers/dma/loongson1-dma.c                        | 665 +++++++++++++++++++++
> > >  4 files changed, 741 insertions(+)
> > > ---
> > > base-commit: a1e7655b77e3391b58ac28256789ea45b1685abb
> > > change-id: 20231120-loongson1-dma-163afe5708b9
> > >
> > > Best regards,
> > > --
> > > Keguang Zhang <keguang.zhang@...il.com>
> > >
> > >
>
>
>
> --
> Best regards,
>
> Keguang Zhang