On Tuesday 20 August 2013 11:08 AM, ujhely...@gmail.com wrote: > From: Matus Ujhelyi <ujhely...@gmail.com> > > Some phy's can be configured to enable wake on lan (e.g. at803x or marvell > 88E1318S). > There is no way how to enable wol on CPSW with such connected phys. This patch > adds this support. It is provided by calling the phy's related code. > > Tested on board with at8030x connected phy. Wol interrupt line is > connected to GPIO0 on am335x. > > Signed-off-by: Matus Ujhelyi <ujhely...@gmail.com> > --- > drivers/net/ethernet/ti/cpsw.c | 24 ++++++++++++++++++++++++ > 1 file changed, 24 insertions(+) > > diff --git a/drivers/net/ethernet/ti/cpsw.c b/drivers/net/ethernet/ti/cpsw.c > index 0fcf212..5d673bf 100644 > --- a/drivers/net/ethernet/ti/cpsw.c > +++ b/drivers/net/ethernet/ti/cpsw.c > @@ -1640,6 +1640,28 @@ static int cpsw_set_settings(struct net_device *ndev, > struct ethtool_cmd *ecmd) > return -EOPNOTSUPP; > } > > +static void cpsw_get_wol(struct net_device *ndev, struct ethtool_wolinfo > *wol) > +{ > + struct cpsw_priv *priv = netdev_priv(ndev); > + int slave_no = cpsw_slave_index(priv); > + wol->supported = 0; > + wol->wolopts = 0;
Please add a blank line after variable declaration before you start updating wol structure as commented by Francois and Sergei. Regards Mugunthan V N -- 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/