Re: [Qemu-devel] [PATCH] xilinx_axienet: Fix bit mask code

2013-06-10 Thread Edgar E. Iglesias
On Sun, Jun 09, 2013 at 10:56:20PM +0200, Stefan Weil wrote: > Obviously the code wanted to mask the lower bits but failed to do so > because of a missing "<". > > cppcheck detected a conditional expression which was always true (1 < 7). Applied, thanks Stefan > > Signed-off-by: Stefan Weil

Re: [Qemu-devel] [PATCH] xilinx_axienet: Fix bit mask code

2013-06-09 Thread Peter Maydell
On 9 June 2013 21:56, Stefan Weil wrote: > diff --git a/hw/net/xilinx_axienet.c b/hw/net/xilinx_axienet.c > index 8989e95..2ca1511 100644 > --- a/hw/net/xilinx_axienet.c > +++ b/hw/net/xilinx_axienet.c > @@ -575,7 +575,7 @@ static void enet_write(void *opaque, hwaddr addr, > break; >

[Qemu-devel] [PATCH] xilinx_axienet: Fix bit mask code

2013-06-09 Thread Stefan Weil
Obviously the code wanted to mask the lower bits but failed to do so because of a missing "<". cppcheck detected a conditional expression which was always true (1 < 7). Signed-off-by: Stefan Weil --- Please review - I did not look for a Xilinx manual to see whether the code was correct at all.