linux-kernel - Re: [PATCH] Add ARCH_MIGHT_HAVE_VGA

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Mon, 2 Feb 2015 09:10:06 +0100
From:	Geert Uytterhoeven <geert@...ux-m68k.org>
To:	Yoshinori Sato <ysato@...rs.sourceforge.jp>
Cc:	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	Linux-Arch <linux-arch@...r.kernel.org>
Subject: Re: [PATCH] Add ARCH_MIGHT_HAVE_VGA_CONSOLE

Hi Sato-san,


On Mon, Feb 2, 2015 at 8:24 AM, Yoshinori Sato
<ysato@...rs.sourceforge.jp> wrote:
> The dependence of VGA_CONSOLE is complicated.
> We need clean up.

Thanks!

> Signed-off-by: Yoshinori Sato <ysato@...rs.sourceforge.jp>
>
> ---
>  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.

> --- a/drivers/video/console/Kconfig
> +++ b/drivers/video/console/rconfig
> @@ -2,14 +2,14 @@
>  # Video configuration
>  #
>
> +config ARCH_MIGHT_HAVE_VGA_CONSOLE
> +        bool
> +
>  menu "Console display driver support"
>
>  config VGA_CONSOLE
>         bool "VGA text console" if EXPERT || !X86
> -       depends on !4xx && !8xx && !SPARC && !M68K && !PARISC && !FRV && \
> -               !SUPERH && !BLACKFIN && !AVR32 && !MN10300 && !CRIS && \
> -               (!ARM || ARCH_FOOTBRIDGE || ARCH_INTEGRATOR || ARCH_NETWINDER) && \
> -               !ARM64
> +       depends on ARCH_MIGHT_HAVE_VGA_CONSOLE
>         default y
>         help
>           Saying Y here will allow you to use Linux in text mode through a

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@...ux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/