On Wed, Jan 09, 2013 at 13:01:03, Shilimkar, Santosh wrote: > On Wednesday 09 January 2013 11:08 AM, Bedia, Vaibhav wrote: > > On Tue, Jan 08, 2013 at 20:51:08, Shilimkar, Santosh wrote: > >> On Monday 31 December 2012 06:37 PM, Vaibhav Bedia wrote: > >>> 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 <vaibhav.bedia@xxxxxx> > >>> Cc: Tony Lingren <tony@xxxxxxxxxxx> > >>> Cc: Santosh Shilimkar <santosh.shilimkar@xxxxxx> > >>> Cc: Benoit Cousson <b-cousson@xxxxxx> > >>> Cc: Paul Walmsley <paul@xxxxxxxxx> > >>> Cc: Kevin Hilman <khilman@xxxxxxxxxxxxxxxxxxx> > >>> Cc: Vaibhav Hiremath <hvaibhav@xxxxxx> > >>> --- > >> 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? Thanks, Vaibhav -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html