diff mbox

[RESEND,1/1] arm: dts: am33xx: correcting dt node unit address for usb

Message ID 1394109094-19205-1-git-send-email-mugunthanvnm@ti.com (mailing list archive)
State New, archived
Headers show

Commit Message

Mugunthan V N March 6, 2014, 12:31 p.m. UTC
DT node's unit address should be its own register offset address to make it a
unique across the system. This patch corrects the incorrect USB entries with
correct register offset for unit address.

Cc: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Cc: Felipe Balbi <balbi@ti.com>
Signed-off-by: Mugunthan V N <mugunthanvnm@ti.com>
---
This patch was posted long back (Aug. 30, 2013), below is the archive of the
orginal patch.
https://patchwork.kernel.org/patch/2852163/

---
 arch/arm/boot/dts/am33xx.dtsi | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Sebastian Andrzej Siewior March 6, 2014, 12:42 p.m. UTC | #1
On 03/06/2014 01:31 PM, Mugunthan V N wrote:
> DT node's unit address should be its own register offset address to make it a
> unique across the system. This patch corrects the incorrect USB entries with
> correct register offset for unit address.
> 
> Cc: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
> Cc: Felipe Balbi <balbi@ti.com>
> Signed-off-by: Mugunthan V N <mugunthanvnm@ti.com>

Acked-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>

Sebastian
Felipe Balbi March 6, 2014, 3:26 p.m. UTC | #2
On Thu, Mar 06, 2014 at 06:01:34PM +0530, Mugunthan V N wrote:
> DT node's unit address should be its own register offset address to make it a
> unique across the system. This patch corrects the incorrect USB entries with
> correct register offset for unit address.
> 
> Cc: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
> Cc: Felipe Balbi <balbi@ti.com>
> Signed-off-by: Mugunthan V N <mugunthanvnm@ti.com>

Acked-by: Felipe Balbi <balbi@ti.com>

> ---
> This patch was posted long back (Aug. 30, 2013), below is the archive of the
> orginal patch.
> https://patchwork.kernel.org/patch/2852163/
> 
> ---
>  arch/arm/boot/dts/am33xx.dtsi | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi
> index 7073429..9770e35 100644
> --- a/arch/arm/boot/dts/am33xx.dtsi
> +++ b/arch/arm/boot/dts/am33xx.dtsi
> @@ -453,7 +453,7 @@
>  			ti,hwmods = "usb_otg_hs";
>  			status = "disabled";
>  
> -			usb_ctrl_mod: control@44e10000 {
> +			usb_ctrl_mod: control@44e10620 {
>  				compatible = "ti,am335x-usb-ctrl-module";
>  				reg = <0x44e10620 0x10
>  					0x44e10648 0x4>;
> @@ -556,7 +556,7 @@
>  					"tx14", "tx15";
>  			};
>  
> -			cppi41dma: dma-controller@07402000 {
> +			cppi41dma: dma-controller@47402000 {
>  				compatible = "ti,am3359-cppi41";
>  				reg =  <0x47400000 0x1000
>  					0x47402000 0x1000
> -- 
> 1.9.0
>
Tony Lindgren March 13, 2014, 8:30 p.m. UTC | #3
* Felipe Balbi <balbi@ti.com> [140306 07:33]:
> On Thu, Mar 06, 2014 at 06:01:34PM +0530, Mugunthan V N wrote:
> > DT node's unit address should be its own register offset address to make it a
> > unique across the system. This patch corrects the incorrect USB entries with
> > correct register offset for unit address.
> > 
> > Cc: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
> > Cc: Felipe Balbi <balbi@ti.com>
> > Signed-off-by: Mugunthan V N <mugunthanvnm@ti.com>
> 
> Acked-by: Felipe Balbi <balbi@ti.com>

Applying into omap-for-v3.15/fixes with cc stable v3.12+. If somebody
wants it in earlier kernels it needs to be updated and posted
to the lists.

Regards,

Tony

> > This patch was posted long back (Aug. 30, 2013), below is the archive of the
> > orginal patch.
> > https://patchwork.kernel.org/patch/2852163/
> > 
> > ---
> >  arch/arm/boot/dts/am33xx.dtsi | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> > 
> > diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi
> > index 7073429..9770e35 100644
> > --- a/arch/arm/boot/dts/am33xx.dtsi
> > +++ b/arch/arm/boot/dts/am33xx.dtsi
> > @@ -453,7 +453,7 @@
> >  			ti,hwmods = "usb_otg_hs";
> >  			status = "disabled";
> >  
> > -			usb_ctrl_mod: control@44e10000 {
> > +			usb_ctrl_mod: control@44e10620 {
> >  				compatible = "ti,am335x-usb-ctrl-module";
> >  				reg = <0x44e10620 0x10
> >  					0x44e10648 0x4>;
> > @@ -556,7 +556,7 @@
> >  					"tx14", "tx15";
> >  			};
> >  
> > -			cppi41dma: dma-controller@07402000 {
> > +			cppi41dma: dma-controller@47402000 {
> >  				compatible = "ti,am3359-cppi41";
> >  				reg =  <0x47400000 0x1000
> >  					0x47402000 0x1000
> > -- 
> > 1.9.0
> > 
> 
> -- 
> balbi



> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
diff mbox

Patch

diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi
index 7073429..9770e35 100644
--- a/arch/arm/boot/dts/am33xx.dtsi
+++ b/arch/arm/boot/dts/am33xx.dtsi
@@ -453,7 +453,7 @@ 
 			ti,hwmods = "usb_otg_hs";
 			status = "disabled";
 
-			usb_ctrl_mod: control@44e10000 {
+			usb_ctrl_mod: control@44e10620 {
 				compatible = "ti,am335x-usb-ctrl-module";
 				reg = <0x44e10620 0x10
 					0x44e10648 0x4>;
@@ -556,7 +556,7 @@ 
 					"tx14", "tx15";
 			};
 
-			cppi41dma: dma-controller@07402000 {
+			cppi41dma: dma-controller@47402000 {
 				compatible = "ti,am3359-cppi41";
 				reg =  <0x47400000 0x1000
 					0x47402000 0x1000