Skip to contentSkip to conversation

ethtool: add names of newer Marvell chips

PATCH Applied Stephen Hemminger 3 participants 4 messages Apr 2, 2010 – Apr 6, 2010

Subsystems (2)
Networking Network drivers
lore
Share & tools
lore

Stephen Hemminger

author
#0 lore raw

PATCH 1/1 ethtool: add names of newer Marvell chips

Fill in names of newer chips.

Signed-off-by Stephen Hemminger <[email protected]>
marvell.c +3 -0
index 6696e0a..af38c21 100644
@@ -184,6 +184,9 @@ static void dump_mac(const u8 *r)
 	case 0xb6:	printf("Yukon-2 EC");	break;
  	case 0xb7:	printf("Yukon-2 FE");	break;
 	case 0xb8:	printf("Yukon-2 FE Plus"); break;
+	case 0xb9:	printf("Yukon Supreme"); break;
+	case 0xba:	printf("Yukon Ultra 2"); break;
+	case 0xbc:	printf("Yukon Optima"); break;
 	default:	printf("(Unknown)");	break;
 	}
Signature · 1 line
1.6.3.3
#1 lore raw

Hi,

marvell.c static void dump_mac(const u8 *r) · line 189

Quoted 2 lines
marvell.c +2 -0
+       case 0xba:      printf("Yukon Ultra 2"); break;
+       case 0xbc:      printf("Yukon Optima"); break;

What about 0xbb?
Is there ant reason for not using 0xbb for Yukon Optima?

Is it something with blackberry (bb)? :-)
Mark

Quoted 27 lines · Stephen Hemminger

Fill in names of newer chips.

Signed-off-by Stephen Hemminger <[email protected]>
marvell.c +3 -1
index 6696e0a..af38c21 100644
@@ -184,6 +184,9 @@ static void dump_mac(const u8 *r)
        case 0xb6:      printf("Yukon-2 EC");   break;
        case 0xb7:      printf("Yukon-2 FE");   break;
        case 0xb8:      printf("Yukon-2 FE Plus"); break;
+       case 0xb9:      printf("Yukon Supreme"); break;
+       case 0xba:      printf("Yukon Ultra 2"); break;
+       case 0xbc:      printf("Yukon Optima"); break;
        default:        printf("(Unknown)");    break;
        }
 
--

1.6.3.3

--
To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

author
#2 lore raw
Quoted 9 lines · On Sat, 3 Apr 2010 10:20:40 +0300 Mark Ryden <[email protected]> wrote:

Hi,

marvell.c static void dump_mac(const u8 *r) · line 189

Quoted 2 lines
marvell.c +2 -0
+       case 0xba:      printf("Yukon Ultra 2"); break;
+       case 0xbc:      printf("Yukon Optima"); break;

What about 0xbb?
Is there ant reason for not using 0xbb for Yukon Optima?

Is it something with blackberry (bb)? :-)

The value comes from a chip register.
The vendor hardware engineers didn't choose to use that version yet.