[PATCH v4 0/4] ARM: OMAP2+: AM33XX: VDD CORE OPP50 support

[Kevin Hilman]

On Tue, Sep 3, 2013 at 11:55 AM, Russ Dill <Russ.Dill at ti.com> wrote:
> On Fri, Aug 30, 2013 at 9:06 AM, Kevin Hilman <khilman at linaro.org> wrote:
>> Well, I was thinking of something much dumber.
>>
>> I was thinking about just _carefully_ writing a single, self-contained C
>> function, with all of its data on the stack (and consts as #defines).
>> Think of it is a step up in readability from straight assembly (which
>> was the stated reason for moving the code to the M3 in the first place.)
>>
>> Kevin
>
> That can only give you your suspend function. You need a resume
> function to undo everything before calling cpu_resume, and the resume
> function needs access to the data that was in the suspend function.

Well, it can be done (and I've seen it done) with various hacks, but I
agree it's not pretty, but it is at least slightly prettier than piles
of asm.  That being said, I saw you posted a more generic solution
that looks like a real solution that can scale.  I'll have a closer
look at that.  Thanks for working on it.

Kevin