Matt Reimer wrote:
> On 6/24/06, Aric Blumer <aric_at_sdgsystems.com> wrote:
>> Paul Sokolovsky wrote:
>> > Hello pHilipp,
>> >
>> > Saturday, June 24, 2006, 9:58:24 AM, you wrote:
>> >
>> >
>> >> Hi!
>> >>
>> >> I'd like to build a kernel that works on both HX4700 and HTC
>> Magician.
>> >>
>> >> Currently it seems impossible because of the flash wakeup code of the
>> >> HX4700 in sleep.S
>> >> Is there any place this code could be moved?
>> >>
>> >
>> > I myself wondered why it was put there - unlike other issue you
>> > reported lately, this snippet seems to be perfectly suitable for ll_pm
>> > hanlers.
>> >
>>
>> We had a discussion about this about a year ago when I put it in. The
>> flash comes out of suspend unwritable, and if something tries to write
>> to it before we have a chance to unlock it, chaos ensues. We tried this
>> elsewhere, but it seemed that other code could always write before it
>> was unlocked.
>
> If hx4700 only uses flash as an mtd block device, couldn't that code
> be moved into the mtd driver's resume()?
We had looked into doing that, but I don't remember why it didn't work.
(I don't keep good notes like you do. :-) ) I do recall that the mtd
code was using the deprecated pm_register method to do suspend/resume,
and that code was not being executed (perhaps because we didn't have
CONFIG_PM_LEGACY=y).
Received on Sat Jul 08 2006 - 14:04:01 EDT
This archive was generated by hypermail 2.2.0 : Sat Jul 08 2006 - 14:06:26 EDT