All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v4 0/3] ARM: l2c: add l2c support for RZ/A1
@ 2017-02-16 16:17 ` Chris Brandt
  0 siblings, 0 replies; 27+ messages in thread
From: Chris Brandt @ 2017-02-16 16:17 UTC (permalink / raw)
  To: Simon Horman, Magnus Damm, Geert Uytterhoeven, Rob Herring,
	Mark Rutland, Russell King, Brad Mouring, Andrey Smirnov,
	Arnd Bergmann, Richard Cochran
  Cc: devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-renesas-soc-u79uwXL29TY76Z2rM5mHXA,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Chris Brandt

The PL310 in the Renesas RZ/A1 SoC (R7S72100) does not have the sideband
signals connected between the CPU and L2C. According the PL310 TRM,
sideband signals are optional.

If a PL310 is added to a system, but the sideband signals are not
connected, some Cortex A9 optimizations cannot be used. In particular,
enabling Full Line Zeros in the CA9 without sidebands connected will
crash the system since the CA9 will expect the L2C to perform operations,
yet the L2C never gets the commands.

This series adds the option to not enable anything in the PL310 that
uses sidebands, and then adds L2C support to the RZ/A1 DT.

v4:
* changed l2x0_bresp_dis to l2x0_bresp_disable
* changed l2x0_flz_dis to l2x0_flz_disable

v3:
* split "arm,pl310-no-sideband" into "arm,early-bresp-disable" and
  "arm,full-line-zero-disable"

v2:
* Added "arm,pl310-no-sideband" to cache-l2x0.c instead of hacking in a
  dummy l2c_write_sec function to keep FLZ from being enabled.


Chris Brandt (3):
  ARM: l2c: allow CA9 optimizations to be disabled
  ARM: shmobile: r7s72100: Enable L2 cache
  ARM: dts: r7s72100: add l2 cache

 Documentation/devicetree/bindings/arm/l2c2x0.txt |  3 +++
 arch/arm/boot/dts/r7s72100.dtsi                  | 11 +++++++++++
 arch/arm/mach-shmobile/setup-r7s72100.c          |  2 ++
 arch/arm/mm/cache-l2x0.c                         | 13 +++++++++++--
 4 files changed, 27 insertions(+), 2 deletions(-)

-- 
2.10.1


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

^ permalink raw reply	[flat|nested] 27+ messages in thread

* [PATCH v4 0/3] ARM: l2c: add l2c support for RZ/A1
@ 2017-02-16 16:17 ` Chris Brandt
  0 siblings, 0 replies; 27+ messages in thread
From: Chris Brandt @ 2017-02-16 16:17 UTC (permalink / raw)
  To: Simon Horman, Magnus Damm, Geert Uytterhoeven, Rob Herring,
	Mark Rutland, Russell King, Brad Mouring, Andrey Smirnov,
	Arnd Bergmann, Richard Cochran
  Cc: devicetree, linux-renesas-soc, linux-arm-kernel, Chris Brandt

The PL310 in the Renesas RZ/A1 SoC (R7S72100) does not have the sideband
signals connected between the CPU and L2C. According the PL310 TRM,
sideband signals are optional.

If a PL310 is added to a system, but the sideband signals are not
connected, some Cortex A9 optimizations cannot be used. In particular,
enabling Full Line Zeros in the CA9 without sidebands connected will
crash the system since the CA9 will expect the L2C to perform operations,
yet the L2C never gets the commands.

This series adds the option to not enable anything in the PL310 that
uses sidebands, and then adds L2C support to the RZ/A1 DT.

v4:
* changed l2x0_bresp_dis to l2x0_bresp_disable
* changed l2x0_flz_dis to l2x0_flz_disable

v3:
* split "arm,pl310-no-sideband" into "arm,early-bresp-disable" and
  "arm,full-line-zero-disable"

v2:
* Added "arm,pl310-no-sideband" to cache-l2x0.c instead of hacking in a
  dummy l2c_write_sec function to keep FLZ from being enabled.


Chris Brandt (3):
  ARM: l2c: allow CA9 optimizations to be disabled
  ARM: shmobile: r7s72100: Enable L2 cache
  ARM: dts: r7s72100: add l2 cache

 Documentation/devicetree/bindings/arm/l2c2x0.txt |  3 +++
 arch/arm/boot/dts/r7s72100.dtsi                  | 11 +++++++++++
 arch/arm/mach-shmobile/setup-r7s72100.c          |  2 ++
 arch/arm/mm/cache-l2x0.c                         | 13 +++++++++++--
 4 files changed, 27 insertions(+), 2 deletions(-)

-- 
2.10.1

^ permalink raw reply	[flat|nested] 27+ messages in thread

* [PATCH v4 0/3] ARM: l2c: add l2c support for RZ/A1
@ 2017-02-16 16:17 ` Chris Brandt
  0 siblings, 0 replies; 27+ messages in thread
From: Chris Brandt @ 2017-02-16 16:17 UTC (permalink / raw)
  To: linux-arm-kernel

The PL310 in the Renesas RZ/A1 SoC (R7S72100) does not have the sideband
signals connected between the CPU and L2C. According the PL310 TRM,
sideband signals are optional.

If a PL310 is added to a system, but the sideband signals are not
connected, some Cortex A9 optimizations cannot be used. In particular,
enabling Full Line Zeros in the CA9 without sidebands connected will
crash the system since the CA9 will expect the L2C to perform operations,
yet the L2C never gets the commands.

This series adds the option to not enable anything in the PL310 that
uses sidebands, and then adds L2C support to the RZ/A1 DT.

v4:
* changed l2x0_bresp_dis to l2x0_bresp_disable
* changed l2x0_flz_dis to l2x0_flz_disable

v3:
* split "arm,pl310-no-sideband" into "arm,early-bresp-disable" and
  "arm,full-line-zero-disable"

v2:
* Added "arm,pl310-no-sideband" to cache-l2x0.c instead of hacking in a
  dummy l2c_write_sec function to keep FLZ from being enabled.


Chris Brandt (3):
  ARM: l2c: allow CA9 optimizations to be disabled
  ARM: shmobile: r7s72100: Enable L2 cache
  ARM: dts: r7s72100: add l2 cache

 Documentation/devicetree/bindings/arm/l2c2x0.txt |  3 +++
 arch/arm/boot/dts/r7s72100.dtsi                  | 11 +++++++++++
 arch/arm/mach-shmobile/setup-r7s72100.c          |  2 ++
 arch/arm/mm/cache-l2x0.c                         | 13 +++++++++++--
 4 files changed, 27 insertions(+), 2 deletions(-)

-- 
2.10.1

^ permalink raw reply	[flat|nested] 27+ messages in thread

* [PATCH v4 1/3] ARM: l2c: allow CA9 optimizations to be disabled
  2017-02-16 16:17 ` Chris Brandt
@ 2017-02-16 16:17   ` Chris Brandt
  -1 siblings, 0 replies; 27+ messages in thread
From: Chris Brandt @ 2017-02-16 16:17 UTC (permalink / raw)
  To: Simon Horman, Magnus Damm, Geert Uytterhoeven, Rob Herring,
	Mark Rutland, Russell King, Brad Mouring, Andrey Smirnov,
	Arnd Bergmann, Richard Cochran
  Cc: devicetree, linux-renesas-soc, linux-arm-kernel, Chris Brandt

If a PL310 is added to a system, but the sideband signals are not
connected, some Cortex A9 optimizations cannot be used. In particular,
enabling Full Line of Zeros in the CA9 without sidebands connected will
crash the system since the CA9 will expect the L2C to perform operations,
yet the L2C never gets the commands. Early BRESP also does not work
without sideband signals.

Signed-off-by: Chris Brandt <chris.brandt@renesas.com>
---
v3:
* changed l2x0_bresp_dis to l2x0_bresp_disable
* changed l2x0_flz_dis to l2x0_flz_disable
v2:
* split "arm,pl310-no-sideband" into "arm,early-bresp-disable" and
  "arm,full-line-zero-disable"
---
 Documentation/devicetree/bindings/arm/l2c2x0.txt |  3 +++
 arch/arm/mm/cache-l2x0.c                         | 13 +++++++++++--
 2 files changed, 14 insertions(+), 2 deletions(-)

diff --git a/Documentation/devicetree/bindings/arm/l2c2x0.txt b/Documentation/devicetree/bindings/arm/l2c2x0.txt
index 917199f..d9650c1 100644
--- a/Documentation/devicetree/bindings/arm/l2c2x0.txt
+++ b/Documentation/devicetree/bindings/arm/l2c2x0.txt
@@ -90,6 +90,9 @@ Optional properties:
 - arm,standby-mode: L2 standby mode enable. Value <0> (forcibly disable),
   <1> (forcibly enable), property absent (OS specific behavior,
   preferably retain firmware settings)
+- arm,early-bresp-disable : Disable the CA9 optimization Early BRESP (PL310)
+- arm,full-line-zero-disable : Disable the CA9 optimization Full line of zero
+  write (PL310)
 
 Example:
 
diff --git a/arch/arm/mm/cache-l2x0.c b/arch/arm/mm/cache-l2x0.c
index 2290be3..808efbb 100644
--- a/arch/arm/mm/cache-l2x0.c
+++ b/arch/arm/mm/cache-l2x0.c
@@ -57,6 +57,9 @@ static unsigned long sync_reg_offset = L2X0_CACHE_SYNC;
 
 struct l2x0_regs l2x0_saved_regs;
 
+static bool l2x0_bresp_disable;
+static bool l2x0_flz_disable;
+
 /*
  * Common code for all cache controllers.
  */
@@ -620,7 +623,7 @@ static void __init l2c310_enable(void __iomem *base, unsigned num_lock)
 	u32 aux = l2x0_saved_regs.aux_ctrl;
 
 	if (rev >= L310_CACHE_ID_RTL_R2P0) {
-		if (cortex_a9) {
+		if (cortex_a9 && !l2x0_bresp_disable) {
 			aux |= L310_AUX_CTRL_EARLY_BRESP;
 			pr_info("L2C-310 enabling early BRESP for Cortex-A9\n");
 		} else if (aux & L310_AUX_CTRL_EARLY_BRESP) {
@@ -629,7 +632,7 @@ static void __init l2c310_enable(void __iomem *base, unsigned num_lock)
 		}
 	}
 
-	if (cortex_a9) {
+	if (cortex_a9 && !l2x0_flz_disable) {
 		u32 aux_cur = readl_relaxed(base + L2X0_AUX_CTRL);
 		u32 acr = get_auxcr();
 
@@ -1200,6 +1203,12 @@ static void __init l2c310_of_parse(const struct device_node *np,
 		*aux_mask &= ~L2C_AUX_CTRL_PARITY_ENABLE;
 	}
 
+	if (of_property_read_bool(np, "arm,early-bresp-disable"))
+		l2x0_bresp_disable = true;
+
+	if (of_property_read_bool(np, "arm,full-line-zero-disable"))
+		l2x0_flz_disable = true;
+
 	prefetch = l2x0_saved_regs.prefetch_ctrl;
 
 	ret = of_property_read_u32(np, "arm,double-linefill", &val);
-- 
2.10.1

^ permalink raw reply related	[flat|nested] 27+ messages in thread

* [PATCH v4 1/3] ARM: l2c: allow CA9 optimizations to be disabled
@ 2017-02-16 16:17   ` Chris Brandt
  0 siblings, 0 replies; 27+ messages in thread
From: Chris Brandt @ 2017-02-16 16:17 UTC (permalink / raw)
  To: linux-arm-kernel

If a PL310 is added to a system, but the sideband signals are not
connected, some Cortex A9 optimizations cannot be used. In particular,
enabling Full Line of Zeros in the CA9 without sidebands connected will
crash the system since the CA9 will expect the L2C to perform operations,
yet the L2C never gets the commands. Early BRESP also does not work
without sideband signals.

Signed-off-by: Chris Brandt <chris.brandt@renesas.com>
---
v3:
* changed l2x0_bresp_dis to l2x0_bresp_disable
* changed l2x0_flz_dis to l2x0_flz_disable
v2:
* split "arm,pl310-no-sideband" into "arm,early-bresp-disable" and
  "arm,full-line-zero-disable"
---
 Documentation/devicetree/bindings/arm/l2c2x0.txt |  3 +++
 arch/arm/mm/cache-l2x0.c                         | 13 +++++++++++--
 2 files changed, 14 insertions(+), 2 deletions(-)

diff --git a/Documentation/devicetree/bindings/arm/l2c2x0.txt b/Documentation/devicetree/bindings/arm/l2c2x0.txt
index 917199f..d9650c1 100644
--- a/Documentation/devicetree/bindings/arm/l2c2x0.txt
+++ b/Documentation/devicetree/bindings/arm/l2c2x0.txt
@@ -90,6 +90,9 @@ Optional properties:
 - arm,standby-mode: L2 standby mode enable. Value <0> (forcibly disable),
   <1> (forcibly enable), property absent (OS specific behavior,
   preferably retain firmware settings)
+- arm,early-bresp-disable : Disable the CA9 optimization Early BRESP (PL310)
+- arm,full-line-zero-disable : Disable the CA9 optimization Full line of zero
+  write (PL310)
 
 Example:
 
diff --git a/arch/arm/mm/cache-l2x0.c b/arch/arm/mm/cache-l2x0.c
index 2290be3..808efbb 100644
--- a/arch/arm/mm/cache-l2x0.c
+++ b/arch/arm/mm/cache-l2x0.c
@@ -57,6 +57,9 @@ static unsigned long sync_reg_offset = L2X0_CACHE_SYNC;
 
 struct l2x0_regs l2x0_saved_regs;
 
+static bool l2x0_bresp_disable;
+static bool l2x0_flz_disable;
+
 /*
  * Common code for all cache controllers.
  */
@@ -620,7 +623,7 @@ static void __init l2c310_enable(void __iomem *base, unsigned num_lock)
 	u32 aux = l2x0_saved_regs.aux_ctrl;
 
 	if (rev >= L310_CACHE_ID_RTL_R2P0) {
-		if (cortex_a9) {
+		if (cortex_a9 && !l2x0_bresp_disable) {
 			aux |= L310_AUX_CTRL_EARLY_BRESP;
 			pr_info("L2C-310 enabling early BRESP for Cortex-A9\n");
 		} else if (aux & L310_AUX_CTRL_EARLY_BRESP) {
@@ -629,7 +632,7 @@ static void __init l2c310_enable(void __iomem *base, unsigned num_lock)
 		}
 	}
 
-	if (cortex_a9) {
+	if (cortex_a9 && !l2x0_flz_disable) {
 		u32 aux_cur = readl_relaxed(base + L2X0_AUX_CTRL);
 		u32 acr = get_auxcr();
 
@@ -1200,6 +1203,12 @@ static void __init l2c310_of_parse(const struct device_node *np,
 		*aux_mask &= ~L2C_AUX_CTRL_PARITY_ENABLE;
 	}
 
+	if (of_property_read_bool(np, "arm,early-bresp-disable"))
+		l2x0_bresp_disable = true;
+
+	if (of_property_read_bool(np, "arm,full-line-zero-disable"))
+		l2x0_flz_disable = true;
+
 	prefetch = l2x0_saved_regs.prefetch_ctrl;
 
 	ret = of_property_read_u32(np, "arm,double-linefill", &val);
-- 
2.10.1

^ permalink raw reply related	[flat|nested] 27+ messages in thread

* [PATCH v4 2/3] ARM: shmobile: r7s72100: Enable L2 cache
  2017-02-16 16:17 ` Chris Brandt
@ 2017-02-16 16:17   ` Chris Brandt
  -1 siblings, 0 replies; 27+ messages in thread
From: Chris Brandt @ 2017-02-16 16:17 UTC (permalink / raw)
  To: Simon Horman, Magnus Damm, Geert Uytterhoeven, Rob Herring,
	Mark Rutland, Russell King, Brad Mouring, Andrey Smirnov,
	Arnd Bergmann, Richard Cochran
  Cc: devicetree, linux-renesas-soc, linux-arm-kernel, Chris Brandt

Even though L2C is specified in the DT, you still need to add the aux
settings in the machine_desc.

Signed-off-by: Chris Brandt <chris.brandt@renesas.com>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
v2:
* removed dummy l2c_write_sec function
---
 arch/arm/mach-shmobile/setup-r7s72100.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/arch/arm/mach-shmobile/setup-r7s72100.c b/arch/arm/mach-shmobile/setup-r7s72100.c
index d46639f..319ca95 100644
--- a/arch/arm/mach-shmobile/setup-r7s72100.c
+++ b/arch/arm/mach-shmobile/setup-r7s72100.c
@@ -26,6 +26,8 @@ static const char *const r7s72100_boards_compat_dt[] __initconst = {
 };
 
 DT_MACHINE_START(R7S72100_DT, "Generic R7S72100 (Flattened Device Tree)")
+	.l2c_aux_val    = 0,
+	.l2c_aux_mask   = ~0,
 	.init_early	= shmobile_init_delay,
 	.init_late	= shmobile_init_late,
 	.dt_compat	= r7s72100_boards_compat_dt,
-- 
2.10.1

^ permalink raw reply related	[flat|nested] 27+ messages in thread

* [PATCH v4 2/3] ARM: shmobile: r7s72100: Enable L2 cache
@ 2017-02-16 16:17   ` Chris Brandt
  0 siblings, 0 replies; 27+ messages in thread
From: Chris Brandt @ 2017-02-16 16:17 UTC (permalink / raw)
  To: linux-arm-kernel

Even though L2C is specified in the DT, you still need to add the aux
settings in the machine_desc.

Signed-off-by: Chris Brandt <chris.brandt@renesas.com>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
v2:
* removed dummy l2c_write_sec function
---
 arch/arm/mach-shmobile/setup-r7s72100.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/arch/arm/mach-shmobile/setup-r7s72100.c b/arch/arm/mach-shmobile/setup-r7s72100.c
index d46639f..319ca95 100644
--- a/arch/arm/mach-shmobile/setup-r7s72100.c
+++ b/arch/arm/mach-shmobile/setup-r7s72100.c
@@ -26,6 +26,8 @@ static const char *const r7s72100_boards_compat_dt[] __initconst = {
 };
 
 DT_MACHINE_START(R7S72100_DT, "Generic R7S72100 (Flattened Device Tree)")
+	.l2c_aux_val    = 0,
+	.l2c_aux_mask   = ~0,
 	.init_early	= shmobile_init_delay,
 	.init_late	= shmobile_init_late,
 	.dt_compat	= r7s72100_boards_compat_dt,
-- 
2.10.1

^ permalink raw reply related	[flat|nested] 27+ messages in thread

* [PATCH v4 3/3] ARM: dts: r7s72100: add l2 cache
  2017-02-16 16:17 ` Chris Brandt
@ 2017-02-16 16:17   ` Chris Brandt
  -1 siblings, 0 replies; 27+ messages in thread
From: Chris Brandt @ 2017-02-16 16:17 UTC (permalink / raw)
  To: Simon Horman, Magnus Damm, Geert Uytterhoeven, Rob Herring,
	Mark Rutland, Russell King, Brad Mouring, Andrey Smirnov,
	Arnd Bergmann, Richard Cochran
  Cc: devicetree, linux-renesas-soc, linux-arm-kernel, Chris Brandt

Note that early-bresp-disable and full-line-zero-disable are required
because the sideband signals between the CPU and L2C were not connected
in this SoC.

Signed-off-by: Chris Brandt <chris.brandt@renesas.com>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
v3:
* split "arm,pl310-no-sideband" into "arm,early-bresp-disable" and
  "arm,full-line-zero-disable"
v2:
* added "arm,pl310-no-sideband"
---
 arch/arm/boot/dts/r7s72100.dtsi | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/arch/arm/boot/dts/r7s72100.dtsi b/arch/arm/boot/dts/r7s72100.dtsi
index 614ba79..ed62e19 100644
--- a/arch/arm/boot/dts/r7s72100.dtsi
+++ b/arch/arm/boot/dts/r7s72100.dtsi
@@ -180,6 +180,7 @@
 			compatible = "arm,cortex-a9";
 			reg = <0>;
 			clock-frequency = <400000000>;
+			next-level-cache = <&L2>;
 		};
 	};
 
@@ -371,6 +372,16 @@
 			<0xe8202000 0x1000>;
 	};
 
+	L2: cache-controller@3ffff000 {
+		compatible = "arm,pl310-cache";
+		reg = <0x3ffff000 0x1000>;
+		interrupts = <GIC_SPI 8 IRQ_TYPE_LEVEL_HIGH>;
+		arm,early-bresp-disable;
+		arm,full-line-zero-disable;
+		cache-unified;
+		cache-level = <2>;
+	};
+
 	i2c0: i2c@fcfee000 {
 		#address-cells = <1>;
 		#size-cells = <0>;
-- 
2.10.1

^ permalink raw reply related	[flat|nested] 27+ messages in thread

* [PATCH v4 3/3] ARM: dts: r7s72100: add l2 cache
@ 2017-02-16 16:17   ` Chris Brandt
  0 siblings, 0 replies; 27+ messages in thread
From: Chris Brandt @ 2017-02-16 16:17 UTC (permalink / raw)
  To: linux-arm-kernel

Note that early-bresp-disable and full-line-zero-disable are required
because the sideband signals between the CPU and L2C were not connected
in this SoC.

Signed-off-by: Chris Brandt <chris.brandt@renesas.com>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
v3:
* split "arm,pl310-no-sideband" into "arm,early-bresp-disable" and
  "arm,full-line-zero-disable"
v2:
* added "arm,pl310-no-sideband"
---
 arch/arm/boot/dts/r7s72100.dtsi | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/arch/arm/boot/dts/r7s72100.dtsi b/arch/arm/boot/dts/r7s72100.dtsi
index 614ba79..ed62e19 100644
--- a/arch/arm/boot/dts/r7s72100.dtsi
+++ b/arch/arm/boot/dts/r7s72100.dtsi
@@ -180,6 +180,7 @@
 			compatible = "arm,cortex-a9";
 			reg = <0>;
 			clock-frequency = <400000000>;
+			next-level-cache = <&L2>;
 		};
 	};
 
@@ -371,6 +372,16 @@
 			<0xe8202000 0x1000>;
 	};
 
+	L2: cache-controller at 3ffff000 {
+		compatible = "arm,pl310-cache";
+		reg = <0x3ffff000 0x1000>;
+		interrupts = <GIC_SPI 8 IRQ_TYPE_LEVEL_HIGH>;
+		arm,early-bresp-disable;
+		arm,full-line-zero-disable;
+		cache-unified;
+		cache-level = <2>;
+	};
+
 	i2c0: i2c at fcfee000 {
 		#address-cells = <1>;
 		#size-cells = <0>;
-- 
2.10.1

^ permalink raw reply related	[flat|nested] 27+ messages in thread

* Re: [PATCH v4 0/3] ARM: l2c: add l2c support for RZ/A1
  2017-02-16 16:17 ` Chris Brandt
  (?)
@ 2017-02-16 16:44     ` Russell King - ARM Linux
  -1 siblings, 0 replies; 27+ messages in thread
From: Russell King - ARM Linux @ 2017-02-16 16:44 UTC (permalink / raw)
  To: Chris Brandt
  Cc: Simon Horman, Magnus Damm, Geert Uytterhoeven, Rob Herring,
	Mark Rutland, Brad Mouring, Andrey Smirnov, Arnd Bergmann,
	Richard Cochran, devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-renesas-soc-u79uwXL29TY76Z2rM5mHXA,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r

On Thu, Feb 16, 2017 at 11:17:39AM -0500, Chris Brandt wrote:
> The PL310 in the Renesas RZ/A1 SoC (R7S72100) does not have the sideband
> signals connected between the CPU and L2C. According the PL310 TRM,
> sideband signals are optional.
> 
> If a PL310 is added to a system, but the sideband signals are not
> connected, some Cortex A9 optimizations cannot be used. In particular,
> enabling Full Line Zeros in the CA9 without sidebands connected will
> crash the system since the CA9 will expect the L2C to perform operations,
> yet the L2C never gets the commands.
> 
> This series adds the option to not enable anything in the PL310 that
> uses sidebands, and then adds L2C support to the RZ/A1 DT.
> 
> v4:
> * changed l2x0_bresp_dis to l2x0_bresp_disable
> * changed l2x0_flz_dis to l2x0_flz_disable

Looks good, thanks.

I'll want to merge patch 1, but I suspect the other two patches need
to go through arm-soc, which will cause a problem due to being merged
independently...  Arnd?

-- 
RMK's Patch system: http://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up
according to speedtest.net.
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [PATCH v4 0/3] ARM: l2c: add l2c support for RZ/A1
@ 2017-02-16 16:44     ` Russell King - ARM Linux
  0 siblings, 0 replies; 27+ messages in thread
From: Russell King - ARM Linux @ 2017-02-16 16:44 UTC (permalink / raw)
  To: Chris Brandt
  Cc: Simon Horman, Magnus Damm, Geert Uytterhoeven, Rob Herring,
	Mark Rutland, Brad Mouring, Andrey Smirnov, Arnd Bergmann,
	Richard Cochran, devicetree, linux-renesas-soc, linux-arm-kernel

On Thu, Feb 16, 2017 at 11:17:39AM -0500, Chris Brandt wrote:
> The PL310 in the Renesas RZ/A1 SoC (R7S72100) does not have the sideband
> signals connected between the CPU and L2C. According the PL310 TRM,
> sideband signals are optional.
> 
> If a PL310 is added to a system, but the sideband signals are not
> connected, some Cortex A9 optimizations cannot be used. In particular,
> enabling Full Line Zeros in the CA9 without sidebands connected will
> crash the system since the CA9 will expect the L2C to perform operations,
> yet the L2C never gets the commands.
> 
> This series adds the option to not enable anything in the PL310 that
> uses sidebands, and then adds L2C support to the RZ/A1 DT.
> 
> v4:
> * changed l2x0_bresp_dis to l2x0_bresp_disable
> * changed l2x0_flz_dis to l2x0_flz_disable

Looks good, thanks.

I'll want to merge patch 1, but I suspect the other two patches need
to go through arm-soc, which will cause a problem due to being merged
independently...  Arnd?

-- 
RMK's Patch system: http://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up
according to speedtest.net.

^ permalink raw reply	[flat|nested] 27+ messages in thread

* [PATCH v4 0/3] ARM: l2c: add l2c support for RZ/A1
@ 2017-02-16 16:44     ` Russell King - ARM Linux
  0 siblings, 0 replies; 27+ messages in thread
From: Russell King - ARM Linux @ 2017-02-16 16:44 UTC (permalink / raw)
  To: linux-arm-kernel

On Thu, Feb 16, 2017 at 11:17:39AM -0500, Chris Brandt wrote:
> The PL310 in the Renesas RZ/A1 SoC (R7S72100) does not have the sideband
> signals connected between the CPU and L2C. According the PL310 TRM,
> sideband signals are optional.
> 
> If a PL310 is added to a system, but the sideband signals are not
> connected, some Cortex A9 optimizations cannot be used. In particular,
> enabling Full Line Zeros in the CA9 without sidebands connected will
> crash the system since the CA9 will expect the L2C to perform operations,
> yet the L2C never gets the commands.
> 
> This series adds the option to not enable anything in the PL310 that
> uses sidebands, and then adds L2C support to the RZ/A1 DT.
> 
> v4:
> * changed l2x0_bresp_dis to l2x0_bresp_disable
> * changed l2x0_flz_dis to l2x0_flz_disable

Looks good, thanks.

I'll want to merge patch 1, but I suspect the other two patches need
to go through arm-soc, which will cause a problem due to being merged
independently...  Arnd?

-- 
RMK's Patch system: http://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up
according to speedtest.net.

^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [PATCH v4 0/3] ARM: l2c: add l2c support for RZ/A1
  2017-02-16 16:44     ` Russell King - ARM Linux
@ 2017-02-16 16:57       ` Arnd Bergmann
  -1 siblings, 0 replies; 27+ messages in thread
From: Arnd Bergmann @ 2017-02-16 16:57 UTC (permalink / raw)
  To: Russell King - ARM Linux
  Cc: Chris Brandt, Simon Horman, Magnus Damm, Geert Uytterhoeven,
	Rob Herring, Mark Rutland, Brad Mouring, Andrey Smirnov,
	Richard Cochran, devicetree, linux-renesas-soc, Linux ARM

On Thu, Feb 16, 2017 at 5:44 PM, Russell King - ARM Linux
<linux@armlinux.org.uk> wrote:
> On Thu, Feb 16, 2017 at 11:17:39AM -0500, Chris Brandt wrote:
>> The PL310 in the Renesas RZ/A1 SoC (R7S72100) does not have the sideband
>> signals connected between the CPU and L2C. According the PL310 TRM,
>> sideband signals are optional.
>>
>> If a PL310 is added to a system, but the sideband signals are not
>> connected, some Cortex A9 optimizations cannot be used. In particular,
>> enabling Full Line Zeros in the CA9 without sidebands connected will
>> crash the system since the CA9 will expect the L2C to perform operations,
>> yet the L2C never gets the commands.
>>
>> This series adds the option to not enable anything in the PL310 that
>> uses sidebands, and then adds L2C support to the RZ/A1 DT.
>>
>> v4:
>> * changed l2x0_bresp_dis to l2x0_bresp_disable
>> * changed l2x0_flz_dis to l2x0_flz_disable
>
> Looks good, thanks.
>
> I'll want to merge patch 1, but I suspect the other two patches need
> to go through arm-soc, which will cause a problem due to being merged
> independently...  Arnd?

I just checked what we have queued up in next/dt for this platform,
and I'm fairly sure that there are no conflicts, so you can pick up all three
and add

Acked-by: Arnd Bergmann <arnd@arndb.de>

     Arnd

^ permalink raw reply	[flat|nested] 27+ messages in thread

* [PATCH v4 0/3] ARM: l2c: add l2c support for RZ/A1
@ 2017-02-16 16:57       ` Arnd Bergmann
  0 siblings, 0 replies; 27+ messages in thread
From: Arnd Bergmann @ 2017-02-16 16:57 UTC (permalink / raw)
  To: linux-arm-kernel

On Thu, Feb 16, 2017 at 5:44 PM, Russell King - ARM Linux
<linux@armlinux.org.uk> wrote:
> On Thu, Feb 16, 2017 at 11:17:39AM -0500, Chris Brandt wrote:
>> The PL310 in the Renesas RZ/A1 SoC (R7S72100) does not have the sideband
>> signals connected between the CPU and L2C. According the PL310 TRM,
>> sideband signals are optional.
>>
>> If a PL310 is added to a system, but the sideband signals are not
>> connected, some Cortex A9 optimizations cannot be used. In particular,
>> enabling Full Line Zeros in the CA9 without sidebands connected will
>> crash the system since the CA9 will expect the L2C to perform operations,
>> yet the L2C never gets the commands.
>>
>> This series adds the option to not enable anything in the PL310 that
>> uses sidebands, and then adds L2C support to the RZ/A1 DT.
>>
>> v4:
>> * changed l2x0_bresp_dis to l2x0_bresp_disable
>> * changed l2x0_flz_dis to l2x0_flz_disable
>
> Looks good, thanks.
>
> I'll want to merge patch 1, but I suspect the other two patches need
> to go through arm-soc, which will cause a problem due to being merged
> independently...  Arnd?

I just checked what we have queued up in next/dt for this platform,
and I'm fairly sure that there are no conflicts, so you can pick up all three
and add

Acked-by: Arnd Bergmann <arnd@arndb.de>

     Arnd

^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [PATCH v4 0/3] ARM: l2c: add l2c support for RZ/A1
  2017-02-16 16:57       ` Arnd Bergmann
@ 2017-02-16 17:21         ` Russell King - ARM Linux
  -1 siblings, 0 replies; 27+ messages in thread
From: Russell King - ARM Linux @ 2017-02-16 17:21 UTC (permalink / raw)
  To: Arnd Bergmann, Chris Brandt
  Cc: Simon Horman, Magnus Damm, Geert Uytterhoeven, Rob Herring,
	Mark Rutland, Brad Mouring, Andrey Smirnov, Richard Cochran,
	devicetree, linux-renesas-soc, Linux ARM

On Thu, Feb 16, 2017 at 05:57:53PM +0100, Arnd Bergmann wrote:
> On Thu, Feb 16, 2017 at 5:44 PM, Russell King - ARM Linux
> <linux@armlinux.org.uk> wrote:
> > On Thu, Feb 16, 2017 at 11:17:39AM -0500, Chris Brandt wrote:
> >> The PL310 in the Renesas RZ/A1 SoC (R7S72100) does not have the sideband
> >> signals connected between the CPU and L2C. According the PL310 TRM,
> >> sideband signals are optional.
> >>
> >> If a PL310 is added to a system, but the sideband signals are not
> >> connected, some Cortex A9 optimizations cannot be used. In particular,
> >> enabling Full Line Zeros in the CA9 without sidebands connected will
> >> crash the system since the CA9 will expect the L2C to perform operations,
> >> yet the L2C never gets the commands.
> >>
> >> This series adds the option to not enable anything in the PL310 that
> >> uses sidebands, and then adds L2C support to the RZ/A1 DT.
> >>
> >> v4:
> >> * changed l2x0_bresp_dis to l2x0_bresp_disable
> >> * changed l2x0_flz_dis to l2x0_flz_disable
> >
> > Looks good, thanks.
> >
> > I'll want to merge patch 1, but I suspect the other two patches need
> > to go through arm-soc, which will cause a problem due to being merged
> > independently...  Arnd?
> 
> I just checked what we have queued up in next/dt for this platform,
> and I'm fairly sure that there are no conflicts, so you can pick up all three
> and add
> 
> Acked-by: Arnd Bergmann <arnd@arndb.de>

Chris, can you put all three in the patch system, preferably with
Arnd's ack for the last two please?

Thanks.

-- 
RMK's Patch system: http://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up
according to speedtest.net.

^ permalink raw reply	[flat|nested] 27+ messages in thread

* [PATCH v4 0/3] ARM: l2c: add l2c support for RZ/A1
@ 2017-02-16 17:21         ` Russell King - ARM Linux
  0 siblings, 0 replies; 27+ messages in thread
From: Russell King - ARM Linux @ 2017-02-16 17:21 UTC (permalink / raw)
  To: linux-arm-kernel

On Thu, Feb 16, 2017 at 05:57:53PM +0100, Arnd Bergmann wrote:
> On Thu, Feb 16, 2017 at 5:44 PM, Russell King - ARM Linux
> <linux@armlinux.org.uk> wrote:
> > On Thu, Feb 16, 2017 at 11:17:39AM -0500, Chris Brandt wrote:
> >> The PL310 in the Renesas RZ/A1 SoC (R7S72100) does not have the sideband
> >> signals connected between the CPU and L2C. According the PL310 TRM,
> >> sideband signals are optional.
> >>
> >> If a PL310 is added to a system, but the sideband signals are not
> >> connected, some Cortex A9 optimizations cannot be used. In particular,
> >> enabling Full Line Zeros in the CA9 without sidebands connected will
> >> crash the system since the CA9 will expect the L2C to perform operations,
> >> yet the L2C never gets the commands.
> >>
> >> This series adds the option to not enable anything in the PL310 that
> >> uses sidebands, and then adds L2C support to the RZ/A1 DT.
> >>
> >> v4:
> >> * changed l2x0_bresp_dis to l2x0_bresp_disable
> >> * changed l2x0_flz_dis to l2x0_flz_disable
> >
> > Looks good, thanks.
> >
> > I'll want to merge patch 1, but I suspect the other two patches need
> > to go through arm-soc, which will cause a problem due to being merged
> > independently...  Arnd?
> 
> I just checked what we have queued up in next/dt for this platform,
> and I'm fairly sure that there are no conflicts, so you can pick up all three
> and add
> 
> Acked-by: Arnd Bergmann <arnd@arndb.de>

Chris, can you put all three in the patch system, preferably with
Arnd's ack for the last two please?

Thanks.

-- 
RMK's Patch system: http://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up
according to speedtest.net.

^ permalink raw reply	[flat|nested] 27+ messages in thread

* RE: [PATCH v4 0/3] ARM: l2c: add l2c support for RZ/A1
  2017-02-16 17:21         ` Russell King - ARM Linux
@ 2017-02-16 17:26           ` Chris Brandt
  -1 siblings, 0 replies; 27+ messages in thread
From: Chris Brandt @ 2017-02-16 17:26 UTC (permalink / raw)
  To: Russell King - ARM Linux, Arnd Bergmann
  Cc: Simon Horman, Magnus Damm, Geert Uytterhoeven, Rob Herring,
	Mark Rutland, Brad Mouring, Andrey Smirnov, Richard Cochran,
	devicetree, linux-renesas-soc, Linux ARM

On Thursday, February 16, 2017, Russell King wrote:
> Chris, can you put all three in the patch system, preferably with Arnd's
> ack for the last two please?

Yes, I will do that now.
Thank you.


Chris

^ permalink raw reply	[flat|nested] 27+ messages in thread

* [PATCH v4 0/3] ARM: l2c: add l2c support for RZ/A1
@ 2017-02-16 17:26           ` Chris Brandt
  0 siblings, 0 replies; 27+ messages in thread
From: Chris Brandt @ 2017-02-16 17:26 UTC (permalink / raw)
  To: linux-arm-kernel

On Thursday, February 16, 2017, Russell King wrote:
> Chris, can you put all three in the patch system, preferably with Arnd's
> ack for the last two please?

Yes, I will do that now.
Thank you.


Chris

^ permalink raw reply	[flat|nested] 27+ messages in thread

* RE: [PATCH v4 0/3] ARM: l2c: add l2c support for RZ/A1
  2017-02-16 17:21         ` Russell King - ARM Linux
  (?)
@ 2017-03-08 18:53           ` Chris Brandt
  -1 siblings, 0 replies; 27+ messages in thread
From: Chris Brandt @ 2017-03-08 18:53 UTC (permalink / raw)
  To: Russell King - ARM Linux, Arnd Bergmann
  Cc: Mark Rutland, Richard Cochran, devicetree, Andrey Smirnov,
	Magnus Damm, linux-renesas-soc, Rob Herring, Geert Uytterhoeven,
	Simon Horman, Linux ARM, Brad Mouring

On Thursday, February 16, 2017, Russell King wrote:
> On Thu, Feb 16, 2017 at 05:57:53PM +0100, Arnd Bergmann wrote:
> > On Thu, Feb 16, 2017 at 5:44 PM, Russell King - ARM Linux
> > <linux@armlinux.org.uk> wrote:
> > > On Thu, Feb 16, 2017 at 11:17:39AM -0500, Chris Brandt wrote:
> > >> The PL310 in the Renesas RZ/A1 SoC (R7S72100) does not have the
> > >> sideband signals connected between the CPU and L2C. According the
> > >> PL310 TRM, sideband signals are optional.
> > >>
> > >> If a PL310 is added to a system, but the sideband signals are not
> > >> connected, some Cortex A9 optimizations cannot be used. In
> > >> particular, enabling Full Line Zeros in the CA9 without sidebands
> > >> connected will crash the system since the CA9 will expect the L2C
> > >> to perform operations, yet the L2C never gets the commands.
> > >>
> > >> This series adds the option to not enable anything in the PL310
> > >> that uses sidebands, and then adds L2C support to the RZ/A1 DT.
> > >>
> > >> v4:
> > >> * changed l2x0_bresp_dis to l2x0_bresp_disable
> > >> * changed l2x0_flz_dis to l2x0_flz_disable
> > >
> > > Looks good, thanks.
> > >
> > > I'll want to merge patch 1, but I suspect the other two patches need
> > > to go through arm-soc, which will cause a problem due to being
> > > merged independently...  Arnd?
> >
> > I just checked what we have queued up in next/dt for this platform,
> > and I'm fairly sure that there are no conflicts, so you can pick up
> > all three and add
> >
> > Acked-by: Arnd Bergmann <arnd@arndb.de>
> 
> Chris, can you put all three in the patch system, preferably with Arnd's
> ack for the last two please?
> 
> Thanks.


These are still sitting in the patch system. Was there something wrong with
them that they would not go into 4.11?


Chris

^ permalink raw reply	[flat|nested] 27+ messages in thread

* RE: [PATCH v4 0/3] ARM: l2c: add l2c support for RZ/A1
@ 2017-03-08 18:53           ` Chris Brandt
  0 siblings, 0 replies; 27+ messages in thread
From: Chris Brandt @ 2017-03-08 18:53 UTC (permalink / raw)
  To: Russell King - ARM Linux, Arnd Bergmann
  Cc: Simon Horman, Magnus Damm, Geert Uytterhoeven, Rob Herring,
	Mark Rutland, Brad Mouring, Andrey Smirnov, Richard Cochran,
	devicetree, linux-renesas-soc, Linux ARM

On Thursday, February 16, 2017, Russell King wrote:
> On Thu, Feb 16, 2017 at 05:57:53PM +0100, Arnd Bergmann wrote:
> > On Thu, Feb 16, 2017 at 5:44 PM, Russell King - ARM Linux
> > <linux@armlinux.org.uk> wrote:
> > > On Thu, Feb 16, 2017 at 11:17:39AM -0500, Chris Brandt wrote:
> > >> The PL310 in the Renesas RZ/A1 SoC (R7S72100) does not have the
> > >> sideband signals connected between the CPU and L2C. According the
> > >> PL310 TRM, sideband signals are optional.
> > >>
> > >> If a PL310 is added to a system, but the sideband signals are not
> > >> connected, some Cortex A9 optimizations cannot be used. In
> > >> particular, enabling Full Line Zeros in the CA9 without sidebands
> > >> connected will crash the system since the CA9 will expect the L2C
> > >> to perform operations, yet the L2C never gets the commands.
> > >>
> > >> This series adds the option to not enable anything in the PL310
> > >> that uses sidebands, and then adds L2C support to the RZ/A1 DT.
> > >>
> > >> v4:
> > >> * changed l2x0_bresp_dis to l2x0_bresp_disable
> > >> * changed l2x0_flz_dis to l2x0_flz_disable
> > >
> > > Looks good, thanks.
> > >
> > > I'll want to merge patch 1, but I suspect the other two patches need
> > > to go through arm-soc, which will cause a problem due to being
> > > merged independently...  Arnd?
> >
> > I just checked what we have queued up in next/dt for this platform,
> > and I'm fairly sure that there are no conflicts, so you can pick up
> > all three and add
> >
> > Acked-by: Arnd Bergmann <arnd@arndb.de>
> 
> Chris, can you put all three in the patch system, preferably with Arnd's
> ack for the last two please?
> 
> Thanks.


These are still sitting in the patch system. Was there something wrong with
them that they would not go into 4.11?


Chris


^ permalink raw reply	[flat|nested] 27+ messages in thread

* [PATCH v4 0/3] ARM: l2c: add l2c support for RZ/A1
@ 2017-03-08 18:53           ` Chris Brandt
  0 siblings, 0 replies; 27+ messages in thread
From: Chris Brandt @ 2017-03-08 18:53 UTC (permalink / raw)
  To: linux-arm-kernel

On Thursday, February 16, 2017, Russell King wrote:
> On Thu, Feb 16, 2017 at 05:57:53PM +0100, Arnd Bergmann wrote:
> > On Thu, Feb 16, 2017 at 5:44 PM, Russell King - ARM Linux
> > <linux@armlinux.org.uk> wrote:
> > > On Thu, Feb 16, 2017 at 11:17:39AM -0500, Chris Brandt wrote:
> > >> The PL310 in the Renesas RZ/A1 SoC (R7S72100) does not have the
> > >> sideband signals connected between the CPU and L2C. According the
> > >> PL310 TRM, sideband signals are optional.
> > >>
> > >> If a PL310 is added to a system, but the sideband signals are not
> > >> connected, some Cortex A9 optimizations cannot be used. In
> > >> particular, enabling Full Line Zeros in the CA9 without sidebands
> > >> connected will crash the system since the CA9 will expect the L2C
> > >> to perform operations, yet the L2C never gets the commands.
> > >>
> > >> This series adds the option to not enable anything in the PL310
> > >> that uses sidebands, and then adds L2C support to the RZ/A1 DT.
> > >>
> > >> v4:
> > >> * changed l2x0_bresp_dis to l2x0_bresp_disable
> > >> * changed l2x0_flz_dis to l2x0_flz_disable
> > >
> > > Looks good, thanks.
> > >
> > > I'll want to merge patch 1, but I suspect the other two patches need
> > > to go through arm-soc, which will cause a problem due to being
> > > merged independently...  Arnd?
> >
> > I just checked what we have queued up in next/dt for this platform,
> > and I'm fairly sure that there are no conflicts, so you can pick up
> > all three and add
> >
> > Acked-by: Arnd Bergmann <arnd@arndb.de>
> 
> Chris, can you put all three in the patch system, preferably with Arnd's
> ack for the last two please?
> 
> Thanks.


These are still sitting in the patch system. Was there something wrong with
them that they would not go into 4.11?


Chris

^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [PATCH v4 0/3] ARM: l2c: add l2c support for RZ/A1
  2017-03-08 18:53           ` Chris Brandt
  (?)
@ 2017-03-08 19:16             ` Russell King - ARM Linux
  -1 siblings, 0 replies; 27+ messages in thread
From: Russell King - ARM Linux @ 2017-03-08 19:16 UTC (permalink / raw)
  To: Chris Brandt
  Cc: Mark Rutland, Richard Cochran, Arnd Bergmann, devicetree,
	Andrey Smirnov, Magnus Damm, linux-renesas-soc, Rob Herring,
	Geert Uytterhoeven, Simon Horman, Linux ARM, Brad Mouring

On Wed, Mar 08, 2017 at 06:53:37PM +0000, Chris Brandt wrote:
> On Thursday, February 16, 2017, Russell King wrote:
> > On Thu, Feb 16, 2017 at 05:57:53PM +0100, Arnd Bergmann wrote:
> > > On Thu, Feb 16, 2017 at 5:44 PM, Russell King - ARM Linux
> > > <linux@armlinux.org.uk> wrote:
> > > > On Thu, Feb 16, 2017 at 11:17:39AM -0500, Chris Brandt wrote:
> > > >> The PL310 in the Renesas RZ/A1 SoC (R7S72100) does not have the
> > > >> sideband signals connected between the CPU and L2C. According the
> > > >> PL310 TRM, sideband signals are optional.
> > > >>
> > > >> If a PL310 is added to a system, but the sideband signals are not
> > > >> connected, some Cortex A9 optimizations cannot be used. In
> > > >> particular, enabling Full Line Zeros in the CA9 without sidebands
> > > >> connected will crash the system since the CA9 will expect the L2C
> > > >> to perform operations, yet the L2C never gets the commands.
> > > >>
> > > >> This series adds the option to not enable anything in the PL310
> > > >> that uses sidebands, and then adds L2C support to the RZ/A1 DT.
> > > >>
> > > >> v4:
> > > >> * changed l2x0_bresp_dis to l2x0_bresp_disable
> > > >> * changed l2x0_flz_dis to l2x0_flz_disable
> > > >
> > > > Looks good, thanks.
> > > >
> > > > I'll want to merge patch 1, but I suspect the other two patches need
> > > > to go through arm-soc, which will cause a problem due to being
> > > > merged independently...  Arnd?
> > >
> > > I just checked what we have queued up in next/dt for this platform,
> > > and I'm fairly sure that there are no conflicts, so you can pick up
> > > all three and add
> > >
> > > Acked-by: Arnd Bergmann <arnd@arndb.de>
> > 
> > Chris, can you put all three in the patch system, preferably with Arnd's
> > ack for the last two please?
> > 
> > Thanks.
> 
> 
> These are still sitting in the patch system.

Exactly right.

> Was there something wrong with them that they would not go into 4.11?

They were submitted way too late.

* The merge window opened on Sunday, 19th February.

* The patches were submitted on Thursday, 16th February just before 6pm
  my local time.

* The last linux-next grab of trees before the merge window happened
  occured around 11pm my local time on Thursday, 16th February.

There's a general requirement that everything for a merge window should
be in linux-next before the merge window.

I generally want material for the merge window to be ready a week before
the merge window opens to allow for proper testing and any re-work that
comes out of that testing.

In any case, anything submitted from Thursday through to the Sunday when
the merge window opens _definitely_ is not going to get queued for the
merge window unless there are exceptional (eg, important bug fix) reasons
to do so.

So, it basically missed the time slot for merging new material, sorry.

I'll re-open my tree for new material in the next few days.

-- 
RMK's Patch system: http://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up
according to speedtest.net.

^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [PATCH v4 0/3] ARM: l2c: add l2c support for RZ/A1
@ 2017-03-08 19:16             ` Russell King - ARM Linux
  0 siblings, 0 replies; 27+ messages in thread
From: Russell King - ARM Linux @ 2017-03-08 19:16 UTC (permalink / raw)
  To: Chris Brandt
  Cc: Arnd Bergmann, Simon Horman, Magnus Damm, Geert Uytterhoeven,
	Rob Herring, Mark Rutland, Brad Mouring, Andrey Smirnov,
	Richard Cochran, devicetree, linux-renesas-soc, Linux ARM

On Wed, Mar 08, 2017 at 06:53:37PM +0000, Chris Brandt wrote:
> On Thursday, February 16, 2017, Russell King wrote:
> > On Thu, Feb 16, 2017 at 05:57:53PM +0100, Arnd Bergmann wrote:
> > > On Thu, Feb 16, 2017 at 5:44 PM, Russell King - ARM Linux
> > > <linux@armlinux.org.uk> wrote:
> > > > On Thu, Feb 16, 2017 at 11:17:39AM -0500, Chris Brandt wrote:
> > > >> The PL310 in the Renesas RZ/A1 SoC (R7S72100) does not have the
> > > >> sideband signals connected between the CPU and L2C. According the
> > > >> PL310 TRM, sideband signals are optional.
> > > >>
> > > >> If a PL310 is added to a system, but the sideband signals are not
> > > >> connected, some Cortex A9 optimizations cannot be used. In
> > > >> particular, enabling Full Line Zeros in the CA9 without sidebands
> > > >> connected will crash the system since the CA9 will expect the L2C
> > > >> to perform operations, yet the L2C never gets the commands.
> > > >>
> > > >> This series adds the option to not enable anything in the PL310
> > > >> that uses sidebands, and then adds L2C support to the RZ/A1 DT.
> > > >>
> > > >> v4:
> > > >> * changed l2x0_bresp_dis to l2x0_bresp_disable
> > > >> * changed l2x0_flz_dis to l2x0_flz_disable
> > > >
> > > > Looks good, thanks.
> > > >
> > > > I'll want to merge patch 1, but I suspect the other two patches need
> > > > to go through arm-soc, which will cause a problem due to being
> > > > merged independently...  Arnd?
> > >
> > > I just checked what we have queued up in next/dt for this platform,
> > > and I'm fairly sure that there are no conflicts, so you can pick up
> > > all three and add
> > >
> > > Acked-by: Arnd Bergmann <arnd@arndb.de>
> > 
> > Chris, can you put all three in the patch system, preferably with Arnd's
> > ack for the last two please?
> > 
> > Thanks.
> 
> 
> These are still sitting in the patch system.

Exactly right.

> Was there something wrong with them that they would not go into 4.11?

They were submitted way too late.

* The merge window opened on Sunday, 19th February.

* The patches were submitted on Thursday, 16th February just before 6pm
  my local time.

* The last linux-next grab of trees before the merge window happened
  occured around 11pm my local time on Thursday, 16th February.

There's a general requirement that everything for a merge window should
be in linux-next before the merge window.

I generally want material for the merge window to be ready a week before
the merge window opens to allow for proper testing and any re-work that
comes out of that testing.

In any case, anything submitted from Thursday through to the Sunday when
the merge window opens _definitely_ is not going to get queued for the
merge window unless there are exceptional (eg, important bug fix) reasons
to do so.

So, it basically missed the time slot for merging new material, sorry.

I'll re-open my tree for new material in the next few days.

-- 
RMK's Patch system: http://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up
according to speedtest.net.

^ permalink raw reply	[flat|nested] 27+ messages in thread

* [PATCH v4 0/3] ARM: l2c: add l2c support for RZ/A1
@ 2017-03-08 19:16             ` Russell King - ARM Linux
  0 siblings, 0 replies; 27+ messages in thread
From: Russell King - ARM Linux @ 2017-03-08 19:16 UTC (permalink / raw)
  To: linux-arm-kernel

On Wed, Mar 08, 2017 at 06:53:37PM +0000, Chris Brandt wrote:
> On Thursday, February 16, 2017, Russell King wrote:
> > On Thu, Feb 16, 2017 at 05:57:53PM +0100, Arnd Bergmann wrote:
> > > On Thu, Feb 16, 2017 at 5:44 PM, Russell King - ARM Linux
> > > <linux@armlinux.org.uk> wrote:
> > > > On Thu, Feb 16, 2017 at 11:17:39AM -0500, Chris Brandt wrote:
> > > >> The PL310 in the Renesas RZ/A1 SoC (R7S72100) does not have the
> > > >> sideband signals connected between the CPU and L2C. According the
> > > >> PL310 TRM, sideband signals are optional.
> > > >>
> > > >> If a PL310 is added to a system, but the sideband signals are not
> > > >> connected, some Cortex A9 optimizations cannot be used. In
> > > >> particular, enabling Full Line Zeros in the CA9 without sidebands
> > > >> connected will crash the system since the CA9 will expect the L2C
> > > >> to perform operations, yet the L2C never gets the commands.
> > > >>
> > > >> This series adds the option to not enable anything in the PL310
> > > >> that uses sidebands, and then adds L2C support to the RZ/A1 DT.
> > > >>
> > > >> v4:
> > > >> * changed l2x0_bresp_dis to l2x0_bresp_disable
> > > >> * changed l2x0_flz_dis to l2x0_flz_disable
> > > >
> > > > Looks good, thanks.
> > > >
> > > > I'll want to merge patch 1, but I suspect the other two patches need
> > > > to go through arm-soc, which will cause a problem due to being
> > > > merged independently...  Arnd?
> > >
> > > I just checked what we have queued up in next/dt for this platform,
> > > and I'm fairly sure that there are no conflicts, so you can pick up
> > > all three and add
> > >
> > > Acked-by: Arnd Bergmann <arnd@arndb.de>
> > 
> > Chris, can you put all three in the patch system, preferably with Arnd's
> > ack for the last two please?
> > 
> > Thanks.
> 
> 
> These are still sitting in the patch system.

Exactly right.

> Was there something wrong with them that they would not go into 4.11?

They were submitted way too late.

* The merge window opened on Sunday, 19th February.

* The patches were submitted on Thursday, 16th February just before 6pm
  my local time.

* The last linux-next grab of trees before the merge window happened
  occured around 11pm my local time on Thursday, 16th February.

There's a general requirement that everything for a merge window should
be in linux-next before the merge window.

I generally want material for the merge window to be ready a week before
the merge window opens to allow for proper testing and any re-work that
comes out of that testing.

In any case, anything submitted from Thursday through to the Sunday when
the merge window opens _definitely_ is not going to get queued for the
merge window unless there are exceptional (eg, important bug fix) reasons
to do so.

So, it basically missed the time slot for merging new material, sorry.

I'll re-open my tree for new material in the next few days.

-- 
RMK's Patch system: http://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up
according to speedtest.net.

^ permalink raw reply	[flat|nested] 27+ messages in thread

* RE: [PATCH v4 0/3] ARM: l2c: add l2c support for RZ/A1
  2017-03-08 19:16             ` Russell King - ARM Linux
  (?)
@ 2017-03-08 19:35               ` Chris Brandt
  -1 siblings, 0 replies; 27+ messages in thread
From: Chris Brandt @ 2017-03-08 19:35 UTC (permalink / raw)
  To: Russell King - ARM Linux
  Cc: Mark Rutland, Richard Cochran, Arnd Bergmann, devicetree,
	Andrey Smirnov, Magnus Damm, linux-renesas-soc, Rob Herring,
	Geert Uytterhoeven, Simon Horman, Linux ARM, Brad Mouring

On Wednesday, March 08, 2017, Russell King wrote:
> > Was there something wrong with them that they would not go into 4.11?
> 
> They were submitted way too late.
> 
> * The merge window opened on Sunday, 19th February.
> 
> * The patches were submitted on Thursday, 16th February just before 6pm
>   my local time.
> 
> * The last linux-next grab of trees before the merge window happened
>   occured around 11pm my local time on Thursday, 16th February.
> 
> There's a general requirement that everything for a merge window should be
> in linux-next before the merge window.
> 
> I generally want material for the merge window to be ready a week before
> the merge window opens to allow for proper testing and any re-work that
> comes out of that testing.
> 
> In any case, anything submitted from Thursday through to the Sunday when
> the merge window opens _definitely_ is not going to get queued for the
> merge window unless there are exceptional (eg, important bug fix) reasons
> to do so.
> 
> So, it basically missed the time slot for merging new material, sorry.

Well damn, that answers it.

Thank you for the detailed explanation.

Cheers

Chris

^ permalink raw reply	[flat|nested] 27+ messages in thread

* RE: [PATCH v4 0/3] ARM: l2c: add l2c support for RZ/A1
@ 2017-03-08 19:35               ` Chris Brandt
  0 siblings, 0 replies; 27+ messages in thread
From: Chris Brandt @ 2017-03-08 19:35 UTC (permalink / raw)
  To: Russell King - ARM Linux
  Cc: Arnd Bergmann, Simon Horman, Magnus Damm, Geert Uytterhoeven,
	Rob Herring, Mark Rutland, Brad Mouring, Andrey Smirnov,
	Richard Cochran, devicetree, linux-renesas-soc, Linux ARM

On Wednesday, March 08, 2017, Russell King wrote:
> > Was there something wrong with them that they would not go into 4.11?
> 
> They were submitted way too late.
> 
> * The merge window opened on Sunday, 19th February.
> 
> * The patches were submitted on Thursday, 16th February just before 6pm
>   my local time.
> 
> * The last linux-next grab of trees before the merge window happened
>   occured around 11pm my local time on Thursday, 16th February.
> 
> There's a general requirement that everything for a merge window should be
> in linux-next before the merge window.
> 
> I generally want material for the merge window to be ready a week before
> the merge window opens to allow for proper testing and any re-work that
> comes out of that testing.
> 
> In any case, anything submitted from Thursday through to the Sunday when
> the merge window opens _definitely_ is not going to get queued for the
> merge window unless there are exceptional (eg, important bug fix) reasons
> to do so.
> 
> So, it basically missed the time slot for merging new material, sorry.

Well damn, that answers it.

Thank you for the detailed explanation.

Cheers

Chris

^ permalink raw reply	[flat|nested] 27+ messages in thread

* [PATCH v4 0/3] ARM: l2c: add l2c support for RZ/A1
@ 2017-03-08 19:35               ` Chris Brandt
  0 siblings, 0 replies; 27+ messages in thread
From: Chris Brandt @ 2017-03-08 19:35 UTC (permalink / raw)
  To: linux-arm-kernel

On Wednesday, March 08, 2017, Russell King wrote:
> > Was there something wrong with them that they would not go into 4.11?
> 
> They were submitted way too late.
> 
> * The merge window opened on Sunday, 19th February.
> 
> * The patches were submitted on Thursday, 16th February just before 6pm
>   my local time.
> 
> * The last linux-next grab of trees before the merge window happened
>   occured around 11pm my local time on Thursday, 16th February.
> 
> There's a general requirement that everything for a merge window should be
> in linux-next before the merge window.
> 
> I generally want material for the merge window to be ready a week before
> the merge window opens to allow for proper testing and any re-work that
> comes out of that testing.
> 
> In any case, anything submitted from Thursday through to the Sunday when
> the merge window opens _definitely_ is not going to get queued for the
> merge window unless there are exceptional (eg, important bug fix) reasons
> to do so.
> 
> So, it basically missed the time slot for merging new material, sorry.

Well damn, that answers it.

Thank you for the detailed explanation.

Cheers

Chris

^ permalink raw reply	[flat|nested] 27+ messages in thread

end of thread, other threads:[~2017-03-08 19:57 UTC | newest]

Thread overview: 27+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-02-16 16:17 [PATCH v4 0/3] ARM: l2c: add l2c support for RZ/A1 Chris Brandt
2017-02-16 16:17 ` Chris Brandt
2017-02-16 16:17 ` Chris Brandt
2017-02-16 16:17 ` [PATCH v4 1/3] ARM: l2c: allow CA9 optimizations to be disabled Chris Brandt
2017-02-16 16:17   ` Chris Brandt
2017-02-16 16:17 ` [PATCH v4 2/3] ARM: shmobile: r7s72100: Enable L2 cache Chris Brandt
2017-02-16 16:17   ` Chris Brandt
2017-02-16 16:17 ` [PATCH v4 3/3] ARM: dts: r7s72100: add l2 cache Chris Brandt
2017-02-16 16:17   ` Chris Brandt
     [not found] ` <20170216161742.29320-1-chris.brandt-zM6kxYcvzFBBDgjK7y7TUQ@public.gmane.org>
2017-02-16 16:44   ` [PATCH v4 0/3] ARM: l2c: add l2c support for RZ/A1 Russell King - ARM Linux
2017-02-16 16:44     ` Russell King - ARM Linux
2017-02-16 16:44     ` Russell King - ARM Linux
2017-02-16 16:57     ` Arnd Bergmann
2017-02-16 16:57       ` Arnd Bergmann
2017-02-16 17:21       ` Russell King - ARM Linux
2017-02-16 17:21         ` Russell King - ARM Linux
2017-02-16 17:26         ` Chris Brandt
2017-02-16 17:26           ` Chris Brandt
2017-03-08 18:53         ` Chris Brandt
2017-03-08 18:53           ` Chris Brandt
2017-03-08 18:53           ` Chris Brandt
2017-03-08 19:16           ` Russell King - ARM Linux
2017-03-08 19:16             ` Russell King - ARM Linux
2017-03-08 19:16             ` Russell King - ARM Linux
2017-03-08 19:35             ` Chris Brandt
2017-03-08 19:35               ` Chris Brandt
2017-03-08 19:35               ` Chris Brandt

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.