Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754913AbbBBUL6 (ORCPT ); Mon, 2 Feb 2015 15:11:58 -0500 Received: from mout.kundenserver.de ([212.227.17.24]:55712 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753762AbbBBULz (ORCPT ); Mon, 2 Feb 2015 15:11:55 -0500 From: Arnd Bergmann To: Yoshinori Sato Cc: Geert Uytterhoeven , "linux-kernel@vger.kernel.org" , Linux-Arch Subject: Re: [PATCH] Add ARCH_MIGHT_HAVE_VGA_CONSOLE Date: Mon, 02 Feb 2015 21:11:50 +0100 Message-ID: <3429563.4px2PR9Zmy@wuerfel> User-Agent: KMail/4.11.5 (Linux/3.16.0-10-generic; KDE/4.11.5; x86_64; ; ) In-Reply-To: <87pp9s48mz.wl-ysato@users.sourceforge.jp> References: <877fw093oa.wl-ysato@users.sourceforge.jp> <87pp9s48mz.wl-ysato@users.sourceforge.jp> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" X-Provags-ID: V03:K0:+TtKZIUAkqo6q3d8gasNOTeEz/ciRMxZIYTCMtF3ztTB5ya/w0+ /a3kd3xBw3ZZlSnfKtRJRxKTtFqKy0Vn1V/WcdbQ8PZPC49o4Yn6EPwtMk+odf8pcxD4JUN EedsEDCkoNEIA4/unk6gkW5vazEIsefqA8l0FvoHc9W1/B/DYd1IaRjhc/yY7h0SIvykh3P tAjtAGXzs5CCNy5krgw3g== X-UI-Out-Filterresults: notjunk:1; Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2892 Lines: 84 On Tuesday 03 February 2015 00:48:20 Yoshinori Sato wrote: > At Mon, 2 Feb 2015 09:10:06 +0100, > Geert Uytterhoeven wrote: > > > > Hi Sato-san, > > > > > > On Mon, Feb 2, 2015 at 8:24 AM, Yoshinori Sato > > wrote: > > > The dependence of VGA_CONSOLE is complicated. > > > We need clean up. > > > > Thanks! > > > > > Signed-off-by: Yoshinori Sato > > > > > > --- > > > arch/alpha/Kconfig | 1 + > > > arch/arc/Kconfig | 1 + > > > arch/arm/Kconfig | 1 + > > > arch/c6x/Kconfig | 1 + > > > arch/hexagon/Kconfig | 1 + > > > arch/ia64/Kconfig | 1 + > > > arch/m32r/Kconfig | 1 + > > > arch/metag/Kconfig | 1 + > > > arch/microblaze/Kconfig | 1 + > > > arch/mips/Kconfig | 1 + > > > arch/nios2/Kconfig | 1 + > > > arch/openrisc/Kconfig | 1 + > > > arch/powerpc/Kconfig | 1 + > > > arch/s390/Kconfig | 1 + > > > arch/score/Kconfig | 1 + > > > arch/tile/Kconfig | 1 + > > > arch/um/Kconfig.common | 1 + > > > arch/unicore32/Kconfig | 1 + > > > arch/x86/Kconfig | 1 + > > > arch/xtensa/Kconfig | 1 + > > > > While I know you just kept the existing logic from > > drivers/video/console/kconfig, > > I doubt all of the architectures listed above do support VGA. > > I guest most of them were just too lazy to update the dependency list. > > > > I think only alpha, arm, ia64, mips, powerpc, and x86 can have VGA. The list looks right to me too. We could be more specific in case of ARM, MIPS and PowerPC and move the symbols into the individual platforms that do support it, but it's fine to leave this up to the architecture maintainers. > Hmm... > If require, Add this flag. > > Signed-off-by: Yoshinori Sato Reviewed-by: Arnd Bergmann Please add this patch to your series with my Reviewed-by line added. except for one thing: > --- a/arch/arm/Kconfig > +++ b/arch/arm/Kconfig > @@ -76,6 +76,7 @@ config ARM > select PERF_USE_VMALLOC > select RTC_LIB > select SYS_SUPPORTS_APM_EMULATION >+ select ARCH_MIGHT_HAVE_VGA_CONSOLE if ARCH_FOOTBRIDGE || ARCH_INTEGRATOR || ARCH_NETWINDER > # Above selects are sorted alphabetically; please add new ones > # according to that. Thanks. > help You did not read the comment in the patch context: This list of selects is sorted alphabetically in order to avoid merge conflicts. Please keep it this way. Thanks, Arnd -- 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/