linux-kernel - Re: [PATCH 1/2] net: cadence: macb: add support for the WOL

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <53C3ADB9.4010309@gmail.com>
Date:	Mon, 14 Jul 2014 15:45:21 +0530
From:	Varka Bhadram <varkabhadram@...il.com>
To:	Jongsung Kim <neidhard.kim@....com>, nicolas.ferre@...el.com
CC:	netdev@...r.kernel.org, linux-kernel@...r.kernel.org,
	davem@...emloft.net
Subject: Re: [PATCH 1/2] net: cadence: macb: add support for the WOL

On 07/14/2014 02:32 PM, Jongsung Kim wrote:
> This patch enables the ethtool utility to control the WOL function
> of the PHY connected to the GEM/MACB. (if supported)
>
> Signed-off-by: Jongsung Kim <neidhard.kim@....com>
> ---
>   drivers/net/ethernet/cadence/macb.c |   26 ++++++++++++++++++++++++++
>   1 files changed, 26 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/net/ethernet/cadence/macb.c b/drivers/net/ethernet/cadence/macb.c
> index e9daa07..7ad8909 100644
> --- a/drivers/net/ethernet/cadence/macb.c
> +++ b/drivers/net/ethernet/cadence/macb.c
> @@ -1742,11 +1742,37 @@ static void macb_get_regs(struct net_device *dev, struct ethtool_regs *regs,
>   	}
>   }
>   
> +static void macb_get_wol(struct net_device *netdev, struct ethtool_wolinfo *wol)
> +{
> +	struct macb *bp = netdev_priv(netdev);
> +	struct phy_device *phydev = bp->phy_dev;
> +
> +	wol->supported = 0;
> +	wol->wolopts = 0;
> +
> +	if (phydev)
> +		phy_ethtool_get_wol(phydev, wol);
> +}
> +
> +static int macb_set_wol(struct net_device *netdev, struct ethtool_wolinfo *wol)
> +{
> +	struct macb *bp = netdev_priv(netdev);
> +	struct phy_device *phydev = bp->phy_dev;
> +	int err = -ENODEV;
> +
> +	if (phydev)
> +		err = phy_ethtool_set_wol(phydev, wol);
> +
> +	return err;
> +}
> +

I think we can do in this way:

if (phydev)
	return phy_ethtool_set_wol(phydev, wol);
else
	return -ENODEV;


we can save err. What do you say ...?

>   const struct ethtool_ops macb_ethtool_ops = {
>   	.get_settings		= macb_get_settings,
>   	.set_settings		= macb_set_settings,
>   	.get_regs_len		= macb_get_regs_len,
>   	.get_regs		= macb_get_regs,
> +	.get_wol		= macb_get_wol,
> +	.set_wol		= macb_set_wol,
>   	.get_link		= ethtool_op_get_link,
>   	.get_ts_info		= ethtool_op_get_ts_info,
>   };


-- 
Regards,
Varka Bhadram.

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