Re: [PATCH v4 3/8] power: power_supply: add battery information struct

[Sebastian Reichel <sre-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>]

[-- Attachment #1: Type: text/plain, Size: 1327 bytes --]

Hi,

On Sat, Jan 21, 2017 at 11:13:59PM -0800, Matt Ranostay wrote:
> Add power_supply_battery_info structure that is used to
> enumerate battery state machine settings.
> 
> Signed-off-by: Matt Ranostay <matt-sk+viVC6FLCDq+mSdOJa79kegs52MxvZ@public.gmane.org>
> ---
>  include/linux/power_supply.h | 12 ++++++++++++
>  1 file changed, 12 insertions(+)
> 
> diff --git a/include/linux/power_supply.h b/include/linux/power_supply.h
> index 3965503315ef..59674a257d5e 100644
> --- a/include/linux/power_supply.h
> +++ b/include/linux/power_supply.h
> @@ -288,6 +288,18 @@ struct power_supply_info {
>  	int use_for_apm;
>  };
>  
> +/*
> + * This is the recommended struct to specify static battery parameters.
> + * Power supply class itself doesn't use this, but it implements access
> + * that most platform drivers should use for consistency.
> + */
> +
> +struct power_supply_battery_info {
> +	int energy;		/* uWh */
> +	int power;		/* uAh */
> +	int nominal_voltage;	/* uV */
> +};
> +
>  extern struct atomic_notifier_head power_supply_notifier;
>  extern int power_supply_reg_notifier(struct notifier_block *nb);
>  extern void power_supply_unreg_notifier(struct notifier_block *nb);

You can merge this into the next patch, which adds the API
for using this struct.

-- Sebastian

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]