[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 13/21] libcacard: fix reported ATR length
From: |
Alon Levy |
Subject: |
Re: [Qemu-devel] [PATCH 13/21] libcacard: fix reported ATR length |
Date: |
Thu, 8 Mar 2012 10:15:30 +0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Thu, Mar 08, 2012 at 03:12:03PM +0800, Wen Congyang wrote:
> At 02/28/2012 06:20 PM, Gerd Hoffmann Wrote:
> > From: Alon Levy <address@hidden>
> >
> > Signed-off-by: Alon Levy <address@hidden>
> > Signed-off-by: Gerd Hoffmann <address@hidden>
> > ---
> > libcacard/vcardt.h | 4 ++--
> > 1 files changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/libcacard/vcardt.h b/libcacard/vcardt.h
> > index 538bdde..d4d8e2e 100644
> > --- a/libcacard/vcardt.h
> > +++ b/libcacard/vcardt.h
> > @@ -26,8 +26,8 @@ typedef struct VCardEmulStruct VCardEmul;
> > #define MAX_CHANNEL 4
> >
> > /* create an ATR with appropriate historical bytes */
> > -#define VCARD_ATR_PREFIX(size) 0x3b, 0x66+(size), 0x00, 0xff, \
> > - 'V', 'C', 'A', 'R', 'D', '_'
> > +#define VCARD_ATR_PREFIX(size) (0x3b, 0x68+(size), 0x00, 0xff, \
> > + 'V', 'C', 'A', 'R', 'D', '_')
> >
> >
> > typedef enum {
>
> qemu build faile with this patch:
This is fixed by a patch on the list:
http://patchwork.ozlabs.org/patch/144276/
Anthony, could you please apply that patch, or should I send a pull request
with it?
Alon
>
> CC libcacard/vcard_emul_nss.o
> cc1: warnings being treated as errors
> vcard_emul_nss.c:528: error: left-hand operand of comma expression has no
> effect
> vcard_emul_nss.c:528: error: left-hand operand of comma expression has no
> effect
> vcard_emul_nss.c:528: error: left-hand operand of comma expression has no
> effect
> vcard_emul_nss.c:528: error: left-hand operand of comma expression has no
> effect
> vcard_emul_nss.c:528: error: left-hand operand of comma expression has no
> effect
> vcard_emul_nss.c:528: error: left-hand operand of comma expression has no
> effect
> vcard_emul_nss.c:528: error: left-hand operand of comma expression has no
> effect
> vcard_emul_nss.c:528: error: left-hand operand of comma expression has no
> effect
> vcard_emul_nss.c:528: error: left-hand operand of comma expression has no
> effect
> vcard_emul_nss.c:528: error: initializer element is not constant
> vcard_emul_nss.c:528: error: (near initialization for ‘nss_atr[0]’)
> make[1]: *** [vcard_emul_nss.o] Error 1
> make: *** [subdir-libcacard] Error 2
>
> I donot know this patch's purpose, so I donot know how to fix it. I guess this
> patch is unnecessary.
>
> Thanks
> Wen Congyang