From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933842AbbBCEmK (ORCPT ); Mon, 2 Feb 2015 23:42:10 -0500 Received: from mail1.asahi-net.or.jp ([202.224.39.197]:52293 "EHLO mail1.asahi-net.or.jp" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932804AbbBCEmH (ORCPT ); Mon, 2 Feb 2015 23:42:07 -0500 Date: Tue, 03 Feb 2015 13:42:02 +0900 Message-ID: <87bnlbwqqt.wl-ysato@users.sourceforge.jp> From: Yoshinori Sato To: Arnd Bergmann Cc: Geert Uytterhoeven , "linux-kernel@vger.kernel.org" , Linux-Arch Subject: Re: [PATCH] Add ARCH_MIGHT_HAVE_VGA_CONSOLE In-Reply-To: <3429563.4px2PR9Zmy@wuerfel> References: <877fw093oa.wl-ysato@users.sourceforge.jp> <87pp9s48mz.wl-ysato@users.sourceforge.jp> <3429563.4px2PR9Zmy@wuerfel> User-Agent: Wanderlust/2.15.9 (Almost Unreal) SEMI-EPG/1.14.7 (Harue) FLIM/1.14.9 (=?ISO-8859-4?Q?Goj=F2?=) APEL/10.8 EasyPG/1.0.0 Emacs/24.4 (x86_64-pc-linux-gnu) MULE/6.0 (HANACHIRUSATO) MIME-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue") Content-Type: text/plain; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org At Mon, 02 Feb 2015 21:11:50 +0100, Arnd Bergmann wrote: > > 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. OK. > 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. Ouch. I will fix v3. > Thanks, > > Arnd -- Yoshinori Sato