lkml.org 
[lkml]   [2018]   [Jul]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v4 5/5] power: supply: ds2760_battery: add devicetree probing
Hi,

On Fri, Jul 06, 2018 at 07:35:52AM +0200, Daniel Mack wrote:
> Add a matching table for devicetree probing, and optionally set the module
> parameter variables from DT properties.
>
> Signed-off-by: Daniel Mack <daniel@zonque.org>
> ---

Acked-for-power-supply: Sebastian Reichel <sebastian.reichel@collabora.co.uk>

-- Sebastian

> drivers/power/supply/ds2760_battery.c | 29 ++++++++++++++++++++++++++-
> 1 file changed, 28 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/power/supply/ds2760_battery.c b/drivers/power/supply/ds2760_battery.c
> index aa406a7c65a1..11bed88a89fa 100644
> --- a/drivers/power/supply/ds2760_battery.c
> +++ b/drivers/power/supply/ds2760_battery.c
> @@ -29,6 +29,7 @@
> #include <linux/power_supply.h>
> #include <linux/suspend.h>
> #include <linux/w1.h>
> +#include <linux/of.h>
>
> static unsigned int cache_time = 1000;
> module_param(cache_time, uint, 0644);
> @@ -705,6 +706,24 @@ static int w1_ds2760_add_slave(struct w1_slave *sl)
>
> psy_cfg.drv_data = di;
>
> + if (dev->of_node) {
> + u32 tmp;
> +
> + psy_cfg.of_node = dev->of_node;
> +
> + if (!of_property_read_bool(dev->of_node, "maxim,pmod-enabled"))
> + pmod_enabled = true;
> +
> + if (!of_property_read_u32(dev->of_node,
> + "maxim,cache-time-ms", &tmp))
> + cache_time = tmp;
> +
> + if (!of_property_read_u32(dev->of_node,
> + "rated-capacity-microamp-hours",
> + &tmp))
> + rated_capacity = tmp / 10; /* property is in mAh */
> + }
> +
> di->charge_status = POWER_SUPPLY_STATUS_UNKNOWN;
>
> sl->family_data = di;
> @@ -719,7 +738,7 @@ static int w1_ds2760_add_slave(struct w1_slave *sl)
>
> ds2760_battery_write_status(di, status);
>
> - /* set rated capacity from module param */
> + /* set rated capacity from module param or device tree */
> if (rated_capacity)
> ds2760_battery_write_rated_capacity(di, rated_capacity);
>
> @@ -769,6 +788,13 @@ static void w1_ds2760_remove_slave(struct w1_slave *sl)
> power_supply_unregister(di->bat);
> }
>
> +#ifdef CONFIG_OF
> +static const struct of_device_id w1_ds2760_of_ids[] = {
> + { .compatible = "maxim,ds2760" },
> + {}
> +};
> +#endif
> +
> static struct w1_family_ops w1_ds2760_fops = {
> .add_slave = w1_ds2760_add_slave,
> .remove_slave = w1_ds2760_remove_slave,
> @@ -778,6 +804,7 @@ static struct w1_family_ops w1_ds2760_fops = {
> static struct w1_family w1_ds2760_family = {
> .fid = W1_FAMILY_DS2760,
> .fops = &w1_ds2760_fops,
> + .of_match_table = of_match_ptr(w1_ds2760_of_ids),
> };
> module_w1_family(w1_ds2760_family);
>
> --
> 2.17.1
>
[unhandled content-type:application/pgp-signature]
\
 
 \ /
  Last update: 2018-07-06 15:44    [W:0.964 / U:0.096 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site