On Wed, Mar 23, 2016 at 16:12:46 +0000, Peter Maydell wrote: > On 17 March 2016 at 13:25, Shannon Zhao <zhaoshengl...@huawei.com> wrote: > > From: Shannon Zhao <shannon.z...@linaro.org> > > > > There is a problem for power button that it will not work if an early > > system_powerdown request happens before guest gpio driver loads. > > > > Here we add the emulation of gpio_key and use it for ARM virt power > > button. > > I tweaked the type names to be 'gpio-key' rather than 'gpio_key', > and added a comment to briefly describe what the device does: > > + * Emulate a (human) keypress -- when the key is triggered by > + * setting the incoming gpio line, the outbound irq line is > + * raised for 100ms before being dropped again. > > Applied to target-arm.next, thanks.
I might be doing something wrong, but aarch64-softmmu @ master only works for me after reverting 94f02c5ea94 "ARM: Virt: Use gpio_key for power button". Bisect log appended. This is what I get when booting aarch64 as per [1]: $ aarch64-softmmu/qemu-system-aarch64 -machine virt -cpu cortex-a57 -machine type=virt \ -nographic -smp 1 -m 2048 -kernel img/aarch64/aarch64-linux-3.15rc2-buildroot.img \ --append "console=ttyAMA0" qemu-system-aarch64: Unknown device 'gpio-key' for default sysbus Aborted (core dumped) Thanks, Emilio [1] http://www.bennee.com/~alex/blog/2014/05/09/running-linux-in-qemus-aarch64-system-emulation-mode/ git bisect start # good: [4829e0378dfb91d55af9dfd741bd09e8f2c4f91a] Merge remote-tracking branch 'remotes/armbru/tags/pull-qapi-2016-03-18' into staging git bisect good 4829e0378dfb91d55af9dfd741bd09e8f2c4f91a # bad: [7acbff99c6c285b3070bf0e768d56f511e2bf346] Update version for v2.6.0-rc1 release git bisect bad 7acbff99c6c285b3070bf0e768d56f511e2bf346 # good: [9fd3c5d556b21e0020d98d4695c84a655aa056f0] tests/test-filter-redirector: Add unit test for filter-redirector git bisect good 9fd3c5d556b21e0020d98d4695c84a655aa056f0 # bad: [e8710c2293c0f4652090b1434603715f2d9a410f] block: m25p80: Removed unused variable git bisect bad e8710c2293c0f4652090b1434603715f2d9a410f # good: [5481531154cf08ed53623a0184f7677a9b98d083] raw: Support BDRV_REQ_FUA git bisect good 5481531154cf08ed53623a0184f7677a9b98d083 # good: [c98d3d79ee387ea6e8fb091299f8562b20022f10] target-mips: use CP0_CHECK for gen_m{f|t}hc0 git bisect good c98d3d79ee387ea6e8fb091299f8562b20022f10 # good: [f4e732a0a773c4e44c2c183a5d63cd850ffb57d1] iotests: Test qemu-img convert -S 0 behavior git bisect good f4e732a0a773c4e44c2c183a5d63cd850ffb57d1 # good: [69bc7f5029db5ea55359f7905e9829777ae5a34f] Merge remote-tracking branch 'remotes/berrange/tags/pull-qcrypto-2016-03-30-1' into staging git bisect good 69bc7f5029db5ea55359f7905e9829777ae5a34f # good: [489ef4c810033e63af570c8a430af8b9858bfa5f] Merge remote-tracking branch 'remotes/lalrae/tags/mips-20160329-2' into staging git bisect good 489ef4c810033e63af570c8a430af8b9858bfa5f # bad: [94f02c5ea9430be935088b2574f72f2dcf902997] ARM: Virt: Use gpio_key for power button git bisect bad 94f02c5ea9430be935088b2574f72f2dcf902997 # good: [e5a8152c9bfd3d40c37711ba5b704f277cbc0c54] hw/gpio: Add the emulation of gpio_key git bisect good e5a8152c9bfd3d40c37711ba5b704f277cbc0c54 # first bad commit: [94f02c5ea9430be935088b2574f72f2dcf902997] ARM: Virt: Use gpio_key for power button