Message ID | 20170117135420.10197-1-abailon@baylibre.com |
---|---|
State | New |
Headers | show |
* Alexandre Bailon <abailon@baylibre.com> [170117 05:55]: > In order to make CPPI 4.1 DMA driver more generic, accesses to USBSS > have been removed. So it is not required anymore to define "glue" > register's address and size in DT. > Remove "glue" register from cppi41dma node. Is this OK to queue later on after all the dependencies have been merged? Regards, Tony -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
* Alexandre Bailon <abailon@baylibre.com> [170117 08:17]: > On 01/17/2017 04:47 PM, Tony Lindgren wrote: > > * Alexandre Bailon <abailon@baylibre.com> [170117 05:55]: > >> In order to make CPPI 4.1 DMA driver more generic, accesses to USBSS > >> have been removed. So it is not required anymore to define "glue" > >> register's address and size in DT. > >> Remove "glue" register from cppi41dma node. > > > > Is this OK to queue later on after all the dependencies have been > > merged? > It should be queued on same time as "[PATCH v2 3/7] dmaengine: cppi41: > Remove usbss_mem", so yes I guess it is OK. Sounds like it will break things then.. Please read about "flag day" changes and why they are not accepted. You should make the driver work with the old and new binding, then we can just drop the old binding later on from the dts files. Regards, Tony -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi index 9e96d60..2466ed5 100644 --- a/arch/arm/boot/dts/am33xx.dtsi +++ b/arch/arm/boot/dts/am33xx.dtsi @@ -659,11 +659,10 @@ cppi41dma: dma-controller@47402000 { compatible = "ti,am3359-cppi41"; - reg = <0x47400000 0x1000 - 0x47402000 0x1000 + reg = <0x47402000 0x1000 0x47403000 0x1000 0x47404000 0x4000>; - reg-names = "glue", "controller", "scheduler", "queuemgr"; + reg-names = "controller", "scheduler", "queuemgr"; interrupts = <17>; interrupt-names = "glue"; #dma-cells = <2>; diff --git a/arch/arm/boot/dts/dm814x.dtsi b/arch/arm/boot/dts/dm814x.dtsi index 9708157..2dd727a 100644 --- a/arch/arm/boot/dts/dm814x.dtsi +++ b/arch/arm/boot/dts/dm814x.dtsi @@ -156,11 +156,10 @@ cppi41dma: dma-controller@47402000 { compatible = "ti,am3359-cppi41"; - reg = <0x47400000 0x1000 - 0x47402000 0x1000 + reg = <0x47402000 0x1000 0x47403000 0x1000 0x47404000 0x4000>; - reg-names = "glue", "controller", "scheduler", "queuemgr"; + reg-names = "controller", "scheduler", "queuemgr"; interrupts = <17>; interrupt-names = "glue"; #dma-cells = <2>; diff --git a/arch/arm/boot/dts/dm816x.dtsi b/arch/arm/boot/dts/dm816x.dtsi index 276211e..bb78511 100644 --- a/arch/arm/boot/dts/dm816x.dtsi +++ b/arch/arm/boot/dts/dm816x.dtsi @@ -493,11 +493,10 @@ cppi41dma: dma-controller@47402000 { compatible = "ti,am3359-cppi41"; - reg = <0x47400000 0x1000 - 0x47402000 0x1000 + reg = <0x47402000 0x1000 0x47403000 0x1000 0x47404000 0x4000>; - reg-names = "glue", "controller", "scheduler", "queuemgr"; + reg-names = "controller", "scheduler", "queuemgr"; interrupts = <17>; interrupt-names = "glue"; #dma-cells = <2>;
In order to make CPPI 4.1 DMA driver more generic, accesses to USBSS have been removed. So it is not required anymore to define "glue" register's address and size in DT. Remove "glue" register from cppi41dma node. Signed-off-by: Alexandre Bailon <abailon@baylibre.com> --- arch/arm/boot/dts/am33xx.dtsi | 5 ++--- arch/arm/boot/dts/dm814x.dtsi | 5 ++--- arch/arm/boot/dts/dm816x.dtsi | 5 ++--- 3 files changed, 6 insertions(+), 9 deletions(-) -- 2.10.2 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html