Re: [Qemu-devel] [PATCH v2 0/8] Add system_powerdown support on ARM for ACPI and DT

[Shannon Zhao <shannon.zhao@linaro.org>]

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 <wei@redhat.com>
> 
> I will provide reviews to each individual patches.
> 
Thanks again. :)

-- 
Shannon