lkml.org 
[lkml]   [2014]   [Jan]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
Subject[PATCH] Power: charger-manager: Fixed coding style
Date
Removed apaces before semicolon and fixed brackets on if statements

Signed-off-by: Jesper Falk <jesperfalk94@gmail.com>
---
drivers/power/charger-manager.c | 19 ++++++++-----------
1 file changed, 8 insertions(+), 11 deletions(-)

diff --git a/drivers/power/charger-manager.c b/drivers/power/charger-manager.c
index 7287c0e..4c5a848 100644
--- a/drivers/power/charger-manager.c
+++ b/drivers/power/charger-manager.c
@@ -327,7 +327,7 @@ static int try_charger_enable(struct charger_manager *cm, bool enable)
cm->charging_start_time = ktime_to_ms(ktime_get());
cm->charging_end_time = 0;

- for (i = 0 ; i < desc->num_charger_regulators ; i++) {
+ for (i = 0; i < desc->num_charger_regulators; i++) {
if (desc->charger_regulators[i].externally_control)
continue;

@@ -345,7 +345,7 @@ static int try_charger_enable(struct charger_manager *cm, bool enable)
cm->charging_start_time = 0;
cm->charging_end_time = ktime_to_ms(ktime_get());

- for (i = 0 ; i < desc->num_charger_regulators ; i++) {
+ for (i = 0; i < desc->num_charger_regulators; i++) {
if (desc->charger_regulators[i].externally_control)
continue;

@@ -1476,20 +1476,17 @@ static int charger_manager_probe(struct platform_device *pdev)
* The following two do not need to be errors.
* Users may intentionally ignore those two features.
*/
- if (desc->fullbatt_uV == 0) {
+ if (desc->fullbatt_uV == 0)
dev_info(&pdev->dev, "Ignoring full-battery voltage threshold as it is not supplied\n");
- }
if (!desc->fullbatt_vchkdrop_ms || !desc->fullbatt_vchkdrop_uV) {
dev_info(&pdev->dev, "Disabling full-battery voltage drop checking mechanism as it is not supplied\n");
desc->fullbatt_vchkdrop_ms = 0;
desc->fullbatt_vchkdrop_uV = 0;
}
- if (desc->fullbatt_soc == 0) {
+ if (desc->fullbatt_soc == 0)
dev_info(&pdev->dev, "Ignoring full-battery soc(state of charge) threshold as it is not supplied\n");
- }
- if (desc->fullbatt_full_capacity == 0) {
+ if (desc->fullbatt_full_capacity == 0)
dev_info(&pdev->dev, "Ignoring full-battery full capacity threshold as it is not supplied\n");
- }

if (!desc->charger_regulators || desc->num_charger_regulators < 1) {
dev_err(&pdev->dev, "charger_regulators undefined\n");
@@ -1676,16 +1673,16 @@ static int charger_manager_remove(struct platform_device *pdev)
cancel_work_sync(&setup_polling);
cancel_delayed_work_sync(&cm_monitor_work);

- for (i = 0 ; i < desc->num_charger_regulators ; i++) {
+ for (i = 0; i < desc->num_charger_regulators; i++) {
struct charger_regulator *charger
= &desc->charger_regulators[i];
- for (j = 0 ; j < charger->num_cables ; j++) {
+ for (j = 0; j < charger->num_cables; j++) {
struct charger_cable *cable = &charger->cables[j];
extcon_unregister_interest(&cable->extcon_dev);
}
}

- for (i = 0 ; i < desc->num_charger_regulators ; i++)
+ for (i = 0; i < desc->num_charger_regulators; i++)
regulator_put(desc->charger_regulators[i].consumer);

power_supply_unregister(&cm->charger_psy);
--
1.8.5.2


\
 
 \ /
  Last update: 2014-01-06 17:21    [W:0.054 / U:0.036 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site