[ Moved Lorenzo up in the addressee list to get his attention ]
On Tue, 22 Apr 2014, Daniel Lezcano wrote:
> On 04/22/2014 05:40 PM, Nicolas Pitre wrote:
> > On Tue, 22 Apr 2014, Daniel Lezcano wrote:
> >
> > > On 04/22/2014 08:17 AM, Abhilash Kesavan wrote:
> > > > +/*
> > > > + * The common v7_exit_coherency_flush API could not be used because of the
> > > > + * Erratum 799270 workaround. This macro is the same as the common one (in
> > > > + * arch/arm/include/asm/cacheflush.h) except for the erratum handling.
> > > > + */
> > > > +#define exynos_v7_exit_coherency_flush(level) \
> > > > + asm volatile( \
> > > > + "stmfd sp!, {fp, ip}\n\t"\
> > > > + "mrc p15, 0, r0, c1, c0, 0 @ get SCTLR\n\t" \
> > > > + "bic r0, r0, #"__stringify(CR_C)"\n\t" \
> > > > + "mcr p15, 0, r0, c1, c0, 0 @ set SCTLR\n\t" \
> > > > + "isb\n\t"\
> > > > + "bl v7_flush_dcache_"__stringify(level)"\n\t" \
> > > > + "clrex\n\t"\
> > > > + "mrc p15, 0, r0, c1, c0, 1 @ get ACTLR\n\t" \
> > > > + "bic r0, r0, #(1 << 6) @ disable local coherency\n\t" \
> > > > + /* Dummy Load of a device register to avoid Erratum 799270 */ \
> > >
> > > Wouldn't make sense to add the erratum in the Kconfig and re-use it in the
> > > generic v7_exit_coherency_flush macro instead of redefining it ?
> >
> > The implementation of the erratum (the dummy device register load) is
> > platform specific I'm afraid.
> >
> > Is TC2 also concerned by this erratum, or is this Samsung specific?
>
> Sounds like it is ARM Cortex-A15MP specific:
>
> http://infocenter.arm.com/help/topic/com.arm.doc.epm028090/cortex_a15_mpcore_software_developers_errata_notice_r2_v12.pdf
>
> Page 31.
The condition for hitting the erratum is: "The L2 cache block has been
idle for 256 or more cycles with no memory requests from any core, no
external snoops, and no ACP requests".
Given that the operation that almost immediately precedes the
problematic mcr is a call to v7_flush_dcache_louis or
v7_flush_dcache_all, is this possible that the condition for the erratum
could ever be met?
If no then we should document that v7_exit_coherency_flush(() is safe
against erratum 799270 and use it here.
> > > > + "ldr r4, [%0]\n\t" \
> > > > + "and r4, r4, #0\n\t" \
> > > > + "orr r0, r0, r4\n\t" \
> > > > + "mcr p15, 0, r0, c1, c0, 1 @ set ACTLR\n\t" \
> > > > + "isb\n\t" \
> > > > + "dsb\n\t" \
> > > > + "ldmfd sp!, {fp, ip}" \
> > > > + : \
> > > > + : "Ir" (S5P_INFORM0) \
> > > > + : "r0", "r1", "r2", "r3", "r4", "r5", "r6", "r7", \
> > > > + "r9", "r10", "lr", "memory")
> > >
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
