* [PATCH v5 1/5] misc: sram: add ability to limit used sram areas
2013-10-22 14:00 [PATCH v5 0/5] ARM: rockchip: add smp functionality Heiko Stübner
@ 2013-10-22 14:01 ` Heiko Stübner
2013-10-22 14:01 ` [PATCH v5 2/5] ARM: rockchip: add snoop-control-unit Heiko Stübner
2013-10-22 14:08 ` [PATCH v5 0/5] ARM: rockchip: add smp functionality Heiko Stübner
2 siblings, 0 replies; 4+ messages in thread
From: Heiko Stübner @ 2013-10-22 14:01 UTC (permalink / raw)
To: linux-arm-kernel
Some SoCs need parts of their sram for special purposes. So while being part
of the peripheral, it should not be part of the genpool controlling the sram.
Therefore add an option available to keep arbitrary portions of the
sram from being part of the pool. If available is not present, all
sram is used.
Suggested-by: Rob Herring <robherring2@gmail.com>
move from marking reserved space to available space
Suggested-by: Matt Sealey <neko@bakuhatsu.net>
Signed-off-by: Heiko Stuebner <heiko@sntech.de>
Tested-by: Ulrich Prinz <ulrich.prinz@googlemail.com>
---
Documentation/devicetree/bindings/misc/sram.txt | 8 ++++
drivers/misc/sram.c | 51 ++++++++++++++++++++---
2 files changed, 53 insertions(+), 6 deletions(-)
diff --git a/Documentation/devicetree/bindings/misc/sram.txt b/Documentation/devicetree/bindings/misc/sram.txt
index 4d0a00e..8be552f 100644
--- a/Documentation/devicetree/bindings/misc/sram.txt
+++ b/Documentation/devicetree/bindings/misc/sram.txt
@@ -8,9 +8,17 @@ Required properties:
- reg : SRAM iomem address range
+Optional properties:
+
+- available: optional list of available chunks inside the sram that the
+ OS can use. Without available, the whole range is usable for the OS.
+ Format is <base size>, <base size>, ...; with base being relative to the
+ reg property base.
+
Example:
sram: sram at 5c000000 {
compatible = "mmio-sram";
reg = <0x5c000000 0x40000>; /* 256 KiB SRAM at address 0x5c000000 */
+ available = <0x100 0x3ff00>;
};
diff --git a/drivers/misc/sram.c b/drivers/misc/sram.c
index afe66571..755e640 100644
--- a/drivers/misc/sram.c
+++ b/drivers/misc/sram.c
@@ -42,6 +42,8 @@ static int sram_probe(struct platform_device *pdev)
struct sram_dev *sram;
struct resource *res;
unsigned long size;
+ const __be32 *avail_list = NULL;
+ int avail_size = 0;
int ret;
res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
@@ -65,12 +67,49 @@ static int sram_probe(struct platform_device *pdev)
if (!sram->pool)
return -ENOMEM;
- ret = gen_pool_add_virt(sram->pool, (unsigned long)virt_base,
- res->start, size, -1);
- if (ret < 0) {
- if (sram->clk)
- clk_disable_unprepare(sram->clk);
- return ret;
+ if (pdev->dev.of_node) {
+ avail_list = of_get_property(pdev->dev.of_node,
+ "available",
+ &avail_size);
+ if (avail_list) {
+ avail_size /= sizeof(*avail_list);
+ if (!avail_size || avail_size % 2) {
+ dev_warn(&pdev->dev, "wrong number of arguments in available property\n");
+ avail_list = NULL;
+ }
+ }
+ }
+
+ if (!avail_list) {
+ ret = gen_pool_add_virt(sram->pool, (unsigned long)virt_base,
+ res->start, size, -1);
+ if (ret < 0) {
+ if (sram->clk)
+ clk_disable_unprepare(sram->clk);
+ return ret;
+ }
+ } else {
+ unsigned int astart;
+ unsigned int asize;
+ int i;
+
+ for (i = 0; i < avail_size; i += 2) {
+ /* get the next available block */
+ astart = be32_to_cpu(*avail_list++);
+ asize = be32_to_cpu(*avail_list++);
+
+ dev_dbg(&pdev->dev, "found available block 0x%x-0x%x\n",
+ astart, astart + asize);
+
+ ret = gen_pool_add_virt(sram->pool,
+ (unsigned long)virt_base + astart,
+ res->start + astart, asize, -1);
+ if (ret < 0) {
+ if (sram->clk)
+ clk_disable_unprepare(sram->clk);
+ return ret;
+ }
+ }
}
platform_set_drvdata(pdev, sram);
--
1.7.10.4
^ permalink raw reply related [flat|nested] 4+ messages in thread* [PATCH v5 0/5] ARM: rockchip: add smp functionality
2013-10-22 14:00 [PATCH v5 0/5] ARM: rockchip: add smp functionality Heiko Stübner
2013-10-22 14:01 ` [PATCH v5 1/5] misc: sram: add ability to limit used sram areas Heiko Stübner
2013-10-22 14:01 ` [PATCH v5 2/5] ARM: rockchip: add snoop-control-unit Heiko Stübner
@ 2013-10-22 14:08 ` Heiko Stübner
2 siblings, 0 replies; 4+ messages in thread
From: Heiko Stübner @ 2013-10-22 14:08 UTC (permalink / raw)
To: linux-arm-kernel
gaah, there I just learned about the arm at kernel.org address and still didn't
remember when digging up this old recipient list - which of course resulted in
Kevin not being included.
Additionally I also had the old devicetree list included.
So I'll do it again shortly :-)
Sorry for the noise
Heiko
Am Dienstag, 22. Oktober 2013, 16:00:13 schrieb Heiko St?bner:
> This series enables the use of the additional cores on Rockchip
> Cortex-A9 SoCs.
>
> To achieve this, add the scu, the needed sram and power-management-unit.
>
> Tested on both a BQ Curie2 (rk3066a / dual core) and
> on a Radxa Rock (rk3188 / quad core).
>
> changes since v4:
> - rebase on top of the recent rk3188 board support
> - implement suggestion from Matt Sealey in moving the sram-limit from
> marking reserved regions to marking available regions - hopefully
> I got the usage right :-)
> - remove __CPUINIT as suggested by Fabio Estevam
>
> changes since v3:
> - address comments from Rob Herring:
> - split the gathering of the reserve-data into a separate loop
> - spelling and style fixes
> - first patch only included for reference, already part of the
> char-misc git tree
>
> changes since v2:
> - rework the sram allocation following the suggestion from Philipp Zabel
>
> changes since v1:
> - add reserved block feature for mmio-sram, to not use two logical
> sram nodes
> - the sram content is kept intact while the device is running, so
> copying the trampoline is only needed once
>
> Heiko Stuebner (5):
> misc: sram: add ability to limit used sram areas
> ARM: rockchip: add snoop-control-unit
> ARM: rockchip: add sram dt nodes and documentation
> ARM: rockchip: add power-management-unit dt node
> ARM: rockchip: add smp bringup code
>
> .../devicetree/bindings/arm/rockchip/pmu.txt | 16 ++
> .../devicetree/bindings/arm/rockchip/smp-sram.txt | 24 +++
> Documentation/devicetree/bindings/misc/sram.txt | 8 +
> arch/arm/boot/dts/rk3066a.dtsi | 6 +
> arch/arm/boot/dts/rk3188.dtsi | 6 +
> arch/arm/boot/dts/rk3xxx.dtsi | 10 +
> arch/arm/mach-rockchip/Kconfig | 1 +
> arch/arm/mach-rockchip/Makefile | 1 +
> arch/arm/mach-rockchip/core.h | 22 +++
> arch/arm/mach-rockchip/headsmp.S | 30 ++++
> arch/arm/mach-rockchip/platsmp.c | 195
> ++++++++++++++++++++ arch/arm/mach-rockchip/rockchip.c |
> 2 +
> drivers/misc/sram.c | 51 ++++-
> 13 files changed, 368 insertions(+), 6 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/arm/rockchip/pmu.txt
> create mode 100644
> Documentation/devicetree/bindings/arm/rockchip/smp-sram.txt create mode
> 100644 arch/arm/mach-rockchip/core.h
> create mode 100644 arch/arm/mach-rockchip/headsmp.S
> create mode 100644 arch/arm/mach-rockchip/platsmp.c
^ permalink raw reply [flat|nested] 4+ messages in thread