Gitweb:     
http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=cc8259a6666de456460bacdd5637f5e2d71790ea
Commit:     cc8259a6666de456460bacdd5637f5e2d71790ea
Parent:     6ed3003c19a96fe18edf8179c4be6fe14abbebbc
Author:     Bjorn Helgaas <[EMAIL PROTECTED]>
AuthorDate: Wed Feb 6 01:40:02 2008 -0800
Committer:  Linus Torvalds <[EMAIL PROTECTED]>
CommitDate: Wed Feb 6 10:41:19 2008 -0800

    simplify pnp_activate_dev() and pnp_disable_dev() return values
    
    Make pnp_activate_dev() and pnp_disable_dev() return only 0 (success) or a
    negative error value, as pci_enable_device() and pci_disable_device() do.
    
    Previously they returned:
    
        0:      device was already active (or disabled)
        1:      we just activated (or disabled) device
        <0:     -EBUSY or error from pnp_start_dev() (or pnp_stop_dev())
    
    Now we return only 0 (device is active or disabled) or <0 (error).
    
    All in-tree callers either ignore the return values or check only for
    errors (negative values).
    
    Signed-off-by: Bjorn Helgaas <[EMAIL PROTECTED]>
    Cc: Adam Belay <[EMAIL PROTECTED]>
    Signed-off-by: Andrew Morton <[EMAIL PROTECTED]>
    Signed-off-by: Linus Torvalds <[EMAIL PROTECTED]>
---
 drivers/pnp/manager.c |    8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/pnp/manager.c b/drivers/pnp/manager.c
index c6b3d4e..6f8f8ed 100644
--- a/drivers/pnp/manager.c
+++ b/drivers/pnp/manager.c
@@ -513,7 +513,7 @@ int pnp_activate_dev(struct pnp_dev *dev)
        int error;
 
        if (dev->active)
-               return 0;       /* the device is already active */
+               return 0;
 
        /* ensure resources are allocated */
        if (pnp_auto_config_dev(dev))
@@ -524,7 +524,7 @@ int pnp_activate_dev(struct pnp_dev *dev)
                return error;
 
        dev->active = 1;
-       return 1;
+       return 0;
 }
 
 /**
@@ -538,7 +538,7 @@ int pnp_disable_dev(struct pnp_dev *dev)
        int error;
 
        if (!dev->active)
-               return 0;       /* the device is already disabled */
+               return 0;
 
        error = pnp_stop_dev(dev);
        if (error)
@@ -551,7 +551,7 @@ int pnp_disable_dev(struct pnp_dev *dev)
        pnp_clean_resource_table(&dev->res);
        up(&pnp_res_mutex);
 
-       return 1;
+       return 0;
 }
 
 /**
-
To unsubscribe from this list: send the line "unsubscribe git-commits-head" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to