linux-kernel - Re: [PATCH 2/2] mfd: remove CONFIG_MFD

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]

Message-ID: <20110905150942.54aecf4e@endymion.delvare>
Date:	Mon, 5 Sep 2011 15:09:42 +0200
From:	Jean Delvare <khali@...ux-fr.org>
To:	Arnd Bergmann <arnd@...db.de>
Cc:	Luciano Coelho <coelho@...com>,
	Randy Dunlap <rdunlap@...otime.net>,
	matti.j.aaltonen@...ia.com, johannes@...solutions.net,
	linux-kernel@...r.kernel.org, sameo@...ux.intel.com,
	mchehab@...radead.org, linux-media@...r.kernel.org,
	linux-omap@...r.kernel.org, Tony Lindgren <tony@...mide.com>,
	Grant Likely <grant.likely@...retlab.ca>
Subject: Re: [PATCH 2/2] mfd: remove CONFIG_MFD_SUPPORT

On Fri, 2 Sep 2011 16:43:36 +0200, Arnd Bergmann wrote:
> We currently have two symbols to control compilation the MFD subsystem,
> MFD_SUPPORT and MFD_CORE. The MFD_SUPPORT is actually not required
> at all, it only hides the submenu when not set, with the effect that
> Kconfig warns about missing dependencies when another driver selects
> an MFD driver while MFD_SUPPORT is disabled. Turning the MFD submenu
> back from menuconfig into a plain menu simplifies the Kconfig syntax
> for those kinds of users and avoids the surprise when the menu
> suddenly appears because another driver was enabled that selects this
> symbol.
> 
> Signed-off-by: Arnd Bergmann <arnd@...db.de>
> ---
>  arch/arm/mach-omap2/Kconfig |    1 -
>  drivers/gpio/Kconfig        |    3 +--
>  drivers/i2c/busses/Kconfig  |    1 -
>  drivers/media/radio/Kconfig |    1 -
>  drivers/mfd/Kconfig         |   22 ++++------------------
>  5 files changed, 5 insertions(+), 23 deletions(-)
> (...)
> --- a/drivers/mfd/Kconfig
> +++ b/drivers/mfd/Kconfig
> @@ -2,23 +2,8 @@
>  # Multifunction miscellaneous devices
>  #
>  
> -menuconfig MFD_SUPPORT
> -	bool "Multifunction device drivers"
> -	depends on HAS_IOMEM
> -	default y
> -	help
> -	  Multifunction devices embed several functions (e.g. GPIOs,
> -	  touchscreens, keyboards, current regulators, power management chips,
> -	  etc...) in one single integrated circuit. They usually talk to the
> -	  main CPU through one or more IRQ lines and low speed data busses (SPI,
> -	  I2C, etc..). They appear as one single device to the main system
> -	  through the data bus and the MFD framework allows for sub devices
> -	  (a.k.a. functions) to appear as discrete platform devices.
> -	  MFDs are typically found on embedded platforms.
> -
> -	  This option alone does not add any kernel code.
> -
> -if MFD_SUPPORT
> +if HAS_IOMEM
> +menu "Multifunction device drivers"
>  
>  config MFD_CORE
>  	tristate

I think I prefer Luciano's proposal, for the same reasons given for the
misc device drivers patch. But here again I'm not the one making the
decision, so it's up to Samuel to decide which patch he wants to apply.


-- 
Jean Delvare
--
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/