From: Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx> Date: Thu, 22 Jul 2010 12:06:39 +1000 > After merging the net tree, today's linux-next build (x86_64 allmodconfig) > produced this warning: > > drivers/net/r6040.c: In function 'r6040_ioctl': > drivers/net/r6040.c:513: warning: passing argument 2 of 'phy_mii_ioctl' from incompatible pointer type > include/linux/phy.h:522: note: expected 'struct ifreq *' but argument is of type 'struct mii_ioctl_data *' > > Introduced by commit 28b041139e344ecd0f144d6205b004ae354cfa1e ("net: > preserve ifreq parameter when calling generic phy_mii_ioctl()") (which > changed the phy_mii_ioctl() API) interacting with commit > 3831861b4ad8fd0ad7110048eb3e155628799d2b ("r6040: implement phylib") > (which added a use of that function). Thanks Stephen, should be fixed as follows: -------------------- r6040: Fix args to phy_mii_ioctl(). Reported by Stephen Rothwell. Signed-off-by: David S. Miller <davem@xxxxxxxxxxxxx> --- drivers/net/r6040.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/net/r6040.c b/drivers/net/r6040.c index 7d482a2..142c381 100644 --- a/drivers/net/r6040.c +++ b/drivers/net/r6040.c @@ -510,7 +510,7 @@ static int r6040_ioctl(struct net_device *dev, struct ifreq *rq, int cmd) if (!lp->phydev) return -EINVAL; - return phy_mii_ioctl(lp->phydev, if_mii(rq), cmd); + return phy_mii_ioctl(lp->phydev, rq, cmd); } static int r6040_rx(struct net_device *dev, int limit) -- 1.7.1.1 -- To unsubscribe from this list: send the line "unsubscribe linux-next" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html