Message ID | 1283187480-23563-1-git-send-email-cam@cs.ualberta.ca |
---|---|
State | New |
Headers | show |
(2010/08/31 1:58), Cam Macdonell wrote: > fixes gcc 4.1 warning > > Signed-off-by: Cam Macdonell <cam@cs.ualberta.ca> > --- > hw/ivshmem.c | 4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/hw/ivshmem.c b/hw/ivshmem.c > index bbb5cba..fa9c684 100644 > --- a/hw/ivshmem.c > +++ b/hw/ivshmem.c > @@ -181,8 +181,8 @@ static void ivshmem_io_writel(void *opaque, target_phys_addr_t addr, > IVShmemState *s = opaque; > > uint64_t write_one = 1; > - uint16_t dest = val >> 16; > - uint16_t vector = val & 0xff; > + int16_t dest = val >> 16; > + int16_t vector = val & 0xff; > > addr &= 0xfc; > Since val is uint32_t, I think this change doesn't make sense. Thanks, H.Seto
diff --git a/hw/ivshmem.c b/hw/ivshmem.c index bbb5cba..fa9c684 100644 --- a/hw/ivshmem.c +++ b/hw/ivshmem.c @@ -181,8 +181,8 @@ static void ivshmem_io_writel(void *opaque, target_phys_addr_t addr, IVShmemState *s = opaque; uint64_t write_one = 1; - uint16_t dest = val >> 16; - uint16_t vector = val & 0xff; + int16_t dest = val >> 16; + int16_t vector = val & 0xff; addr &= 0xfc;
fixes gcc 4.1 warning Signed-off-by: Cam Macdonell <cam@cs.ualberta.ca> --- hw/ivshmem.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-)