Commit 4c93566e authored by David S. Miller's avatar David S. Miller

[TULIP] DMFE: Fix SROM parsing regression.

Changeset 16b110c3 (dmfe warning fix)
bothed up the offsets read from the SROM so that it doesn't read the
same datums it used to.

The change made transformations like turning:

	"srom + 34"

into

	"(__le32 *)srom + 34/4"

which doesn't work because 4 does not divide evenly
into 34 so we're using a different pointer offset
than in the original code.

I've changed theses cases in dmfe_parse_srom() to
consistently use "(type *)(srom + offset)" preserving
the offsets from the original code.
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent acea6852
...@@ -1909,7 +1909,7 @@ static void dmfe_parse_srom(struct dmfe_board_info * db) ...@@ -1909,7 +1909,7 @@ static void dmfe_parse_srom(struct dmfe_board_info * db)
if ( ( (int) srom[18] & 0xff) == SROM_V41_CODE) { if ( ( (int) srom[18] & 0xff) == SROM_V41_CODE) {
/* SROM V4.01 */ /* SROM V4.01 */
/* Get NIC support media mode */ /* Get NIC support media mode */
db->NIC_capability = le16_to_cpup((__le16 *)srom + 34/2); db->NIC_capability = le16_to_cpup((__le16 *) (srom + 34));
db->PHY_reg4 = 0; db->PHY_reg4 = 0;
for (tmp_reg = 1; tmp_reg < 0x10; tmp_reg <<= 1) { for (tmp_reg = 1; tmp_reg < 0x10; tmp_reg <<= 1) {
switch( db->NIC_capability & tmp_reg ) { switch( db->NIC_capability & tmp_reg ) {
...@@ -1921,8 +1921,8 @@ static void dmfe_parse_srom(struct dmfe_board_info * db) ...@@ -1921,8 +1921,8 @@ static void dmfe_parse_srom(struct dmfe_board_info * db)
} }
/* Media Mode Force or not check */ /* Media Mode Force or not check */
dmfe_mode = le32_to_cpup((__le32 *)srom + 34/4) & dmfe_mode = (le32_to_cpup((__le32 *) (srom + 34)) &
le32_to_cpup((__le32 *)srom + 36/4); le32_to_cpup((__le32 *) (srom + 36)));
switch(dmfe_mode) { switch(dmfe_mode) {
case 0x4: dmfe_media_mode = DMFE_100MHF; break; /* 100MHF */ case 0x4: dmfe_media_mode = DMFE_100MHF; break; /* 100MHF */
case 0x2: dmfe_media_mode = DMFE_10MFD; break; /* 10MFD */ case 0x2: dmfe_media_mode = DMFE_10MFD; break; /* 10MFD */
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment