diff options
author | Hayes Wang <hayeswang@realtek.com> | 2021-02-19 17:04:41 +0800 |
---|---|---|
committer | Jakub Kicinski <kuba@kernel.org> | 2021-02-23 12:36:02 -0800 |
commit | c79515e47935c747282c6ed2ee5b2ef039756eeb (patch) | |
tree | b776cd518406712772d53fd77370b1ee5238be59 /drivers/net | |
parent | 7a0ae61acde2cebd69665837170405eced86a6c7 (diff) | |
download | linux-stable-c79515e47935c747282c6ed2ee5b2ef039756eeb.tar.gz linux-stable-c79515e47935c747282c6ed2ee5b2ef039756eeb.tar.bz2 linux-stable-c79515e47935c747282c6ed2ee5b2ef039756eeb.zip |
r8152: check if the pointer of the function exists
Return error code if autosuspend_en, eee_get, or eee_set don't exist.
Signed-off-by: Hayes Wang <hayeswang@realtek.com>
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Diffstat (limited to 'drivers/net')
-rw-r--r-- | drivers/net/usb/r8152.c | 13 |
1 files changed, 13 insertions, 0 deletions
diff --git a/drivers/net/usb/r8152.c b/drivers/net/usb/r8152.c index 4bfee289aa6f..baa63ea2590a 100644 --- a/drivers/net/usb/r8152.c +++ b/drivers/net/usb/r8152.c @@ -5757,6 +5757,9 @@ static int rtl8152_runtime_suspend(struct r8152 *tp) struct net_device *netdev = tp->netdev; int ret = 0; + if (!tp->rtl_ops.autosuspend_en) + return -EBUSY; + set_bit(SELECTIVE_SUSPEND, &tp->flags); smp_mb__after_atomic(); @@ -6156,6 +6159,11 @@ rtl_ethtool_get_eee(struct net_device *net, struct ethtool_eee *edata) struct r8152 *tp = netdev_priv(net); int ret; + if (!tp->rtl_ops.eee_get) { + ret = -EOPNOTSUPP; + goto out; + } + ret = usb_autopm_get_interface(tp->intf); if (ret < 0) goto out; @@ -6178,6 +6186,11 @@ rtl_ethtool_set_eee(struct net_device *net, struct ethtool_eee *edata) struct r8152 *tp = netdev_priv(net); int ret; + if (!tp->rtl_ops.eee_set) { + ret = -EOPNOTSUPP; + goto out; + } + ret = usb_autopm_get_interface(tp->intf); if (ret < 0) goto out; |