From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:59474) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZwKE3-00080Z-Ev for qemu-devel@nongnu.org; Tue, 10 Nov 2015 20:29:28 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZwKDy-000543-PZ for qemu-devel@nongnu.org; Tue, 10 Nov 2015 20:29:27 -0500 Received: from mail-wm0-x235.google.com ([2a00:1450:400c:c09::235]:37296) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZwKDy-00053x-Ep for qemu-devel@nongnu.org; Tue, 10 Nov 2015 20:29:22 -0500 Received: by wmww144 with SMTP id w144so25114228wmw.0 for ; Tue, 10 Nov 2015 17:29:21 -0800 (PST) References: <1446128855-26637-1-git-send-email-shannon.zhao@linaro.org> <56422B7A.3070107@redhat.com> <564259EC.2000308@redhat.com> From: Shannon Zhao Message-ID: <564299E0.8070405@linaro.org> Date: Wed, 11 Nov 2015 09:29:04 +0800 MIME-Version: 1.0 In-Reply-To: <564259EC.2000308@redhat.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v2 0/8] Add system_powerdown support on ARM for ACPI and DT List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Wei Huang , qemu-devel@nongnu.org, peter.maydell@linaro.org, mst@redhat.com, imammedo@redhat.com Cc: peter.huangpeng@huawei.com, zhaoshenglong@huawei.com On 2015/11/11 4:56, Wei Huang wrote: > >> >> Compared with v1, this version doesn't work with system_powerdown >> command. In my setup, the testing environment is exactly same for both >> v1 and v2. So something changed in v2 caused system_powerdown fail to work. >> > > I found this was caused by the change of "_HID" name for GPIO device. It > was changed from "LNRO0009" (v1) to "ARMH0061" (v2), which doesn't match > with my stock guest kernel PL061 driver. After changing the guest > kernel, it is working again. So: > Hi Wei, Thanks very much for your help. The reason why I change the _HID is based on the _HID 0f PL011 which is ARMH0011. About the _HID of ARM company's devices, I have to say that I didn't see which _HID they should be at any public place. I heard(maybe it's not correct) there is a _HID list of ARM devices. If so, I think ARM should publish them in public, otherwise the only thing we can do is to guess or refer to existing _HID. BTW, I think you could use this kernel patches[1] which adds ACPI support for AMBA device and drop the previous one. [1] https://lkml.org/lkml/2015/9/30/392 > Tested-by: Wei Huang > > I will provide reviews to each individual patches. > Thanks again. :) -- Shannon