[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 3/3] hw/block/pflash_cfi02: Remove unneeded variable assignme
From: |
Laurent Vivier |
Subject: |
Re: [PATCH 3/3] hw/block/pflash_cfi02: Remove unneeded variable assignment |
Date: |
Tue, 18 Feb 2020 20:16:33 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.4.1 |
Le 15/02/2020 à 17:15, Philippe Mathieu-Daudé a écrit :
> Fix warning reported by Clang static code analyzer:
>
> CC hw/block/pflash_cfi02.o
> hw/block/pflash_cfi02.c:311:5: warning: Value stored to 'ret' is never read
> ret = -1;
> ^ ~~
>
> Reported-by: Clang Static Analyzer
> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
> ---
> hw/block/pflash_cfi02.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/hw/block/pflash_cfi02.c b/hw/block/pflash_cfi02.c
> index 7c4744c020..12f18d401a 100644
> --- a/hw/block/pflash_cfi02.c
> +++ b/hw/block/pflash_cfi02.c
> @@ -308,7 +308,6 @@ static uint64_t pflash_read(void *opaque, hwaddr offset,
> unsigned int width)
> hwaddr boff;
> uint64_t ret;
>
> - ret = -1;
> /* Lazy reset to ROMD mode after a certain amount of read accesses */
> if (!pfl->rom_mode && pfl->wcycle == 0 &&
> ++pfl->read_counter > PFLASH_LAZY_ROMD_THRESHOLD) {
>
Applied to my trivial-patches branch.
Thanks,
Laurent