Messages in this thread | | | Date | Fri, 10 Nov 2017 13:57:19 +0900 | From | Stephen Hemminger <> | Subject | Re: [PATCH] drivers: hv: balloon: remove extraneous assignment to region_start |
| |
On Wed, 18 Oct 2017 12:52:12 +0100 Colin King <colin.king@canonical.com> wrote:
> From: Colin Ian King <colin.king@canonical.com> > > 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@canonical.com>
LGTM
Acked-by: Stephen Hemminger <sthemmin@microsoft.com>
| |