linux-kernel - Re: Power: s3c_adc_battery.c backup battery query

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <CA+E=qVdOhTr9iXR9w9UdAQ6q0XuqQLunxyaxgcfd0Y1Tn8hzpA@mail.gmail.com>
Date:	Wed, 22 Aug 2012 10:34:04 +0300
From:	Vasily Khoruzhick <anarsoul@...il.com>
To:	anish kumar <yesanishhere@...il.com>
Cc:	linux-kernel@...r.kernel.org, lars@...afoo.de, cbou@...l.ru,
	dwmw2@...radead.org, kzjeef@...il.com
Subject: Re: Power: s3c_adc_battery.c backup battery query

On Wed, Aug 22, 2012 at 8:05 AM, anish kumar <yesanishhere@...il.com> wrote:
> Hello,

Hi

> I am trying to write a generic batttery driver using IIO and I have
> one some below
> questions:
>
> Why do we have the representation of backup battery in this
> driver(s3c_adc_battery.c) and when does the
> s3c_adc_backup_bat_get_property function gets called?

Because backup battery differs a bit from main battery (for example,
it has no current monitoring capability).
Maybe it was not a good decision, as it's possible to handle it via
platform data info as you suggested.

> As I understand, it is as this:It is called when it is registered
> and when the user wants to know the backup battery status(sysfs)
> and we don't do a workqueue scheduling as it will consume more power.
>
> static struct s3c_adc_bat backup_bat = {
>         .psy = {
>                 .name           = "backup-battery",
>                 .type           = POWER_SUPPLY_TYPE_BATTERY,
>                 .properties     = s3c_adc_backup_bat_props,
>                 .num_properties = ARRAY_SIZE(s3c_adc_backup_bat_props),
>                 .get_property   = s3c_adc_backup_bat_get_property,
>                 .use_for_apm    = 1,
>         },
> };
>
> I am stuck with mulitple *get_property callbacks.As we don't know how many
> batteries system has and consequently we don't know how many *get_property
> callbacks to be implemented.
>
> So below is my plan:
> What I am trying to do is to have a single callback(*get_property) and manage
> everything in this callback and will use the name of the psy to
> distinguish between
> the different batteries which system has and driver will receive this
> information using
> platform data.Hope it makes sense.

Just send a patch for review :)

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