2017-11-11 17:48:58

by Arvind Yadav

[permalink] [raw]
Subject: [PATCH] powercap: Remove reassignment

It is not necessary to reassignment of 'result'.
Here, result is being initialized zero and then updated with
seed_constraint_attributes().
class_register is enough to return successful and error value.

Signed-off-by: Arvind Yadav <[email protected]>
---
drivers/powercap/powercap_sys.c | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/drivers/powercap/powercap_sys.c b/drivers/powercap/powercap_sys.c
index 5b10b50..64b2b25 100644
--- a/drivers/powercap/powercap_sys.c
+++ b/drivers/powercap/powercap_sys.c
@@ -673,15 +673,13 @@ EXPORT_SYMBOL_GPL(powercap_unregister_control_type);

static int __init powercap_init(void)
{
- int result = 0;
+ int result;

result = seed_constraint_attributes();
if (result)
return result;

- result = class_register(&powercap_class);
-
- return result;
+ return class_register(&powercap_class);
}

device_initcall(powercap_init);
--
2.7.4


From 1586781243977082175@xxx Thu Dec 14 17:28:38 +0000 2017
X-GM-THRID: 1586533440851029553
X-Gmail-Labels: Inbox,Category Forums