Commit 3a4d000a authored by Yogesh Hegde's avatar Yogesh Hegde Committed by Greg Kroah-Hartman

staging: rtl8192e: Remove unused rf_set_sens variable

The rf_set_sens variable is declared but never set, so it is always NULL.
This commit cleans up the unused rf_set_sens variable and removes the
associated code that relied on it.

Specifically, the following changes have been made:
- Removed the `range->sensitivity` assignment, which was never used.
- Removed the sensitivity level get implementation, which always returns
  an error code.
- Removed the sensitivity level set implementation, which always returns
  an error code.
- Removed the `rf_set_sens` variable.
Suggested-by: default avatarPhilipp Hortmann <philipp.g.hortmann@gmail.com>
Signed-off-by: default avatarYogesh Hegde <yogi.kernel@gmail.com>
Tested-by: default avatarPhilipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/5a9767fab76b1836ea7881994ffb3593c1ab12bf.1682358035.git.yogi.kernel@gmail.comSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 77f2760e
...@@ -231,7 +231,6 @@ struct r8192_priv { ...@@ -231,7 +231,6 @@ struct r8192_priv {
struct rt_stats stats; struct rt_stats stats;
struct iw_statistics wstats; struct iw_statistics wstats;
short (*rf_set_sens)(struct net_device *dev, short sens);
u8 (*rf_set_chan)(struct net_device *dev, u8 ch); u8 (*rf_set_chan)(struct net_device *dev, u8 ch);
struct rx_desc *rx_ring[MAX_RX_QUEUE]; struct rx_desc *rx_ring[MAX_RX_QUEUE];
......
...@@ -311,10 +311,6 @@ static int _rtl92e_wx_get_range(struct net_device *dev, ...@@ -311,10 +311,6 @@ static int _rtl92e_wx_get_range(struct net_device *dev,
/* ~130 Mb/s real (802.11n) */ /* ~130 Mb/s real (802.11n) */
range->throughput = 130 * 1000 * 1000; range->throughput = 130 * 1000 * 1000;
if (priv->rf_set_sens != NULL)
/* signal level threshold range */
range->sensitivity = priv->max_sens;
range->max_qual.qual = 100; range->max_qual.qual = 100;
range->max_qual.level = 0; range->max_qual.level = 0;
range->max_qual.noise = 0; range->max_qual.noise = 0;
...@@ -813,10 +809,7 @@ static int _rtl92e_wx_get_sens(struct net_device *dev, ...@@ -813,10 +809,7 @@ static int _rtl92e_wx_get_sens(struct net_device *dev,
{ {
struct r8192_priv *priv = rtllib_priv(dev); struct r8192_priv *priv = rtllib_priv(dev);
if (priv->rf_set_sens == NULL) return -1;
return -1; /* we have not this support for this radio */
wrqu->sens.value = priv->sens;
return 0;
} }
static int _rtl92e_wx_set_sens(struct net_device *dev, static int _rtl92e_wx_set_sens(struct net_device *dev,
...@@ -831,15 +824,7 @@ static int _rtl92e_wx_set_sens(struct net_device *dev, ...@@ -831,15 +824,7 @@ static int _rtl92e_wx_set_sens(struct net_device *dev,
return 0; return 0;
mutex_lock(&priv->wx_mutex); mutex_lock(&priv->wx_mutex);
if (priv->rf_set_sens == NULL) { err = -1;
err = -1; /* we have not this support for this radio */
goto exit;
}
if (priv->rf_set_sens(dev, wrqu->sens.value) == 0)
priv->sens = wrqu->sens.value;
else
err = -EINVAL;
exit: exit:
mutex_unlock(&priv->wx_mutex); mutex_unlock(&priv->wx_mutex);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment