diff options
| author | Alex Williamson <alex.williamson@hp.com> | 2005-12-21 15:21:36 -0700 |
|---|---|---|
| committer | Tony Luck <tony.luck@intel.com> | 2006-01-03 11:16:17 -0800 |
| commit | 408045afbdb46e109a1a44e67af688e9ddf7ad66 (patch) | |
| tree | 4bcdc550aeca5a3cb2c561f664be9427f86ecf8e /arch/ia64/pci | |
| parent | 88026842b0a760145aa71d69e74fbc9ec118ca44 (diff) | |
| download | kernel-common-408045afbdb46e109a1a44e67af688e9ddf7ad66.tar.gz kernel-common-408045afbdb46e109a1a44e67af688e9ddf7ad66.tar.bz2 kernel-common-408045afbdb46e109a1a44e67af688e9ddf7ad66.zip | |
[IA64] incorrect return from ia64_pci_legacy_write()
The function ia64_pci_legacy_write() returns 0 for everything
except errors. This return value gets sent back to the user from
pci_write_legacy_io(), making it look like every write fails. The trivial
patch below copies the behavior of the SGI sn machvec and does what
would be expected from something implementing a write() function.
Signed-off-by: Alex Williamson <alex.williamson@hp.com>
Signed-off-by: Tony Luck <tony.luck@intel.com>
Diffstat (limited to 'arch/ia64/pci')
| -rw-r--r-- | arch/ia64/pci/pci.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/ia64/pci/pci.c b/arch/ia64/pci/pci.c index 20d76fae24e8..30dbc98bf0b3 100644 --- a/arch/ia64/pci/pci.c +++ b/arch/ia64/pci/pci.c @@ -700,7 +700,7 @@ int ia64_pci_legacy_read(struct pci_bus *bus, u16 port, u32 *val, u8 size) */ int ia64_pci_legacy_write(struct pci_dev *bus, u16 port, u32 val, u8 size) { - int ret = 0; + int ret = size; switch (size) { case 1: |