Re: [PATCH RESEND v9 0/2] Add support for Loongson1 APB DMA

From: Jiaxun Yang
Date: Mon Jul 15 2024 - 03:00:51 EST




在2024年7月15日七月 下午2:39,Huacai Chen写道:
[...]
>
>> You said that you've accepted my suggestion, which means you recognize
>> 'loongson' as the better name for the drivers.
> No, I don't think so, this is just a compromise to keep consistency.

Folks, can we settle on this topic?

Is this naming really important? As long as people can read actual chip name from
kernel code & documents, I think both are acceptable.

I suggest let this patch go as is. And if anyone want to unify the naming, they can
propose a treewide patch.

Otherwise, we are going nowhere.

Thanks
- Jiaxun

>
>
>
> Huacai
>
>> Moreover, Loongson1 and Loongson2 belong to different SoC series.
>> To be honest, I can't see why Loongson1 APB DMA should give up this
>> intuitive and comprehensible naming.
>> Thanks for your review!
>> >
>> > Huacai
>> >
>> > On Thu, Jul 11, 2024 at 6:57 PM Keguang Zhang via B4 Relay
>> > <devnull+keguang.zhang.gmail.com@xxxxxxxxxx> wrote:
>> > >
>> > > Add the driver and dt-binding document for Loongson1 APB DMA.
>> > >
>> > > ---
>> > > Changes in v9:
>> > > - Fix all the errors and warnings when building with W=1 and C=1
>> > > - Link to v8: https://lore.kernel.org/r/20240607-loongson1-dma-v8-0-f9992d257250@xxxxxxxxx
>> > >
>> > > Changes in v8:
>> > > - Change 'interrupts' property to an items list
>> > > - Link to v7: https://lore.kernel.org/r/20240329-loongson1-dma-v7-0-37db58608de5@xxxxxxxxx
>> > >
>> > > Changes in v7:
>> > > - Change the comptible to 'loongson,ls1*-apbdma' (suggested by Huacai Chen)
>> > > - Update the title and description part accordingly
>> > > - Rename the file to loongson,ls1b-apbdma.yaml
>> > > - Add a compatible string for LS1A
>> > > - Delete minItems of 'interrupts'
>> > > - Change patterns of 'interrupt-names' to const
>> > > - Rename the file to loongson1-apb-dma.c to keep the consistency
>> > > - Update Kconfig and Makefile accordingly
>> > > - Link to v6: https://lore.kernel.org/r/20240316-loongson1-dma-v6-0-90de2c3cc928@xxxxxxxxx
>> > >
>> > > Changes in 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
>> > >
>> > > Changes in 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
>> > >
>> > > Changes in 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.
>> > >
>> > > Changes in v3:
>> > > - Rename ls1x_dma_filter_fn to ls1x_dma_filter.
>> > >
>> > > Changes in 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 APB DMA
>> > > dmaengine: Loongson1: Add Loongson-1 APB DMA driver
>> > >
>> > > .../bindings/dma/loongson,ls1b-apbdma.yaml | 67 +++
>> > > drivers/dma/Kconfig | 9 +
>> > > drivers/dma/Makefile | 1 +
>> > > drivers/dma/loongson1-apb-dma.c | 665 +++++++++++++++++++++
>> > > 4 files changed, 742 insertions(+)
>> > > ---
>> > > base-commit: d35b2284e966c0bef3e2182a5c5ea02177dd32e4
>> > > change-id: 20231120-loongson1-dma-163afe5708b9
>> > >
>> > > Best regards,
>> > > --
>> > > Keguang Zhang <keguang.zhang@xxxxxxxxx>
>> > >
>> > >
>> > >
>>
>>
>>
>> --
>> Best regards,
>>
>> Keguang Zhang

--
- Jiaxun