* [PATCH 0/2] update the L2 cache settings on Meson8/Meson8b
@ 2017-10-31 22:23 Martin Blumenstingl
2017-10-31 22:23 ` [PATCH 1/2] ARM: dts: meson8b: add more L2 cache settings Martin Blumenstingl
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Martin Blumenstingl @ 2017-10-31 22:23 UTC (permalink / raw)
To: linux-arm-kernel
The L2 cache settings on our mainline kernel did not match the
configuration from Amlogic's vendor kernel.
This was boot-tested on a Meson8 (actually Meson8m2, but both use
the same CPU cores and L2 cache configuration) and a Meson8b board.
Martin Blumenstingl (2):
ARM: dts: meson8b: add more L2 cache settings
ARM: dts: meson8: add more L2 cache settings
arch/arm/boot/dts/meson8.dtsi | 3 +++
arch/arm/boot/dts/meson8b.dtsi | 3 +++
2 files changed, 6 insertions(+)
--
2.15.0
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH 1/2] ARM: dts: meson8b: add more L2 cache settings
2017-10-31 22:23 [PATCH 0/2] update the L2 cache settings on Meson8/Meson8b Martin Blumenstingl
@ 2017-10-31 22:23 ` Martin Blumenstingl
2017-10-31 22:23 ` [PATCH 2/2] ARM: dts: meson8: " Martin Blumenstingl
2017-12-06 19:44 ` [PATCH 0/2] update the L2 cache settings on Meson8/Meson8b Kevin Hilman
2 siblings, 0 replies; 4+ messages in thread
From: Martin Blumenstingl @ 2017-10-31 22:23 UTC (permalink / raw)
To: linux-arm-kernel
Amlogic's vendor kernel prints these PL310 L2 cache controller settings
during boot:
8 ways, 2048 sets, CACHE_ID 0x4100a0c9, Cache size: 524288 B
AUX_CTRL 0x7ec60001, PERFETCH_CTRL 0x75000007, POWER_CTRL 0x00000000
TAG_LATENCY 0x00000111, DATA_LATENCY 0x00000222
Add the "prefetch-data", "prefetch-instr" and "arm,shared-override"
properties to get the same L2 cache controller configuration as the
vendor kernel.
Four differences still remain:
- L310_AUX_CTRL_EARLY_BRESP is enabled by the vendor kernel, however
this is only supported on Cortex-A9 cores (Meson8b has Cortex-A5 cores
though)
- L310_AUX_CTRL_NS_INT_CTRL is currently not supported by the cache-l2x0
driver
- bit 23 is set by the vendor kernel, but this is defined in cache-l2x0.h
- L310_AUX_CTRL_FULL_LINE_ZERO is enabled by the vendor kernel which is
also only supported on Cortex-A9 cores
Signed-off-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com>
---
arch/arm/boot/dts/meson8b.dtsi | 3 +++
1 file changed, 3 insertions(+)
diff --git a/arch/arm/boot/dts/meson8b.dtsi b/arch/arm/boot/dts/meson8b.dtsi
index 554c58e67b01..71466077890e 100644
--- a/arch/arm/boot/dts/meson8b.dtsi
+++ b/arch/arm/boot/dts/meson8b.dtsi
@@ -223,6 +223,9 @@
arm,data-latency = <3 3 3>;
arm,tag-latency = <2 2 2>;
arm,filter-ranges = <0x100000 0xc0000000>;
+ prefetch-data = <1>;
+ prefetch-instr = <1>;
+ arm,shared-override;
};
&pwm_ab {
--
2.15.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH 2/2] ARM: dts: meson8: add more L2 cache settings
2017-10-31 22:23 [PATCH 0/2] update the L2 cache settings on Meson8/Meson8b Martin Blumenstingl
2017-10-31 22:23 ` [PATCH 1/2] ARM: dts: meson8b: add more L2 cache settings Martin Blumenstingl
@ 2017-10-31 22:23 ` Martin Blumenstingl
2017-12-06 19:44 ` [PATCH 0/2] update the L2 cache settings on Meson8/Meson8b Kevin Hilman
2 siblings, 0 replies; 4+ messages in thread
From: Martin Blumenstingl @ 2017-10-31 22:23 UTC (permalink / raw)
To: linux-arm-kernel
Amlogic's vendor kernel prints these PL310 L2 cache controller settings
during boot:
8 ways, 4096 sets, CACHE_ID 0x4100a0c9, Cache size: 1048576 B
AUX_CTRL 0x7ec80001, PERFETCH_CTRL 0x71000007, POWER_CTRL 0x00000000
TAG_LATENCY 0x00000111, DATA_LATENCY 0x00000222
Add the "prefetch-data", "prefetch-instr" and "arm,shared-override"
properties to get the same L2 cache controller configuration as the
vendor kernel.
Two differences still remain:
- L310_AUX_CTRL_NS_INT_CTRL is currently not supported by the cache-l2x0
driver
- bit 23 is set by the vendor kernel, but this is defined in cache-l2x0.h
Signed-off-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com>
---
arch/arm/boot/dts/meson8.dtsi | 3 +++
1 file changed, 3 insertions(+)
diff --git a/arch/arm/boot/dts/meson8.dtsi b/arch/arm/boot/dts/meson8.dtsi
index bedcbebc2433..264f2b80e2af 100644
--- a/arch/arm/boot/dts/meson8.dtsi
+++ b/arch/arm/boot/dts/meson8.dtsi
@@ -313,6 +313,9 @@
arm,data-latency = <3 3 3>;
arm,tag-latency = <2 2 2>;
arm,filter-ranges = <0x100000 0xc0000000>;
+ prefetch-data = <1>;
+ prefetch-instr = <1>;
+ arm,shared-override;
};
&pwm_ab {
--
2.15.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH 0/2] update the L2 cache settings on Meson8/Meson8b
2017-10-31 22:23 [PATCH 0/2] update the L2 cache settings on Meson8/Meson8b Martin Blumenstingl
2017-10-31 22:23 ` [PATCH 1/2] ARM: dts: meson8b: add more L2 cache settings Martin Blumenstingl
2017-10-31 22:23 ` [PATCH 2/2] ARM: dts: meson8: " Martin Blumenstingl
@ 2017-12-06 19:44 ` Kevin Hilman
2 siblings, 0 replies; 4+ messages in thread
From: Kevin Hilman @ 2017-12-06 19:44 UTC (permalink / raw)
To: linux-arm-kernel
Martin Blumenstingl <martin.blumenstingl@googlemail.com> writes:
> The L2 cache settings on our mainline kernel did not match the
> configuration from Amlogic's vendor kernel.
>
> This was boot-tested on a Meson8 (actually Meson8m2, but both use
> the same CPU cores and L2 cache configuration) and a Meson8b board.
Boot tested on meson8b-odroidc1, and applied to v4.16/dt,
Thanks,
Kevin
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2017-12-06 19:44 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-10-31 22:23 [PATCH 0/2] update the L2 cache settings on Meson8/Meson8b Martin Blumenstingl
2017-10-31 22:23 ` [PATCH 1/2] ARM: dts: meson8b: add more L2 cache settings Martin Blumenstingl
2017-10-31 22:23 ` [PATCH 2/2] ARM: dts: meson8: " Martin Blumenstingl
2017-12-06 19:44 ` [PATCH 0/2] update the L2 cache settings on Meson8/Meson8b Kevin Hilman
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).