Re: [Qemu-devel] [PATCH 00/16] usb-ccid (v18)

2011-02-07 Thread Eric Blake
[adding libvir-list as well] On 02/07/2011 08:44 AM, Alon Levy wrote: >>> I guess I'll wait a little longer for more feedback? Should I split >>> the enum property separately? it's only used by ccid-card-emualted atm. >> >> The only non-cosmetic concern I have about your series is the enum >> pro

Re: [Qemu-devel] [PATCH 00/16] usb-ccid (v18)

2011-02-07 Thread Alon Levy
On Mon, Feb 07, 2011 at 07:12:52AM -0600, Anthony Liguori wrote: > On 02/07/2011 05:03 AM, Alon Levy wrote: > >Hi, > > > > I have a v19 waiting to be sent, but it's basically the same just > >with patches folded were appropriate, and the fixes Markus pointed > >out for the Enumeration property typ

Re: [Qemu-devel] [PATCH 00/16] usb-ccid (v18)

2011-02-07 Thread Alon Levy
Hi, I have a v19 waiting to be sent, but it's basically the same just with patches folded were appropriate, and the fixes Markus pointed out for the Enumeration property type. I guess I'll wait a little longer for more feedback? Should I split the enum property separately? it's only used by cci

Re: [Qemu-devel] [PATCH 00/16] usb-ccid (v18)

2011-02-07 Thread Anthony Liguori
On 02/07/2011 05:03 AM, Alon Levy wrote: Hi, I have a v19 waiting to be sent, but it's basically the same just with patches folded were appropriate, and the fixes Markus pointed out for the Enumeration property type. I guess I'll wait a little longer for more feedback? Should I split the en

Re: [Qemu-devel] [PATCH 00/16] usb-ccid (v18)

2011-02-07 Thread Alon Levy
On Mon, Feb 07, 2011 at 08:56:30AM -0700, Eric Blake wrote: > [adding libvir-list as well] > > On 02/07/2011 08:44 AM, Alon Levy wrote: > >>> I guess I'll wait a little longer for more feedback? Should I split > >>> the enum property separately? it's only used by ccid-card-emualted atm. > >> > >>

Re: [Qemu-devel] [PATCH 00/16] usb-ccid (v18)

2011-02-07 Thread Anthony Liguori
On 02/07/2011 09:56 AM, Eric Blake wrote: [adding libvir-list as well] On 02/07/2011 08:44 AM, Alon Levy wrote: I guess I'll wait a little longer for more feedback? Should I split the enum property separately? it's only used by ccid-card-emualted atm. The only non-cosmetic conc

[Qemu-devel] [PATCH 00/16] usb-ccid (v18)

2011-02-03 Thread Alon Levy
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