linux-kernel - Re: [PATCH] asus-wmi: use ASUS_WMI_METHODID

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Wed, 20 Jun 2012 11:50:32 +0800
From:	Alex Hung <alex.hung@...onical.com>
To:	Alex Hung <alex.hung@...onical.com>
CC:	corentincj@...aif.net, mjg@...hat.com,
	acpi4asus-user@...ts.sourceforge.net,
	platform-driver-x86@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] asus-wmi: use ASUS_WMI_METHODID_DSTS2 as default DSTS
 ID.

Hi All,

The attached is acpi and dmi dumps and dmesg logs of before and after 
applying the patch for references.

Best Regards,
Alex Hung

On 06/20/2012 11:47 AM, Alex Hung wrote:
> According to responses from the BIOS team, ASUS_WMI_METHODID_DSTS2
> (0x53545344) will be used as future DSTS ID. In addition, calling
> asus_wmi_evaluate_method(ASUS_WMI_METHODID_DSTS2, 0, 0, NULL) returns
> ASUS_WMI_UNSUPPORTED_METHOD in new ASUS laptop PCs. This patch fixes
> no DSTS ID will be assigned in this case.
>
> Signed-off-by: Alex Hung<alex.hung@...onical.com>
> ---
>   drivers/platform/x86/asus-wmi.c |    7 +------
>   1 files changed, 1 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/platform/x86/asus-wmi.c b/drivers/platform/x86/asus-wmi.c
> index 25e3093..0f69a97 100644
> --- a/drivers/platform/x86/asus-wmi.c
> +++ b/drivers/platform/x86/asus-wmi.c
> @@ -1461,14 +1461,9 @@ static int asus_wmi_platform_init(struct asus_wmi *asus)
>   	 */
>   	if (!asus_wmi_evaluate_method(ASUS_WMI_METHODID_DSTS, 0, 0, NULL))
>   		asus->dsts_id = ASUS_WMI_METHODID_DSTS;
> -	else if (!asus_wmi_evaluate_method(ASUS_WMI_METHODID_DSTS2, 0, 0, NULL))
> +	else
>   		asus->dsts_id = ASUS_WMI_METHODID_DSTS2;
>
> -	if (!asus->dsts_id) {
> -		pr_err("Can't find DSTS");
> -		return -ENODEV;
> -	}
> -
>   	/* CWAP allow to define the behavior of the Fn+F2 key,
>   	 * this method doesn't seems to be present on Eee PCs */
>   	if (asus->driver->quirks->wapf>= 0)


Download attachment "asus-wmi-dump.tar" of type "application/x-tar" (136990 bytes)