linux-kernel - Re: [PATCH] power_supply: Add support for WM8350 PMU

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] [thread-next>] [day] [month] [year] [list]

Message-Id: <20081110154330.f95ec239.akpm@linux-foundation.org>
Date:	Mon, 10 Nov 2008 15:43:30 -0800
From:	Andrew Morton <akpm@...ux-foundation.org>
To:	Mark Brown <broonie@...nsource.wolfsonmicro.com>
Cc:	cbouatmailru@...il.com, sameo@...nedhand.com,
	linux-kernel@...r.kernel.org, broonie@...nsource.wolfsonmicro.com
Subject: Re: [PATCH] power_supply: Add support for WM8350 PMU

On Thu,  6 Nov 2008 15:15:22 +0000
Mark Brown <broonie@...nsource.wolfsonmicro.com> wrote:

> This patch adds support for the PMU provided by the WM8350 which
> implements battery, line and USB supplies including a battery charger.
> The hardware functions largely autonomously, with minimal software
> control required to initiate fast charging.
> 
> Support for configuration of the USB supply is not yet implemented.
> This means that the hardware will remain in the mode configured at
> startup, by default limiting the current drawn from USB to 100mA.
> 
> This driver was originally written by Liam Girdwood with subsequent
> updates for submission by Mark Brown.

An idle question...

> +		supply = "Battery";
> +		supply = "USB";
> +		supply = "Line";
> +		supply = "Battery and USB";

> +		charge = "Charger Off";
> +		charge = "Trickle Charging";
> +		charge = "Fast Charging";

> +		dev_err(wm8350->dev, "battery too hot\n");
> +		dev_err(wm8350->dev, "battery too cold\n");
> +		dev_err(wm8350->dev, "battery failed\n");
> +		dev_err(wm8350->dev, "charger timeout\n");
> +		dev_dbg(wm8350->dev, "charger stopped\n");
> +		dev_dbg(wm8350->dev, "charger started\n");
> +		dev_dbg(wm8350->dev, "fast charger ready\n");
> +		dev_warn(wm8350->dev, "battery < 3.9V\n");
> +		dev_warn(wm8350->dev, "battery < 3.1V\n");
> +		dev_warn(wm8350->dev, "battery < 2.85V\n");

The above constitutes a sort-of userspace interface.  A number of the
things which are being reported there are fairly common to the
technology and other drivers will be able to report them.

But afaict this interface is unique to this particular driver.  Has any
thought been given to standardising this interface across all similar
drivers?  Too messy?

--
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/