| Message ID | bef0ae334d0e58b26e826d295ad1a8d6c8fda639.1479974100.git.kirtika@chromium.org (mailing list archive) |
|---|---|
| State | Changes Requested |
| Delegated to: | Kalle Valo |
| Headers | show |
diff --git a/drivers/net/wireless/ath/ath10k/ahb.c b/drivers/net/wireless/ath/ath10k/ahb.c index 766c63b..8078d64 100644 --- a/drivers/net/wireless/ath/ath10k/ahb.c +++ b/drivers/net/wireless/ath/ath10k/ahb.c @@ -130,11 +130,8 @@ static void ath10k_ahb_clock_deinit(struct ath10k *ar) static int ath10k_ahb_clock_enable(struct ath10k *ar) { struct ath10k_ahb *ar_ahb = ath10k_ahb_priv(ar); - struct device *dev; int ret; - dev = &ar_ahb->pdev->dev; - if (IS_ERR_OR_NULL(ar_ahb->cmd_clk) || IS_ERR_OR_NULL(ar_ahb->ref_clk) || IS_ERR_OR_NULL(ar_ahb->rtc_clk)) {
Commit 8beff219c528 introduced ath10k_ahb_clock_enable() which defines and sets 'struct device* dev' but does not use it. Compiling with W=1 gives the following warning, fix it. drivers/net/wireless/ath/ath10k/ahb.c: In function ‘ath10k_ahb_clock_enable’: drivers/net/wireless/ath/ath10k/ahb.c:133:17: warning: variable ‘dev’ set but not used [-Wunused-but-set-variable] This is a harmless warning, and is only being fixed to reduce the noise with W=1 in the kernel. Fixes: 8beff219c528("ath10k: add clock ctrl related functions in ahb") Cc: Raja Mani <rmani@qti.qualcomm.com> Cc: Kalle Valo <kvalo@qca.qualcomm.com> Signed-off-by: Kirtika Ruchandani <kirtika@chromium.org> --- drivers/net/wireless/ath/ath10k/ahb.c | 3 --- 1 file changed, 3 deletions(-) -- 2.8.0.rc3.226.g39d4020