ppc4xx: Fixed compilation warning in 4xx_enet.c
authorAlessio Centazzo <centazzo@gmail.com>
Sat, 11 Jul 2009 18:56:06 +0000 (11:56 -0700)
committerBen Warren <biggerbadderben@gmail.com>
Thu, 23 Jul 2009 05:53:45 +0000 (22:53 -0700)
commit0544c63681d2ea3607faf374e9c56f101e365b42
treecde444dd83ac90232f33dd2f4348814cf152f2b1
parent09133f8580f0106429ba3600f1855bd3577ae58b
ppc4xx: Fixed compilation warning in 4xx_enet.c

This patch fixes a compilation warning for some Ethernet PHY-less
PPC4xx platforms (440SPE based ones) and a potential compilation
error for 440SP platforms (use of undefined 'ethgroup' variable).
In the original code and in case of 440SPE platforms, 'ethgroup'
is initialized to -1 and never modified.  Later in the function,
within an #ifdef statement, an 'if statement' executes code only
if 'ethgroup' is set to 4, therefore it is harmless to avoid
executing the 'if statement' by removing the CONFIG_440SPE from
the affected #ifdefs.  In case of 440SP platforms  with on-board
Ethernet PHY, 'ethgroup' is undefined but used (there are not such
platforms in the repository yet). All other architectures are not
affected by this change.

Signed-off-by: Alessio Centazzo acpatin@yahoo.com
Acked-by: Stefan Roese <sr@denx.de>
Signed-off-by: Ben Warren <biggerbadderben@gmail.com>
drivers/net/4xx_enet.c