In patch v5 [1] of Introduce CanoKey QEMU I said that canokey-qemu was incompatible with qemu-xhci.
kraxel kindly suggested[2] that it should be the problem of usb_wakeup So I fixed it in this patch set. Now that the v5 patch has been in the process of git PULL [3], I think it would be better to post a new patch set instead of sending out v6, which would make maintainer's tree back and forth. This patch should be applied after [1]. As for the CCID ZLP issue, it is described in the comment and commit message. I added a commit in https://github.com/canokeys/canokey-qemu to export the EP num in the header, so hw/usb/canokey.c in qemu could use it for CTAPHID quirks. If you want to compile this version when --enable-canokey, make sure to use the latest libcanokey-qemu.so The CI result for this PATCH is at [4]. The failure for amd64-debian-container seems irrelevent to this patchset. [1] https://lore.kernel.org/qemu-devel/YoY5k0PQny8WtAHi@Sun/ [2] https://lore.kernel.org/qemu-devel/20220609095659.ulgk64bx3nlqz...@sirius.home.kraxel.org/ [3] https://lore.kernel.org/qemu-devel/20220610092043.1874654-1-kra...@redhat.com/ [4] https://gitlab.com/ZenithalHourlyRate/qemu/-/pipelines/561801062 Hongren (Zenithal) Zheng (3): hw/usb/canokey: Fix CCID ZLP hw/usb/canokey: fix compatibility of qemu-xhci docs/system/devices/usb/canokey: remove limitations on qemu-xhci docs/system/devices/canokey.rst | 10 ---------- hw/usb/canokey.c | 35 +++++++++++++++++++++++++++++---- hw/usb/canokey.h | 1 + 3 files changed, 32 insertions(+), 14 deletions(-) -- 2.35.1
