qemu-arm
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-arm] [Qemu devel PATCH] msf2: Remove dead code reported by Cov


From: Peter Maydell
Subject: Re: [Qemu-arm] [Qemu devel PATCH] msf2: Remove dead code reported by Coverity
Date: Tue, 17 Oct 2017 16:28:15 +0100

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 ?

>          s->fifo_depth = 8;
> -    } else {
> -        s->fifo_depth = 4;
>      }
>  }

thanks
-- PMM



reply via email to

[Prev in Thread] Current Thread [Next in Thread]