[PATCH] Ensure pcf50606_global is set
laforge at openmoko.org
Sat Jun 2 18:21:07 CEST 2007
On Sat, Jun 02, 2007 at 12:22:57AM +0200, andrzej zaborowski wrote:
> With a non-modular kernel, the s3c_udc is initalised before PCF50606.
> If the UDC is drawing power from the host, gta01_udc_vbus_draw() is
> called back and it will try to tell the PCF50606 (which is
> uninitialised) to enable fast-charge.
> The patch also eliminates the race between code that wants to use
> pcf50606_global before setting up I2C, by reordering the actions in
> pcf50606_detect(). The race is not only potential because I've hit it.
> The pcf50606_global usage is generally ugly :)
tell me ;)
But it was the 'best' solution I could come up with for now. It's hard
to think of devices that have multiple of the same PMU's, but inevitably
they will sooner or later exist, though.
Also, without a proper abstraction level and infrastructure (i.e. power
resources associated with each device, and power provides (pmu's) I
doubt it makes sense to clean this up any further.
But if you have ideas, please let me know.
> BTW I also had to disable PCF50606_FEAT_WDT in platform_data to make
> pcf50606 work, not sure why.
that's strange. This is on a GTA01?
- Harald Welte <laforge at openmoko.org> http://openmoko.org/
Software for the world's first truly open Free Software mobile phone
More information about the openmoko-kernel