[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH] Use signed 16-bit values for ivshmem register w
From: |
Hidetoshi Seto |
Subject: |
[Qemu-devel] Re: [PATCH] Use signed 16-bit values for ivshmem register writes |
Date: |
Fri, 03 Sep 2010 10:53:59 +0900 |
User-agent: |
Mozilla/5.0 (Windows; U; Windows NT 6.0; ja; rv:1.9.2.8) Gecko/20100802 Thunderbird/3.1.2 |
(2010/08/31 1:58), Cam Macdonell wrote:
> fixes gcc 4.1 warning
>
> Signed-off-by: Cam Macdonell <address@hidden>
> ---
> 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
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- [Qemu-devel] Re: [PATCH] Use signed 16-bit values for ivshmem register writes,
Hidetoshi Seto <=