From: "Bedia, Vaibhav" <vaibhav.bedia@ti.com>
To: "Shilimkar, Santosh" <santosh.shilimkar@ti.com>
Cc: "linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>,
"tony@atomide.com" <tony@atomide.com>,
"khilman@deeprootsystems.com" <khilman@deeprootsystems.com>,
"Cousson, Benoit" <b-cousson@ti.com>,
Paul Walmsley <paul@pwsan.com>,
"Hiremath, Vaibhav" <hvaibhav@ti.com>
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 [thread overview]
Message-ID: <B5906170F1614E41A8A28DE3B8D121433ECDD089@DBDE01.ent.ti.com> (raw)
In-Reply-To: <50EC3964.3060708@ti.com>
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@ti.com>
> > Cc: Tony Lingren <tony@atomide.com>
> > Cc: Santosh Shilimkar <santosh.shilimkar@ti.com>
> > Cc: Benoit Cousson <b-cousson@ti.com>
> > Cc: Paul Walmsley <paul@pwsan.com>
> > Cc: Kevin Hilman <khilman@deeprootsystems.com>
> > Cc: Vaibhav Hiremath <hvaibhav@ti.com>
> > ---
> 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
WARNING: multiple messages have this Message-ID (diff)
From: vaibhav.bedia@ti.com (Bedia, Vaibhav)
To: linux-arm-kernel@lists.infradead.org
Subject: [RFC v2 14/18] ARM: OMAP2+: AM33XX: control: Add some control module registers and APIs
Date: Wed, 9 Jan 2013 05:38:25 +0000 [thread overview]
Message-ID: <B5906170F1614E41A8A28DE3B8D121433ECDD089@DBDE01.ent.ti.com> (raw)
In-Reply-To: <50EC3964.3060708@ti.com>
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@ti.com>
> > Cc: Tony Lingren <tony@atomide.com>
> > Cc: Santosh Shilimkar <santosh.shilimkar@ti.com>
> > Cc: Benoit Cousson <b-cousson@ti.com>
> > Cc: Paul Walmsley <paul@pwsan.com>
> > Cc: Kevin Hilman <khilman@deeprootsystems.com>
> > Cc: Vaibhav Hiremath <hvaibhav@ti.com>
> > ---
> 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
next prev parent reply other threads:[~2013-01-09 5:38 UTC|newest]
Thread overview: 148+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-12-31 13:06 [RFC v2 00/18] ARM: OMAP2+: AM33XX: Add suspend-resume support Vaibhav Bedia
2012-12-31 13:06 ` Vaibhav Bedia
2012-12-31 13:06 ` [RFC v2 01/18] mailbox: OMAP2+: Add support for AM33XX Vaibhav Bedia
2012-12-31 13:06 ` Vaibhav Bedia
2013-01-01 18:25 ` Tony Lindgren
2013-01-01 18:25 ` Tony Lindgren
2013-01-02 11:09 ` Bedia, Vaibhav
2013-01-02 11:09 ` Bedia, Vaibhav
2013-01-08 13:53 ` Santosh Shilimkar
2013-01-08 13:53 ` Santosh Shilimkar
2013-01-09 5:16 ` Bedia, Vaibhav
2013-01-09 5:16 ` Bedia, Vaibhav
2012-12-31 13:06 ` [RFC v2 02/18] mailbox: Add an API for flushing the FIFO Vaibhav Bedia
2012-12-31 13:06 ` Vaibhav Bedia
2013-01-08 13:56 ` Santosh Shilimkar
2013-01-08 13:56 ` Santosh Shilimkar
2013-01-09 5:16 ` Bedia, Vaibhav
2013-01-09 5:16 ` Bedia, Vaibhav
2012-12-31 13:06 ` [RFC v2 03/18] memory: emif: Move EMIF related header file to include/linux/ Vaibhav Bedia
2012-12-31 13:06 ` Vaibhav Bedia
2013-01-08 14:04 ` Santosh Shilimkar
2013-01-08 14:04 ` Santosh Shilimkar
2013-01-09 5:31 ` Bedia, Vaibhav
2013-01-09 5:31 ` Bedia, Vaibhav
2012-12-31 13:06 ` [RFC v2 04/18] ARM: OMAP2+: AM33XX: CM: Get rid of unncessary header inclusions Vaibhav Bedia
2012-12-31 13:06 ` Vaibhav Bedia
2013-01-08 15:00 ` Santosh Shilimkar
2013-01-08 15:00 ` Santosh Shilimkar
2012-12-31 13:06 ` [RFC v2 05/18] ARM: OMAP2+: AM33XX: CM/PRM: Use __ASSEMBLER__ macros in header files Vaibhav Bedia
2012-12-31 13:06 ` Vaibhav Bedia
2013-01-08 15:01 ` Santosh Shilimkar
2013-01-08 15:01 ` Santosh Shilimkar
2013-01-09 5:31 ` Bedia, Vaibhav
2013-01-09 5:31 ` Bedia, Vaibhav
2012-12-31 13:06 ` [RFC v2 06/18] ARM: OMAP2+: AM33XX: hwmod: Register OCMC RAM hwmod Vaibhav Bedia
2012-12-31 13:06 ` Vaibhav Bedia
2013-01-08 15:04 ` Santosh Shilimkar
2013-01-08 15:04 ` Santosh Shilimkar
2012-12-31 13:07 ` [RFC v2 07/18] ARM: OMAP2+: AM33XX: hwmod: Update TPTC0 hwmod with the right flags Vaibhav Bedia
2012-12-31 13:07 ` Vaibhav Bedia
2013-01-08 15:05 ` Santosh Shilimkar
2013-01-08 15:05 ` Santosh Shilimkar
2013-01-09 6:02 ` Bedia, Vaibhav
2013-01-09 6:02 ` Bedia, Vaibhav
2013-02-11 23:33 ` Kevin Hilman
2013-02-11 23:33 ` Kevin Hilman
2013-02-13 10:56 ` Bedia, Vaibhav
2013-02-13 10:56 ` Bedia, Vaibhav
2012-12-31 13:07 ` [RFC v2 08/18] ARM: OMAP2+: AM33XX: hwmod: Fixup cpgmac0 hwmod entry Vaibhav Bedia
2012-12-31 13:07 ` Vaibhav Bedia
2013-01-08 15:08 ` Santosh Shilimkar
2013-01-08 15:08 ` Santosh Shilimkar
2012-12-31 13:07 ` [RFC v2 09/18] ARM: OMAP2+: AM33XX: hwmod: Update the WKUP-M3 hwmod with reset status bit Vaibhav Bedia
2012-12-31 13:07 ` Vaibhav Bedia
2013-01-08 15:09 ` Santosh Shilimkar
2013-01-08 15:09 ` Santosh Shilimkar
2012-12-31 13:07 ` [RFC v2 10/18] ARM: OMAP2+: AM33XX: Update the hardreset API Vaibhav Bedia
2012-12-31 13:07 ` Vaibhav Bedia
2013-01-08 15:10 ` Santosh Shilimkar
2013-01-08 15:10 ` Santosh Shilimkar
2012-12-31 13:07 ` [RFC v2 11/18] ARM: DTS: AM33XX: Add nodes for OCMC RAM and WKUP-M3 Vaibhav Bedia
2012-12-31 13:07 ` Vaibhav Bedia
2013-01-08 15:12 ` Santosh Shilimkar
2013-01-08 15:12 ` Santosh Shilimkar
2012-12-31 13:07 ` [RFC v2 12/18] ARM: OMAP2+: timer: Add suspend-resume callbacks for clockevent device Vaibhav Bedia
2012-12-31 13:07 ` Vaibhav Bedia
2013-01-08 15:15 ` Santosh Shilimkar
2013-01-08 15:15 ` Santosh Shilimkar
2013-01-11 4:37 ` Bedia, Vaibhav
2013-01-11 4:37 ` Bedia, Vaibhav
2013-01-17 18:45 ` Jon Hunter
2013-01-17 18:45 ` Jon Hunter
2013-01-18 5:25 ` Santosh Shilimkar
2013-01-18 5:25 ` Santosh Shilimkar
2013-01-21 7:22 ` Bedia, Vaibhav
2013-01-21 7:22 ` Bedia, Vaibhav
2013-01-30 17:46 ` Jon Hunter
2013-01-30 17:46 ` Jon Hunter
2013-01-31 11:17 ` Bedia, Vaibhav
2013-01-31 11:17 ` Bedia, Vaibhav
2013-01-17 18:40 ` Jon Hunter
2013-01-17 18:40 ` Jon Hunter
2013-01-21 7:22 ` Bedia, Vaibhav
2013-01-21 7:22 ` Bedia, Vaibhav
2012-12-31 13:07 ` [RFC v2 13/18] ARM: OMAP2+: AM33XX: timer: Interchance clkevt and clksrc timers Vaibhav Bedia
2012-12-31 13:07 ` Vaibhav Bedia
2013-01-08 15:17 ` Santosh Shilimkar
2013-01-08 15:17 ` Santosh Shilimkar
2013-01-09 5:31 ` Bedia, Vaibhav
2013-01-09 5:31 ` Bedia, Vaibhav
2013-01-17 19:09 ` Jon Hunter
2013-01-17 19:09 ` Jon Hunter
2013-01-24 22:30 ` Jon Hunter
2013-01-24 22:30 ` Jon Hunter
2013-01-30 17:48 ` Jon Hunter
2013-01-30 17:48 ` Jon Hunter
2013-01-30 17:49 ` Jon Hunter
2013-01-30 17:49 ` Jon Hunter
2013-01-31 11:29 ` Bedia, Vaibhav
2013-01-31 11:29 ` Bedia, Vaibhav
2012-12-31 13:07 ` [RFC v2 14/18] ARM: OMAP2+: AM33XX: control: Add some control module registers and APIs Vaibhav Bedia
2012-12-31 13:07 ` Vaibhav Bedia
2013-01-08 15:21 ` Santosh Shilimkar
2013-01-08 15:21 ` Santosh Shilimkar
2013-01-09 5:38 ` Bedia, Vaibhav [this message]
2013-01-09 5:38 ` Bedia, Vaibhav
2013-01-09 7:31 ` Santosh Shilimkar
2013-01-09 7:31 ` Santosh Shilimkar
2013-01-09 9:37 ` Bedia, Vaibhav
2013-01-09 9:37 ` Bedia, Vaibhav
2013-01-09 9:53 ` Santosh Shilimkar
2013-01-09 9:53 ` Santosh Shilimkar
2012-12-31 13:07 ` [RFC v2 15/18] ARM: OMAP2+: AM33XX: Add assembly code for PM operations Vaibhav Bedia
2012-12-31 13:07 ` Vaibhav Bedia
2012-12-31 13:07 ` [RFC v2 16/18] ARM: OMAP2+: AM33XX: Basic suspend resume support Vaibhav Bedia
2012-12-31 13:07 ` Vaibhav Bedia
2013-01-17 14:27 ` Peter Korsgaard
2013-01-17 14:27 ` Peter Korsgaard
2013-01-21 10:37 ` Bedia, Vaibhav
2013-01-21 10:37 ` Bedia, Vaibhav
2013-01-22 12:50 ` Peter Korsgaard
2013-01-22 12:50 ` Peter Korsgaard
2013-02-12 1:27 ` Kevin Hilman
2013-02-12 1:27 ` Kevin Hilman
2013-02-13 13:43 ` Bedia, Vaibhav
2013-02-13 13:43 ` Bedia, Vaibhav
2013-02-18 16:11 ` Kevin Hilman
2013-02-18 16:11 ` Kevin Hilman
2013-02-20 9:21 ` Bedia, Vaibhav
2013-02-20 9:21 ` Bedia, Vaibhav
2013-02-20 14:30 ` Kevin Hilman
2013-02-20 14:30 ` Kevin Hilman
2013-04-03 11:52 ` Daniel Mack
2013-04-03 11:52 ` Daniel Mack
2013-04-04 8:47 ` Bedia, Vaibhav
2013-04-04 8:47 ` Bedia, Vaibhav
2012-12-31 13:07 ` [RFC v2 17/18] ARM: OMAP2+: AM33XX: Select Mailbox when PM is enabled Vaibhav Bedia
2012-12-31 13:07 ` Vaibhav Bedia
2013-01-08 15:22 ` Santosh Shilimkar
2013-01-08 15:22 ` Santosh Shilimkar
2013-01-09 5:38 ` Bedia, Vaibhav
2013-01-09 5:38 ` Bedia, Vaibhav
2012-12-31 13:07 ` [RFC v2 18/18] ARM: OMAP2+: AM33XX: Hookup AM33XX PM code into OMAP builds Vaibhav Bedia
2012-12-31 13:07 ` Vaibhav Bedia
2013-01-08 15:31 ` [RFC v2 00/18] ARM: OMAP2+: AM33XX: Add suspend-resume support Santosh Shilimkar
2013-01-08 15:31 ` Santosh Shilimkar
2013-01-09 5:38 ` Bedia, Vaibhav
2013-01-09 5:38 ` Bedia, Vaibhav
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=B5906170F1614E41A8A28DE3B8D121433ECDD089@DBDE01.ent.ti.com \
--to=vaibhav.bedia@ti.com \
--cc=b-cousson@ti.com \
--cc=hvaibhav@ti.com \
--cc=khilman@deeprootsystems.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-omap@vger.kernel.org \
--cc=paul@pwsan.com \
--cc=santosh.shilimkar@ti.com \
--cc=tony@atomide.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.