Driver core: switch firmware_class to uevent_suppress.
authorCornelia Huck <cornelia.huck@de.ibm.com>
Thu, 29 Mar 2007 09:12:14 +0000 (11:12 +0200)
committerGreg Kroah-Hartman <gregkh@suse.de>
Fri, 27 Apr 2007 17:57:29 +0000 (10:57 -0700)
Use uevent_suppress instead of returning an error code in
firmware_uevent(). Get rid of the now unneeded FW_STATUS_READY
and FW_STATUS_READY_NOHOTPLUG.

Signed-off-by: Cornelia Huck <cornelia.huck@de.ibm.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/base/firmware_class.c

index c0a979a5074bbec75ef691cfe1f13b269c1332fb..97ab5bd1c4d62ebb72ef9babc07f2c35571c484b 100644 (file)
@@ -31,8 +31,6 @@ enum {
        FW_STATUS_LOADING,
        FW_STATUS_DONE,
        FW_STATUS_ABORT,
-       FW_STATUS_READY,
-       FW_STATUS_READY_NOHOTPLUG,
 };
 
 static int loading_timeout = 60;       /* In seconds */
@@ -96,9 +94,6 @@ static int firmware_uevent(struct device *dev, char **envp, int num_envp,
        struct firmware_priv *fw_priv = dev_get_drvdata(dev);
        int i = 0, len = 0;
 
-       if (!test_bit(FW_STATUS_READY, &fw_priv->status))
-               return -ENODEV;
-
        if (add_uevent_var(envp, num_envp, &i, buffer, buffer_size, &len,
                           "FIRMWARE=%s", fw_priv->fw_id))
                return -ENOMEM;
@@ -333,6 +328,7 @@ static int fw_register_device(struct device **dev_p, const char *fw_name,
        f_dev->parent = device;
        f_dev->class = &firmware_class;
        dev_set_drvdata(f_dev, fw_priv);
+       f_dev->uevent_suppress = 1;
        retval = device_register(f_dev);
        if (retval) {
                printk(KERN_ERR "%s: device_register failed\n",
@@ -382,9 +378,7 @@ static int fw_setup_device(struct firmware *fw, struct device **dev_p,
        }
 
        if (uevent)
-                set_bit(FW_STATUS_READY, &fw_priv->status);
-        else
-                set_bit(FW_STATUS_READY_NOHOTPLUG, &fw_priv->status);
+               f_dev->uevent_suppress = 0;
        *dev_p = f_dev;
        goto out;