| Message ID | 20240617-ep93xx-v10-0-662e640ed811@maquefel.me |
|---|---|
| Headers | show |
| Series | ep93xx device tree conversion | expand |
On Mon, Jun 17, 2024 at 11:38 AM Nikita Shubin via B4 Relay <devnull+nikita.shubin.maquefel.me@kernel.org> wrote: > > The goal is to recieve ACKs for all patches in series to merge it via Arnd branch. 'receive' > Unfortunately, CLK subsystem suddenly went silent on clk portion of series V2 reroll, > tried to ping them for about a month but no luck. > > Link: https://lore.kernel.org/r/20240408-ep93xx-clk-v2-1-adcd68c13753@maquefel.me > > Some changes since last version (v9) - see "Changes in v10", mostly > cosmetic. ... > Patches should be formated with '--histogram' 'formatted' ... > Changes in v10: > > Reordered SoB tags to make sure they appear before Rb and Acked tags. This is not required. The importance is only the order of SoBs themselves. If they are interleaved with other tags, it's fine. ... Hopefully to see this series being eventually applied soon. Arnd? (Do we have all necessary subsystem maintainers' tags, btw?)
On Mon, 17 Jun 2024 12:36:34 +0300 Nikita Shubin via B4 Relay wrote:
> The goal is to recieve ACKs for all patches in series to merge it via Arnd branch.
Why? The usual process is for every subsystem to accept the relevant
patches, and then they converge during the merge window.
Hello Andy! On Mon, 2024-06-17 at 12:58 +0200, Andy Shevchenko wrote: > On Mon, Jun 17, 2024 at 11:38 AM Nikita Shubin via B4 Relay > <devnull+nikita.shubin.maquefel.me@kernel.org> wrote: > > > > The goal is to recieve ACKs for all patches in series to merge it > > via Arnd branch. > > 'receive' > > > Unfortunately, CLK subsystem suddenly went silent on clk portion of > > series V2 reroll, > > tried to ping them for about a month but no luck. > > > > Link: > > https://lore.kernel.org/r/20240408-ep93xx-clk-v2-1-adcd68c13753@maquefel.me > > > > Some changes since last version (v9) - see "Changes in v10", mostly > > cosmetic. > > ... > > > Patches should be formated with '--histogram' > > 'formatted' > > ... > > > Changes in v10: > > > > Reordered SoB tags to make sure they appear before Rb and Acked > > tags. > > This is not required. The importance is only the order of SoBs > themselves. If they are interleaved with other tags, it's fine. Ah - ok. Just saw someone was complaining about b4 reordering them. > > ... > > > Hopefully to see this series being eventually applied soon. > Arnd? (Do we have all necessary subsystem maintainers' tags, btw?) > > As i see from my perspective only three left: Clk subsystem: - clk: ep93xx: add DT support for Cirrus EP93xx DMA subsystem (but the only request from Vinod, as far as i remember, was fixing commits titles): - dmaengine: cirrus: Convert to DT for Cirrus EP93xx - dmaengine: cirrus: remove platform code Beside that tags missing on platform code removal (which can be Acked by Arnd himself i believe) and dtsi/dts files (same ?).
On Tue, 2024-06-18 at 07:33 -0700, Jakub Kicinski wrote: > On Mon, 17 Jun 2024 12:36:34 +0300 Nikita Shubin via B4 Relay wrote: > > The goal is to recieve ACKs for all patches in series to merge it > > via Arnd branch. > > Why? The usual process is for every subsystem to accept the relevant > patches, and then they converge during the merge window. It was decided from the very beginning of these series, mostly because it's a full conversion of platform code to DT and it seemed not convenient to maintain compatibility with both platform and DT. Generally i think it's too late to ask such a question, when just a few patches left.
On Tue, 18 Jun 2024 19:33:49 +0300 Nikita Shubin wrote: > On Tue, 2024-06-18 at 07:33 -0700, Jakub Kicinski wrote: > > On Mon, 17 Jun 2024 12:36:34 +0300 Nikita Shubin via B4 Relay wrote: > > > The goal is to recieve ACKs for all patches in series to merge it > > > via Arnd branch. > > > > Why? The usual process is for every subsystem to accept the relevant > > patches, and then they converge during the merge window. > > It was decided from the very beginning of these series, mostly because > it's a full conversion of platform code to DT and it seemed not > convenient to maintain compatibility with both platform and DT. > > Generally i think it's too late to ask such a question, when just a few > patches left. Put the relevant information in the cover letter. Justification why you can't follow normal merging rules is very relevant.
On Tue, 2024-06-18 at 19:20 +0300, Nikita Shubin wrote: > Hello Andy! > On Mon, 2024-06-17 at 12:58 +0200, Andy Shevchenko wrote: > > On Mon, Jun 17, 2024 at 11:38 AM Nikita Shubin via B4 Relay > > <devnull+nikita.shubin.maquefel.me@kernel.org> wrote: > > > > > > The goal is to recieve ACKs for all patches in series to merge it > > > via Arnd branch. > > > > 'receive' > > > > > Unfortunately, CLK subsystem suddenly went silent on clk portion > > > of > > > series V2 reroll, > > > tried to ping them for about a month but no luck. > > > > > > Link: > > > https://lore.kernel.org/r/20240408-ep93xx-clk-v2-1-adcd68c13753@maquefel.me > > > > > > Some changes since last version (v9) - see "Changes in v10", > > > mostly > > > cosmetic. > > > > ... > > > > > Patches should be formated with '--histogram' > > > > 'formatted' > > > > ... > > > > > Changes in v10: > > > > > > Reordered SoB tags to make sure they appear before Rb and Acked > > > tags. > > > > This is not required. The importance is only the order of SoBs > > themselves. If they are interleaved with other tags, it's fine. > > Ah - ok. Just saw someone was complaining about b4 reordering them. > > > > > ... > > > > > > Hopefully to see this series being eventually applied soon. > > Arnd? (Do we have all necessary subsystem maintainers' tags, btw?) > > > > > > As i see from my perspective only three left: > > Clk subsystem: > > - clk: ep93xx: add DT support for Cirrus EP93xx > > DMA subsystem (but the only request from Vinod, as far as i remember, > was fixing commits titles): > > - dmaengine: cirrus: Convert to DT for Cirrus EP93xx > - dmaengine: cirrus: remove platform code > > Beside that tags missing on platform code removal (which can be Acked > by Arnd himself i believe) and dtsi/dts files (same ?). Vinod acked the above two patches: https://lore.kernel.org/all/ZnkIp8bOcZK3yVKP@matsya/ https://lore.kernel.org/all/ZnkImp8BtTdxl7O3@matsya/ so only: - clk: ep93xx: add DT support for Cirrus EP93xx https://lore.kernel.org/all/20240617-ep93xx-v10-3-662e640ed811@maquefel.me/ left. Hope Stephen will find some time for this one.
Hello, On Thu, Jun 27, 2024 at 11:29:44AM +0300, Nikita Shubin wrote: > On Tue, 2024-06-18 at 19:20 +0300, Nikita Shubin wrote: > > Hello Andy! > > On Mon, 2024-06-17 at 12:58 +0200, Andy Shevchenko wrote: > > > On Mon, Jun 17, 2024 at 11:38 AM Nikita Shubin via B4 Relay > > > <devnull+nikita.shubin.maquefel.me@kernel.org> wrote: > > > > > > > > The goal is to recieve ACKs for all patches in series to merge it > > > > via Arnd branch. > > > > > > 'receive' > > > > > > > Unfortunately, CLK subsystem suddenly went silent on clk portion > > > > of > > > > series V2 reroll, > > > > tried to ping them for about a month but no luck. > > > > > > > > Link: > > > > https://lore.kernel.org/r/20240408-ep93xx-clk-v2-1-adcd68c13753@maquefel.me > > > > > > > > Some changes since last version (v9) - see "Changes in v10", > > > > mostly > > > > cosmetic. > > > > > > ... > > > > > > > Patches should be formated with '--histogram' > > > > > > 'formatted' > > > > > > ... > > > > > > > Changes in v10: > > > > > > > > Reordered SoB tags to make sure they appear before Rb and Acked > > > > tags. > > > > > > This is not required. The importance is only the order of SoBs > > > themselves. If they are interleaved with other tags, it's fine. > > > > Ah - ok. Just saw someone was complaining about b4 reordering them. > > > > > > > > ... > > > > > > > > > Hopefully to see this series being eventually applied soon. > > > Arnd? (Do we have all necessary subsystem maintainers' tags, btw?) > > > > > > > > > > As i see from my perspective only three left: > > > > Clk subsystem: > > > > - clk: ep93xx: add DT support for Cirrus EP93xx > > > > DMA subsystem (but the only request from Vinod, as far as i remember, > > was fixing commits titles): > > > > - dmaengine: cirrus: Convert to DT for Cirrus EP93xx > > - dmaengine: cirrus: remove platform code > > > > Beside that tags missing on platform code removal (which can be Acked > > by Arnd himself i believe) and dtsi/dts files (same ?). > > Vinod acked the above two patches: > > https://lore.kernel.org/all/ZnkIp8bOcZK3yVKP@matsya/ > https://lore.kernel.org/all/ZnkImp8BtTdxl7O3@matsya/ > > so only: > > - clk: ep93xx: add DT support for Cirrus EP93xx > > https://lore.kernel.org/all/20240617-ep93xx-v10-3-662e640ed811@maquefel.me/ > > left. > > Hope Stephen will find some time for this one. As we're approaching the merge window and this is still unclear, I applied the pwm bits (i.e. patches 12, 13). If I understand correctly, patch 33 isn't suitable for application yet as it has a dependency on pinctrl changes in that series. (side note: Your patches are signed, but that doesn't bring any benefit if the receivers don't have your key. I didn't find it neither on keys.openpgp.org nor in the kernel pgp key collection.) Best regards Uwe
Arnd, Are we continuing this patch series ? You are silent since last version submit, which makes me a bit worried. If you suddenly changed your mind please let us know, cause anyway we have no possibility to merge these series without you. On Fri, 2024-07-05 at 11:21 +0200, Uwe Kleine-König wrote: > Hello, > > On Thu, Jun 27, 2024 at 11:29:44AM +0300, Nikita Shubin wrote: > > On Tue, 2024-06-18 at 19:20 +0300, Nikita Shubin wrote: > > > Hello Andy! > > > On Mon, 2024-06-17 at 12:58 +0200, Andy Shevchenko wrote: > > > > On Mon, Jun 17, 2024 at 11:38 AM Nikita Shubin via B4 Relay > > > > <devnull+nikita.shubin.maquefel.me@kernel.org> wrote: > > > > > > > > > > The goal is to recieve ACKs for all patches in series to > > > > > merge it > > > > > via Arnd branch. > > > > > > > > 'receive' > > > > > > > > > Unfortunately, CLK subsystem suddenly went silent on clk > > > > > portion > > > > > of > > > > > series V2 reroll, > > > > > tried to ping them for about a month but no luck. > > > > > > > > > > Link: > > > > > https://lore.kernel.org/r/20240408-ep93xx-clk-v2-1-adcd68c13753@maquefel.me > > > > > > > > > > Some changes since last version (v9) - see "Changes in v10", > > > > > mostly > > > > > cosmetic. > > > > > > > > ... > > > > > > > > > Patches should be formated with '--histogram' > > > > > > > > 'formatted' > > > > > > > > ... > > > > > > > > > Changes in v10: > > > > > > > > > > Reordered SoB tags to make sure they appear before Rb and > > > > > Acked > > > > > tags. > > > > > > > > This is not required. The importance is only the order of SoBs > > > > themselves. If they are interleaved with other tags, it's fine. > > > > > > Ah - ok. Just saw someone was complaining about b4 reordering > > > them. > > > > > > > > > > > ... > > > > > > > > > > > > Hopefully to see this series being eventually applied soon. > > > > Arnd? (Do we have all necessary subsystem maintainers' tags, > > > > btw?) > > > > > > > > > > > > > > As i see from my perspective only three left: > > > > > > Clk subsystem: > > > > > > - clk: ep93xx: add DT support for Cirrus EP93xx > > > > > > DMA subsystem (but the only request from Vinod, as far as i > > > remember, > > > was fixing commits titles): > > > > > > - dmaengine: cirrus: Convert to DT for Cirrus EP93xx > > > - dmaengine: cirrus: remove platform code > > > > > > Beside that tags missing on platform code removal (which can be > > > Acked > > > by Arnd himself i believe) and dtsi/dts files (same ?). > > > > Vinod acked the above two patches: > > > > https://lore.kernel.org/all/ZnkIp8bOcZK3yVKP@matsya/ > > https://lore.kernel.org/all/ZnkImp8BtTdxl7O3@matsya/ > > > > so only: > > > > - clk: ep93xx: add DT support for Cirrus EP93xx > > > > https://lore.kernel.org/all/20240617-ep93xx-v10-3-662e640ed811@maquefel.me/ > > > > left. > > > > Hope Stephen will find some time for this one. > > As we're approaching the merge window and this is still unclear, I > applied the pwm bits (i.e. patches 12, 13). If I understand > correctly, > patch 33 isn't suitable for application yet as it has a dependency on > pinctrl changes in that series. > > (side note: Your patches are signed, but that doesn't bring any > benefit > if the receivers don't have your key. I didn't find it neither on > keys.openpgp.org nor in the kernel pgp key collection.) > > Best regards > Uwe
On Fri, Jul 5, 2024 at 3:21 AM Uwe Kleine-König <u.kleine-koenig@baylibre.com> wrote: > > Hello, > > On Thu, Jun 27, 2024 at 11:29:44AM +0300, Nikita Shubin wrote: > > On Tue, 2024-06-18 at 19:20 +0300, Nikita Shubin wrote: > > > Hello Andy! > > > On Mon, 2024-06-17 at 12:58 +0200, Andy Shevchenko wrote: > > > > On Mon, Jun 17, 2024 at 11:38 AM Nikita Shubin via B4 Relay > > > > <devnull+nikita.shubin.maquefel.me@kernel.org> wrote: > > > > > > > > > > The goal is to recieve ACKs for all patches in series to merge it > > > > > via Arnd branch. > > > > > > > > 'receive' > > > > > > > > > Unfortunately, CLK subsystem suddenly went silent on clk portion > > > > > of > > > > > series V2 reroll, > > > > > tried to ping them for about a month but no luck. > > > > > > > > > > Link: > > > > > https://lore.kernel.org/r/20240408-ep93xx-clk-v2-1-adcd68c13753@maquefel.me > > > > > > > > > > Some changes since last version (v9) - see "Changes in v10", > > > > > mostly > > > > > cosmetic. > > > > > > > > ... > > > > > > > > > Patches should be formated with '--histogram' > > > > > > > > 'formatted' > > > > > > > > ... > > > > > > > > > Changes in v10: > > > > > > > > > > Reordered SoB tags to make sure they appear before Rb and Acked > > > > > tags. > > > > > > > > This is not required. The importance is only the order of SoBs > > > > themselves. If they are interleaved with other tags, it's fine. > > > > > > Ah - ok. Just saw someone was complaining about b4 reordering them. > > > > > > > > > > > ... > > > > > > > > > > > > Hopefully to see this series being eventually applied soon. > > > > Arnd? (Do we have all necessary subsystem maintainers' tags, btw?) > > > > > > > > > > > > > > As i see from my perspective only three left: > > > > > > Clk subsystem: > > > > > > - clk: ep93xx: add DT support for Cirrus EP93xx > > > > > > DMA subsystem (but the only request from Vinod, as far as i remember, > > > was fixing commits titles): > > > > > > - dmaengine: cirrus: Convert to DT for Cirrus EP93xx > > > - dmaengine: cirrus: remove platform code > > > > > > Beside that tags missing on platform code removal (which can be Acked > > > by Arnd himself i believe) and dtsi/dts files (same ?). > > > > Vinod acked the above two patches: > > > > https://lore.kernel.org/all/ZnkIp8bOcZK3yVKP@matsya/ > > https://lore.kernel.org/all/ZnkImp8BtTdxl7O3@matsya/ > > > > so only: > > > > - clk: ep93xx: add DT support for Cirrus EP93xx > > > > https://lore.kernel.org/all/20240617-ep93xx-v10-3-662e640ed811@maquefel.me/ > > > > left. > > > > Hope Stephen will find some time for this one. > > As we're approaching the merge window and this is still unclear, I > applied the pwm bits (i.e. patches 12, 13). If I understand correctly, > patch 33 isn't suitable for application yet as it has a dependency on > pinctrl changes in that series. Now causing an error in linux-next: Documentation/devicetree/bindings/pwm/cirrus,ep9301-pwm.example.dts:18:18: fatal error: dt-bindings/clock/cirrus,ep9301-syscon.h: No such file or directory 18 | #include <dt-bindings/clock/cirrus,ep9301-syscon.h> | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ compilation terminated. make[2]: *** [scripts/Makefile.lib:442: Documentation/devicetree/bindings/pwm/cirrus,ep9301-pwm.example.dtb] Error 1
Hello Rob, On Tue, Jul 09, 2024 at 07:58:42AM -0600, Rob Herring wrote: > On Fri, Jul 5, 2024 at 3:21 AM Uwe Kleine-König > <u.kleine-koenig@baylibre.com> wrote: > > As we're approaching the merge window and this is still unclear, I > > applied the pwm bits (i.e. patches 12, 13). If I understand correctly, > > patch 33 isn't suitable for application yet as it has a dependency on > > pinctrl changes in that series. > > Now causing an error in linux-next: > > Documentation/devicetree/bindings/pwm/cirrus,ep9301-pwm.example.dts:18:18: > fatal error: dt-bindings/clock/cirrus,ep9301-syscon.h: No such file or > directory > 18 | #include <dt-bindings/clock/cirrus,ep9301-syscon.h> > | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > compilation terminated. > make[2]: *** [scripts/Makefile.lib:442: > Documentation/devicetree/bindings/pwm/cirrus,ep9301-pwm.example.dtb] > Error 1 Oh, I thought I had tested that, but obviously I didn't. I'll drop them again. Thanks for letting me know. Best regards Uwe
On Mon, Jul 8, 2024, at 09:34, Nikita Shubin wrote: > Arnd, > > Are we continuing this patch series ? > > You are silent since last version submit, which makes me a bit worried. > > If you suddenly changed your mind please let us know, cause anyway we > have no possibility to merge these series without you. Hi Nikita, I definitely still want to merge your work, I was just not paying attention while there were others commenting on it, and I don't know what the current state is. If you are ready to have some or all of the patches included in the next merge window, can you send either the set of patches that were reviewed to soc@kernel.org for me to pick up, or prepare a pull request to that address? Arnd
Hello Arnd! On Wed, 2024-07-10 at 14:31 +0200, Arnd Bergmann wrote: > On Mon, Jul 8, 2024, at 09:34, Nikita Shubin wrote: > > Arnd, > > > > Are we continuing this patch series ? > > > > You are silent since last version submit, which makes me a bit > > worried. > > > > If you suddenly changed your mind please let us know, cause anyway > > we > > have no possibility to merge these series without you. > > Hi Nikita, > > I definitely still want to merge your work, I was just not paying > attention while there were others commenting on it, and I don't > know what the current state is. If you are ready to have some > or all of the patches included in the next merge window, can > you send either the set of patches that were reviewed to > soc@kernel.org for me to pick up, or prepare a pull request > to that address? > > Arnd Thanks for support! We still have a minor issue but AFAIS only a single patch left, but hoping to settle this one with Stephen soon.
The goal is to recieve ACKs for all patches in series to merge it via Arnd branch. Unfortunately, CLK subsystem suddenly went silent on clk portion of series V2 reroll, tried to ping them for about a month but no luck. Link: https://lore.kernel.org/r/20240408-ep93xx-clk-v2-1-adcd68c13753@maquefel.me Some changes since last version (v9) - see "Changes in v10", mostly cosmetic. Following patches require attention from Stephen Boyd or clk subsystem: - clk: ep93xx: add DT support for Cirrus EP93xx Vinod, i have replaced titles s/dma/dmaengine/ in: - dmaengine: cirrus: Convert to DT for Cirrus EP93xx - dmaengine: cirrus: remove platform code Patches should be formated with '--histogram' --- Changes in v10: Reordered SoB tags to make sure they appear before Rb and Acked tags. dmaengine: cirrus: Convert to DT for Cirrus EP93xx - s/dma/dmaengine/ title dmaengine: cirrus: remove platform code - s/dma/dmaengine/ title soc: Add SoC driver for Cirrus ep93xx: - added __init for ep93xx_adev_alloc(), ep93xx_controller_register() - added static, __initconst for pinctrl_names[] - clk revision for SPI is now resolved here through differently named clk device - more verbose Kconfig description clk: ep93xx: add DT support for Cirrus EP93xx: - dropped includes - dropped ep93xx_soc_table[] - add different named clk and dropped involved includes - moved pll's and fclk, hclk, pclk init to separate function - fixed ep93xx_clk_ids[] explicit lines - Link to v9: https://lore.kernel.org/r/20240326-ep93xx-v9-0-156e2ae5dfc8@maquefel.me - Link to v2 clk: https://lore.kernel.org/r/20240408-ep93xx-clk-v2-1-adcd68c13753@maquefel.me Changes in v9: ARM: dts: add Cirrus EP93XX SoC .dtsi - added #interrupt-cells to gpio nodes with interrupts-controller - fixed EOF ARM: dts: ep93xx: Add EDB9302 DT - Alexander Sverdlin: fixed bug in Device Tree resulting in CS4271 not working input: keypad: ep93xx: add DT support for Cirrus EP93xx - fixed identation and type - Link to v8: https://lore.kernel.org/r/20240226-ep93xx-v8-0-3136dca7238f@maquefel.me/ Changes in v8: soc: Add SoC driver for Cirrus ep93xx - fixed freeing adev instead of rdev - use __free() and no_free_ptr() for rdev allocation - s/of_device_get_match_data()/device_get_match_data()/ ata: pata_ep93xx: add device tree support - more appropriate usage of dev_err_probe() pinctrl: add a Cirrus ep93xx SoC pin controller - 8 per row in ide_9312_pins mtd: rawnand: add support for ts72xx - fwnode_handle_put() for fwnode in ts72xx_nand_remove() - Link to v7: https://lore.kernel.org/r/20240118-ep93xx-v7-0-d953846ae771@maquefel.me Changes in v7: mtd: rawnand: add support for ts72xx - fixed KConfig description ARM: ep93xx: Add terminator to gpiod_lookup_table - + Reported-by, Fixes ARM: ep93xx: add regmap aux_dev - + trailing comma - - #include <linux/spinlock.h> clk: ep93xx: add DT support for Cirrus EP93xx - dropped unused defines - return from default in ep93xx_mux_get_parent() - use guard() in ep93xx_mux_set_parent_lock() - <math.h> header for abs_diff() - fixed comments pinctrl: add a Cirrus ep93xx SoC pin controller - dropped comments for DEVCFG defines - <linux/array_size.h> for ARRAY_SIZE() - + default in ep93xx_get_group_name() - correct cast for id->driver_data - s/device_set_of_node_from_dev()/device_set_node()/ power: reset: Add a driver for the ep93xx reset - Add <linux/container_of.h>, <linux/errno.h>, <linux/slab.h> - Add <linux/module.h>, <linux/mod_devicetable.h> - Remove <platform_device.h> spi: ep93xx: add DT support for Cirrus EP93xx - Replace with ret = dev_err_probe(...); ata: pata_ep93xx: add device tree support - fixed wrong rebase with some partes leaked in "ata: pata_ep93xx: remove legacy pinctrl use" - fix dma_request_chan() error processing dma: cirrus: Convert to DT for Cirrus EP93xx - fixed commit message (dropped explicit "only") - fixed clk_get() processing to defer probe and log spamming - refactor ep93xx_m2p_dma_filter() - dropped blank line in ep93xx_m2p_dma_of_xlate() - refactor ep93xx_m2m_dma_of_xlate() dma: cirrus: remove platform code - s/dma/DMA/ in commit message soc: Add SoC driver for Cirrus ep93xx - add period - use cleanup and guard() for spinlocking - correct cast for device_get_match_data() - dropped dev_info() with SoC revision - i can't find it anywhere since 2.6 :/, don't know why i was so sured that ep93xx always printed that ata: pata_ep93xx: remove legacy pinctrl use - made error handling in DMA as Uwe suggested - Link to v6: https://lore.kernel.org/r/20231212-ep93xx-v6-0-c307b8ac9aa8@maquefel.me Changes in v6: - clk: ep93xx: add DT support for Cirrus EP93xx - s/spin_lock_irqsave()/guard()/ - refactor index check in ep93xx_mux_set_parent_lock() to something more readable - use in_range in ep93xx_mux_set_parent_lock()/ep93xx_ddiv_set_rate() - use GENMASK() in ep93xx_ddiv_recalc_rate() - comment reserved bit in ep93xx_ddiv_set_rate() - move out from loop ClkDiv value assigment - some style fixes Andy, i was i asked to set index of XTALI explicitly, i am not setting ddiv_pdata there becouse only XTALI is jnown in advance, and i think setting them in one place is more convenient. - pinctrl: add a Cirrus ep93xx SoC pin controller - drop OF from Kconfig - droped linux/of.h include - add space to */ where it is applicable - add coma in multiline assigment - "return NULL" as default case in ep93xx_get_group_name() - fixed casting id->driver_data - use device_set_of_node_from_dev() - use dev_err_probe() - power: reset: Add a driver for the ep93xx reset - drop linux/of.h include - soc: Add SoC driver for Cirrus ep93xx - s/GPL-2.0/GPL-2.0-only/ - drop linux/kernel.h include - + blank line before linux/soc/cirrus/ep93xx.h - + blank line after ep93xx_get_soc_rev() - + coma for pinctrl_names - valid casting to int for of_device_get_match_data() return value - mtd: rawnand: add support for ts72xx - return as part of switch case - s/iowrite8/iowrite8_rep/ - net: cirrus: add DT support for Cirrus EP93xx - fix header sorting - dma: cirrus: Convert to DT for Cirrus EP93xx - use devm_clk_get - use is_slave_direction Changes in v5: - gpio: ep93xx: split device in multiple - ordered headers - use irqd_to_hwirq() - s/platform_get_irq()/platform_get_irq_optional()/ - [PATCH v4 02/42] ARM: ep93xx: add swlocked prototypes - replaced with ARM: ep93xx: add regmap aux_dev - [PATCH v4 03/42] dt-bindings: clock: Add Cirrus EP93xx - fixed identation - removed EP93XX_CLK_END - and dropped it - clock bindings moved to syscon with renaming to cirrus,ep9301-syscon.h - clk: ep93xx: add DT support for Cirrus EP93xx - convert to auxiliary and use parent device tree node - moved all clocks except XTALI here - used devm version everywhere and *_parent_hw() instead of passing name where it's possible - unfortunately devm_clk_hw_register_fixed_rate doesn't have a parent index version - [PATCH v4 05/42] dt-bindings: pinctrl: Add Cirrus EP93xx - "unevaluatedProperties: false" for pins - returned "additionalProperties: false" where it was - and dropped it - pinctrl: add a Cirrus ep93xx SoC pin controller - sorted includes - convert to auxiliary and use parent device tree node - power: reset: Add a driver for the ep93xx reset - convert to auxiliary device - dt-bindings: soc: Add Cirrus EP93xx - dropped all ref to reboot, clk, pinctrl subnodes - added pins, as it's now used for pinctrl - added #clock-cells, as it's now used for clk - dt-bindings: pwm: Add Cirrus EP93xx - $ref to pwm.yaml - fixed 'pwm-cells' - s/additionalProperties/unevaluatedProperties/ - soc: Add SoC driver for Cirrus ep93xx - removed clocks, they are moved to clk auxiliary driver, as we dropped the clk dt node - removed all swlocked exported functions - dropped static spinlock - added instantiating auxiliary reboot, clk, pinctrl - dt-bindings: spi: Add Cirrus EP93xx - Document DMA support - spi: ep93xx: add DT support for Cirrus EP93xx - dropped CONFIG_OF and SPI/DMA platform data entirely - s/master/host/ - reworked DMA setup so we can use probe defer - dt-bindings: dma: Add Cirrus EP93xx - dropped bindings header (moved ports description to YAML) - changed '#dma-cells' to 2, we use port, direction in cells so we can drop platform code completely - dma: cirrus: add DT support for Cirrus EP93xx - dropped platform probing completely - dropped struct ep93xx_dma_data replaced with internal struct ep93xx_dma_chan_cfg with port/direction - added xlate functions for m2m/m2p - we require filters to set dma_cfg before hw_setup - dt-bindings: ata: Add Cirrus EP93xx - Document DMA support - ata: pata_ep93xx: add device tree support - drop DMA platform header with data - use DMA OF so we can defer probing until DMA is up - ARM: dts: add Cirrus EP93XX SoC .dtsi - ARM: dts: ep93xx: add ts7250 board - ARM: dts: ep93xx: Add EDB9302 DT - replaced "eclk: clock-controller" to syscon reference - replaced "pinctrl: pinctrl" to syscon reference - gpios are now "enabled" by default - reworked i2s node - change all dma nodes and refs - new additions to I2S - Document DMA - Document Audio Port usage - drop legacy DMA support - Link to v4: https://lore.kernel.org/r/20230915-ep93xx-v4-0-a1d779dcec10@maquefel.me Changes in v4: - gpio: ep93xx: split device in multiple - s/generic_handle_irq/generic_handle_domain_irq/ - s/int offset/irq_hw_number_t offset/ though now it looks a bit odd to me - drop i = 0 - drop 'error' - use dev_err_probe withour printing devname once again dt-bindings: clock: Add Cirrus EP93xx - renamed cirrus,ep93xx-clock.h -> cirrus,ep9301-clk.h clk: ep93xx: add DT support for Cirrus EP93xx - drop unused includes - use .name only for xtali, pll1, pll2 parents - convert // to /* - pass clk_parent_data instead of char* clock name dt-bindings: pinctrl: Add Cirrus EP93xx - s/additionalProperties/unevaluatedProperties/ dt-bindings: soc: Add Cirrus EP93xx - move syscon to soc directory - add vendor prefix - make reboot same style as pinctrl, clk - use absolute path for ref - expand example soc: Add SoC driver for Cirrus ep93xx - s/0xf0000000/GENMASK(31, 28)/ - s/ret/ep93xx_chip_revision(map)/ - drop symbol exports - convert to platform driver dt-bindings: rtc: Add Cirrus EP93xx - allOf: with $ref to rtc.yaml - s/additionalProperties/unevaluatedProperties/ dt-bindings: watchdog: Add Cirrus EP93x - drop description - reword power: reset: Add a driver for the ep93xx reset - lets use 'GPL-2.0+' instead of '(GPL-2.0)' - s/of_device/of/ - drop mdelay with warning - return 0 at the end net: cirrus: add DT support for Cirrus EP93xx - fix leaking np mtd: nand: add support for ts72xx - +bits.h - drop comment - ok to fwnode_get_next_child_node - use goto to put handle and nand and report error ARM: dts: add Cirrus EP93XX SoC .dtsi - add simple-bus for ebi, as we don't require to setup anything - add arm,pl011 compatible to uart nodes - drop i2c-gpio, as it's isn't used anywhere ARM: dts: ep93xx: add ts7250 board - generic node name for temperature-sensor - drop i2c - move nand, rtc, watchdog to ebi node - Link to v3: https://lore.kernel.org/r/20230605-ep93xx-v3-0-3d63a5f1103e@maquefel.me --- Alexander Sverdlin (3): ASoC: ep93xx: Drop legacy DMA support ARM: dts: ep93xx: Add EDB9302 DT ASoC: cirrus: edb93xx: Delete driver Nikita Shubin (35): gpio: ep93xx: split device in multiple ARM: ep93xx: add regmap aux_dev clk: ep93xx: add DT support for Cirrus EP93xx pinctrl: add a Cirrus ep93xx SoC pin controller power: reset: Add a driver for the ep93xx reset dt-bindings: soc: Add Cirrus EP93xx soc: Add SoC driver for Cirrus ep93xx dt-bindings: dma: Add Cirrus EP93xx dmaengine: cirrus: Convert to DT for Cirrus EP93xx dt-bindings: watchdog: Add Cirrus EP93x watchdog: ep93xx: add DT support for Cirrus EP93xx dt-bindings: pwm: Add Cirrus EP93xx pwm: ep93xx: add DT support for Cirrus EP93xx dt-bindings: spi: Add Cirrus EP93xx spi: ep93xx: add DT support for Cirrus EP93xx dt-bindings: net: Add Cirrus EP93xx net: cirrus: add DT support for Cirrus EP93xx dt-bindings: mtd: Add ts7200 nand-controller mtd: rawnand: add support for ts72xx dt-bindings: ata: Add Cirrus EP93xx ata: pata_ep93xx: add device tree support dt-bindings: input: Add Cirrus EP93xx keypad input: keypad: ep93xx: add DT support for Cirrus EP93xx wdt: ts72xx: add DT support for ts72xx gpio: ep93xx: add DT support for gpio-ep93xx ASoC: dt-bindings: ep93xx: Document DMA support ASoC: dt-bindings: ep93xx: Document Audio Port support ARM: dts: add Cirrus EP93XX SoC .dtsi ARM: dts: ep93xx: add ts7250 board ARM: ep93xx: DT for the Cirrus ep93xx SoC platforms pwm: ep93xx: drop legacy pinctrl ata: pata_ep93xx: remove legacy pinctrl use ARM: ep93xx: delete all boardfiles ARM: ep93xx: soc: drop defines dmaengine: cirrus: remove platform code .../bindings/arm/cirrus/cirrus,ep9301.yaml | 38 + .../bindings/ata/cirrus,ep9312-pata.yaml | 42 + .../bindings/dma/cirrus,ep9301-dma-m2m.yaml | 84 ++ .../bindings/dma/cirrus,ep9301-dma-m2p.yaml | 144 ++ .../bindings/input/cirrus,ep9307-keypad.yaml | 87 ++ .../devicetree/bindings/mtd/technologic,nand.yaml | 45 + .../devicetree/bindings/net/cirrus,ep9301-eth.yaml | 59 + .../devicetree/bindings/pwm/cirrus,ep9301-pwm.yaml | 53 + .../bindings/soc/cirrus/cirrus,ep9301-syscon.yaml | 94 ++ .../bindings/sound/cirrus,ep9301-i2s.yaml | 16 + .../devicetree/bindings/spi/cirrus,ep9301-spi.yaml | 70 + .../bindings/watchdog/cirrus,ep9301-wdt.yaml | 42 + arch/arm/Makefile | 1 - arch/arm/boot/dts/cirrus/Makefile | 4 + arch/arm/boot/dts/cirrus/ep93xx-bk3.dts | 125 ++ arch/arm/boot/dts/cirrus/ep93xx-edb9302.dts | 181 +++ arch/arm/boot/dts/cirrus/ep93xx-ts7250.dts | 145 ++ arch/arm/boot/dts/cirrus/ep93xx.dtsi | 444 ++++++ arch/arm/mach-ep93xx/Kconfig | 20 +- arch/arm/mach-ep93xx/Makefile | 11 - arch/arm/mach-ep93xx/clock.c | 733 ---------- arch/arm/mach-ep93xx/core.c | 1018 -------------- arch/arm/mach-ep93xx/dma.c | 114 -- arch/arm/mach-ep93xx/edb93xx.c | 368 ----- arch/arm/mach-ep93xx/ep93xx-regs.h | 38 - arch/arm/mach-ep93xx/gpio-ep93xx.h | 111 -- arch/arm/mach-ep93xx/hardware.h | 25 - arch/arm/mach-ep93xx/irqs.h | 76 -- arch/arm/mach-ep93xx/platform.h | 42 - arch/arm/mach-ep93xx/soc.h | 212 --- arch/arm/mach-ep93xx/timer-ep93xx.c | 143 -- arch/arm/mach-ep93xx/ts72xx.c | 422 ------ arch/arm/mach-ep93xx/ts72xx.h | 94 -- arch/arm/mach-ep93xx/vision_ep9307.c | 321 ----- drivers/ata/pata_ep93xx.c | 107 +- drivers/clk/Kconfig | 8 + drivers/clk/Makefile | 1 + drivers/clk/clk-ep93xx.c | 834 ++++++++++++ drivers/dma/ep93xx_dma.c | 287 +++- drivers/gpio/gpio-ep93xx.c | 345 ++--- drivers/input/keyboard/ep93xx_keypad.c | 74 +- drivers/mtd/nand/raw/Kconfig | 6 + drivers/mtd/nand/raw/Makefile | 1 + drivers/mtd/nand/raw/technologic-nand-controller.c | 222 +++ drivers/net/ethernet/cirrus/ep93xx_eth.c | 63 +- drivers/pinctrl/Kconfig | 7 + drivers/pinctrl/Makefile | 1 + drivers/pinctrl/pinctrl-ep93xx.c | 1434 ++++++++++++++++++++ drivers/power/reset/Kconfig | 10 + drivers/power/reset/Makefile | 1 + drivers/power/reset/ep93xx-restart.c | 84 ++ drivers/pwm/pwm-ep93xx.c | 26 +- drivers/soc/Kconfig | 1 + drivers/soc/Makefile | 1 + drivers/soc/cirrus/Kconfig | 17 + drivers/soc/cirrus/Makefile | 2 + drivers/soc/cirrus/soc-ep93xx.c | 252 ++++ drivers/spi/spi-ep93xx.c | 66 +- drivers/watchdog/ep93xx_wdt.c | 8 + drivers/watchdog/ts72xx_wdt.c | 8 + include/dt-bindings/clock/cirrus,ep9301-syscon.h | 46 + include/linux/platform_data/dma-ep93xx.h | 94 -- include/linux/platform_data/eth-ep93xx.h | 10 - include/linux/platform_data/keypad-ep93xx.h | 32 - include/linux/platform_data/spi-ep93xx.h | 15 - include/linux/soc/cirrus/ep93xx.h | 47 +- sound/soc/cirrus/Kconfig | 9 - sound/soc/cirrus/Makefile | 4 - sound/soc/cirrus/edb93xx.c | 116 -- sound/soc/cirrus/ep93xx-i2s.c | 19 - sound/soc/cirrus/ep93xx-pcm.c | 19 +- 71 files changed, 5149 insertions(+), 4550 deletions(-) --- base-commit: 2df0193e62cf887f373995fb8a91068562784adc change-id: 20230605-ep93xx-01c76317e2d2 Best regards,