ramips: use switch statement instead of multiple if-else-if in the rt288x pci code

SVN-Revision: 17972
owl
Gabor Juhos 2009-10-07 11:37:03 +00:00
parent 37f5a64938
commit fe360a232b
1 changed files with 22 additions and 12 deletions

View File

@ -80,12 +80,17 @@ static int rt2880_pci_config_read(struct pci_bus *bus, unsigned int devfn,
config_access(PCI_ACCESS_READ, bus, devfn, where, &data); config_access(PCI_ACCESS_READ, bus, devfn, where, &data);
spin_unlock_irqrestore(&rt2880_pci_lock, flags); spin_unlock_irqrestore(&rt2880_pci_lock, flags);
if (size == 1) switch (size) {
case 1:
*val = (data >> ((where & 3) << 3)) & 0xff; *val = (data >> ((where & 3) << 3)) & 0xff;
else if (size == 2) break;
case 2:
*val = (data >> ((where & 3) << 3)) & 0xffff; *val = (data >> ((where & 3) << 3)) & 0xffff;
else break;
case 4:
*val = data; *val = data;
break;
}
return PCIBIOS_SUCCESSFUL; return PCIBIOS_SUCCESSFUL;
} }
@ -97,16 +102,21 @@ static int rt2880_pci_config_write(struct pci_bus *bus, unsigned int devfn,
u32 data = 0; u32 data = 0;
spin_lock_irqsave(&rt2880_pci_lock, flags); spin_lock_irqsave(&rt2880_pci_lock, flags);
if (size == 4) {
data = val; switch (size) {
} else { case 1:
config_access(PCI_ACCESS_READ, bus, devfn, where, &data); config_access(PCI_ACCESS_READ, bus, devfn, where, &data);
if (size == 1) data = (data & ~(0xff << ((where & 3) << 3))) |
data = (data & ~(0xff << ((where & 3) << 3))) | (val << ((where & 3) << 3));
(val << ((where & 3) << 3)); break;
else if (size == 2) case 2:
data = (data & ~(0xffff << ((where & 3) << 3))) | config_access(PCI_ACCESS_READ, bus, devfn, where, &data);
(val << ((where & 3) << 3)); data = (data & ~(0xffff << ((where & 3) << 3))) |
(val << ((where & 3) << 3));
break;
case 4:
data = val;
break;
} }
config_access(PCI_ACCESS_WRITE, bus, devfn, where, &data); config_access(PCI_ACCESS_WRITE, bus, devfn, where, &data);