All of lore.kernel.org
 help / color / mirror / Atom feed
From: Vaibhav Hiremath <hvaibhav@ti.com>
To: Vaibhav Bedia <vaibhav.bedia@ti.com>
Cc: linux-arm-kernel@lists.infradead.org, linux-omap@vger.kernel.org,
	khilman@ti.com, paul@pwsan.com, b-cousson@ti.com,
	tony@atomide.com
Subject: Re: [PATCH 06/15] ARM: OMAP2+: hwmod: Enable OCMCRAM registration in AM33XX
Date: Mon, 5 Nov 2012 12:53:59 +0530	[thread overview]
Message-ID: <5097698F.1080609@ti.com> (raw)
In-Reply-To: <1351859566-24818-7-git-send-email-vaibhav.bedia@ti.com>



On 11/2/2012 6:02 PM, Vaibhav Bedia wrote:
> The hwmod data for OCMCRAM in AM33XX was commented out.
> This data is needed by the power management code, hence
> uncomment the same and register the OCP interface for it.
> 
> Signed-off-by: Vaibhav Bedia <vaibhav.bedia@ti.com>
> ---
>  arch/arm/mach-omap2/omap_hwmod_33xx_data.c |   11 ++++++++++-
>  1 files changed, 10 insertions(+), 1 deletions(-)
> 
> diff --git a/arch/arm/mach-omap2/omap_hwmod_33xx_data.c b/arch/arm/mach-omap2/omap_hwmod_33xx_data.c
> index 2e470ce..ec3fbb2 100644
> --- a/arch/arm/mach-omap2/omap_hwmod_33xx_data.c
> +++ b/arch/arm/mach-omap2/omap_hwmod_33xx_data.c
> @@ -415,7 +415,6 @@ static struct omap_hwmod am33xx_adc_tsc_hwmod = {
>   *    - cEFUSE (doesn't fall under any ocp_if)
>   *    - clkdiv32k
>   *    - debugss
> - *    - ocmc ram
>   *    - ocp watch point
>   *    - aes0
>   *    - sha0
> @@ -481,6 +480,7 @@ static struct omap_hwmod am33xx_debugss_hwmod = {
>  		},
>  	},
>  };
> +#endif
>  
>  /* ocmcram */
>  static struct omap_hwmod_class am33xx_ocmcram_hwmod_class = {
> @@ -501,6 +501,7 @@ static struct omap_hwmod am33xx_ocmcram_hwmod = {
>  	},
>  };
>  
> +#if 0

Can you cut-n-paste the ocmcram hwmod entry outside of #if and resubmit
it again?

Thanks,
Vaibhav

>  /* ocpwp */
>  static struct omap_hwmod_class am33xx_ocpwp_hwmod_class = {
>  	.name		= "ocpwp",
> @@ -3331,6 +3332,13 @@ static struct omap_hwmod_ocp_if am33xx_l3_s__usbss = {
>  	.flags		= OCPIF_SWSUP_IDLE,
>  };
>  
> +/* l3 main -> ocmc */
> +static struct omap_hwmod_ocp_if am33xx_l3_main__ocmc = {
> +	.master		= &am33xx_l3_main_hwmod,
> +	.slave		= &am33xx_ocmcram_hwmod,
> +	.user		= OCP_USER_MPU | OCP_USER_SDMA,
> +};
> +
>  static struct omap_hwmod_ocp_if *am33xx_hwmod_ocp_ifs[] __initdata = {
>  	&am33xx_l4_fw__emif_fw,
>  	&am33xx_l3_main__emif,
> @@ -3401,6 +3409,7 @@ static struct omap_hwmod_ocp_if *am33xx_hwmod_ocp_ifs[] __initdata = {
>  	&am33xx_l3_main__tptc0,
>  	&am33xx_l3_main__tptc1,
>  	&am33xx_l3_main__tptc2,
> +	&am33xx_l3_main__ocmc,
>  	&am33xx_l3_s__usbss,
>  	&am33xx_l4_hs__cpgmac0,
>  	&am33xx_cpgmac0__mdio,
> 

WARNING: multiple messages have this Message-ID (diff)
From: hvaibhav@ti.com (Vaibhav Hiremath)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 06/15] ARM: OMAP2+: hwmod: Enable OCMCRAM registration in AM33XX
Date: Mon, 5 Nov 2012 12:53:59 +0530	[thread overview]
Message-ID: <5097698F.1080609@ti.com> (raw)
In-Reply-To: <1351859566-24818-7-git-send-email-vaibhav.bedia@ti.com>



On 11/2/2012 6:02 PM, Vaibhav Bedia wrote:
> The hwmod data for OCMCRAM in AM33XX was commented out.
> This data is needed by the power management code, hence
> uncomment the same and register the OCP interface for it.
> 
> Signed-off-by: Vaibhav Bedia <vaibhav.bedia@ti.com>
> ---
>  arch/arm/mach-omap2/omap_hwmod_33xx_data.c |   11 ++++++++++-
>  1 files changed, 10 insertions(+), 1 deletions(-)
> 
> diff --git a/arch/arm/mach-omap2/omap_hwmod_33xx_data.c b/arch/arm/mach-omap2/omap_hwmod_33xx_data.c
> index 2e470ce..ec3fbb2 100644
> --- a/arch/arm/mach-omap2/omap_hwmod_33xx_data.c
> +++ b/arch/arm/mach-omap2/omap_hwmod_33xx_data.c
> @@ -415,7 +415,6 @@ static struct omap_hwmod am33xx_adc_tsc_hwmod = {
>   *    - cEFUSE (doesn't fall under any ocp_if)
>   *    - clkdiv32k
>   *    - debugss
> - *    - ocmc ram
>   *    - ocp watch point
>   *    - aes0
>   *    - sha0
> @@ -481,6 +480,7 @@ static struct omap_hwmod am33xx_debugss_hwmod = {
>  		},
>  	},
>  };
> +#endif
>  
>  /* ocmcram */
>  static struct omap_hwmod_class am33xx_ocmcram_hwmod_class = {
> @@ -501,6 +501,7 @@ static struct omap_hwmod am33xx_ocmcram_hwmod = {
>  	},
>  };
>  
> +#if 0

Can you cut-n-paste the ocmcram hwmod entry outside of #if and resubmit
it again?

Thanks,
Vaibhav

>  /* ocpwp */
>  static struct omap_hwmod_class am33xx_ocpwp_hwmod_class = {
>  	.name		= "ocpwp",
> @@ -3331,6 +3332,13 @@ static struct omap_hwmod_ocp_if am33xx_l3_s__usbss = {
>  	.flags		= OCPIF_SWSUP_IDLE,
>  };
>  
> +/* l3 main -> ocmc */
> +static struct omap_hwmod_ocp_if am33xx_l3_main__ocmc = {
> +	.master		= &am33xx_l3_main_hwmod,
> +	.slave		= &am33xx_ocmcram_hwmod,
> +	.user		= OCP_USER_MPU | OCP_USER_SDMA,
> +};
> +
>  static struct omap_hwmod_ocp_if *am33xx_hwmod_ocp_ifs[] __initdata = {
>  	&am33xx_l4_fw__emif_fw,
>  	&am33xx_l3_main__emif,
> @@ -3401,6 +3409,7 @@ static struct omap_hwmod_ocp_if *am33xx_hwmod_ocp_ifs[] __initdata = {
>  	&am33xx_l3_main__tptc0,
>  	&am33xx_l3_main__tptc1,
>  	&am33xx_l3_main__tptc2,
> +	&am33xx_l3_main__ocmc,
>  	&am33xx_l3_s__usbss,
>  	&am33xx_l4_hs__cpgmac0,
>  	&am33xx_cpgmac0__mdio,
> 

  parent reply	other threads:[~2012-11-05  7:24 UTC|newest]

Thread overview: 218+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-11-02 12:32 [RFC 00/15] Add basic suspend-resume support for AM33XX Vaibhav Bedia
2012-11-02 12:32 ` Vaibhav Bedia
2012-11-02 12:32 ` [PATCH 01/15] ARM: OMAP2+: mailbox: Add an API for flushing the FIFO Vaibhav Bedia
2012-11-02 12:32   ` Vaibhav Bedia
2012-11-02 19:00   ` Tony Lindgren
2012-11-02 19:00     ` Tony Lindgren
2012-11-03  8:24     ` Bedia, Vaibhav
2012-11-03  8:24       ` Bedia, Vaibhav
2012-11-03 16:03   ` Santosh Shilimkar
2012-11-03 16:03     ` Santosh Shilimkar
2012-11-04 15:26     ` Bedia, Vaibhav
2012-11-04 15:26       ` Bedia, Vaibhav
2012-11-05 14:59       ` Santosh Shilimkar
2012-11-05 14:59         ` Santosh Shilimkar
2012-11-02 12:32 ` [PATCH 02/15] ARM: OMAP2+: mailbox: Add support for AM33XX Vaibhav Bedia
2012-11-02 12:32   ` Vaibhav Bedia
2012-11-03  0:14   ` Russ Dill
2012-11-03  0:14     ` Russ Dill
2012-11-03  8:39     ` Bedia, Vaibhav
2012-11-03  8:39       ` Bedia, Vaibhav
2012-11-03 13:48     ` Bedia, Vaibhav
2012-11-03 13:48       ` Bedia, Vaibhav
2012-11-03 16:10   ` Santosh Shilimkar
2012-11-03 16:10     ` Santosh Shilimkar
2012-11-04 15:26     ` Bedia, Vaibhav
2012-11-04 15:26       ` Bedia, Vaibhav
2012-11-05 15:00       ` Santosh Shilimkar
2012-11-05 15:00         ` Santosh Shilimkar
2012-11-02 12:32 ` [PATCH 03/15] ARM: OMAP: mailbox: Convert to device_initcall Vaibhav Bedia
2012-11-02 12:32   ` Vaibhav Bedia
2012-11-03 16:12   ` Santosh Shilimkar
2012-11-03 16:12     ` Santosh Shilimkar
2012-11-02 12:32 ` [PATCH 04/15] ARM: OMAP2+: hwmod: Update the reset API for AM33XX Vaibhav Bedia
2012-11-02 12:32   ` Vaibhav Bedia
2012-11-05  6:58   ` Vaibhav Hiremath
2012-11-05  6:58     ` Vaibhav Hiremath
2012-11-05 17:57     ` Bedia, Vaibhav
2012-11-05 17:57       ` Bedia, Vaibhav
2012-11-06  6:06       ` Hiremath, Vaibhav
2012-11-06  6:06         ` Hiremath, Vaibhav
2012-11-06  7:19         ` Bedia, Vaibhav
2012-11-06  7:19           ` Bedia, Vaibhav
2012-11-02 12:32 ` [PATCH 05/15] ARM: OMAP2+: AM33XX: Update WKUP_M3 hwmod entry for reset status Vaibhav Bedia
2012-11-02 12:32   ` Vaibhav Bedia
2012-11-03 16:15   ` Santosh Shilimkar
2012-11-03 16:15     ` Santosh Shilimkar
2012-11-04 15:26     ` Bedia, Vaibhav
2012-11-04 15:26       ` Bedia, Vaibhav
2012-11-05  6:59   ` Vaibhav Hiremath
2012-11-05  6:59     ` Vaibhav Hiremath
2012-11-02 12:32 ` [PATCH 06/15] ARM: OMAP2+: hwmod: Enable OCMCRAM registration in AM33XX Vaibhav Bedia
2012-11-02 12:32   ` Vaibhav Bedia
2012-11-03 16:16   ` Santosh Shilimkar
2012-11-03 16:16     ` Santosh Shilimkar
2012-11-04 15:26     ` Bedia, Vaibhav
2012-11-04 15:26       ` Bedia, Vaibhav
2012-11-05  7:23   ` Vaibhav Hiremath [this message]
2012-11-05  7:23     ` Vaibhav Hiremath
2012-11-05 17:57     ` Bedia, Vaibhav
2012-11-05 17:57       ` Bedia, Vaibhav
2012-11-06  6:07       ` Hiremath, Vaibhav
2012-11-06  6:07         ` Hiremath, Vaibhav
2012-11-02 12:32 ` [PATCH 07/15] ARM: OMAP2+: hwmod: Update the hwmod data for TPTCs " Vaibhav Bedia
2012-11-02 12:32   ` Vaibhav Bedia
2012-11-05  7:19   ` Vaibhav Hiremath
2012-11-05  7:19     ` Vaibhav Hiremath
2012-11-02 12:32 ` [PATCH 08/15] ARM: OMAP2+: hwmod: Fix the omap_hwmod_addr_space for CPGMAC0 Vaibhav Bedia
2012-11-02 12:32   ` Vaibhav Bedia
2012-11-03 16:18   ` Santosh Shilimkar
2012-11-03 16:18     ` Santosh Shilimkar
2012-11-04 15:26     ` Bedia, Vaibhav
2012-11-04 15:26       ` Bedia, Vaibhav
2012-11-05  9:10     ` Bedia, Vaibhav
2012-11-05  9:10       ` Bedia, Vaibhav
2012-11-06  9:29       ` Vaibhav Hiremath
2012-11-06  9:29         ` Vaibhav Hiremath
2012-11-06 10:09         ` Bedia, Vaibhav
2012-11-06 10:09           ` Bedia, Vaibhav
2012-11-06 13:08           ` Hiremath, Vaibhav
2012-11-06 13:08             ` Hiremath, Vaibhav
2012-11-06 13:46             ` Bedia, Vaibhav
2012-11-06 13:46               ` Bedia, Vaibhav
2012-11-06 13:50               ` Benoit Cousson
2012-11-06 13:50                 ` Benoit Cousson
2012-11-06 13:56                 ` Bedia, Vaibhav
2012-11-06 13:56                   ` Bedia, Vaibhav
2012-11-02 12:32 ` [PATCH 09/15] ARM: OMAP: AM33XX: Remove unnecessary include and use __ASSEMBLER__ macros Vaibhav Bedia
2012-11-02 12:32   ` Vaibhav Bedia
2012-11-03 16:29   ` Santosh Shilimkar
2012-11-03 16:29     ` Santosh Shilimkar
2012-11-04 15:26     ` Bedia, Vaibhav
2012-11-04 15:26       ` Bedia, Vaibhav
2012-11-02 12:32 ` [PATCH 10/15] ARM: OMAP2+: control: Add some AM33XX Control module registers Vaibhav Bedia
2012-11-02 12:32   ` Vaibhav Bedia
2012-11-02 12:32 ` [PATCH 11/15] ARM: OMAP: timer: Interchange clksrc and clkevt for AM33XX Vaibhav Bedia
2012-11-02 12:32   ` Vaibhav Bedia
2012-11-03 11:43   ` Kevin Hilman
2012-11-03 11:43     ` Kevin Hilman
2012-11-03 12:47     ` Bedia, Vaibhav
2012-11-03 12:47       ` Bedia, Vaibhav
2012-11-03 13:04       ` Kevin Hilman
2012-11-03 13:04         ` Kevin Hilman
2012-11-03 13:48         ` Bedia, Vaibhav
2012-11-03 13:48           ` Bedia, Vaibhav
2012-11-05 18:03           ` Kevin Hilman
2012-11-05 18:03             ` Kevin Hilman
2012-11-05 21:59             ` Santosh Shilimkar
2012-11-05 21:59               ` Santosh Shilimkar
2012-11-06 14:38               ` Bedia, Vaibhav
2012-11-06 14:38                 ` Bedia, Vaibhav
2012-11-08 13:15               ` Bedia, Vaibhav
2012-11-08 13:15                 ` Bedia, Vaibhav
2012-11-06 14:33             ` Bedia, Vaibhav
2012-11-06 14:33               ` Bedia, Vaibhav
2012-11-03 16:22   ` Kevin Hilman
2012-11-03 16:22     ` Kevin Hilman
2012-11-05  4:40     ` Bedia, Vaibhav
2012-11-05  4:40       ` Bedia, Vaibhav
2012-11-03 16:31   ` Santosh Shilimkar
2012-11-03 16:31     ` Santosh Shilimkar
2012-11-04 15:26     ` Bedia, Vaibhav
2012-11-04 15:26       ` Bedia, Vaibhav
2012-11-08 20:41   ` Jon Hunter
2012-11-08 20:41     ` Jon Hunter
2012-11-12 22:54     ` Tony Lindgren
2012-11-12 22:54       ` Tony Lindgren
2012-11-02 12:32 ` [PATCH 12/15] ARM: OMAP: timer: Add suspend-resume callbacks for clockevent device Vaibhav Bedia
2012-11-02 12:32   ` Vaibhav Bedia
2012-11-03 12:15   ` Kevin Hilman
2012-11-03 12:15     ` Kevin Hilman
2012-11-03 13:17     ` Bedia, Vaibhav
2012-11-03 13:17       ` Bedia, Vaibhav
2012-11-03 13:41       ` Kevin Hilman
2012-11-03 13:41         ` Kevin Hilman
2012-11-03 14:03         ` Bedia, Vaibhav
2012-11-03 14:03           ` Bedia, Vaibhav
2012-11-05 21:20       ` Jon Hunter
2012-11-05 21:20         ` Jon Hunter
2012-11-06  9:38         ` Bedia, Vaibhav
2012-11-06  9:38           ` Bedia, Vaibhav
2012-11-06 16:02           ` Jon Hunter
2012-11-06 16:02             ` Jon Hunter
2012-11-03 15:52   ` Santosh Shilimkar
2012-11-03 15:52     ` Santosh Shilimkar
2012-11-04 15:25     ` Bedia, Vaibhav
2012-11-04 15:25       ` Bedia, Vaibhav
2012-11-05 14:55       ` Santosh Shilimkar
2012-11-05 14:55         ` Santosh Shilimkar
2012-11-05 21:04   ` Jon Hunter
2012-11-05 21:04     ` Jon Hunter
2012-11-06  7:32     ` Bedia, Vaibhav
2012-11-06  7:32       ` Bedia, Vaibhav
2012-11-06 16:00       ` Jon Hunter
2012-11-06 16:00         ` Jon Hunter
2012-11-02 12:32 ` [PATCH 13/15] ARM: DTS: AM33XX: Add nodes for OCMCRAM and Mailbox Vaibhav Bedia
2012-11-02 12:32   ` Vaibhav Bedia
2012-11-03 12:16   ` Kevin Hilman
2012-11-03 12:16     ` Kevin Hilman
2012-11-03 13:17     ` Bedia, Vaibhav
2012-11-03 13:17       ` Bedia, Vaibhav
2012-11-03 15:54   ` Santosh Shilimkar
2012-11-03 15:54     ` Santosh Shilimkar
2012-11-04 15:26     ` Bedia, Vaibhav
2012-11-04 15:26       ` Bedia, Vaibhav
2012-11-05 14:53       ` Santosh Shilimkar
2012-11-05 14:53         ` Santosh Shilimkar
2012-11-05 17:57         ` Bedia, Vaibhav
2012-11-05 17:57           ` Bedia, Vaibhav
2012-11-05 19:29           ` Kevin Hilman
2012-11-05 19:29             ` Kevin Hilman
2012-11-05 21:19             ` Santosh Shilimkar
2012-11-05 21:19               ` Santosh Shilimkar
2012-11-05 21:45               ` Santosh Shilimkar
2012-11-05 21:45                 ` Santosh Shilimkar
2012-11-06  5:08                 ` Bedia, Vaibhav
2012-11-06  5:08                   ` Bedia, Vaibhav
2012-11-05 14:58       ` Santosh Shilimkar
2012-11-05 14:58         ` Santosh Shilimkar
2012-11-02 12:32 ` [PATCH 14/15] ARM: OMAP2+: omap2plus_defconfig: Enable Mailbox Vaibhav Bedia
2012-11-02 12:32   ` Vaibhav Bedia
2012-11-03 12:20   ` Kevin Hilman
2012-11-03 12:20     ` Kevin Hilman
2012-11-03 13:17     ` Bedia, Vaibhav
2012-11-03 13:17       ` Bedia, Vaibhav
2012-11-03 13:43       ` Kevin Hilman
2012-11-03 13:43         ` Kevin Hilman
2012-11-02 12:32 ` [PATCH 15/15] ARM: OMAP2+: AM33XX: Basic suspend resume support Vaibhav Bedia
2012-11-02 12:32   ` Vaibhav Bedia
2012-11-02 13:11   ` Bedia, Vaibhav
2012-11-02 13:11     ` Bedia, Vaibhav
2012-11-03 16:57   ` Santosh Shilimkar
2012-11-03 16:57     ` Santosh Shilimkar
2012-11-04 15:26     ` Bedia, Vaibhav
2012-11-04 15:26       ` Bedia, Vaibhav
2012-11-05 17:40   ` Kevin Hilman
2012-11-05 17:40     ` Kevin Hilman
2012-11-05 21:52     ` Santosh Shilimkar
2012-11-05 21:52       ` Santosh Shilimkar
2012-11-06 12:29       ` Bedia, Vaibhav
2012-11-06 12:29         ` Bedia, Vaibhav
2012-11-06 12:38         ` Santosh Shilimkar
2012-11-06 12:38           ` Santosh Shilimkar
2012-11-06 13:00           ` Bedia, Vaibhav
2012-11-06 13:00             ` Bedia, Vaibhav
2012-11-06 10:40     ` Bedia, Vaibhav
2012-11-06 10:40       ` Bedia, Vaibhav
2012-11-07  1:06       ` Kevin Hilman
2012-11-07  1:06         ` Kevin Hilman
2012-11-07 13:25         ` Bedia, Vaibhav
2012-11-07 13:25           ` Bedia, Vaibhav
2012-11-07 17:15           ` Kevin Hilman
2012-11-07 17:15             ` Kevin Hilman
2012-11-08 13:05             ` Bedia, Vaibhav
2012-11-08 13:05               ` Bedia, Vaibhav
2012-11-02 22:16 ` [RFC 00/15] Add basic suspend-resume support for AM33XX Daniel Mack
2012-11-02 22:16   ` Daniel Mack
2012-11-03  8:39   ` Bedia, Vaibhav
2012-11-03  8:39     ` 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=5097698F.1080609@ti.com \
    --to=hvaibhav@ti.com \
    --cc=b-cousson@ti.com \
    --cc=khilman@ti.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=paul@pwsan.com \
    --cc=tony@atomide.com \
    --cc=vaibhav.bedia@ti.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.