On Wed, Mar 21, 2012 at 10:35:53PM +0530, Ashish Chavan wrote:

> > > Actually I don't have enough details about the problem that you
> > > mentioned with this. I am waiting for inputs from some body else. Can

> > ...and didn't bother asking any questions so it's unlikely anyone will
> > say anything...

> No, actually it's not like that. You mentioned that the sequence should
> be changed, which will fix an issue that the driver has with suspend and
> resume. I asked other users of this driver about the issue and waiting
> for their feedback. I just wanted to have clear picture about the issue
> before I commit any change there. Just to make sure that only changing
> of sequence is enough or some thing extra need to be done.

No, I didn't say that the sequence should be changed.  I said that it
should be implemented using the relevant framework features rather than
being open coded.  The sequence itself is fine from a framework point of
view.