Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Date:   Sat, 4 May 2019 21:25:10 +0200 (CEST)
From:   Julia Lawall <julia.lawall@lip6.fr>
To:     Vatsala Narang <vatsalanarang@gmail.com>
cc:     gregkh@linuxfoundation.org, hadess@hadess.net, hdegoede@redhat.com,
        devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 2/7] staging: rtl8723bs: core: Remove return in void
 function.
In-Reply-To: <20190504184358.25632-1-vatsalanarang@gmail.com>
Message-ID: <alpine.DEB.2.21.1905042124130.2506@hadrien>
References: <20190504184358.25632-1-vatsalanarang@gmail.com>
User-Agent: Alpine 2.21 (DEB 202 2017-01-01)
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

> @@ -5828,8 +5822,6 @@ void survey_timer_hdl(struct timer_list *t)
>
>
>  exit_survey_timer_hdl:
> -
> -	return;
>  }

Are you sure that you compiled this code?  I'm not sure that it is possible
to have a label without a subsequent statement.

julia