Re: [UPDATED v3][PATCH 0/7] regulator: voltage and currentregulator framework

[Krzysztof Helt]

On Thu, 06 Mar 2008 18:10:30 +0000
Liam Girdwood <lg@xxxxxxxxxxxxxxxxxxxxxxxxxxx> wrote:

> Updated with comments received from v2. I've also added more in code
> comments to hopefully make the review a little easier to read through.
> 
> This patch series provides a generic framework to allow device drivers
> to control voltage and current regulators on SoC based devices (e.g.
> phones, gps, media players).
> 

>  4. Userspace.
> 
>   The framework also exports a lot of useful voltage/current data to
> userspace via sysfs. This could be used to monitor device and regulator
> power and status.
> 

IMO, this work should be merged into kernel hwmon project. The hwmon
project has already sysfs API to read voltages and currents. Hwmon guys should
be able to extend the API to cover also possibility to change (set) voltages
and currents (as they do currently with fans).

The advantage is that monitoring of regulators (reading voltages and currents)
would be instantially supported by all existing lm-sensors tool.

You should send your patches to kernel hwmon list: lm-sensors@xxxxxxxxxxxxxx

Kind regards,
Krzysztof

----------------------------------------------------------------------
Gdzie warto jechac na urlop? Sprawdz!
kliknij >>> http://link.interia.pl/f1e32

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