Commit 53207aa1 authored by Linus Walleij's avatar Linus Walleij Committed by Lee Jones

power: ab8500: Require device tree

The core AB8500 driver and the whole platform is completely
dependent on being probed from device tree so remove the
non-DT probe paths.
Signed-off-by: default avatarLinus Walleij <linus.walleij@linaro.org>
Acked-by: default avatarSebastian Reichel <sebastian.reichel@collabora.com>
Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
parent a38fd874
...@@ -698,7 +698,7 @@ config BATTERY_GAUGE_LTC2941 ...@@ -698,7 +698,7 @@ config BATTERY_GAUGE_LTC2941
config AB8500_BM config AB8500_BM
bool "AB8500 Battery Management Driver" bool "AB8500 Battery Management Driver"
depends on AB8500_CORE && AB8500_GPADC && (IIO = y) depends on AB8500_CORE && AB8500_GPADC && (IIO = y) && OF
help help
Say Y to include support for AB8500 battery management. Say Y to include support for AB8500 battery management.
......
...@@ -1008,12 +1008,10 @@ static int ab8500_btemp_probe(struct platform_device *pdev) ...@@ -1008,12 +1008,10 @@ static int ab8500_btemp_probe(struct platform_device *pdev)
} }
di->bm = plat; di->bm = plat;
if (np) { ret = ab8500_bm_of_probe(dev, np, di->bm);
ret = ab8500_bm_of_probe(dev, np, di->bm); if (ret) {
if (ret) { dev_err(dev, "failed to get battery information\n");
dev_err(dev, "failed to get battery information\n"); return ret;
return ret;
}
} }
/* get parent data */ /* get parent data */
......
...@@ -3360,15 +3360,12 @@ static int ab8500_charger_probe(struct platform_device *pdev) ...@@ -3360,15 +3360,12 @@ static int ab8500_charger_probe(struct platform_device *pdev)
} }
di->bm = plat; di->bm = plat;
if (np) { ret = ab8500_bm_of_probe(dev, np, di->bm);
ret = ab8500_bm_of_probe(dev, np, di->bm); if (ret) {
if (ret) { dev_err(dev, "failed to get battery information\n");
dev_err(dev, "failed to get battery information\n"); return ret;
return ret; }
} di->autopower_cfg = of_property_read_bool(np, "autopower_cfg");
di->autopower_cfg = of_property_read_bool(np, "autopower_cfg");
} else
di->autopower_cfg = false;
/* get parent data */ /* get parent data */
di->dev = dev; di->dev = dev;
......
...@@ -3043,12 +3043,10 @@ static int ab8500_fg_probe(struct platform_device *pdev) ...@@ -3043,12 +3043,10 @@ static int ab8500_fg_probe(struct platform_device *pdev)
} }
di->bm = plat; di->bm = plat;
if (np) { ret = ab8500_bm_of_probe(dev, np, di->bm);
ret = ab8500_bm_of_probe(dev, np, di->bm); if (ret) {
if (ret) { dev_err(dev, "failed to get battery information\n");
dev_err(dev, "failed to get battery information\n"); return ret;
return ret;
}
} }
mutex_init(&di->cc_lock); mutex_init(&di->cc_lock);
......
...@@ -1997,12 +1997,10 @@ static int abx500_chargalg_probe(struct platform_device *pdev) ...@@ -1997,12 +1997,10 @@ static int abx500_chargalg_probe(struct platform_device *pdev)
} }
di->bm = plat; di->bm = plat;
if (np) { ret = ab8500_bm_of_probe(&pdev->dev, np, di->bm);
ret = ab8500_bm_of_probe(&pdev->dev, np, di->bm); if (ret) {
if (ret) { dev_err(&pdev->dev, "failed to get battery information\n");
dev_err(&pdev->dev, "failed to get battery information\n"); return ret;
return ret;
}
} }
/* get device struct and parent */ /* get device struct and parent */
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment