[PATCH] Power: charger-manager: Fixed coding style

From: Jesper Falk
Date: Mon Jan 06 2014 - 10:46:31 EST


Removed apaces before semicolon and fixed brackets on if statements

Signed-off-by: Jesper Falk <jesperfalk94@xxxxxxxxx>
---
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

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/