Re: [PATCH v2] nfc: s3fwrn5: remove unnecessary label

[patchwork-bot+netdevbpf]

Hello:

This patch was applied to netdev/net-next.git (refs/heads/master):

On Tue,  6 Apr 2021 09:59:54 +0800 you wrote:
> From: wengjianfeng 
> 
> In function s3fwrn5_nci_post_setup, the variable ret is assigned then
> goto out label, which just return ret, so we use return to replace it.
> Other goto sentences are similar, we use return sentences to replace
> goto sentences and delete out label.
> 
> [...]

Here is the summary with links:
  - [v2] nfc: s3fwrn5: remove unnecessary label
https://git.kernel.org/netdev/net-next/c/b58c4649d94e

You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html

Re: [PATCH v2] nfc: s3fwrn5: remove unnecessary label

[Krzysztof Kozlowski]

On 06/04/2021 03:59, samirweng1979 wrote:
> From: wengjianfeng 
> 
> In function s3fwrn5_nci_post_setup, the variable ret is assigned then
> goto out label, which just return ret, so we use return to replace it.
> Other goto sentences are similar, we use return sentences to replace
> goto sentences and delete out label.
> 
> Signed-off-by: wengjianfeng 
> ---
>  drivers/nfc/s3fwrn5/core.c | 12 
>  1 file changed, 4 insertions(+), 8 deletions(-)


Reviewed-by: Krzysztof Kozlowski 


Best regards,
Krzysztof

[PATCH v2] nfc: s3fwrn5: remove unnecessary label

[samirweng1979]

From: wengjianfeng 

In function s3fwrn5_nci_post_setup, the variable ret is assigned then
goto out label, which just return ret, so we use return to replace it.
Other goto sentences are similar, we use return sentences to replace
goto sentences and delete out label.

Signed-off-by: wengjianfeng 
---
 drivers/nfc/s3fwrn5/core.c | 12 
 1 file changed, 4 insertions(+), 8 deletions(-)

diff --git a/drivers/nfc/s3fwrn5/core.c b/drivers/nfc/s3fwrn5/core.c
index c00b7a0..865d3e3 100644
--- a/drivers/nfc/s3fwrn5/core.c
+++ b/drivers/nfc/s3fwrn5/core.c
@@ -124,13 +124,12 @@ static int s3fwrn5_nci_post_setup(struct nci_dev *ndev)
 
if (s3fwrn5_firmware_init(info)) {
//skip bootloader mode
-   ret = 0;
-   goto out;
+   return 0;
}
 
ret = s3fwrn5_firmware_update(info);
if (ret < 0)
-   goto out;
+   return ret;
 
/* NCI core reset */
 
@@ -139,12 +138,9 @@ static int s3fwrn5_nci_post_setup(struct nci_dev *ndev)
 
ret = nci_core_reset(info->ndev);
if (ret < 0)
-   goto out;
-
-   ret = nci_core_init(info->ndev);
+   return ret;
 
-out:
-   return ret;
+   return nci_core_init(info->ndev);
 }
 
 static struct nci_ops s3fwrn5_nci_ops = {
-- 
1.9.1