mbox series

[0/3] AM654/J721e: Sync UDMA bindings

Message ID 20200622065555.16639-1-vigneshr@ti.com
Headers show
Series AM654/J721e: Sync UDMA bindings | expand

Message

Vignesh Raghavendra June 22, 2020, 6:55 a.m. UTC
UDMA DT bindings have deviated from kernel's DT for AM654 and J721e.
This series updates UDMA driver and sync DT bindings

Tested OSPI and CPSW on AM654 and J721e after the changes

Lokesh,

Patch 2/3 and 3/3 should ideally go as a single patch to avoid breaking of
functionality but, that makes harder to review. Feel free to squash them
while merging or I can repost with 2/3 and 3/3 squashed

Vignesh Raghavendra (3):
  dma: ti: Add static PSIL endpoint information
  dma: ti: k3-udma: Update driver to use static endpoint Data
  arm: dts: k3-am65/j721e: Sync DMA DT bindings from Kernel DT

 arch/arm/dts/k3-am65-mcu.dtsi                 |  44 ++++
 arch/arm/dts/k3-am654-base-board-u-boot.dtsi  | 121 +---------
 .../k3-j721e-common-proc-board-u-boot.dtsi    | 118 +---------
 arch/arm/dts/k3-j721e-mcu-wakeup.dtsi         |  43 ++++
 drivers/dma/ti/Kconfig                        |   4 +
 drivers/dma/ti/Makefile                       |   1 +
 drivers/dma/ti/k3-psil-am654.c                | 175 ++++++++++++++
 drivers/dma/ti/k3-psil-j721e.c                | 222 ++++++++++++++++++
 drivers/dma/ti/k3-psil-priv.h                 |  43 ++++
 drivers/dma/ti/k3-psil.c                      |  42 ++++
 drivers/dma/ti/k3-psil.h                      |  67 ++++++
 drivers/dma/ti/k3-udma.c                      | 145 ++++++++----
 include/dt-bindings/dma/k3-udma.h             |  31 ---
 13 files changed, 768 insertions(+), 288 deletions(-)
 create mode 100644 drivers/dma/ti/k3-psil-am654.c
 create mode 100644 drivers/dma/ti/k3-psil-j721e.c
 create mode 100644 drivers/dma/ti/k3-psil-priv.h
 create mode 100644 drivers/dma/ti/k3-psil.c
 create mode 100644 drivers/dma/ti/k3-psil.h
 delete mode 100644 include/dt-bindings/dma/k3-udma.h

Comments

Grygorii Strashko June 24, 2020, 8:14 a.m. UTC | #1
On 22/06/2020 09:55, Vignesh Raghavendra wrote:
> UDMA DT bindings have deviated from kernel's DT for AM654 and J721e.
> This series updates UDMA driver and sync DT bindings
> 
> Tested OSPI and CPSW on AM654 and J721e after the changes
> 
> Lokesh,
> 
> Patch 2/3 and 3/3 should ideally go as a single patch to avoid breaking of
> functionality but, that makes harder to review. Feel free to squash them
> while merging or I can repost with 2/3 and 3/3 squashed

I think it should be ok, but It  has to be mentioned in Patch 2 description
that it will break boot.

> 
> Vignesh Raghavendra (3):
>    dma: ti: Add static PSIL endpoint information
>    dma: ti: k3-udma: Update driver to use static endpoint Data
>    arm: dts: k3-am65/j721e: Sync DMA DT bindings from Kernel DT
> 
>   arch/arm/dts/k3-am65-mcu.dtsi                 |  44 ++++
>   arch/arm/dts/k3-am654-base-board-u-boot.dtsi  | 121 +---------
>   .../k3-j721e-common-proc-board-u-boot.dtsi    | 118 +---------
>   arch/arm/dts/k3-j721e-mcu-wakeup.dtsi         |  43 ++++
>   drivers/dma/ti/Kconfig                        |   4 +
>   drivers/dma/ti/Makefile                       |   1 +
>   drivers/dma/ti/k3-psil-am654.c                | 175 ++++++++++++++
>   drivers/dma/ti/k3-psil-j721e.c                | 222 ++++++++++++++++++
>   drivers/dma/ti/k3-psil-priv.h                 |  43 ++++
>   drivers/dma/ti/k3-psil.c                      |  42 ++++
>   drivers/dma/ti/k3-psil.h                      |  67 ++++++
>   drivers/dma/ti/k3-udma.c                      | 145 ++++++++----
>   include/dt-bindings/dma/k3-udma.h             |  31 ---
>   13 files changed, 768 insertions(+), 288 deletions(-)
>   create mode 100644 drivers/dma/ti/k3-psil-am654.c
>   create mode 100644 drivers/dma/ti/k3-psil-j721e.c
>   create mode 100644 drivers/dma/ti/k3-psil-priv.h
>   create mode 100644 drivers/dma/ti/k3-psil.c
>   create mode 100644 drivers/dma/ti/k3-psil.h
>   delete mode 100644 include/dt-bindings/dma/k3-udma.h
> 

Thank you.
Reviewed-by: Grygorii Strashko <grygorii.strashko at ti.com>