diff options
author | John Baldwin <jhb@FreeBSD.org> | 2008-08-05 21:04:00 +0000 |
---|---|---|
committer | John Baldwin <jhb@FreeBSD.org> | 2008-08-05 21:04:00 +0000 |
commit | 73492bc0b174300ef6b14a95ae6540b54ea309db (patch) | |
tree | 94a8a5af96d5446089fb10d14442aa43e06bb5f7 /sys/dev/pci/pci.c | |
parent | a37f97161c18d004eb92618d49354ef9284f9a4a (diff) | |
download | src-73492bc0b174300ef6b14a95ae6540b54ea309db.tar.gz src-73492bc0b174300ef6b14a95ae6540b54ea309db.zip |
Remove the second check for a 64-bit BAR value on a 32-bit system in
pci_add_map(). First, this condition is already handled earlier in
the function. Second, as written the check would never fire as the
'start' value was overwritten with a long value (rman_get_start() returns
long) before the comparison was done.
Discussed with: imp
MFC after: 2 weeks
Notes
Notes:
svn path=/head/; revision=181336
Diffstat (limited to 'sys/dev/pci/pci.c')
-rw-r--r-- | sys/dev/pci/pci.c | 20 |
1 files changed, 1 insertions, 19 deletions
diff --git a/sys/dev/pci/pci.c b/sys/dev/pci/pci.c index f91258622fac..b3168dc2446a 100644 --- a/sys/dev/pci/pci.c +++ b/sys/dev/pci/pci.c @@ -2404,26 +2404,8 @@ pci_add_map(device_t pcib, device_t bus, device_t dev, */ resource_list_delete(rl, type, reg); start = 0; - } else { + } else start = rman_get_start(res); - if ((u_long)start != start) { - /* - * Wait a minute! This platform can't do this - * address. - */ - device_printf(bus, - "pci%d:%d.%d.%x bar %#x start %#jx, too many bits.", - pci_get_domain(dev), b, s, f, reg, - (uintmax_t)start); - - /* - * Delete this resource and zero the BAR. - */ - resource_list_release(rl, bus, dev, type, reg, res); - resource_list_delete(rl, type, reg); - start = 0; - } - } pci_write_config(dev, reg, start, 4); if (ln2range == 64) pci_write_config(dev, reg + 4, start >> 32, 4); |