[PATCH] ARM: MM: Add ARM_L1_CACHE_SHIFT_BY6 for handle inside each ARCH Kconfig

Russell King - ARM Linux linux at arm.linux.org.uk
Wed Jan 27 18:50:11 EST 2010


On Thu, Jan 28, 2010 at 08:34:11AM +0900, Kukjin Kim wrote:
> This patch adds ARM_L1_CACHE_SHIFT_BY6 into arch/arm/Kconfig. Basically,
> ARM L1 cache line size is 32Bytes. Therefore, whenever new ARCH which
> has 64Bytes L1 cache line size is added, have to set ARM_L1_CACHE_SHIFT
> to 6. That is, have to add/modify arch/arm/mm/Kconfig.
> This patch is for handle it in each ARCH Kconfig.

(a) please don't introduce new stuff which doesn't appear to be used;
    if you decide not to send the patch which uses whatever you've
    added, it goes unnoticed and just adds unnecessary bloat to the
    kernel.

(b) please drop the 'BY' from the name.

(c) it would make sense to convert the existing users (which would
    sort point a.)



More information about the linux-arm-kernel mailing list