diff mbox

powerpc/85xx:DTS: Fix tbi node location for Px020RDB

Message ID 1307501379-3160-1-git-send-email-prabhakar@freescale.com (mailing list archive)
State Rejected
Delegated to: Kumar Gala
Headers show

Commit Message

Prabhakar Kushwaha June 8, 2011, 2:49 a.m. UTC
ten-bit interface (TBI) module is part of SoC not board.

Move tbi entries from board related dts files to Si dts.

Signed-off-by: Prabhakar Kushwaha <prabhakar@freescale.com>
---
 Based upon http://git.kernel.org/pub/scm/linux/kernel/git/galak/powerpc.git (branch next)

 arch/powerpc/boot/dts/p1020rdb.dts            |    9 ---------
 arch/powerpc/boot/dts/p1020rdb_camp_core0.dts |    8 --------
 arch/powerpc/boot/dts/p1020si.dtsi            |    6 +++++-
 arch/powerpc/boot/dts/p2020rdb.dts            |    8 --------
 arch/powerpc/boot/dts/p2020rdb_camp_core0.dts |    8 --------
 arch/powerpc/boot/dts/p2020si.dtsi            |    6 +++++-
 6 files changed, 10 insertions(+), 35 deletions(-)

Comments

Grant Likely June 8, 2011, 2:57 p.m. UTC | #1
On Wed, Jun 08, 2011 at 08:19:39AM +0530, Prabhakar Kushwaha wrote:
> ten-bit interface (TBI) module is part of SoC not board.
> 
> Move tbi entries from board related dts files to Si dts.
> 
> Signed-off-by: Prabhakar Kushwaha <prabhakar@freescale.com>

Looks okay to me.

g.

> ---
>  Based upon http://git.kernel.org/pub/scm/linux/kernel/git/galak/powerpc.git (branch next)
> 
>  arch/powerpc/boot/dts/p1020rdb.dts            |    9 ---------
>  arch/powerpc/boot/dts/p1020rdb_camp_core0.dts |    8 --------
>  arch/powerpc/boot/dts/p1020si.dtsi            |    6 +++++-
>  arch/powerpc/boot/dts/p2020rdb.dts            |    8 --------
>  arch/powerpc/boot/dts/p2020rdb_camp_core0.dts |    8 --------
>  arch/powerpc/boot/dts/p2020si.dtsi            |    6 +++++-
>  6 files changed, 10 insertions(+), 35 deletions(-)
> 
> diff --git a/arch/powerpc/boot/dts/p1020rdb.dts b/arch/powerpc/boot/dts/p1020rdb.dts
> index d6a8ae4..a4e5d6c 100644
> --- a/arch/powerpc/boot/dts/p1020rdb.dts
> +++ b/arch/powerpc/boot/dts/p1020rdb.dts
> @@ -211,14 +211,6 @@
>  			};
>  		};
>  
> -		mdio@25000 {
> -
> -			tbi0: tbi-phy@11 {
> -				reg = <0x11>;
> -				device_type = "tbi-phy";
> -			};
> -		};
> -
>  		enet0: ethernet@b0000 {
>  			fixed-link = <1 1 1000 0 0>;
>  			phy-connection-type = "rgmii-id";
> @@ -227,7 +219,6 @@
>  
>  		enet1: ethernet@b1000 {
>  			phy-handle = <&phy0>;
> -			tbi-handle = <&tbi0>;
>  			phy-connection-type = "sgmii";
>  
>  		};
> diff --git a/arch/powerpc/boot/dts/p1020rdb_camp_core0.dts b/arch/powerpc/boot/dts/p1020rdb_camp_core0.dts
> index f0bf7f4..abab234 100644
> --- a/arch/powerpc/boot/dts/p1020rdb_camp_core0.dts
> +++ b/arch/powerpc/boot/dts/p1020rdb_camp_core0.dts
> @@ -114,20 +114,12 @@
>  			};
>  		};
>  
> -		mdio@25000 {
> -			tbi0: tbi-phy@11 {
> -				reg = <0x11>;
> -				device_type = "tbi-phy";
> -			};
> -		};
> -
>  		enet0: ethernet@b0000 {
>  			status = "disabled";
>  		};
>  
>  		enet1: ethernet@b1000 {
>  			phy-handle = <&phy0>;
> -			tbi-handle = <&tbi0>;
>  			phy-connection-type = "sgmii";
>  		};
>  
> diff --git a/arch/powerpc/boot/dts/p1020si.dtsi b/arch/powerpc/boot/dts/p1020si.dtsi
> index 5c5acb6..7844d2e 100644
> --- a/arch/powerpc/boot/dts/p1020si.dtsi
> +++ b/arch/powerpc/boot/dts/p1020si.dtsi
> @@ -190,7 +190,10 @@
>  			#size-cells = <0>;
>  			compatible = "fsl,etsec2-tbi";
>  			reg = <0x25000 0x1000 0xb1030 0x4>;
> -
> +			tbi0: tbi-phy@11 {
> +				reg = <0x11>;
> +				device_type = "tbi-phy";
> +			};
>  		};
>  
>  		enet0: ethernet@b0000 {
> @@ -229,6 +232,7 @@
>  			fsl,num_tx_queues = <0x8>;
>  			local-mac-address = [ 00 00 00 00 00 00 ];
>  			interrupt-parent = <&mpic>;
> +			tbi-handle = <&tbi0>;
>  
>  			queue-group@0 {
>  				#address-cells = <1>;
> diff --git a/arch/powerpc/boot/dts/p2020rdb.dts b/arch/powerpc/boot/dts/p2020rdb.dts
> index 1d7a05f..2941cbb 100644
> --- a/arch/powerpc/boot/dts/p2020rdb.dts
> +++ b/arch/powerpc/boot/dts/p2020rdb.dts
> @@ -213,13 +213,6 @@
>  				};
>  		};
>  
> -		mdio@25520 {
> -			tbi0: tbi-phy@11 {
> -				reg = <0x11>;
> -				device_type = "tbi-phy";
> -			};
> -		};
> -
>  		mdio@26520 {
>  			status = "disabled";
>  		};
> @@ -243,7 +236,6 @@
>  		};
>  
>  		enet1: ethernet@25000 {
> -			tbi-handle = <&tbi0>;
>  			phy-handle = <&phy0>;
>  			phy-connection-type = "sgmii";
>  		};
> diff --git a/arch/powerpc/boot/dts/p2020rdb_camp_core0.dts b/arch/powerpc/boot/dts/p2020rdb_camp_core0.dts
> index fc8dddd..4641f3d 100644
> --- a/arch/powerpc/boot/dts/p2020rdb_camp_core0.dts
> +++ b/arch/powerpc/boot/dts/p2020rdb_camp_core0.dts
> @@ -124,13 +124,6 @@
>  			};
>  		};
>  
> -		mdio@25520 {
> -			tbi0: tbi-phy@11 {
> -				reg = <0x11>;
> -				device_type = "tbi-phy";
> -			};
> -		};
> -
>  		mdio@26520 {
>  			status = "disabled";
>  		};
> @@ -140,7 +133,6 @@
>  		};
>  
>  		enet1: ethernet@25000 {
> -			tbi-handle = <&tbi0>;
>  			phy-handle = <&phy0>;
>  			phy-connection-type = "sgmii";
>  
> diff --git a/arch/powerpc/boot/dts/p2020si.dtsi b/arch/powerpc/boot/dts/p2020si.dtsi
> index 6def17f..9317075 100644
> --- a/arch/powerpc/boot/dts/p2020si.dtsi
> +++ b/arch/powerpc/boot/dts/p2020si.dtsi
> @@ -235,6 +235,10 @@
>  			#size-cells = <0>;
>  			compatible = "fsl,gianfar-tbi";
>  			reg = <0x26520 0x20>;
> +			tbi0: tbi-phy@11 {
> +				reg = <0x11>;
> +				device_type = "tbi-phy";
> +			};
>  		};
>  
>  		mdio@26520 {
> @@ -270,7 +274,7 @@
>  			local-mac-address = [ 00 00 00 00 00 00 ];
>  			interrupts = <35 2 36 2 40 2>;
>  			interrupt-parent = <&mpic>;
> -
> +			tbi-handle = <&tbi0>;
>  		};
>  
>  		enet2: ethernet@26000 {
> -- 
> 1.7.3
> 
> 
> _______________________________________________
> Linuxppc-dev mailing list
> Linuxppc-dev@lists.ozlabs.org
> https://lists.ozlabs.org/listinfo/linuxppc-dev
Kumar Gala June 22, 2011, 11:34 a.m. UTC | #2
On Jun 7, 2011, at 9:49 PM, Prabhakar Kushwaha wrote:

> ten-bit interface (TBI) module is part of SoC not board.
> 
> Move tbi entries from board related dts files to Si dts.
> 
> Signed-off-by: Prabhakar Kushwaha <prabhakar@freescale.com>
> ---
> Based upon http://git.kernel.org/pub/scm/linux/kernel/git/galak/powerpc.git (branch next)
> 
> arch/powerpc/boot/dts/p1020rdb.dts            |    9 ---------
> arch/powerpc/boot/dts/p1020rdb_camp_core0.dts |    8 --------
> arch/powerpc/boot/dts/p1020si.dtsi            |    6 +++++-
> arch/powerpc/boot/dts/p2020rdb.dts            |    8 --------
> arch/powerpc/boot/dts/p2020rdb_camp_core0.dts |    8 --------
> arch/powerpc/boot/dts/p2020si.dtsi            |    6 +++++-
> 6 files changed, 10 insertions(+), 35 deletions(-)
> 
> diff --git a/arch/powerpc/boot/dts/p1020rdb.dts b/arch/powerpc/boot/dts/p1020rdb.dts
> index d6a8ae4..a4e5d6c 100644
> --- a/arch/powerpc/boot/dts/p1020rdb.dts
> +++ b/arch/powerpc/boot/dts/p1020rdb.dts
> @@ -211,14 +211,6 @@
> 			};
> 		};
> 
> -		mdio@25000 {
> -
> -			tbi0: tbi-phy@11 {
> -				reg = <0x11>;
> -				device_type = "tbi-phy";
> -			};
> -		};
> -
> 		enet0: ethernet@b0000 {
> 			fixed-link = <1 1 1000 0 0>;
> 			phy-connection-type = "rgmii-id";
> @@ -227,7 +219,6 @@
> 
> 		enet1: ethernet@b1000 {
> 			phy-handle = <&phy0>;
> -			tbi-handle = <&tbi0>;
> 			phy-connection-type = "sgmii";
> 
> 		};

I'm not sure we should do this.  The phy address we pick is board specific so it should NOT be in .dtsi

- k
Grant Likely June 22, 2011, 4:34 p.m. UTC | #3
On Wed, Jun 22, 2011 at 5:34 AM, Kumar Gala <galak@kernel.crashing.org> wrote:
>
> On Jun 7, 2011, at 9:49 PM, Prabhakar Kushwaha wrote:
>
>> ten-bit interface (TBI) module is part of SoC not board.
>>
>> Move tbi entries from board related dts files to Si dts.
>>
>> Signed-off-by: Prabhakar Kushwaha <prabhakar@freescale.com>
>> ---
>> Based upon http://git.kernel.org/pub/scm/linux/kernel/git/galak/powerpc.git (branch next)
>>
>> arch/powerpc/boot/dts/p1020rdb.dts            |    9 ---------
>> arch/powerpc/boot/dts/p1020rdb_camp_core0.dts |    8 --------
>> arch/powerpc/boot/dts/p1020si.dtsi            |    6 +++++-
>> arch/powerpc/boot/dts/p2020rdb.dts            |    8 --------
>> arch/powerpc/boot/dts/p2020rdb_camp_core0.dts |    8 --------
>> arch/powerpc/boot/dts/p2020si.dtsi            |    6 +++++-
>> 6 files changed, 10 insertions(+), 35 deletions(-)
>>
>> diff --git a/arch/powerpc/boot/dts/p1020rdb.dts b/arch/powerpc/boot/dts/p1020rdb.dts
>> index d6a8ae4..a4e5d6c 100644
>> --- a/arch/powerpc/boot/dts/p1020rdb.dts
>> +++ b/arch/powerpc/boot/dts/p1020rdb.dts
>> @@ -211,14 +211,6 @@
>>                       };
>>               };
>>
>> -             mdio@25000 {
>> -
>> -                     tbi0: tbi-phy@11 {
>> -                             reg = <0x11>;
>> -                             device_type = "tbi-phy";
>> -                     };
>> -             };
>> -
>>               enet0: ethernet@b0000 {
>>                       fixed-link = <1 1 1000 0 0>;
>>                       phy-connection-type = "rgmii-id";
>> @@ -227,7 +219,6 @@
>>
>>               enet1: ethernet@b1000 {
>>                       phy-handle = <&phy0>;
>> -                     tbi-handle = <&tbi0>;
>>                       phy-connection-type = "sgmii";
>>
>>               };
>
> I'm not sure we should do this.  The phy address we pick is board specific so it should NOT be in .dtsi

Ah, okay.  I was under the impression that the tbi address was
internal to the SoC, and not configurable.

g.
Aggrwal Poonam-B10812 June 23, 2011, 7:24 a.m. UTC | #4
> -----Original Message-----
> From: linuxppc-dev-bounces+poonam.aggrwal=freescale.com@lists.ozlabs.org
> [mailto:linuxppc-dev-
> bounces+poonam.aggrwal=freescale.com@lists.ozlabs.org] On Behalf Of Kumar
> Gala
> Sent: Wednesday, June 22, 2011 5:04 PM
> To: Kushwaha Prabhakar-B32579
> Cc: meet2prabhu@gmail.com; devicetree-discuss@lists.ozlabs.org; linuxppc-
> dev@lists.ozlabs.org
> Subject: Re: [PATCH] powerpc/85xx:DTS: Fix tbi node location for Px020RDB
> 
> 
> On Jun 7, 2011, at 9:49 PM, Prabhakar Kushwaha wrote:
> 
> > ten-bit interface (TBI) module is part of SoC not board.
> >
> > Move tbi entries from board related dts files to Si dts.
> >
> > Signed-off-by: Prabhakar Kushwaha <prabhakar@freescale.com>
> > ---
> > Based upon
> http://git.kernel.org/pub/scm/linux/kernel/git/galak/powerpc.git (branch
> next)
> >
> > arch/powerpc/boot/dts/p1020rdb.dts            |    9 ---------
> > arch/powerpc/boot/dts/p1020rdb_camp_core0.dts |    8 --------
> > arch/powerpc/boot/dts/p1020si.dtsi            |    6 +++++-
> > arch/powerpc/boot/dts/p2020rdb.dts            |    8 --------
> > arch/powerpc/boot/dts/p2020rdb_camp_core0.dts |    8 --------
> > arch/powerpc/boot/dts/p2020si.dtsi            |    6 +++++-
> > 6 files changed, 10 insertions(+), 35 deletions(-)
> >
> > diff --git a/arch/powerpc/boot/dts/p1020rdb.dts
> b/arch/powerpc/boot/dts/p1020rdb.dts
> > index d6a8ae4..a4e5d6c 100644
> > --- a/arch/powerpc/boot/dts/p1020rdb.dts
> > +++ b/arch/powerpc/boot/dts/p1020rdb.dts
> > @@ -211,14 +211,6 @@
> > 			};
> > 		};
> >
> > -		mdio@25000 {
> > -
> > -			tbi0: tbi-phy@11 {
> > -				reg = <0x11>;
> > -				device_type = "tbi-phy";
> > -			};
> > -		};
> > -
> > 		enet0: ethernet@b0000 {
> > 			fixed-link = <1 1 1000 0 0>;
> > 			phy-connection-type = "rgmii-id";
> > @@ -227,7 +219,6 @@
> >
> > 		enet1: ethernet@b1000 {
> > 			phy-handle = <&phy0>;
> > -			tbi-handle = <&tbi0>;
> > 			phy-connection-type = "sgmii";
> >
> > 		};
> 
> I'm not sure we should do this.  The phy address we pick is board
> specific so it should NOT be in .dtsi
The TBI phy and it's address is internal to SOC.

Regards
Poonam
> 
> - k
> _______________________________________________
> Linuxppc-dev mailing list
> Linuxppc-dev@lists.ozlabs.org
> https://lists.ozlabs.org/listinfo/linuxppc-dev
Kumar Gala June 23, 2011, 10:27 a.m. UTC | #5
On Jun 23, 2011, at 2:24 AM, Aggrwal Poonam-B10812 wrote:

> 
> 
>> -----Original Message-----
>> From: linuxppc-dev-bounces+poonam.aggrwal=freescale.com@lists.ozlabs.org
>> [mailto:linuxppc-dev-
>> bounces+poonam.aggrwal=freescale.com@lists.ozlabs.org] On Behalf Of Kumar
>> Gala
>> Sent: Wednesday, June 22, 2011 5:04 PM
>> To: Kushwaha Prabhakar-B32579
>> Cc: meet2prabhu@gmail.com; devicetree-discuss@lists.ozlabs.org; linuxppc-
>> dev@lists.ozlabs.org
>> Subject: Re: [PATCH] powerpc/85xx:DTS: Fix tbi node location for Px020RDB
>> 
>> 
>> On Jun 7, 2011, at 9:49 PM, Prabhakar Kushwaha wrote:
>> 
>>> ten-bit interface (TBI) module is part of SoC not board.
>>> 
>>> Move tbi entries from board related dts files to Si dts.
>>> 
>>> Signed-off-by: Prabhakar Kushwaha <prabhakar@freescale.com>
>>> ---
>>> Based upon
>> http://git.kernel.org/pub/scm/linux/kernel/git/galak/powerpc.git (branch
>> next)
>>> 
>>> arch/powerpc/boot/dts/p1020rdb.dts            |    9 ---------
>>> arch/powerpc/boot/dts/p1020rdb_camp_core0.dts |    8 --------
>>> arch/powerpc/boot/dts/p1020si.dtsi            |    6 +++++-
>>> arch/powerpc/boot/dts/p2020rdb.dts            |    8 --------
>>> arch/powerpc/boot/dts/p2020rdb_camp_core0.dts |    8 --------
>>> arch/powerpc/boot/dts/p2020si.dtsi            |    6 +++++-
>>> 6 files changed, 10 insertions(+), 35 deletions(-)
>>> 
>>> diff --git a/arch/powerpc/boot/dts/p1020rdb.dts
>> b/arch/powerpc/boot/dts/p1020rdb.dts
>>> index d6a8ae4..a4e5d6c 100644
>>> --- a/arch/powerpc/boot/dts/p1020rdb.dts
>>> +++ b/arch/powerpc/boot/dts/p1020rdb.dts
>>> @@ -211,14 +211,6 @@
>>> 			};
>>> 		};
>>> 
>>> -		mdio@25000 {
>>> -
>>> -			tbi0: tbi-phy@11 {
>>> -				reg = <0x11>;
>>> -				device_type = "tbi-phy";
>>> -			};
>>> -		};
>>> -
>>> 		enet0: ethernet@b0000 {
>>> 			fixed-link = <1 1 1000 0 0>;
>>> 			phy-connection-type = "rgmii-id";
>>> @@ -227,7 +219,6 @@
>>> 
>>> 		enet1: ethernet@b1000 {
>>> 			phy-handle = <&phy0>;
>>> -			tbi-handle = <&tbi0>;
>>> 			phy-connection-type = "sgmii";
>>> 
>>> 		};
>> 
>> I'm not sure we should do this.  The phy address we pick is board
>> specific so it should NOT be in .dtsi
> The TBI phy and it's address is internal to SOC.

But it is configured to NOT conflict with external PHY addresses.  This the setting is board specific in my opinion.

- k
diff mbox

Patch

diff --git a/arch/powerpc/boot/dts/p1020rdb.dts b/arch/powerpc/boot/dts/p1020rdb.dts
index d6a8ae4..a4e5d6c 100644
--- a/arch/powerpc/boot/dts/p1020rdb.dts
+++ b/arch/powerpc/boot/dts/p1020rdb.dts
@@ -211,14 +211,6 @@ 
 			};
 		};
 
-		mdio@25000 {
-
-			tbi0: tbi-phy@11 {
-				reg = <0x11>;
-				device_type = "tbi-phy";
-			};
-		};
-
 		enet0: ethernet@b0000 {
 			fixed-link = <1 1 1000 0 0>;
 			phy-connection-type = "rgmii-id";
@@ -227,7 +219,6 @@ 
 
 		enet1: ethernet@b1000 {
 			phy-handle = <&phy0>;
-			tbi-handle = <&tbi0>;
 			phy-connection-type = "sgmii";
 
 		};
diff --git a/arch/powerpc/boot/dts/p1020rdb_camp_core0.dts b/arch/powerpc/boot/dts/p1020rdb_camp_core0.dts
index f0bf7f4..abab234 100644
--- a/arch/powerpc/boot/dts/p1020rdb_camp_core0.dts
+++ b/arch/powerpc/boot/dts/p1020rdb_camp_core0.dts
@@ -114,20 +114,12 @@ 
 			};
 		};
 
-		mdio@25000 {
-			tbi0: tbi-phy@11 {
-				reg = <0x11>;
-				device_type = "tbi-phy";
-			};
-		};
-
 		enet0: ethernet@b0000 {
 			status = "disabled";
 		};
 
 		enet1: ethernet@b1000 {
 			phy-handle = <&phy0>;
-			tbi-handle = <&tbi0>;
 			phy-connection-type = "sgmii";
 		};
 
diff --git a/arch/powerpc/boot/dts/p1020si.dtsi b/arch/powerpc/boot/dts/p1020si.dtsi
index 5c5acb6..7844d2e 100644
--- a/arch/powerpc/boot/dts/p1020si.dtsi
+++ b/arch/powerpc/boot/dts/p1020si.dtsi
@@ -190,7 +190,10 @@ 
 			#size-cells = <0>;
 			compatible = "fsl,etsec2-tbi";
 			reg = <0x25000 0x1000 0xb1030 0x4>;
-
+			tbi0: tbi-phy@11 {
+				reg = <0x11>;
+				device_type = "tbi-phy";
+			};
 		};
 
 		enet0: ethernet@b0000 {
@@ -229,6 +232,7 @@ 
 			fsl,num_tx_queues = <0x8>;
 			local-mac-address = [ 00 00 00 00 00 00 ];
 			interrupt-parent = <&mpic>;
+			tbi-handle = <&tbi0>;
 
 			queue-group@0 {
 				#address-cells = <1>;
diff --git a/arch/powerpc/boot/dts/p2020rdb.dts b/arch/powerpc/boot/dts/p2020rdb.dts
index 1d7a05f..2941cbb 100644
--- a/arch/powerpc/boot/dts/p2020rdb.dts
+++ b/arch/powerpc/boot/dts/p2020rdb.dts
@@ -213,13 +213,6 @@ 
 				};
 		};
 
-		mdio@25520 {
-			tbi0: tbi-phy@11 {
-				reg = <0x11>;
-				device_type = "tbi-phy";
-			};
-		};
-
 		mdio@26520 {
 			status = "disabled";
 		};
@@ -243,7 +236,6 @@ 
 		};
 
 		enet1: ethernet@25000 {
-			tbi-handle = <&tbi0>;
 			phy-handle = <&phy0>;
 			phy-connection-type = "sgmii";
 		};
diff --git a/arch/powerpc/boot/dts/p2020rdb_camp_core0.dts b/arch/powerpc/boot/dts/p2020rdb_camp_core0.dts
index fc8dddd..4641f3d 100644
--- a/arch/powerpc/boot/dts/p2020rdb_camp_core0.dts
+++ b/arch/powerpc/boot/dts/p2020rdb_camp_core0.dts
@@ -124,13 +124,6 @@ 
 			};
 		};
 
-		mdio@25520 {
-			tbi0: tbi-phy@11 {
-				reg = <0x11>;
-				device_type = "tbi-phy";
-			};
-		};
-
 		mdio@26520 {
 			status = "disabled";
 		};
@@ -140,7 +133,6 @@ 
 		};
 
 		enet1: ethernet@25000 {
-			tbi-handle = <&tbi0>;
 			phy-handle = <&phy0>;
 			phy-connection-type = "sgmii";
 
diff --git a/arch/powerpc/boot/dts/p2020si.dtsi b/arch/powerpc/boot/dts/p2020si.dtsi
index 6def17f..9317075 100644
--- a/arch/powerpc/boot/dts/p2020si.dtsi
+++ b/arch/powerpc/boot/dts/p2020si.dtsi
@@ -235,6 +235,10 @@ 
 			#size-cells = <0>;
 			compatible = "fsl,gianfar-tbi";
 			reg = <0x26520 0x20>;
+			tbi0: tbi-phy@11 {
+				reg = <0x11>;
+				device_type = "tbi-phy";
+			};
 		};
 
 		mdio@26520 {
@@ -270,7 +274,7 @@ 
 			local-mac-address = [ 00 00 00 00 00 00 ];
 			interrupts = <35 2 36 2 40 2>;
 			interrupt-parent = <&mpic>;
-
+			tbi-handle = <&tbi0>;
 		};
 
 		enet2: ethernet@26000 {