Merge tag 'pm+acpi-4.1-rc1-2' of git://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm

Pull more power management and ACPI updates from Rafael Wysocki:
 "These are fixes mostly (intel_pstate, ACPI core, ACPI EC driver,
  cpupower tool), a new CPU ID for the Intel RAPL driver and one
  intel_pstate driver improvement that didn't make it to my previous
  pull requests due to timing.

  Specifics:

   - Fix a build warning in the intel_pstate driver showing up in
     non-SMP builds (Borislav Petkov)

   - Change one of the intel_pstate's P-state selection parameters for
     Baytrail and Cherrytrail CPUs to significantly improve performance
     at the cost of a small increase in energy consumption (Kristen
     Carlson Accardi)

   - Fix a NULL pointer dereference in the ACPI EC driver due to an
     unsafe list walk in the query handler removal routine (Chris
     Bainbridge)

   - Get rid of a false-positive lockdep warning in the ACPI container
     hot-remove code (Rafael J Wysocki)

   - Prevent the ACPI device enumeration code from creating device
     objects of a wrong type in some cases (Rafael J Wysocki)

   - Add Skylake processors support to the Intel RAPL power capping
     driver (Brian Bian)

   - Drop the stale MAINTAINERS entry for the ACPI dock driver that is
     regarded as part of the ACPI core and maintained along with it now
     (Chao Yu)

   - Fix cpupower tool breakage caused by a library API change in libpci
     3.3.0 (Lucas Stach)"

* tag 'pm+acpi-4.1-rc1-2' of git://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm:
  ACPI / scan: Add a scan handler for PRP0001
  ACPI / scan: Annotate physical_node_lock in acpi_scan_is_offline()
  ACPI / EC: fix NULL pointer dereference in acpi_ec_remove_query_handler()
  MAINTAINERS: remove maintainship entry of docking station driver
  powercap / RAPL: Add support for Intel Skylake processors
  cpufreq: intel_pstate: Fix an annoying !CONFIG_SMP warning
  intel_pstate: Change the setpoint for Atom params
  cpupower: fix breakage from libpci API change

1 files changed, 33 insertions, 6 deletions

diff --git a/drivers/acpi/scan.c b/drivers/acpi/scan.c
index 69bc0d888c01..03141aa4ea95 100644
--- a/drivers/acpi/scan.c
+++ b/drivers/acpi/scan.c
@@ -375,7 +375,11 @@ bool acpi_scan_is_offline(struct acpi_device *adev, bool uevent)
         struct acpi_device_physical_node *pn;
         bool offline = true;
 
-        mutex_lock(&adev->physical_node_lock);
+        /*
+         * acpi_container_offline() calls this for all of the container's
+         * children under the container's physical_node_lock lock.
+         */
+        mutex_lock_nested(&adev->physical_node_lock, SINGLE_DEPTH_NESTING);
 
         list_for_each_entry(pn, &adev->physical_node_list, node)
                 if (device_supports_offline(pn->dev) && !pn->dev->offline) {
@@ -2388,9 +2392,6 @@ static void acpi_default_enumeration(struct acpi_device *device)
         struct list_head resource_list;
         bool is_spi_i2c_slave = false;
 
-        if (!device->pnp.type.platform_id || device->handler)
-                return;
-
         /*
          * Do not enemerate SPI/I2C slaves as they will be enuerated by their
          * respective parents.
@@ -2403,6 +2404,29 @@ static void acpi_default_enumeration(struct acpi_device *device)
                 acpi_create_platform_device(device);
 }
 
+static const struct acpi_device_id generic_device_ids[] = {
+        {"PRP0001", },
+        {"", },
+};
+
+static int acpi_generic_device_attach(struct acpi_device *adev,
+                                      const struct acpi_device_id *not_used)
+{
+        /*
+         * Since PRP0001 is the only ID handled here, the test below can be
+         * unconditional.
+         */
+        if (adev->data.of_compatible)
+                acpi_default_enumeration(adev);
+
+        return 1;
+}
+
+static struct acpi_scan_handler generic_device_handler = {
+        .ids = generic_device_ids,
+        .attach = acpi_generic_device_attach,
+};
+
 static int acpi_scan_attach_handler(struct acpi_device *device)
 {
         struct acpi_hardware_id *hwid;
@@ -2428,8 +2452,6 @@ static int acpi_scan_attach_handler(struct acpi_device *device)
                                 break;
                 }
         }
-        if (!ret)
-                acpi_default_enumeration(device);
 
         return ret;
 }
@@ -2471,6 +2493,9 @@ static void acpi_bus_attach(struct acpi_device *device)
                 ret = device_attach(&device->dev);
                 if (ret < 0)
                         return;
+
+                if (!ret && device->pnp.type.platform_id)
+                        acpi_default_enumeration(device);
         }
         device->flags.visited = true;
 
@@ -2629,6 +2654,8 @@ int __init acpi_scan_init(void)
         acpi_pnp_init();
         acpi_int340x_thermal_init();
 
+        acpi_scan_add_handler(&generic_device_handler);
+
         mutex_lock(&acpi_scan_lock);
         /*
          * Enumerate devices in the ACPI namespace.