diff options
| author | Richard Spiegel <richard.spiegel@amd.corp-partner.google.com> | 2018-08-07 07:53:36 -0700 |
|---|---|---|
| committer | Martin Roth <martinroth@google.com> | 2018-08-08 15:30:04 +0000 |
| commit | 38ad45c89387641493c9b35e4894922764a15e41 (patch) | |
| tree | 4a0461967475e41e4ef89c20a4639ba2cf1a1403 /src | |
| parent | d3131e416417b537a0e93f72d8f0c6cd3d27e092 (diff) | |
| download | coreboot-38ad45c89387641493c9b35e4894922764a15e41.tar.xz | |
device/pci_device.c: Remove unused variable attr setting
In procedure pci_get_resource, when setting an IO mapped base address,
variable attr is &= with PCI_BASE_ADDRESS_IO_ATTR_MASK. However, in this
particular code flow variable attr is not used later. Remove the line.
BUG=b:112253891
TEST=Build and boot grunt.
Change-Id: Ia4fdda1be92d22017a7a913a911db15aaa440b69
Signed-off-by: Richard Spiegel <richard.spiegel@silverbackltd.com>
Reviewed-on: https://review.coreboot.org/27928
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Martin Roth <martinroth@google.com>
Diffstat (limited to 'src')
| -rw-r--r-- | src/device/pci_device.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/src/device/pci_device.c b/src/device/pci_device.c index 593838909c..b4a8a5ce0e 100644 --- a/src/device/pci_device.c +++ b/src/device/pci_device.c @@ -249,7 +249,6 @@ struct resource *pci_get_resource(struct device *dev, unsigned long index) resource->flags = 0; } else if (attr & PCI_BASE_ADDRESS_SPACE_IO) { /* An I/O mapped base address. */ - attr &= PCI_BASE_ADDRESS_IO_ATTR_MASK; resource->flags |= IORESOURCE_IO; /* I don't want to deal with 32bit I/O resources. */ resource->limit = 0xffff; |