You need to sign in or sign up before continuing.
提交 c50762a8 编写于 作者: B Bjorn Helgaas

PCI: Remove unused assignments

Remove variables and assignments that are never used.

Found by Krzysztof using cppcheck, e.g.,

  $ cppcheck --enable=all --force
  uselessAssignmentPtrArg drivers/pci/proc.c:102 Assignment of function parameter has no effect outside the function. Did you forget dereferencing it?
  unreadVariable drivers/pci/setup-bus.c:1528 Variable 'old_flags' is assigned a value that is never used.
Reported-by: NKrzysztof Wilczyński <kw@linux.com>
Link: https://lore.kernel.org/r/20220313192933.434746-2-helgaas@kernel.orgSigned-off-by: NBjorn Helgaas <bhelgaas@google.com>
上级 cb2d0f84
...@@ -754,8 +754,6 @@ static ssize_t pci_read_config(struct file *filp, struct kobject *kobj, ...@@ -754,8 +754,6 @@ static ssize_t pci_read_config(struct file *filp, struct kobject *kobj,
u8 val; u8 val;
pci_user_read_config_byte(dev, off, &val); pci_user_read_config_byte(dev, off, &val);
data[off - init_off] = val; data[off - init_off] = val;
off++;
--size;
} }
pci_config_pm_runtime_put(dev); pci_config_pm_runtime_put(dev);
...@@ -818,11 +816,8 @@ static ssize_t pci_write_config(struct file *filp, struct kobject *kobj, ...@@ -818,11 +816,8 @@ static ssize_t pci_write_config(struct file *filp, struct kobject *kobj,
size -= 2; size -= 2;
} }
if (size) { if (size)
pci_user_write_config_byte(dev, off, data[off - init_off]); pci_user_write_config_byte(dev, off, data[off - init_off]);
off++;
--size;
}
pci_config_pm_runtime_put(dev); pci_config_pm_runtime_put(dev);
......
...@@ -99,9 +99,7 @@ static ssize_t proc_bus_pci_read(struct file *file, char __user *buf, ...@@ -99,9 +99,7 @@ static ssize_t proc_bus_pci_read(struct file *file, char __user *buf,
unsigned char val; unsigned char val;
pci_user_read_config_byte(dev, pos, &val); pci_user_read_config_byte(dev, pos, &val);
__put_user(val, buf); __put_user(val, buf);
buf++;
pos++; pos++;
cnt--;
} }
pci_config_pm_runtime_put(dev); pci_config_pm_runtime_put(dev);
...@@ -176,9 +174,7 @@ static ssize_t proc_bus_pci_write(struct file *file, const char __user *buf, ...@@ -176,9 +174,7 @@ static ssize_t proc_bus_pci_write(struct file *file, const char __user *buf,
unsigned char val; unsigned char val;
__get_user(val, buf); __get_user(val, buf);
pci_user_write_config_byte(dev, pos, val); pci_user_write_config_byte(dev, pos, val);
buf++;
pos++; pos++;
cnt--;
} }
pci_config_pm_runtime_put(dev); pci_config_pm_runtime_put(dev);
......
...@@ -1525,7 +1525,7 @@ static void pci_bridge_release_resources(struct pci_bus *bus, ...@@ -1525,7 +1525,7 @@ static void pci_bridge_release_resources(struct pci_bus *bus,
{ {
struct pci_dev *dev = bus->self; struct pci_dev *dev = bus->self;
struct resource *r; struct resource *r;
unsigned int old_flags = 0; unsigned int old_flags;
struct resource *b_res; struct resource *b_res;
int idx = 1; int idx = 1;
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册