Patchwork [v3,1/2] powerpc/44x: Add new compatible value for EMAC node of APM821XX dts file.

login
register
mail settings
Submitter Duc Dang
Date March 5, 2012, 10:57 a.m.
Message ID <1330945062-15760-1-git-send-email-dhdang@apm.com>
Download mbox | patch
Permalink /patch/144651/
State Accepted
Delegated to: Josh Boyer
Headers show

Comments

Duc Dang - March 5, 2012, 10:57 a.m.
This compatible value will be used to distinguish some special features of APM821XX EMAC: no half duplex mode support, configuring jumbo frame.

Signed-off-by: Duc Dang <dhdang@apm.com>
---
v3:
 No change since v1/v2 patch sets. Added for completeness.

 arch/powerpc/boot/dts/bluestone.dts |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
David Miller - March 6, 2012, 10:08 p.m.
From: Duc Dang <dhdang@apm.com>
Date: Mon,  5 Mar 2012 17:57:42 +0700

> This compatible value will be used to distinguish some special features of APM821XX EMAC: no half duplex mode support, configuring jumbo frame.
> 
> Signed-off-by: Duc Dang <dhdang@apm.com>

Applied.
Josh Boyer - March 6, 2012, 10:15 p.m.
On Tue, Mar 6, 2012 at 5:08 PM, David Miller <davem@davemloft.net> wrote:
> From: Duc Dang <dhdang@apm.com>
> Date: Mon,  5 Mar 2012 17:57:42 +0700
>
>> This compatible value will be used to distinguish some special features of APM821XX EMAC: no half duplex mode support, configuring jumbo frame.
>>
>> Signed-off-by: Duc Dang <dhdang@apm.com>
>
> Applied.

So I guess we're taking these through your tree?  I guess that works out
well enough since the other patches I was going to send to Ben aren't
panning out anyway.

josh

Patch

diff --git a/arch/powerpc/boot/dts/bluestone.dts b/arch/powerpc/boot/dts/bluestone.dts
index 2a56a0d..74876f7 100644
--- a/arch/powerpc/boot/dts/bluestone.dts
+++ b/arch/powerpc/boot/dts/bluestone.dts
@@ -222,7 +222,7 @@ 
 
 			EMAC0: ethernet@ef600c00 {
 				device_type = "network";
-				compatible = "ibm,emac4sync";
+				compatible = "ibm,emac-apm821xx", "ibm,emac4sync";
 				interrupt-parent = <&EMAC0>;
 				interrupts = <0x0 0x1>;
 				#interrupt-cells = <1>;