On Tue, Jan 29, 2013 at 03:40:38PM +0530, srinidhi kasagar wrote:
- Added l2x0_quirks to manage the errata in cpu_idle path. Tried to address
Russell's comment on this, but could not completely. Because, neither I can
keep the #ifdef CONFIG_PL310_ERRATA_769419 nor remove it entirely since
the platform A suffers from this bug, whereas B do not but shares the same
defconfig.
Yes you can and I covered this situation in my email. In that case,
you _define_ CONFIG_PL310_ERRATA_769419 such that the barrier _always_
gets included both both platforms.
As I already said, there is _no_ point making the barrier conditional;
it's probably more expensive to make it conditional than just to execute
it every time. But we still might as well optimize it away if we are
running _only_ on platforms which _do_ _not_ have that errata.