PNPACPI: continue after _CRS and _PRS errors
authorBjorn Helgaas <bjorn.helgaas@hp.com>
Mon, 28 Apr 2008 22:33:50 +0000 (16:33 -0600)
committerLen Brown <len.brown@intel.com>
Tue, 29 Apr 2008 07:22:15 +0000 (03:22 -0400)
Keep going and register the device even if we have trouble parsing
_CRS or _PRS.  A parsing problem might mean we ignore some resources
the device is using, or we might not be able to change its resources.
But we should still take note of anything we *could* parse correctly.

Also remove reference to dev_id because I plan to remove it soon.

Signed-off-by: Bjorn Helgaas <bjorn.helgaas@hp.com>
Signed-off-by: Len Brown <len.brown@intel.com>
drivers/pnp/pnpacpi/core.c

index c283a9a70d8321063d92bd4a3c40607255f4459d..53f91068d0b0b19ad33748d02c848d62c535852a 100644 (file)
@@ -213,8 +213,7 @@ static int __init pnpacpi_add_device(struct acpi_device *device)
                                                          &dev->res);
                if (ACPI_FAILURE(status) && (status != AE_NOT_FOUND)) {
                        pnp_err("PnPACPI: METHOD_NAME__CRS failure for %s",
-                               dev_id->id);
-                       goto err1;
+                               acpi_device_hid(device));
                }
        }
 
@@ -223,8 +222,7 @@ static int __init pnpacpi_add_device(struct acpi_device *device)
                                                            dev);
                if (ACPI_FAILURE(status) && (status != AE_NOT_FOUND)) {
                        pnp_err("PnPACPI: METHOD_NAME__PRS failure for %s",
-                               dev_id->id);
-                       goto err1;
+                               acpi_device_hid(device));
                }
        }
 
@@ -252,8 +250,6 @@ static int __init pnpacpi_add_device(struct acpi_device *device)
        num++;
 
        return AE_OK;
-err1:
-       kfree(dev_id);
 err:
        kfree(dev);
        return -EINVAL;