[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [Qemu devel PATCH] msf2: Remove dead code reported by C
From: |
sundeep subbaraya |
Subject: |
Re: [Qemu-devel] [Qemu devel PATCH] msf2: Remove dead code reported by Coverity |
Date: |
Wed, 18 Oct 2017 08:53:46 +0530 |
Hi Peter,
On Wed, Oct 18, 2017 at 8:45 AM, sundeep subbaraya <address@hidden>
wrote:
> Hi Peter,
>
> On Tue, Oct 17, 2017 at 8:58 PM, Peter Maydell <address@hidden>
> wrote:
>
>> On 16 October 2017 at 18:54, Subbaraya Sundeep <address@hidden>
>> wrote:
>> > Fixed incorrect frame size mask, validated maximum frame
>> > size in spi_write and removed dead code.
>> >
>> > Signed-off-by: Subbaraya Sundeep <address@hidden>
>> > ---
>> > hw/ssi/mss-spi.c | 8 +++++---
>> > 1 file changed, 5 insertions(+), 3 deletions(-)
>> >
>> > diff --git a/hw/ssi/mss-spi.c b/hw/ssi/mss-spi.c
>> > index 5a8e308..1e49cbc 100644
>> > --- a/hw/ssi/mss-spi.c
>> > +++ b/hw/ssi/mss-spi.c
>> > @@ -76,9 +76,10 @@
>> > #define C_BIGFIFO (1 << 29)
>> > #define C_RESET (1 << 31)
>> >
>> > -#define FRAMESZ_MASK 0x1F
>> > +#define FRAMESZ_MASK 0x3F
>> > #define FMCOUNT_MASK 0x00FFFF00
>> > #define FMCOUNT_SHIFT 8
>> > +#define FRAMESZ_MAX 32
>> >
>> > static void txfifo_reset(MSSSpiState *s)
>> > {
>> > @@ -106,8 +107,6 @@ static void set_fifodepth(MSSSpiState *s)
>> > s->fifo_depth = 16;
>> > } else if (size <= 32) {
>>
>> Shouldn't this just be "} else {" now ?
>>
>> Yes. I will send v2.
>
Also I will log guest error if frame size is greater than 32.
Thanks,
Sundeep
>
> Thanks,
> Sundeep
>
> > s->fifo_depth = 8;
>> > - } else {
>> > - s->fifo_depth = 4;
>> > }
>> > }
>>
>> thanks
>> -- PMM
>>
>
>