On Wednesday 09 January 2013 03:07 PM, Bedia, Vaibhav wrote:

On Wed, Jan 09, 2013 at 13:01:03, Shilimkar, Santosh wrote:

quoted

On Wednesday 09 January 2013 11:08 AM, Bedia, Vaibhav wrote:

quoted

On Tue, Jan 08, 2013 at 20:51:08, Shilimkar, Santosh wrote:

quoted

On Monday 31 December 2012 06:37 PM, Vaibhav Bedia wrote:

quoted

Add minimal APIs for writing to the IPC and the M3_TXEV registers
in the Control module. These will be used in a subsequent patch which
adds suspend-resume support for AM33XX.

Signed-off-by: Vaibhav Bedia <redacted>
Cc: Tony Lingren <tony@atomide.com>
Cc: Santosh Shilimkar <redacted>
Cc: Benoit Cousson <redacted>
Cc: Paul Walmsley <paul@pwsan.com>
Cc: Kevin Hilman <redacted>
Cc: Vaibhav Hiremath <redacted>
---

On Control module, we are trying to move driver/module
specific code to respective drivers. Can you not
add below code to ipc related driver component.

If not, then patch as such is fine with me.

I had it in the pm33xx.c in the previous version. Kevin had asked me to
move it to control.c. Should I revert move it back there?

pm33xx.c is not the right place. I was hoping to move to some driver.
If that is not possible then leave it in control module.

I think I'll leave it here for now. Could I have you ack in that case?

Yep.
Acked-by: Santosh Shilimkar <redacted>