Re: [PATCH] drivers: hv: balloon: remove extraneous assignment to region_start

From: Stephen Hemminger
Date: Thu Nov 09 2017 - 23:58:14 EST


On Wed, 18 Oct 2017 12:52:12 +0100
Colin King <colin.king@xxxxxxxxxxxxx> wrote:

> From: Colin Ian King <colin.king@xxxxxxxxxxxxx>
>
> The variable region_start is assigned twice, the first value is
> never read and redundant, so can be removed. We can clean up the
> code further by assigning rg_start directly rather than using the
> temporary variable region_start which can then be removed. Cleans
> up the clang warning:
>
> drivers/hv/hv_balloon.c:976:3: warning: Value stored to 'region_start'
> is never read
>
> Signed-off-by: Colin Ian King <colin.king@xxxxxxxxxxxxx>

LGTM

Acked-by: Stephen Hemminger <sthemmin@xxxxxxxxxxxxx>