From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753586AbbIWJPv (ORCPT ); Wed, 23 Sep 2015 05:15:51 -0400 Received: from foss.arm.com ([217.140.101.70]:33764 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753105AbbIWJPr (ORCPT ); Wed, 23 Sep 2015 05:15:47 -0400 Subject: Re: [PATCH V3 1/2] ACPI / EC: Fix broken big-endian 64bit platforms using 'global_lock' To: Viresh Kumar , "gregkh@linuxfoundation.org" References: <9b705747a138c96c26faee5218f7b47403195b28.1442305897.git.viresh.kumar@linaro.org> Cc: Sudeep Holla , "linaro-kernel@lists.linaro.org" , Rafael Wysocki , "sboyd@codeaurora.org" , "arnd@arndb.de" , "stable@vger.kernel.org" , Len Brown , "open list:ACPI" , open list , "Zheng, Lv" From: Sudeep Holla Message-ID: <56026DBE.3090603@arm.com> Date: Wed, 23 Sep 2015 10:15:42 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.2.0 MIME-Version: 1.0 In-Reply-To: <9b705747a138c96c26faee5218f7b47403195b28.1442305897.git.viresh.kumar@linaro.org> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 15/09/15 09:34, Viresh Kumar wrote: > global_lock is defined as an unsigned long and accessing only its lower > 32 bits from sysfs is incorrect, as we need to consider other 32 bits > for big endian 64 bit systems. > > Fix that by making global_lock an u32 instead. > > Cc: # v4.1+ > Signed-off-by: Viresh Kumar > > --- > Its marked just for # v4.1+, because arm64 has the first 64 big-endian > platform with ACPI. And ACPI support for that is mainlined recently > only (Arnd Bergmann). > Just to clarify, we don't support big-endian with ACPI on ARM64. We mandate use of EFI for ACPI on ARM64 and EFI spec mandates only little endian. Regards, Sudeep