On 12/06/2012 05:16 PM, Cong Ding wrote:
the ".config_intr" is defined twice in both line 208 and 212.

Signed-off-by: Cong Ding <ding...@gmail.com>
---
  drivers/net/phy/smsc.c |    1 -
  1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/drivers/net/phy/smsc.c b/drivers/net/phy/smsc.c
index 16dceed..5cee6bd 100644
--- a/drivers/net/phy/smsc.c
+++ b/drivers/net/phy/smsc.c
@@ -205,7 +205,6 @@ static struct phy_driver smsc_phy_driver[] = {
        /* basic functions */
        .config_aneg    = genphy_config_aneg,
        .read_status    = lan87xx_read_status,
-       .config_intr    = smsc_phy_config_intr,
/* IRQ related */
        .ack_interrupt  = smsc_phy_ack_interrupt,

Hi Cong,

That looks like a mistake from my previous patch to that file. Copy and paste fail.

The line should read:
.config_init    = smsc_phy_config_init,

I can submit a patch once I get off work unless you beat me to it.

regards,
Patrick
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to