| Message ID | 1354038820-11095-1-git-send-email-mugunthanvnm@ti.com |
|---|---|
| State | Accepted, archived |
| Delegated to: | David Miller |
| Headers | show |
On Tue, Nov 27, 2012 at 11:23:40PM +0530, Mugunthan V N wrote: > CC drivers/net/ethernet/ti/cpsw.o > drivers/net/ethernet/ti/cpsw.c: In function 'cpsw_ndo_ioctl': > drivers/net/ethernet/ti/cpsw.c:881:20: warning: unused variable 'priv' > > The build warning is generated when CPTS is not selected in Kernel Build. > Fixing by passing the net_device pointer to cpts IOCTL instead of passing priv > > Signed-off-by: Mugunthan V N <mugunthanvnm@ti.com> Thanks for this fix. Acked-by: Richard Cochran <richardcochran@gmail.com> -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
From: Mugunthan V N <mugunthanvnm@ti.com> Date: Tue, 27 Nov 2012 23:23:40 +0530 > CC drivers/net/ethernet/ti/cpsw.o > drivers/net/ethernet/ti/cpsw.c: In function 'cpsw_ndo_ioctl': > drivers/net/ethernet/ti/cpsw.c:881:20: warning: unused variable 'priv' > > The build warning is generated when CPTS is not selected in Kernel Build. > Fixing by passing the net_device pointer to cpts IOCTL instead of passing priv > > Signed-off-by: Mugunthan V N <mugunthanvnm@ti.com> Applied. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/net/ethernet/ti/cpsw.c b/drivers/net/ethernet/ti/cpsw.c index 02c2477..c9714e1 100644 --- a/drivers/net/ethernet/ti/cpsw.c +++ b/drivers/net/ethernet/ti/cpsw.c @@ -812,8 +812,9 @@ static void cpsw_hwtstamp_v2(struct cpsw_priv *priv) __raw_writel(ETH_P_1588, &priv->regs->ts_ltype); } -static int cpsw_hwtstamp_ioctl(struct cpsw_priv *priv, struct ifreq *ifr) +static int cpsw_hwtstamp_ioctl(struct net_device *dev, struct ifreq *ifr) { + struct cpsw_priv *priv = netdev_priv(dev); struct cpts *cpts = &priv->cpts; struct hwtstamp_config cfg; @@ -878,14 +879,12 @@ static int cpsw_hwtstamp_ioctl(struct cpsw_priv *priv, struct ifreq *ifr) static int cpsw_ndo_ioctl(struct net_device *dev, struct ifreq *req, int cmd) { - struct cpsw_priv *priv = netdev_priv(dev); - if (!netif_running(dev)) return -EINVAL; #ifdef CONFIG_TI_CPTS if (cmd == SIOCSHWTSTAMP) - return cpsw_hwtstamp_ioctl(priv, req); + return cpsw_hwtstamp_ioctl(dev, req); #endif return -ENOTSUPP; }
CC drivers/net/ethernet/ti/cpsw.o drivers/net/ethernet/ti/cpsw.c: In function 'cpsw_ndo_ioctl': drivers/net/ethernet/ti/cpsw.c:881:20: warning: unused variable 'priv' The build warning is generated when CPTS is not selected in Kernel Build. Fixing by passing the net_device pointer to cpts IOCTL instead of passing priv Signed-off-by: Mugunthan V N <mugunthanvnm@ti.com> --- drivers/net/ethernet/ti/cpsw.c | 7 +++---- 1 files changed, 3 insertions(+), 4 deletions(-)