diff options
| author | YueHaibing <yuehaibing@huawei.com> | 2019-07-24 22:08:50 +0800 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2019-08-25 10:50:10 +0200 |
| commit | d0d12e58188156a2162ae54c23fe089122c0c31e (patch) | |
| tree | 8694f1373899ca7345ef55b9dcb9c5b30cd284f5 | |
| parent | 990ea5ad6744ae919b5e48dfde6b098fd12604f0 (diff) | |
| download | linux-d0d12e58188156a2162ae54c23fe089122c0c31e.tar.gz linux-d0d12e58188156a2162ae54c23fe089122c0c31e.tar.bz2 linux-d0d12e58188156a2162ae54c23fe089122c0c31e.zip | |
xen/pciback: remove set but not used variable 'old_state'
[ Upstream commit 09e088a4903bd0dd911b4f1732b250130cdaffed ]
Fixes gcc '-Wunused-but-set-variable' warning:
drivers/xen/xen-pciback/conf_space_capability.c: In function pm_ctrl_write:
drivers/xen/xen-pciback/conf_space_capability.c:119:25: warning:
variable old_state set but not used [-Wunused-but-set-variable]
It is never used so can be removed.
Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: YueHaibing <yuehaibing@huawei.com>
Reviewed-by: Boris Ostrovsky <boris.ostrovsky@oracle.com>
Signed-off-by: Juergen Gross <jgross@suse.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
| -rw-r--r-- | drivers/xen/xen-pciback/conf_space_capability.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/xen/xen-pciback/conf_space_capability.c b/drivers/xen/xen-pciback/conf_space_capability.c index 73427d8e0116..e5694133ebe5 100644 --- a/drivers/xen/xen-pciback/conf_space_capability.c +++ b/drivers/xen/xen-pciback/conf_space_capability.c @@ -116,13 +116,12 @@ static int pm_ctrl_write(struct pci_dev *dev, int offset, u16 new_value, { int err; u16 old_value; - pci_power_t new_state, old_state; + pci_power_t new_state; err = pci_read_config_word(dev, offset, &old_value); if (err) goto out; - old_state = (pci_power_t)(old_value & PCI_PM_CTRL_STATE_MASK); new_state = (pci_power_t)(new_value & PCI_PM_CTRL_STATE_MASK); new_value &= PM_OK_BITS; |