On Fri, Jan 17, 2014 at 03:46:40PM +0100, Rafael J. Wysocki wrote: > @@ -415,11 +472,12 @@ static int acpi_lpss_platform_notify(str > return 0; > } > > - if (action == BUS_NOTIFY_ADD_DEVICE) > + if (action == BUS_NOTIFY_ADD_DEVICE) { > ret = sysfs_create_group(&pdev->dev.kobj, &lpss_attr_group); > - else if (action == BUS_NOTIFY_DEL_DEVICE) > + pdev->dev.power.set_latency_tolerance = acpi_lpss_set_ltr; While trying to test this I noticed that BUS_NOTIFY_ADD_DEVICE happens after call to dpm_sysfs_add(), so LTR field is never exposed to the userspace. > + } else if (action == BUS_NOTIFY_DEL_DEVICE) { > sysfs_remove_group(&pdev->dev.kobj, &lpss_attr_group); > - > + } > return ret; > } > -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html