|
From: | Gerd Hoffmann |
Subject: | Re: [Qemu-devel] [PATCH 0/5] usb-ccid (v14) |
Date: | Mon, 10 Jan 2011 13:44:32 +0100 |
User-agent: | Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.13) Gecko/20101208 Red Hat/3.1.7-3.el6_0 Thunderbird/3.1.7 |
On 01/08/11 11:28, Alon Levy wrote:
This patchset adds three new devices, usb-ccid, ccid-card-passthru and ccid-card-emulated, providing a CCID bus, a simple passthru protocol implementing card requiring a client, and a standalone emulated card. It also introduces a new directory libcaccard with CAC card emulation, CAC is a type of ISO 7816 smart card. Tree for pull: git://anongit.freedesktop.org/~alon/qemu usb_ccid.v14
One nit left which I didn't notice on previous reviews: It can't be disabled at compile time, there is no --disable-smartcard switch. The nss detection needs some tweaks too. The usual configure behavior in qemu is this:
[ no smarccard option specified ] autodetect, i.e. enable smartcard if nss support is found, otherwise disable (i.e. what your patch does now). --disable-smartcard turn off smartcard support. --enable-smartcard force smartcard support on, if nss isn't present abort configure with an error message. Otherwise it looks works nicely. Good job. cheers, Gerd
[Prev in Thread] | Current Thread | [Next in Thread] |