From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Bedia, Vaibhav" Subject: RE: [RFC v2 14/18] ARM: OMAP2+: AM33XX: control: Add some control module registers and APIs Date: Wed, 9 Jan 2013 05:38:25 +0000 Message-ID: References: <1356959231-17335-1-git-send-email-vaibhav.bedia@ti.com> <1356959231-17335-15-git-send-email-vaibhav.bedia@ti.com> <50EC3964.3060708@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT Return-path: Received: from comal.ext.ti.com ([198.47.26.152]:33857 "EHLO comal.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751060Ab3AIFid convert rfc822-to-8bit (ORCPT ); Wed, 9 Jan 2013 00:38:33 -0500 In-Reply-To: <50EC3964.3060708@ti.com> Content-Language: en-US Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: "Shilimkar, Santosh" Cc: "linux-omap@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "tony@atomide.com" , "khilman@deeprootsystems.com" , "Cousson, Benoit" , Paul Walmsley , "Hiremath, Vaibhav" 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 > > Cc: Tony Lingren > > Cc: Santosh Shilimkar > > Cc: Benoit Cousson > > Cc: Paul Walmsley > > Cc: Kevin Hilman > > Cc: Vaibhav Hiremath > > --- > 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? Regards, Vaibhav From mboxrd@z Thu Jan 1 00:00:00 1970 From: vaibhav.bedia@ti.com (Bedia, Vaibhav) Date: Wed, 9 Jan 2013 05:38:25 +0000 Subject: [RFC v2 14/18] ARM: OMAP2+: AM33XX: control: Add some control module registers and APIs In-Reply-To: <50EC3964.3060708@ti.com> References: <1356959231-17335-1-git-send-email-vaibhav.bedia@ti.com> <1356959231-17335-15-git-send-email-vaibhav.bedia@ti.com> <50EC3964.3060708@ti.com> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org 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 > > Cc: Tony Lingren > > Cc: Santosh Shilimkar > > Cc: Benoit Cousson > > Cc: Paul Walmsley > > Cc: Kevin Hilman > > Cc: Vaibhav Hiremath > > --- > 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? Regards, Vaibhav