diff options
author | Bjorn Helgaas <bjorn.helgaas@hp.com> | 2006-03-21 23:20:28 -0800 |
---|---|---|
committer | Paul Mackerras <paulus@samba.org> | 2006-03-23 14:41:04 +1100 |
commit | d56a3e380085bf45cc2adba80c4e6490ba644231 (patch) | |
tree | f5eba0a77b9b1f4bf7f28952b653467cf1f523af /drivers/macintosh | |
parent | 6ea671a12f66b9d4d0f35fce957a71a6849295f2 (diff) | |
download | op-kernel-dev-d56a3e380085bf45cc2adba80c4e6490ba644231.zip op-kernel-dev-d56a3e380085bf45cc2adba80c4e6490ba644231.tar.gz |
[PATCH] macintosh: tidy-up driver_register() return values
Remove the assumption that driver_register() returns the number of devices
bound to the driver. In fact, it returns zero for success or a negative
error value.
All callers of macio_register_driver() either ignore the return value or
return it as the return value of a module_init() function.
Signed-off-by: Bjorn Helgaas <bjorn.helgaas@hp.com>
Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Paul Mackerras <paulus@samba.org>
Diffstat (limited to 'drivers/macintosh')
-rw-r--r-- | drivers/macintosh/macio_asic.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/macintosh/macio_asic.c b/drivers/macintosh/macio_asic.c index 69596f6..431bd37 100644 --- a/drivers/macintosh/macio_asic.c +++ b/drivers/macintosh/macio_asic.c @@ -550,15 +550,12 @@ static void macio_pci_add_devices(struct macio_chip *chip) */ int macio_register_driver(struct macio_driver *drv) { - int count = 0; - /* initialize common driver fields */ drv->driver.name = drv->name; drv->driver.bus = &macio_bus_type; /* register with core */ - count = driver_register(&drv->driver); - return count ? count : 1; + return driver_register(&drv->driver); } /** |