On Wed, 2012-08-15 at 10:36 +0800, Fengguang Wu wrote: > > -#define L1_CACHE_BYTES L2_CACHE_BYTES > > +#define L1_CACHE_SHIFT L2_CACHE_SHIFT > > +#define L1_CACHE_BYTES (1 << L2_CACHE_SHIFT) > > Nitpick: the last line could better be: > > +#define L1_CACHE_BYTES (1 << L1_CACHE_SHIFT) > > Reviewed-by: Fengguang Wu <fengguang...@intel.com>
Yes, I noticed that after sending the patch. Should I push this through the c6x tree? --Mark -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/