* [PATCH] ARM: imx: i.mx6d/q: disable the double linefill feature of PL310
@ 2013-09-13 7:48 Jason Liu
2013-09-13 16:46 ` Dirk Behme
2013-09-16 2:47 ` Shawn Guo
0 siblings, 2 replies; 4+ messages in thread
From: Jason Liu @ 2013-09-13 7:48 UTC (permalink / raw)
To: linux-arm-kernel
The L2 cache controller(PL310) version on the i.MX6D/Q is r3p1-50rel0
The L2 cache controller(PL310) version on the i.MX6DL/SOLO/SL is r3p2
But according to ARM PL310 errata: 752271
ID: 752271: Double linefill feature can cause data corruption
Fault Status: Present in: r3p0, r3p1, r3p1-50rel0. Fixed in r3p2
Workaround: The only workaround to this erratum is to disable the
double linefill feature. This is the default behavior.
without this patch, you will meet the following error when run the
memtester application at: http://pyropus.ca/software/memtester/
FAILURE: 0x00100000 != 0x00200000 at offset 0x01365664.
FAILURE: 0x00100000 != 0x00200000 at offset 0x01365668.
FAILURE: 0x00100000 != 0x00200000 at offset 0x0136566c.
FAILURE: 0x00100000 != 0x00200000 at offset 0x01365670.
FAILURE: 0x00100000 != 0x00200000 at offset 0x01365674.
FAILURE: 0x00100000 != 0x00200000 at offset 0x01365678.
FAILURE: 0x00100000 != 0x00200000 at offset 0x0136567c.
Signed-off-by: Jason Liu <r64343@freescale.com>
---
arch/arm/mach-imx/system.c | 13 ++++++++++++-
1 file changed, 12 insertions(+), 1 deletion(-)
diff --git a/arch/arm/mach-imx/system.c b/arch/arm/mach-imx/system.c
index 64ff37e..0c3fa48 100644
--- a/arch/arm/mach-imx/system.c
+++ b/arch/arm/mach-imx/system.c
@@ -116,7 +116,18 @@ void __init imx_init_l2cache(void)
/* Configure the L2 PREFETCH and POWER registers */
val = readl_relaxed(l2x0_base + L2X0_PREFETCH_CTRL);
- val |= 0x70800000;
+ val |= 0x30000000;
+ /*
+ * The L2 cache controller(PL310) version on the i.MX6D/Q is r3p1-50rel0
+ * The L2 cache controller(PL310) version on the i.MX6DL/SOLO/SL is r3p2
+ * But according to ARM PL310 errata: 752271
+ * ID: 752271: Double linefill feature can cause data corruption
+ * Fault Status: Present in: r3p0, r3p1, r3p1-50rel0. Fixed in r3p2
+ * Workaround: The only workaround to this erratum is to disable the
+ * double linefill feature. This is the default behavior.
+ */
+ if (!of_machine_is_compatible("fsl,imx6q"))
+ val |= 0x40800000;
writel_relaxed(val, l2x0_base + L2X0_PREFETCH_CTRL);
val = L2X0_DYNAMIC_CLK_GATING_EN | L2X0_STNDBY_MODE_EN;
writel_relaxed(val, l2x0_base + L2X0_POWER_CTRL);
--
1.7.10.4
^ permalink raw reply related [flat|nested] 4+ messages in thread* [PATCH] ARM: imx: i.mx6d/q: disable the double linefill feature of PL310
2013-09-13 7:48 [PATCH] ARM: imx: i.mx6d/q: disable the double linefill feature of PL310 Jason Liu
@ 2013-09-13 16:46 ` Dirk Behme
2013-09-16 1:52 ` Jason Liu
2013-09-16 2:47 ` Shawn Guo
1 sibling, 1 reply; 4+ messages in thread
From: Dirk Behme @ 2013-09-13 16:46 UTC (permalink / raw)
To: linux-arm-kernel
Am 13.09.2013 09:48, schrieb Jason Liu:
> The L2 cache controller(PL310) version on the i.MX6D/Q is r3p1-50rel0
> The L2 cache controller(PL310) version on the i.MX6DL/SOLO/SL is r3p2
> But according to ARM PL310 errata: 752271
> ID: 752271: Double linefill feature can cause data corruption
> Fault Status: Present in: r3p0, r3p1, r3p1-50rel0. Fixed in r3p2
> Workaround: The only workaround to this erratum is to disable the
> double linefill feature. This is the default behavior.
>
> without this patch, you will meet the following error when run the
> memtester application at: http://pyropus.ca/software/memtester/
>
> FAILURE: 0x00100000 != 0x00200000 at offset 0x01365664.
> FAILURE: 0x00100000 != 0x00200000 at offset 0x01365668.
> FAILURE: 0x00100000 != 0x00200000 at offset 0x0136566c.
> FAILURE: 0x00100000 != 0x00200000 at offset 0x01365670.
> FAILURE: 0x00100000 != 0x00200000 at offset 0x01365674.
> FAILURE: 0x00100000 != 0x00200000 at offset 0x01365678.
> FAILURE: 0x00100000 != 0x00200000 at offset 0x0136567c.
>
> Signed-off-by: Jason Liu <r64343@freescale.com>
> ---
> arch/arm/mach-imx/system.c | 13 ++++++++++++-
> 1 file changed, 12 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm/mach-imx/system.c b/arch/arm/mach-imx/system.c
> index 64ff37e..0c3fa48 100644
> --- a/arch/arm/mach-imx/system.c
> +++ b/arch/arm/mach-imx/system.c
> @@ -116,7 +116,18 @@ void __init imx_init_l2cache(void)
>
> /* Configure the L2 PREFETCH and POWER registers */
> val = readl_relaxed(l2x0_base + L2X0_PREFETCH_CTRL);
> - val |= 0x70800000;
> + val |= 0x30000000;
> + /*
> + * The L2 cache controller(PL310) version on the i.MX6D/Q is r3p1-50rel0
> + * The L2 cache controller(PL310) version on the i.MX6DL/SOLO/SL is r3p2
> + * But according to ARM PL310 errata: 752271
> + * ID: 752271: Double linefill feature can cause data corruption
> + * Fault Status: Present in: r3p0, r3p1, r3p1-50rel0. Fixed in r3p2
> + * Workaround: The only workaround to this erratum is to disable the
> + * double linefill feature. This is the default behavior.
> + */
> + if (!of_machine_is_compatible("fsl,imx6q"))
> + val |= 0x40800000;
I've been thinking about this issue some time, too. Therefore two
questions:
* Why do you set the 0x8 in 0x40800000 only in the non-imx6q case
here? This bit shouldn't be affected by the errata and therefore could
be set in all cases?
* To make it more clear what we do here (and in case somebody, e.g. a
boot loader?) has already set bit 30, I would do something like:
val |= 0x70800000;
if (!of_machine_is_compatible("fsl,imx6q"))
val &= ~(1 << 30);
What do you think?
Best regards
Dirk
^ permalink raw reply [flat|nested] 4+ messages in thread* [PATCH] ARM: imx: i.mx6d/q: disable the double linefill feature of PL310
2013-09-13 16:46 ` Dirk Behme
@ 2013-09-16 1:52 ` Jason Liu
0 siblings, 0 replies; 4+ messages in thread
From: Jason Liu @ 2013-09-16 1:52 UTC (permalink / raw)
To: linux-arm-kernel
2013/9/14 Dirk Behme <dirk.behme@gmail.com>:
> Am 13.09.2013 09:48, schrieb Jason Liu:
>
>> The L2 cache controller(PL310) version on the i.MX6D/Q is r3p1-50rel0
>> The L2 cache controller(PL310) version on the i.MX6DL/SOLO/SL is r3p2
>> But according to ARM PL310 errata: 752271
>> ID: 752271: Double linefill feature can cause data corruption
>> Fault Status: Present in: r3p0, r3p1, r3p1-50rel0. Fixed in r3p2
>> Workaround: The only workaround to this erratum is to disable the
>> double linefill feature. This is the default behavior.
>>
>> without this patch, you will meet the following error when run the
>> memtester application at: http://pyropus.ca/software/memtester/
>>
>> FAILURE: 0x00100000 != 0x00200000 at offset 0x01365664.
>> FAILURE: 0x00100000 != 0x00200000 at offset 0x01365668.
>> FAILURE: 0x00100000 != 0x00200000 at offset 0x0136566c.
>> FAILURE: 0x00100000 != 0x00200000 at offset 0x01365670.
>> FAILURE: 0x00100000 != 0x00200000 at offset 0x01365674.
>> FAILURE: 0x00100000 != 0x00200000 at offset 0x01365678.
>> FAILURE: 0x00100000 != 0x00200000 at offset 0x0136567c.
>>
>> Signed-off-by: Jason Liu <r64343@freescale.com>
>> ---
>> arch/arm/mach-imx/system.c | 13 ++++++++++++-
>> 1 file changed, 12 insertions(+), 1 deletion(-)
>>
>> diff --git a/arch/arm/mach-imx/system.c b/arch/arm/mach-imx/system.c
>> index 64ff37e..0c3fa48 100644
>> --- a/arch/arm/mach-imx/system.c
>> +++ b/arch/arm/mach-imx/system.c
>> @@ -116,7 +116,18 @@ void __init imx_init_l2cache(void)
>>
>> /* Configure the L2 PREFETCH and POWER registers */
>> val = readl_relaxed(l2x0_base + L2X0_PREFETCH_CTRL);
>> - val |= 0x70800000;
>> + val |= 0x30000000;
>> + /*
>> + * The L2 cache controller(PL310) version on the i.MX6D/Q is
>> r3p1-50rel0
>> + * The L2 cache controller(PL310) version on the i.MX6DL/SOLO/SL
>> is r3p2
>> + * But according to ARM PL310 errata: 752271
>> + * ID: 752271: Double linefill feature can cause data corruption
>> + * Fault Status: Present in: r3p0, r3p1, r3p1-50rel0. Fixed in
>> r3p2
>> + * Workaround: The only workaround to this erratum is to disable
>> the
>> + * double linefill feature. This is the default behavior.
>> + */
>> + if (!of_machine_is_compatible("fsl,imx6q"))
>> + val |= 0x40800000;
>
>
> I've been thinking about this issue some time, too. Therefore two questions:
>
> * Why do you set the 0x8 in 0x40800000 only in the non-imx6q case here? This
> bit shouldn't be affected by the errata and therefore could be set in all
> cases?
I supposed that you means the bit 23 in the prefetch control register,
by looking at the register definition:
[23] Incr double Linefill enable:
0 The L2CC does not issue INCR 8x64-bit read bursts to L3 on
reads that miss in the L2 cache. This is the default.
1 The L2CC can issue INCR 8x64-bit read bursts to L3 on
reads that miss in the L2 cache.
Since we need disable the double Linefill, we need disable this bit
too on the i.MX6Q.
>
> * To make it more clear what we do here (and in case somebody, e.g. a boot
> loader?) has already set bit 30, I would do something like:
>
> val |= 0x70800000;
> if (!of_machine_is_compatible("fsl,imx6q"))
> val &= ~(1 << 30);
val &= ~( 1 << 30 | 1 << 23);
>
> What do you think?
Yes, this is better.
Jason Liu
>
> Best regards
>
> Dirk
>
>
>
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH] ARM: imx: i.mx6d/q: disable the double linefill feature of PL310
2013-09-13 7:48 [PATCH] ARM: imx: i.mx6d/q: disable the double linefill feature of PL310 Jason Liu
2013-09-13 16:46 ` Dirk Behme
@ 2013-09-16 2:47 ` Shawn Guo
1 sibling, 0 replies; 4+ messages in thread
From: Shawn Guo @ 2013-09-16 2:47 UTC (permalink / raw)
To: linux-arm-kernel
Jason,
On Fri, Sep 13, 2013 at 03:48:52PM +0800, Jason Liu wrote:
> @@ -116,7 +116,18 @@ void __init imx_init_l2cache(void)
>
> /* Configure the L2 PREFETCH and POWER registers */
> val = readl_relaxed(l2x0_base + L2X0_PREFETCH_CTRL);
> - val |= 0x70800000;
> + val |= 0x30000000;
> + /*
> + * The L2 cache controller(PL310) version on the i.MX6D/Q is r3p1-50rel0
> + * The L2 cache controller(PL310) version on the i.MX6DL/SOLO/SL is r3p2
> + * But according to ARM PL310 errata: 752271
> + * ID: 752271: Double linefill feature can cause data corruption
> + * Fault Status: Present in: r3p0, r3p1, r3p1-50rel0. Fixed in r3p2
> + * Workaround: The only workaround to this erratum is to disable the
> + * double linefill feature. This is the default behavior.
> + */
> + if (!of_machine_is_compatible("fsl,imx6q"))
Can we use cpu_is_imx6q() here? I think we may not want to look at
device tree, if the kernel can figure this out by itself.
Shawn
> + val |= 0x40800000;
> writel_relaxed(val, l2x0_base + L2X0_PREFETCH_CTRL);
> val = L2X0_DYNAMIC_CLK_GATING_EN | L2X0_STNDBY_MODE_EN;
> writel_relaxed(val, l2x0_base + L2X0_POWER_CTRL);
> --
> 1.7.10.4
>
>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2013-09-16 2:47 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-09-13 7:48 [PATCH] ARM: imx: i.mx6d/q: disable the double linefill feature of PL310 Jason Liu
2013-09-13 16:46 ` Dirk Behme
2013-09-16 1:52 ` Jason Liu
2013-09-16 2:47 ` Shawn Guo
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).