Re: [patch 5/6] drivers/acpi/hardware/hwsleep.c: fix warning msg

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



this one is obsolete b/c i dropped the patch it repairs.

we're going to come up with a more clever GTS/BFS patch
in response to Bjorn's suggestion.

thanks,
-Len

--
Len Brown, Intel Open Source Technology Center

On Fri, 9 Jan 2009, akpm@xxxxxxxxxxxxxxxxxxxx wrote:

> From: Valdis Kletnieks <valdis.kletnieks@xxxxxx>
> 
> In mmotm1230 the linux.next patch, most likely via this commit:
> 
> commit ecd196686aee5028f71ee09072c8581c7293a847
> Author: Len Brown <len.brown=40intel.com>
> Date:   Fri Dec 19 03:53:37 2008 -0500
> 
>     ACPICA: disable ACPI 2.0 _GTS/_BFS support
> 
> added some warning messages:
> 
>   CC      drivers/acpi/hardware/hwsleep.o
> drivers/acpi/hardware/hwsleep.c: In function`acpi_enter_sleep_state':
> drivers/acpi/hardware/hwsleep.c:231: warning: unused variable`arg'
> drivers/acpi/hardware/hwsleep.c:230: warning: unused variable`arg_list'
> drivers/acpi/hardware/hwsleep.c: In function`acpi_leave_sleep_state_prep':
> drivers/acpi/hardware/hwsleep.c:471: warning: unused variable`arg'
> drivers/acpi/hardware/hwsleep.c:470: warning: unused variable`arg_list'
> 
> If we're going to ifdef-wrap the code, let's do the variables involved
> as well, so we don't get warnings.
> 
> Signed-off-by: Valdis Kletnieks <valdis.kletnieks@xxxxxx>
> Cc: Len Brown <lenb@xxxxxxxxxx>
> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
> ---
> 
>  drivers/acpi/hardware/hwsleep.c |    4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff -puN drivers/acpi/hardware/hwsleep.c~drivers-acpi-hardware-hwsleepc-fix-warning-msg drivers/acpi/hardware/hwsleep.c
> --- a/drivers/acpi/hardware/hwsleep.c~drivers-acpi-hardware-hwsleepc-fix-warning-msg
> +++ a/drivers/acpi/hardware/hwsleep.c
> @@ -227,8 +227,10 @@ acpi_status asmlinkage acpi_enter_sleep_
>  	struct acpi_bit_register_info *sleep_type_reg_info;
>  	struct acpi_bit_register_info *sleep_enable_reg_info;
>  	u32 in_value;
> +#ifdef	ACPI_20_GTS_BFS
>  	struct acpi_object_list arg_list;
>  	union acpi_object arg;
> +#endif
>  	acpi_status status;
>  
>  	ACPI_FUNCTION_TRACE(acpi_enter_sleep_state);
> @@ -467,8 +469,10 @@ ACPI_EXPORT_SYMBOL(acpi_enter_sleep_stat
>   ******************************************************************************/
>  acpi_status acpi_leave_sleep_state_prep(u8 sleep_state)
>  {
> +#ifdef	ACPI_20_GTS_BFS
>  	struct acpi_object_list arg_list;
>  	union acpi_object arg;
> +#endif
>  	acpi_status status;
>  	struct acpi_bit_register_info *sleep_type_reg_info;
>  	struct acpi_bit_register_info *sleep_enable_reg_info;
> _
> 
--
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

[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux