Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753991AbbK3Uoc (ORCPT ); Mon, 30 Nov 2015 15:44:32 -0500 Received: from mail-pa0-f52.google.com ([209.85.220.52]:35812 "EHLO mail-pa0-f52.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751652AbbK3Uob (ORCPT ); Mon, 30 Nov 2015 15:44:31 -0500 Date: Mon, 30 Nov 2015 12:44:28 -0800 From: Brian Norris To: Arnd Bergmann Cc: linux-mtd@lists.infradead.org, David Woodhouse , linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH] mtd: cfi: enforce valid geometry configuration Message-ID: <20151130204428.GM64635@google.com> References: <3747078.dk5V5gxuXa@wuerfel> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <3747078.dk5V5gxuXa@wuerfel> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1152 Lines: 22 On Mon, Nov 23, 2015 at 02:39:33PM +0100, Arnd Bergmann wrote: > MTD allows compile-time configuration of the possible CFI geometry > settings that are allowed by the kernel, but that includes a couple of > invalid configurations, where no bank width or no interleave setting > is allowed. These are then caught with a compile-time warning: > > include/linux/mtd/cfi.h:76:2: warning: #warning No CONFIG_MTD_CFI_Ix selected. No NOR chip support can work. > include/linux/mtd/map.h:145:2: warning: #warning "No CONFIG_MTD_MAP_BANK_WIDTH_xx selected. No NOR chip support can work" > > This is a bit annoying for randconfig tests, and can be avoided if > we change the Kconfig logic to always select the simplest configuration > when no other one is enabled. > > Signed-off-by: Arnd Bergmann Pushed to l2-mtd.git. Thanks for fixing this one! I guess an ugly fix is better than nothing here. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/