| Message ID | 20240711-loongson1-dma-v9-0-5ce8b5e85a56@gmail.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | Add support for Loongson1 APB DMA | expand |
Hi, Keguang, I accept your suggestion about the cpufreq driver naming, and now it is upstream: https://git.kernel.org/pub/scm/linux/kernel/git/vireshk/pm.git/commit/?h=cpufreq/arm/linux-next&id=ccf51454145bffd98e31cdbe54a4262473c609e2 I still hope you can accept my suggestion about the dma driver naming. I know you hope me rename LS2X_APB_DMA to LOONGSON2_APB_DMA, but as I said before, renaming an existing Kconfig option will break config files. See an example: Commit a50a3f4b6a313dc76912bd4ad3b8b4f4b4 introduce PREEMPT_RT and rename PREEMPT to PREEMPT_LL, but then commit b8d3349803ba34afda429e87a837fd95a9 rename it back because of config files broken. Huacai On Thu, Jul 11, 2024 at 6:57 PM Keguang Zhang via B4 Relay <devnull+keguang.zhang.gmail.com@kernel.org> 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@gmail.com > > Changes in v8: > - Change 'interrupts' property to an items list > - Link to v7: https://lore.kernel.org/r/20240329-loongson1-dma-v7-0-37db58608de5@gmail.com > > 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@gmail.com > > 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@gmail.com> > > >
On Fri, Jul 12, 2024 at 12:22 PM Huacai Chen <chenhuacai@kernel.org> wrote: > > Hi, Keguang, > > I accept your suggestion about the cpufreq driver naming, and now it > is upstream: > https://git.kernel.org/pub/scm/linux/kernel/git/vireshk/pm.git/commit/?h=cpufreq/arm/linux-next&id=ccf51454145bffd98e31cdbe54a4262473c609e2 > > I still hope you can accept my suggestion about the dma driver naming. > > I know you hope me rename LS2X_APB_DMA to LOONGSON2_APB_DMA, but as I > said before, renaming an existing Kconfig option will break config > files. > > See an example: > Commit a50a3f4b6a313dc76912bd4ad3b8b4f4b4 introduce PREEMPT_RT and > rename PREEMPT to PREEMPT_LL, but then commit > b8d3349803ba34afda429e87a837fd95a9 rename it back because of config > files broken. > Hi Huacai, I understand the breaking issue of the Kconfig option, so you can keep LS2X_APB_DMA. You said that you've accepted my suggestion, which means you recognize 'loongson' as the better name for the drivers. 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@kernel.org> 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@gmail.com > > > > Changes in v8: > > - Change 'interrupts' property to an items list > > - Link to v7: https://lore.kernel.org/r/20240329-loongson1-dma-v7-0-37db58608de5@gmail.com > > > > 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@gmail.com > > > > 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@gmail.com> > > > > > > -- Best regards, Keguang Zhang
On Fri, Jul 12, 2024 at 2:23 PM Keguang Zhang <keguang.zhang@gmail.com> wrote: > > On Fri, Jul 12, 2024 at 12:22 PM Huacai Chen <chenhuacai@kernel.org> wrote: > > > > Hi, Keguang, > > > > I accept your suggestion about the cpufreq driver naming, and now it > > is upstream: > > https://git.kernel.org/pub/scm/linux/kernel/git/vireshk/pm.git/commit/?h=cpufreq/arm/linux-next&id=ccf51454145bffd98e31cdbe54a4262473c609e2 > > > > I still hope you can accept my suggestion about the dma driver naming. > > > > I know you hope me rename LS2X_APB_DMA to LOONGSON2_APB_DMA, but as I > > said before, renaming an existing Kconfig option will break config > > files. > > > > See an example: > > Commit a50a3f4b6a313dc76912bd4ad3b8b4f4b4 introduce PREEMPT_RT and > > rename PREEMPT to PREEMPT_LL, but then commit > > b8d3349803ba34afda429e87a837fd95a9 rename it back because of config > > files broken. > > > Hi Huacai, > I understand the breaking issue of the Kconfig option, so you can keep > LS2X_APB_DMA. LS2X_APB_DMA with loongson2-apb-dma.c? Even if I accept this, can you accept LS1X_APB_DMA with loongson1-apb-dma.c? > 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. 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@kernel.org> 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@gmail.com > > > > > > Changes in v8: > > > - Change 'interrupts' property to an items list > > > - Link to v7: https://lore.kernel.org/r/20240329-loongson1-dma-v7-0-37db58608de5@gmail.com > > > > > > 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@gmail.com > > > > > > 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@gmail.com> > > > > > > > > > > > > > -- > Best regards, > > Keguang Zhang
On Mon, Jul 15, 2024 at 2:39 PM Huacai Chen <chenhuacai@kernel.org> wrote: > > On Fri, Jul 12, 2024 at 2:23 PM Keguang Zhang <keguang.zhang@gmail.com> wrote: > > > > On Fri, Jul 12, 2024 at 12:22 PM Huacai Chen <chenhuacai@kernel.org> wrote: > > > > > > Hi, Keguang, > > > > > > I accept your suggestion about the cpufreq driver naming, and now it > > > is upstream: > > > https://git.kernel.org/pub/scm/linux/kernel/git/vireshk/pm.git/commit/?h=cpufreq/arm/linux-next&id=ccf51454145bffd98e31cdbe54a4262473c609e2 > > > > > > I still hope you can accept my suggestion about the dma driver naming. > > > > > > I know you hope me rename LS2X_APB_DMA to LOONGSON2_APB_DMA, but as I > > > said before, renaming an existing Kconfig option will break config > > > files. > > > > > > See an example: > > > Commit a50a3f4b6a313dc76912bd4ad3b8b4f4b4 introduce PREEMPT_RT and > > > rename PREEMPT to PREEMPT_LL, but then commit > > > b8d3349803ba34afda429e87a837fd95a9 rename it back because of config > > > files broken. > > > > > Hi Huacai, > > I understand the breaking issue of the Kconfig option, so you can keep > > LS2X_APB_DMA. > LS2X_APB_DMA with loongson2-apb-dma.c? Even if I accept this, can you > accept LS1X_APB_DMA with loongson1-apb-dma.c? > > > 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. > Sorry. The naming 'Loongson1' is the real consistency that I need to maintain. Thanks! > > > 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@kernel.org> 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@gmail.com > > > > > > > > Changes in v8: > > > > - Change 'interrupts' property to an items list > > > > - Link to v7: https://lore.kernel.org/r/20240329-loongson1-dma-v7-0-37db58608de5@gmail.com > > > > > > > > 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@gmail.com > > > > > > > > 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@gmail.com> > > > > > > > > > > > > > > > > > > > > -- > > Best regards, > > > > Keguang Zhang
在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@kernel.org> 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@gmail.com >> > > >> > > Changes in v8: >> > > - Change 'interrupts' property to an items list >> > > - Link to v7: https://lore.kernel.org/r/20240329-loongson1-dma-v7-0-37db58608de5@gmail.com >> > > >> > > 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@gmail.com >> > > >> > > 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@gmail.com> >> > > >> > > >> > > >> >> >> >> -- >> Best regards, >> >> Keguang Zhang
On Mon, Jul 15, 2024 at 3:00 PM Jiaxun Yang <jiaxun.yang@flygoat.com> wrote: > > > > 在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. Renaming still breaks config files. Huacai > > 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@kernel.org> 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@gmail.com > >> > > > >> > > Changes in v8: > >> > > - Change 'interrupts' property to an items list > >> > > - Link to v7: https://lore.kernel.org/r/20240329-loongson1-dma-v7-0-37db58608de5@gmail.com > >> > > > >> > > 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@gmail.com > >> > > > >> > > 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@gmail.com> > >> > > > >> > > > >> > > > >> > >> > >> > >> -- > >> Best regards, > >> > >> Keguang Zhang > > -- > - Jiaxun
在2024年7月16日七月 下午5:40，Huacai Chen写道： > On Mon, Jul 15, 2024 at 3:00 PM Jiaxun Yang <jiaxun.yang@flygoat.com> wrote: >> >> >> >> 在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. > Renaming still breaks config files. This is trival with treewide sed :-) Thanks - Jiaxun
On Tue, Jul 16, 2024 at 9:12 PM Jiaxun Yang <jiaxun.yang@flygoat.com> wrote: > > > > 在2024年7月16日七月 下午5:40，Huacai Chen写道： > > On Mon, Jul 15, 2024 at 3:00 PM Jiaxun Yang <jiaxun.yang@flygoat.com> wrote: > >> > >> > >> > >> 在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. > > Renaming still breaks config files. > > This is trival with treewide sed :-) Please read the commit message of b8d3349803ba34afda429e87a837fd95a carefully. Huacai > > Thanks > - Jiaxun > > -- > - Jiaxun
在2024年7月17日七月 下午9:06，Huacai Chen写道： > On Tue, Jul 16, 2024 at 9:12 PM Jiaxun Yang <jiaxun.yang@flygoat.com> wrote: >> >> >> >> 在2024年7月16日七月 下午5:40，Huacai Chen写道： >> > On Mon, Jul 15, 2024 at 3:00 PM Jiaxun Yang <jiaxun.yang@flygoat.com> wrote: >> >> >> >> >> >> >> >> 在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. >> > Renaming still breaks config files. >> >> This is trival with treewide sed :-) > Please read the commit message of b8d3349803ba34afda429e87a837fd95a carefully. We don't have 114 defconfigs don't we? Those symbols are not frequently specified by down stream users either. I think Keguang had tried his best on resolving all reasonable comments. Naming is a matter of preference after all, I think we should give Keguang some respect here. Thanks - Jiaxun > > Huacai > >> >> Thanks >> - Jiaxun >> >> -- >> - Jiaxun
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@gmail.com Changes in v8: - Change 'interrupts' property to an items list - Link to v7: https://lore.kernel.org/r/20240329-loongson1-dma-v7-0-37db58608de5@gmail.com 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@gmail.com 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,