diff mbox

[1/2] ACPI/Sleep: Add init_nvs_nosave_s3() function

Message ID 1359359225-28223-1-git-send-email-tianyu.lan@intel.com (mailing list archive)
State Changes Requested, archived
Headers show

Commit Message

lan,Tianyu Jan. 28, 2013, 7:47 a.m. UTC
commit 1bad2f19f7f7 introduces nvs_nosave_s3 flag to just ban
saving nvs in ths s3 rather than both of s3 and s4. This patch
is to add init_nvs_nosave_s3() to set nvs_nosave_s3 to true for
those machines in the nonvs blacklist which only don't need to
saving nvs during s3 rather than both s3 and s4.

Signed-off-by: Lan Tianyu <tianyu.lan@intel.com>
---
 drivers/acpi/sleep.c |    6 ++++++
 1 file changed, 6 insertions(+)

Comments

Rafael Wysocki Jan. 28, 2013, 12:21 p.m. UTC | #1
On Monday, January 28, 2013 03:47:04 PM Lan Tianyu wrote:
> commit 1bad2f19f7f7 introduces nvs_nosave_s3 flag to just ban
> saving nvs in ths s3 rather than both of s3 and s4. This patch
> is to add init_nvs_nosave_s3() to set nvs_nosave_s3 to true for
> those machines in the nonvs blacklist which only don't need to
> saving nvs during s3 rather than both s3 and s4.
> 
> Signed-off-by: Lan Tianyu <tianyu.lan@intel.com>

Please fold [2/2] into [1/2] and resend.

Thanks,
Rafael


> ---
>  drivers/acpi/sleep.c |    6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/drivers/acpi/sleep.c b/drivers/acpi/sleep.c
> index 2fcc67d..405b962 100644
> --- a/drivers/acpi/sleep.c
> +++ b/drivers/acpi/sleep.c
> @@ -141,6 +141,12 @@ static int __init init_nvs_nosave(const struct dmi_system_id *d)
>  	return 0;
>  }
>  
> +static int __init init_nvs_nosave_s3(const struct dmi_system_id *d)
> +{
> +	acpi_nvs_nosave_s3();
> +	return 0;
> +}
> +
>  static struct dmi_system_id __initdata acpisleep_dmi_table[] = {
>  	{
>  	.callback = init_old_suspend_ordering,
>
Tianyu Lan Jan. 28, 2013, 1:05 p.m. UTC | #2
? 2013/1/28 20:21, Rafael J. Wysocki ??:
> On Monday, January 28, 2013 03:47:04 PM Lan Tianyu wrote:
>> commit 1bad2f19f7f7 introduces nvs_nosave_s3 flag to just ban
>> saving nvs in ths s3 rather than both of s3 and s4. This patch
>> is to add init_nvs_nosave_s3() to set nvs_nosave_s3 to true for
>> those machines in the nonvs blacklist which only don't need to
>> saving nvs during s3 rather than both s3 and s4.
>>
>> Signed-off-by: Lan Tianyu <tianyu.lan@intel.com>
>
> Please fold [2/2] into [1/2] and resend.
>
Ok. I get it and will update.
> Thanks,
> Rafael
>
>
>> ---
>>   drivers/acpi/sleep.c |    6 ++++++
>>   1 file changed, 6 insertions(+)
>>
>> diff --git a/drivers/acpi/sleep.c b/drivers/acpi/sleep.c
>> index 2fcc67d..405b962 100644
>> --- a/drivers/acpi/sleep.c
>> +++ b/drivers/acpi/sleep.c
>> @@ -141,6 +141,12 @@ static int __init init_nvs_nosave(const struct dmi_system_id *d)
>>   	return 0;
>>   }
>>
>> +static int __init init_nvs_nosave_s3(const struct dmi_system_id *d)
>> +{
>> +	acpi_nvs_nosave_s3();
>> +	return 0;
>> +}
>> +
>>   static struct dmi_system_id __initdata acpisleep_dmi_table[] = {
>>   	{
>>   	.callback = init_old_suspend_ordering,
>>
diff mbox

Patch

diff --git a/drivers/acpi/sleep.c b/drivers/acpi/sleep.c
index 2fcc67d..405b962 100644
--- a/drivers/acpi/sleep.c
+++ b/drivers/acpi/sleep.c
@@ -141,6 +141,12 @@  static int __init init_nvs_nosave(const struct dmi_system_id *d)
 	return 0;
 }
 
+static int __init init_nvs_nosave_s3(const struct dmi_system_id *d)
+{
+	acpi_nvs_nosave_s3();
+	return 0;
+}
+
 static struct dmi_system_id __initdata acpisleep_dmi_table[] = {
 	{
 	.callback = init_old_suspend_ordering,