Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Organization: Red Hat UK Ltd. Registered Address: Red Hat UK Ltd, Amberley
        Place, 107-111 Peascod Street, Windsor, Berkshire, SI4 1TE, United
        Kingdom.
        Registered in England and Wales under Company Registration No. 3798903
From:   David Howells <dhowells@redhat.com>
In-Reply-To: <20210303064028.895953-1-weiyongjun1@huawei.com>
References: <20210303064028.895953-1-weiyongjun1@huawei.com>
To:     Wei Yongjun <weiyongjun1@huawei.com>
Cc:     dhowells@redhat.com, Hulk Robot <hulkci@huawei.com>,
        linux-afs@lists.infradead.org, linux-kernel@vger.kernel.org,
        kernel-janitors@vger.kernel.org
Subject: Re: [PATCH -next] afs: Remove unused variable in afs_req_issue_op()
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-ID: <2025007.1614759802.1@warthog.procyon.org.uk>
Date:   Wed, 03 Mar 2021 08:23:22 +0000
Message-ID: <2025008.1614759802@warthog.procyon.org.uk>
Precedence: bulk

Wei Yongjun <weiyongjun1@huawei.com> wrote:

> GCC reports the following warning with W=1:
> 
> fs/afs/file.c:291:6: warning:
>  unused variable 'ret' [-Wunused-variable]
>   291 |  int ret;
>       |      ^~~
> 
> After commit 799fbdf96cd51, this variable is not used in function
> afs_req_issue_op(), so this commit remove it to fix the warning.
> 
> Fixes: 799fbdf96cd51 ("afs: Use new fscache read helper API")
> Reported-by: Hulk Robot <hulkci@huawei.com>
> Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com>

I've already fixed that and updated my branch, thanks.  linux-next should pick
it up on the next merging.

David