[PATCH 0/5] Power supply and resume ordering meddling
Michael 'Mickey' Lauer
mickey at openmoko.org
Mon Jun 2 15:41:28 CEST 2008
On Monday 02 June 2008 14:27:37 Andy Green wrote:
> The following series fixes some bugs around pcf50633 charging;
> implements the red LED on the AUX key being lit when pcf50633
> decides to charge; introduces a new method for establishing
> resume dependency between drivers; implements this new method
> to make jbt6k74 dependent on pcf50633 AND glamo resume before
> it resumes, instead of using a sleepy workqueue.
> Let the complaints begin!
Here we go:
LED on AUX for charging is very suboptimal. Conceptually, the power button has
to do with power, so if at all, the power LED needs to be lit on charging. (I
know it wasn't your idea, Andy. For my releases, I will revert this patch.)
a) This hardcoded behaviour should be configurable
(CONFIG_HARDCODED_CHARGING_LED) for people who rather want userland to be in
control of the LEDs.
b) It looks this implementation is completely ignoring the LED class driver
which already implements a charging trigger. Why?
More information about the openmoko-kernel