Message ID | 1357440509-28108-8-git-send-email-aaron.lu@intel.com (mailing list archive) |
---|---|
State | Superseded, archived |
Headers | show |
On Sun, Jan 06, 2013 at 10:48:27AM +0800, Aaron Lu wrote: > Expose pm qos flags to user space so that user has a chance to disable > pm features like power off, if he/she has a broken platform or devices > or simply does not like this pm feature. > > This flag is exposed to user space only for ata device or atapi device > that is zero power capable. For normal atapi device, it will never be > powered off. > > Signed-off-by: Aaron Lu <aaron.lu@intel.com> > --- > drivers/ata/libata-acpi.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/drivers/ata/libata-acpi.c b/drivers/ata/libata-acpi.c > index e832bf6..5b67be3 100644 > --- a/drivers/ata/libata-acpi.c > +++ b/drivers/ata/libata-acpi.c > @@ -17,6 +17,7 @@ > #include <linux/pci.h> > #include <linux/slab.h> > #include <linux/pm_runtime.h> > +#include <linux/pm_qos.h> > #include <scsi/scsi_device.h> > #include "libata.h" > > @@ -1022,6 +1023,8 @@ static void ata_acpi_unregister_power_resource(struct ata_device *dev) > void ata_acpi_bind(struct ata_device *dev) > { > ata_acpi_register_power_resource(dev); > + if (dev->class == ATA_DEV_ATA || zpodd_dev_enabled(dev)) > + dev_pm_qos_expose_flags(&dev->sdev->sdev_gendev, 0); > } Why from ata_acpi_bind()?
On 01/08/2013 02:43 AM, Tejun Heo wrote: > On Sun, Jan 06, 2013 at 10:48:27AM +0800, Aaron Lu wrote: >> Expose pm qos flags to user space so that user has a chance to disable >> pm features like power off, if he/she has a broken platform or devices >> or simply does not like this pm feature. >> >> This flag is exposed to user space only for ata device or atapi device >> that is zero power capable. For normal atapi device, it will never be >> powered off. >> >> Signed-off-by: Aaron Lu <aaron.lu@intel.com> >> --- >> drivers/ata/libata-acpi.c | 3 +++ >> 1 file changed, 3 insertions(+) >> >> diff --git a/drivers/ata/libata-acpi.c b/drivers/ata/libata-acpi.c >> index e832bf6..5b67be3 100644 >> --- a/drivers/ata/libata-acpi.c >> +++ b/drivers/ata/libata-acpi.c >> @@ -17,6 +17,7 @@ >> #include <linux/pci.h> >> #include <linux/slab.h> >> #include <linux/pm_runtime.h> >> +#include <linux/pm_qos.h> >> #include <scsi/scsi_device.h> >> #include "libata.h" >> >> @@ -1022,6 +1023,8 @@ static void ata_acpi_unregister_power_resource(struct ata_device *dev) >> void ata_acpi_bind(struct ata_device *dev) >> { >> ata_acpi_register_power_resource(dev); >> + if (dev->class == ATA_DEV_ATA || zpodd_dev_enabled(dev)) >> + dev_pm_qos_expose_flags(&dev->sdev->sdev_gendev, 0); >> } > > Why from ata_acpi_bind()? I think I should add a check to see if platform can power off the device before export this qos flag. The check is done by ACPI, and the qos flag is for the scsi device. So looks like this is the proper place? Thanks, Aaron -- To unsubscribe from this list: send the line "unsubscribe linux-pm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/ata/libata-acpi.c b/drivers/ata/libata-acpi.c index e832bf6..5b67be3 100644 --- a/drivers/ata/libata-acpi.c +++ b/drivers/ata/libata-acpi.c @@ -17,6 +17,7 @@ #include <linux/pci.h> #include <linux/slab.h> #include <linux/pm_runtime.h> +#include <linux/pm_qos.h> #include <scsi/scsi_device.h> #include "libata.h" @@ -1022,6 +1023,8 @@ static void ata_acpi_unregister_power_resource(struct ata_device *dev) void ata_acpi_bind(struct ata_device *dev) { ata_acpi_register_power_resource(dev); + if (dev->class == ATA_DEV_ATA || zpodd_dev_enabled(dev)) + dev_pm_qos_expose_flags(&dev->sdev->sdev_gendev, 0); } void ata_acpi_unbind(struct ata_device *dev)
Expose pm qos flags to user space so that user has a chance to disable pm features like power off, if he/she has a broken platform or devices or simply does not like this pm feature. This flag is exposed to user space only for ata device or atapi device that is zero power capable. For normal atapi device, it will never be powered off. Signed-off-by: Aaron Lu <aaron.lu@intel.com> --- drivers/ata/libata-acpi.c | 3 +++ 1 file changed, 3 insertions(+)