All of lore.kernel.org
 help / color / mirror / Atom feed
From: Magnus Damm <magnus.damm@gmail.com>
To: Russell King - ARM Linux <linux@arm.linux.org.uk>
Cc: Grant Likely <grant.likely@secretlab.ca>,
	Arnd Bergmann <arnd@arndb.de>, Kevin Hilman <khilman@ti.com>,
	davinci-linux-open-source@linux.davincidsp.com,
	Tony Lindgren <tony@atomide.com>, Sekhar Nori <nsekhar@ti.com>,
	linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	Paul Mundt <lethal@linux-sh.org>
Subject: Re: [RFC PATCH] Consolidate SRAM support
Date: Sat, 16 Apr 2011 13:11:56 +0900	[thread overview]
Message-ID: <BANLkTim_HWU28idaCFKgJLfxMuBwwb+Nig@mail.gmail.com> (raw)
In-Reply-To: <20110415154113.GE4423@n2100.arm.linux.org.uk>

Hi Russell,

[CC Paul Mundt]

On Sat, Apr 16, 2011 at 12:41 AM, Russell King - ARM Linux
<linux@arm.linux.org.uk> wrote:
> On Fri, Apr 15, 2011 at 09:32:01AM -0600, Grant Likely wrote:
>> Yes, once the infrastructure is in place, powerpc can do its own
>> migration to the new code.  I vote for putting it in lib at the
>> outset.
>
> I don't agree with stuffing non-arch directories with code which people
> haven't already agreed should be shared.  As I've already said, in my
> experience it's hard to get agreement in the first place and even when
> you can the API generally needs to be changed from what you first think
> would be reasonable.
>
> So, lets wait to see whether the SH folk reply.

The SH arch is using NUMA for on-chip SRAM on some CPUs, but for
SH-Mobile ARM we have no software support at this point.

The SH/ARM hardware has a bunch of different on-chip memories in
place, and they all have individual power management support through
both clock gating and power domains. I assume other vendors have
similar setups.

I'd like to have some refcounting in place for the SRAM code if
possible, which would trickle down to runtime pm get/put which in turn
would allow us to control the power dynamically. Not sure how easy
that would be to accomplish though.

Paul may have some ideas on how to share the code between ARM and SH.

Thanks,

/ magnus
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

WARNING: multiple messages have this Message-ID (diff)
From: magnus.damm@gmail.com (Magnus Damm)
To: linux-arm-kernel@lists.infradead.org
Subject: [RFC PATCH] Consolidate SRAM support
Date: Sat, 16 Apr 2011 13:11:56 +0900	[thread overview]
Message-ID: <BANLkTim_HWU28idaCFKgJLfxMuBwwb+Nig@mail.gmail.com> (raw)
In-Reply-To: <20110415154113.GE4423@n2100.arm.linux.org.uk>

Hi Russell,

[CC Paul Mundt]

On Sat, Apr 16, 2011 at 12:41 AM, Russell King - ARM Linux
<linux@arm.linux.org.uk> wrote:
> On Fri, Apr 15, 2011 at 09:32:01AM -0600, Grant Likely wrote:
>> Yes, once the infrastructure is in place, powerpc can do its own
>> migration to the new code. ?I vote for putting it in lib at the
>> outset.
>
> I don't agree with stuffing non-arch directories with code which people
> haven't already agreed should be shared. ?As I've already said, in my
> experience it's hard to get agreement in the first place and even when
> you can the API generally needs to be changed from what you first think
> would be reasonable.
>
> So, lets wait to see whether the SH folk reply.

The SH arch is using NUMA for on-chip SRAM on some CPUs, but for
SH-Mobile ARM we have no software support at this point.

The SH/ARM hardware has a bunch of different on-chip memories in
place, and they all have individual power management support through
both clock gating and power domains. I assume other vendors have
similar setups.

I'd like to have some refcounting in place for the SRAM code if
possible, which would trickle down to runtime pm get/put which in turn
would allow us to control the power dynamically. Not sure how easy
that would be to accomplish though.

Paul may have some ideas on how to share the code between ARM and SH.

Thanks,

/ magnus

  reply	other threads:[~2011-04-16  4:11 UTC|newest]

Thread overview: 144+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-04-15 13:06 [RFC PATCH] Consolidate SRAM support Russell King - ARM Linux
2011-04-15 13:06 ` Russell King - ARM Linux
2011-04-15 13:39 ` Rob Herring
2011-04-15 13:39   ` Rob Herring
2011-04-15 14:02   ` Ithamar R. Adema
2011-04-15 14:02     ` Ithamar R. Adema
2011-04-15 14:40   ` Arnd Bergmann
2011-04-15 14:40     ` Arnd Bergmann
2011-04-15 15:26     ` Russell King - ARM Linux
2011-04-15 15:26       ` Russell King - ARM Linux
2011-04-15 15:32       ` Grant Likely
2011-04-15 15:32         ` Grant Likely
2011-04-15 15:41         ` Russell King - ARM Linux
2011-04-15 15:41           ` Russell King - ARM Linux
2011-04-16  4:11           ` Magnus Damm [this message]
2011-04-16  4:11             ` Magnus Damm
2011-04-17 17:47           ` Arnd Bergmann
2011-04-17 17:47             ` Arnd Bergmann
2011-04-15 16:03   ` Russell King - ARM Linux
2011-04-15 16:03     ` Russell King - ARM Linux
2011-04-15 16:18     ` Nguyen Dinh-R00091
2011-04-15 16:18       ` Nguyen Dinh-R00091
2011-04-15 16:20       ` Russell King - ARM Linux
2011-04-15 16:20         ` Russell King - ARM Linux
2011-04-15 16:58         ` Russell King - ARM Linux
2011-04-15 16:58           ` Russell King - ARM Linux
2011-04-15 19:20           ` Nguyen Dinh-R00091
2011-04-15 19:20             ` Nguyen Dinh-R00091
2011-04-15 19:40             ` Russell King - ARM Linux
2011-04-15 19:40               ` Russell King - ARM Linux
2011-04-15 20:06               ` Nguyen Dinh-R00091
2011-04-15 20:06                 ` Nguyen Dinh-R00091
2011-04-15 13:52 ` Eduardo Valentin
2011-04-15 13:52   ` Eduardo Valentin
2011-04-15 15:24   ` Russell King - ARM Linux
2011-04-15 15:24     ` Russell King - ARM Linux
2011-04-15 14:50 ` Detlef Vollmann
2011-04-15 14:50   ` Detlef Vollmann
2011-04-15 15:37   ` Russell King - ARM Linux
2011-04-15 15:37     ` Russell King - ARM Linux
2011-04-15 18:12     ` Detlef Vollmann
2011-04-15 18:12       ` Detlef Vollmann
2011-04-15 18:21       ` Russell King - ARM Linux
2011-04-15 18:21         ` Russell King - ARM Linux
2011-04-15 16:04   ` Nicolas Ferre
2011-04-15 16:04     ` Nicolas Ferre
2011-04-15 18:14     ` Detlef Vollmann
2011-04-15 18:14       ` Detlef Vollmann
2011-04-16 11:27       ` Detlef Vollmann
2011-04-16 11:27         ` Detlef Vollmann
2011-04-15 18:31 ` Jean-Christophe PLAGNIOL-VILLARD
2011-04-15 18:31   ` Jean-Christophe PLAGNIOL-VILLARD
2011-04-15 18:38   ` [PATCH 1/2] at91: 9260 and 9g20 add support of join SRAM Memory Mapping Jean-Christophe PLAGNIOL-VILLARD
2011-04-15 18:39   ` [PATCH 2/2] at91: add generic allocator support for sram Jean-Christophe PLAGNIOL-VILLARD
2011-04-15 20:11 ` [RFC PATCH] Consolidate SRAM support Uwe Kleine-König
2011-04-15 20:11   ` Uwe Kleine-König
2011-04-15 20:19   ` Russell King - ARM Linux
2011-04-15 20:19     ` Russell King - ARM Linux
2011-04-15 20:22     ` Uwe Kleine-König
2011-04-15 20:22       ` Uwe Kleine-König
2011-04-16 13:01 ` Haojian Zhuang
2011-04-16 13:01   ` Haojian Zhuang
2011-04-16 13:09   ` Russell King - ARM Linux
2011-04-16 13:09     ` Russell King - ARM Linux
2011-04-18  6:48     ` Tony Lindgren
2011-04-18  6:48       ` Tony Lindgren
2011-04-18  7:00       ` Tomi Valkeinen
2011-04-18  7:00         ` Tomi Valkeinen
2011-04-18  8:17         ` Tony Lindgren
2011-04-18  8:17           ` Tony Lindgren
2011-04-19 14:16           ` Tomi Valkeinen
2011-04-19 14:16             ` Tomi Valkeinen
2011-04-20  5:27             ` Tony Lindgren
2011-04-20  5:27               ` Tony Lindgren
2011-04-18  8:52 ` [RFC PATCH v2] " Russell King - ARM Linux
2011-04-18  8:52   ` Russell King - ARM Linux
2011-04-18  9:31   ` Haojian Zhuang
2011-04-18  9:31     ` Haojian Zhuang
2011-04-18 11:33   ` Tony Lindgren
2011-04-18 11:33     ` Tony Lindgren
2011-04-18 13:50   ` Detlef Vollmann
2011-04-18 13:50     ` Detlef Vollmann
2011-04-18 16:12   ` Nori, Sekhar
2011-04-18 16:12     ` Nori, Sekhar
2011-04-18 16:18     ` Russell King - ARM Linux
2011-04-18 16:18       ` Russell King - ARM Linux
2011-04-19 16:01   ` Jean-Christophe PLAGNIOL-VILLARD
2011-04-19 16:01     ` Jean-Christophe PLAGNIOL-VILLARD
2011-04-19 16:18     ` Russell King - ARM Linux
2011-04-19 16:18       ` Russell King - ARM Linux
2011-04-19 19:05       ` Jean-Christophe PLAGNIOL-VILLARD
2011-04-19 19:05         ` Jean-Christophe PLAGNIOL-VILLARD
2011-04-19 23:20         ` Russell King - ARM Linux
2011-04-19 23:20           ` Russell King - ARM Linux
2011-04-20  4:06           ` Jean-Christophe PLAGNIOL-VILLARD
2011-04-20  4:06             ` Jean-Christophe PLAGNIOL-VILLARD
2011-04-19  1:23 ` [RFC PATCH] " Linus Walleij
2011-04-19  1:23   ` Linus Walleij
2011-05-12 17:45 ` [RFC PATCH v3] " Russell King - ARM Linux
2011-05-12 17:45   ` Russell King - ARM Linux
2011-05-12 18:35   ` Jean-Christophe PLAGNIOL-VILLARD
2011-05-12 18:35     ` Jean-Christophe PLAGNIOL-VILLARD
2011-05-13  7:30   ` Jean Pihet
2011-05-13  7:30     ` Jean Pihet
2011-05-13  9:11     ` Russell King - ARM Linux
2011-05-13  9:11       ` Russell King - ARM Linux
2011-05-13  9:25       ` Jean Pihet
2011-05-13  9:25         ` Jean Pihet
2011-05-13  9:19   ` Santosh Shilimkar
2011-05-13  9:19     ` Santosh Shilimkar
2011-05-17 13:06   ` Nori, Sekhar
2011-05-17 13:06     ` Nori, Sekhar
2011-05-17 21:41     ` [PATCH 0/9] Ben Gardiner
2011-05-17 21:41       ` [PATCH 1/9] davinci: pm: fix compiler errors and kernel panics from sram consolidation Ben Gardiner
2011-05-17 21:41       ` [PATCH 2/9] davinci: sram: ioremap the davinci_soc_info specified sram regions Ben Gardiner
2011-05-18  4:51         ` Jean-Christophe PLAGNIOL-VILLARD
2011-05-18 12:11           ` Nori, Sekhar
2011-05-18 12:14             ` Russell King - ARM Linux
2011-05-18 15:58               ` Jean-Christophe PLAGNIOL-VILLARD
2011-05-25 23:16                 ` Jean-Christophe PLAGNIOL-VILLARD
2011-05-18 13:18         ` Nori, Sekhar
2011-05-18 13:36           ` Ben Gardiner
2011-05-17 21:41       ` [PATCH 3/9] davinci: da850: remove the SRAM_VIRT iotable entry Ben Gardiner
2011-05-17 21:42       ` [PATCH 4/9] davinci: dm355: " Ben Gardiner
2011-05-17 21:42       ` [PATCH 5/9] davinci: dm365: " Ben Gardiner
2011-05-17 21:42       ` [PATCH 6/9] davinci: dm644x: " Ben Gardiner
2011-05-17 21:42       ` [PATCH 7/9] davinci: dm646x: " Ben Gardiner
2011-05-17 21:42       ` [PATCH 8/9] davinci: remove definition of SRAM_VIRT Ben Gardiner
2011-05-18 10:40         ` Sergei Shtylyov
2011-05-17 21:42       ` [PATCH 9/9] davinci: da850: changed SRAM allocator to shared ram Ben Gardiner
2011-05-18 10:29         ` Sergei Shtylyov
2011-05-18 12:33           ` Ben Gardiner
2011-05-26  1:02   ` [RFC PATCH v4] Consolidate SRAM support Jean-Christophe PLAGNIOL-VILLARD
2011-05-26  1:02     ` Jean-Christophe PLAGNIOL-VILLARD
     [not found]     ` <1306371777-20431-1-git-send-email-plagnioj-sclMFOaUSTBWk0Htik3J/w@public.gmane.org>
2011-05-31 17:09       ` Nori, Sekhar
2011-05-31 17:09         ` Nori, Sekhar
2011-05-31 21:21         ` Russell King - ARM Linux
2011-05-31 21:21           ` Russell King - ARM Linux
2011-06-29 13:12     ` Nori, Sekhar
2011-06-29 13:12       ` Nori, Sekhar
2011-07-08 16:51     ` Nori, Sekhar
2011-07-08 16:51       ` Nori, Sekhar
2011-07-08 16:58       ` Nori, Sekhar
2011-07-08 16:58         ` Nori, Sekhar

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=BANLkTim_HWU28idaCFKgJLfxMuBwwb+Nig@mail.gmail.com \
    --to=magnus.damm@gmail.com \
    --cc=arnd@arndb.de \
    --cc=davinci-linux-open-source@linux.davincidsp.com \
    --cc=grant.likely@secretlab.ca \
    --cc=khilman@ti.com \
    --cc=lethal@linux-sh.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=linux@arm.linux.org.uk \
    --cc=nsekhar@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.