Re: [Qemu-devel] [PATCH v2 00/10] ccid and libcacard fixes for windows/mingw

2013-04-17 Thread Alon Levy
On Mon, 2013-04-15 at 15:31 +0200, Marc-André Lureau wrote:
 Hi
 
 
 On Wed, Mar 27, 2013 at 9:36 PM, Alon Levy al...@redhat.com wrote:
 This series:
 1. fixes windows guests to show the ccid device
 2. changes libcacard to use glib
 3. makes libcacard build under mingw
 4. does some cleanups
 
 It contains a few patches already posted to the list (the two
 Jim Meyering patches) which were already acked.
 
 I'll make a pull request once this had some time to be
 reviewed.
 
 Tested with a fedora and windows 7 guest.
 
 The main non cleanup patches are:
   hw/usb/dev-smartcard-reader: support windows guest
   libcacard: correct T0 historical bytes size
 
 
 The patch series doesn't do what you describe.

So you want me to update this cover letter that doesn't get committed,
no problem, I will in a follow up message, but meanwhile could you take
a look at the updated patches per your requests from v1?

 Many patches are missing from the v1:
 http://lists.nongnu.org/archive/html/qemu-devel/2013-03/msg02925.html 

Many patches are missing on purpose, because they are either yours and
so I acked them implicitly by sending the v1 patch series, or they are
mine and have already been acked by me.

 
 
 -- 
 Marc-André Lureau





Re: [Qemu-devel] [PATCH v2 00/10] ccid and libcacard fixes for windows/mingw

2013-04-15 Thread Marc-André Lureau
Hi


On Wed, Mar 27, 2013 at 9:36 PM, Alon Levy al...@redhat.com wrote:

 This series:
 1. fixes windows guests to show the ccid device
 2. changes libcacard to use glib
 3. makes libcacard build under mingw
 4. does some cleanups

 It contains a few patches already posted to the list (the two Jim Meyering
 patches) which were already acked.

 I'll make a pull request once this had some time to be reviewed.

 Tested with a fedora and windows 7 guest.

 The main non cleanup patches are:
   hw/usb/dev-smartcard-reader: support windows guest
   libcacard: correct T0 historical bytes size


The patch series doesn't do what you describe. Many patches are missing
from the v1:
http://lists.nongnu.org/archive/html/qemu-devel/2013-03/msg02925.html

-- 
Marc-André Lureau


[Qemu-devel] [PATCH v2 00/10] ccid and libcacard fixes for windows/mingw

2013-03-27 Thread Alon Levy
This series:
1. fixes windows guests to show the ccid device
2. changes libcacard to use glib
3. makes libcacard build under mingw
4. does some cleanups

It contains a few patches already posted to the list (the two Jim Meyering 
patches) which were already acked.

I'll make a pull request once this had some time to be reviewed.

Tested with a fedora and windows 7 guest.

The main non cleanup patches are:
  hw/usb/dev-smartcard-reader: support windows guest
  libcacard: correct T0 historical bytes size

v2:
 only resent patches that were not acked before.
 did all the changes per Marc Andre's comments. Please look at the g_debug one, 
I'm not sure
 about the ifdeffery there (not if it works, but if it looks ok).


Alon Levy (10):
  libcacard: change default ATR
  ccid-card-passthru: add atr check
  ccid-card-passthru, dev-smartcard-reader: add debug environment
variables
  dev-smartcard-reader: define structs for CCID_Parameter internals
  dev-smartcard-reader: change default protocol to T=0
  dev-smartcard-reader: copy atr protocol to ccid parameters
  libcacard/vreader: add debugging messages for apdu
  libcacard: move atr setting from macro to function
  dev-smartcard-reader: empty implementation for Mechanical (fail
correctly)
  libcacard/cac: change big switch functions to single return point

 Makefile.objs |   1 +
 hw/ccid-card-passthru.c   |  61 
 hw/usb/dev-smartcard-reader.c | 130 --
 include/qemu-common.h |   5 ++
 libcacard/cac.c   |  80 +-
 libcacard/cac.h   |   8 +++
 libcacard/vcard_emul_nss.c|  14 -
 libcacard/vcardt.c|  40 +
 libcacard/vcardt.h|   5 --
 libcacard/vcardt_internal.h   |   6 ++
 libcacard/vreader.c   |  77 +
 util/cutils.c |  23 
 12 files changed, 384 insertions(+), 66 deletions(-)
 create mode 100644 libcacard/vcardt.c
 create mode 100644 libcacard/vcardt_internal.h

-- 
1.8.1.4