| Message ID | 20190527153424.10268-1-yuehaibing@huawei.com (mailing list archive) |
|---|---|
| State | New, archived |
| Delegated to: | Moritz Fischer |
| Headers | show |
| Series | [-next] fpga: dfl: fme: Remove set but not used variable 'fme' | expand |
On 19-05-27 23:34:24, YueHaibing wrote: > Fixes gcc '-Wunused-but-set-variable' warning: > > drivers/fpga/dfl-fme-main.c: In function fme_dev_destroy: > drivers/fpga/dfl-fme-main.c:216:18: warning: variable fme set but not used [-Wunused-but-set-variable] > > It's never used since introduction in commit 29de76240e86 ("fpga: > dfl: fme: add partial reconfiguration sub feature support") > > Signed-off-by: YueHaibing <yuehaibing@huawei.com> Acked-by: Ben Widawsky <ben@bwidawsk.net> [snip]
diff --git a/drivers/fpga/dfl-fme-main.c b/drivers/fpga/dfl-fme-main.c index 086ad2420ade..cb7c4e258298 100644 --- a/drivers/fpga/dfl-fme-main.c +++ b/drivers/fpga/dfl-fme-main.c @@ -213,10 +213,8 @@ static int fme_dev_init(struct platform_device *pdev) static void fme_dev_destroy(struct platform_device *pdev) { struct dfl_feature_platform_data *pdata = dev_get_platdata(&pdev->dev); - struct dfl_fme *fme; mutex_lock(&pdata->lock); - fme = dfl_fpga_pdata_get_private(pdata); dfl_fpga_pdata_set_private(pdata, NULL); mutex_unlock(&pdata->lock); }
Fixes gcc '-Wunused-but-set-variable' warning: drivers/fpga/dfl-fme-main.c: In function fme_dev_destroy: drivers/fpga/dfl-fme-main.c:216:18: warning: variable fme set but not used [-Wunused-but-set-variable] It's never used since introduction in commit 29de76240e86 ("fpga: dfl: fme: add partial reconfiguration sub feature support") Signed-off-by: YueHaibing <yuehaibing@huawei.com> --- drivers/fpga/dfl-fme-main.c | 2 -- 1 file changed, 2 deletions(-)