[PATCH] ARM: cns3xxx: Add support for L2 Cache Controller

Anton Vorontsov cbouatmailru at gmail.com
Thu Jul 7 12:51:11 EDT 2011


On Wed, Jul 06, 2011 at 01:10:39PM -0500, Rob Herring wrote:
[...]
> > +	 * 1 cycle of setup latency, 2 cycles of read and write accesses latency
> > +	 */
> > +	val = readl(base + L2X0_DATA_LATENCY_CTRL);
> > +	val &= 0xfffff888;
> 
> You're missing a "val |= 0x110" or your comment is wrong.

Thanks for spotting this.

These values were taken from the BSP* code, which was tested
the most (and apparently works), I tend to leave the value as is
and fixup the comment.

Thanks!

* The BSP does not use L2X0 driver, instead, it contains 'L2CC'
  driver with these values hard-coded. But nowadays the L2CC driver
  is unneeded as L2X0 supports PL310 controllers.

-- 
Anton Vorontsov
Email: cbouatmailru at gmail.com



More information about the linux-arm-kernel mailing list