diff mbox series

rt2x00: remove unused delayed work data from link description

Message ID 20240515152824.143537-1-dmantipov@yandex.ru (mailing list archive)
State Accepted
Commit 2785ea9673a7305abeea87111849a4e04b0f4626
Delegated to: Kalle Valo
Headers show
Series rt2x00: remove unused delayed work data from link description | expand

Commit Message

Dmitry Antipov May 15, 2024, 3:28 p.m. UTC
Remove unused 'agc_work' and 'vco_work' members of 'struct link'.
This follows commit d96324703ffa ("rt2x00: merge agc and vco works
with link tuner") where AGC and VCO calibration code was completely
redesigned. Compile tested only.

Signed-off-by: Dmitry Antipov <dmantipov@yandex.ru>
---
 drivers/net/wireless/ralink/rt2x00/rt2x00.h | 10 ----------
 1 file changed, 10 deletions(-)

Comments

Stanislaw Gruszka May 18, 2024, 9:32 a.m. UTC | #1
On Wed, May 15, 2024 at 06:28:24PM +0300, Dmitry Antipov wrote:
> Remove unused 'agc_work' and 'vco_work' members of 'struct link'.
> This follows commit d96324703ffa ("rt2x00: merge agc and vco works
> with link tuner") where AGC and VCO calibration code was completely
> redesigned. Compile tested only.
> 
> Signed-off-by: Dmitry Antipov <dmantipov@yandex.ru>

Acked-by: Stanislaw Gruszka <stf_xl@wp.pl>


> ---
>  drivers/net/wireless/ralink/rt2x00/rt2x00.h | 10 ----------
>  1 file changed, 10 deletions(-)
> 
> diff --git a/drivers/net/wireless/ralink/rt2x00/rt2x00.h b/drivers/net/wireless/ralink/rt2x00/rt2x00.h
> index 82af01448a0a..bb648f95dfdd 100644
> --- a/drivers/net/wireless/ralink/rt2x00/rt2x00.h
> +++ b/drivers/net/wireless/ralink/rt2x00/rt2x00.h
> @@ -335,16 +335,6 @@ struct link {
>  	struct delayed_work watchdog_work;
>  	unsigned int watchdog_interval;
>  	unsigned int watchdog;
> -
> -	/*
> -	 * Work structure for scheduling periodic AGC adjustments.
> -	 */
> -	struct delayed_work agc_work;
> -
> -	/*
> -	 * Work structure for scheduling periodic VCO calibration.
> -	 */
> -	struct delayed_work vco_work;
>  };
>  
>  enum rt2x00_delayed_flags {
> -- 
> 2.45.0
>
Kalle Valo May 20, 2024, 5:06 a.m. UTC | #2
Dmitry Antipov <dmantipov@yandex.ru> wrote:

> Remove unused 'agc_work' and 'vco_work' members of 'struct link'.
> This follows commit d96324703ffa ("rt2x00: merge agc and vco works
> with link tuner") where AGC and VCO calibration code was completely
> redesigned. Compile tested only.
> 
> Signed-off-by: Dmitry Antipov <dmantipov@yandex.ru>
> Acked-by: Stanislaw Gruszka <stf_xl@wp.pl>

Patch applied to wireless-next.git, thanks.

2785ea9673a7 wifi: rt2x00: remove unused delayed work data from link description
diff mbox series

Patch

diff --git a/drivers/net/wireless/ralink/rt2x00/rt2x00.h b/drivers/net/wireless/ralink/rt2x00/rt2x00.h
index 82af01448a0a..bb648f95dfdd 100644
--- a/drivers/net/wireless/ralink/rt2x00/rt2x00.h
+++ b/drivers/net/wireless/ralink/rt2x00/rt2x00.h
@@ -335,16 +335,6 @@  struct link {
 	struct delayed_work watchdog_work;
 	unsigned int watchdog_interval;
 	unsigned int watchdog;
-
-	/*
-	 * Work structure for scheduling periodic AGC adjustments.
-	 */
-	struct delayed_work agc_work;
-
-	/*
-	 * Work structure for scheduling periodic VCO calibration.
-	 */
-	struct delayed_work vco_work;
 };
 
 enum rt2x00_delayed_flags {