diff mbox series

[linux-next] ARM: omap1: remove redundant variables err

Message ID 20220830142136.299373-1-cui.jinpeng2@zte.com.cn (mailing list archive)
State New, archived
Headers show
Series [linux-next] ARM: omap1: remove redundant variables err | expand

Commit Message

CGEL Aug. 30, 2022, 2:21 p.m. UTC 
From: Jinpeng Cui <cui.jinpeng2@zte.com.cn>

Rturn value directly from platform_device_register()
instead of getting value from redundant variable err.

Reported-by: Zeal Robot <zealci@zte.com.cn>
Signed-off-by: Jinpeng Cui <cui.jinpeng2@zte.com.cn>
---
 arch/arm/mach-omap1/board-ams-delta.c | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

Comments

Janusz Krzysztofik Sept. 5, 2022, 8:46 p.m. UTC | #1 
On Tuesday, 30 August 2022 16:21:36 CEST cgel.zte@gmail.com wrote:
> From: Jinpeng Cui <cui.jinpeng2@zte.com.cn>
>
> Rturn value directly from platform_device_register()
> instead of getting value from redundant variable err.
>
> Reported-by: Zeal Robot <zealci@zte.com.cn>
> Signed-off-by: Jinpeng Cui <cui.jinpeng2@zte.com.cn>

Acked-by: Janusz Krzysztofik <jmkrzyszt@gmail.com>

Note: The same change, form a different author @zte.com.cn, with a different
wording in commit message and description, was submitted a few hours before
this one.  However, I decided to respond with my Ack to this one because
the other one was not sent to linux-omap@vger.kernel.org, then it didn't
reach OMAP patchwork.  If my decision is not in line with some
recommendations and I should rather prefer the submission sent earlier then
please let me know and I'll respond with my Ack to it as well.

Thanks,
Janusz


> ---
>  arch/arm/mach-omap1/board-ams-delta.c | 6 +-----
>  1 file changed, 1 insertion(+), 5 deletions(-)
>
> diff --git a/arch/arm/mach-omap1/board-ams-delta.c b/arch/arm/mach-omap1/board-ams-delta.c
> index 651c28d81132..e5531ef0fbde 100644
> --- a/arch/arm/mach-omap1/board-ams-delta.c
> +++ b/arch/arm/mach-omap1/board-ams-delta.c
> @@ -822,8 +822,6 @@ static int __init modem_nreset_init(void)
>   */
>  static int __init ams_delta_modem_init(void)
>  {
> -     int err;
> -
>       if (!machine_is_ams_delta())
>               return -ENODEV;
>
> @@ -832,9 +830,7 @@ static int __init ams_delta_modem_init(void)
>       /* Initialize the modem_nreset regulator consumer before use */
>       modem_priv.regulator = ERR_PTR(-ENODEV);
>
> -     err = platform_device_register(&ams_delta_modem_device);
> -
> -     return err;
> +     return platform_device_register(&ams_delta_modem_device);
>  }
>  arch_initcall_sync(ams_delta_modem_init);
>
>
Tony Lindgren March 27, 2023, 7:11 a.m. UTC | #2 
* cgel.zte@gmail.com <cgel.zte@gmail.com> [220830 17:14]:
> From: Jinpeng Cui <cui.jinpeng2@zte.com.cn>
> 
> Rturn value directly from platform_device_register()
> instead of getting value from redundant variable err.

Sorry looks like this is still pending, applying into
omap-for-v6.4/omap1 thanks.

Tony
diff mbox series

Patch

diff --git a/arch/arm/mach-omap1/board-ams-delta.c b/arch/arm/mach-omap1/board-ams-delta.c
index 651c28d81132..e5531ef0fbde 100644
--- a/arch/arm/mach-omap1/board-ams-delta.c
+++ b/arch/arm/mach-omap1/board-ams-delta.c
@@ -822,8 +822,6 @@  static int __init modem_nreset_init(void)
  */
 static int __init ams_delta_modem_init(void)
 {
-	int err;
-
 	if (!machine_is_ams_delta())
 		return -ENODEV;
 
@@ -832,9 +830,7 @@  static int __init ams_delta_modem_init(void)
 	/* Initialize the modem_nreset regulator consumer before use */
 	modem_priv.regulator = ERR_PTR(-ENODEV);
 
-	err = platform_device_register(&ams_delta_modem_device);
-
-	return err;
+	return platform_device_register(&ams_delta_modem_device);
 }
 arch_initcall_sync(ams_delta_modem_init);