From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-8.3 required=3.0 tests=DKIM_INVALID,DKIM_SIGNED, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_PASS,USER_AGENT_MUTT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 1EC5FC43381 for ; Mon, 11 Mar 2019 14:08:36 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id C81FE20657 for ; Mon, 11 Mar 2019 14:08:35 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="signature verification failed" (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="DWCCku4u" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727696AbfCKOIe (ORCPT ); Mon, 11 Mar 2019 10:08:34 -0400 Received: from mail-ot1-f67.google.com ([209.85.210.67]:34520 "EHLO mail-ot1-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727444AbfCKOIe (ORCPT ); Mon, 11 Mar 2019 10:08:34 -0400 Received: by mail-ot1-f67.google.com with SMTP id r19so526896otn.1 for ; Mon, 11 Mar 2019 07:08:33 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=sender:date:from:to:cc:subject:message-id:reply-to:references :mime-version:content-disposition:in-reply-to:user-agent; bh=e+iJq1fm5brP4Mz0sv1AjbZytSLNxjS8DbEH7+dp1Zc=; b=DWCCku4uI9YasFZhdf4zUmiJoMlCX132Fo6QAesRsRc2dh7oXUmbfcX2YXHJ6xIvpQ JPpm89DYcEZRA3gc+WcF9gn49U1ezpzA4iNV0NwO/03TmOl/4IPZa/N9+A5RKREEP0c5 5SzXp/ZRan/avvGQY4nQCAqmEyoekA05ibyejAlwsRynViPl8zdJo914nKZzgWO6R8Sn XbBYbhr619HRbQXQQRjzFywMZBROQKD7r+N7F1bU1s9CewUHAsFtpV26Mwm0YcncFIpT OBTXQP8aeFL4plhXKb47rYpWtXu/1fnIcU9FntZzOLwDZaQCp59e8mdNxxIKlp6p02zd XMJQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:sender:date:from:to:cc:subject:message-id :reply-to:references:mime-version:content-disposition:in-reply-to :user-agent; bh=e+iJq1fm5brP4Mz0sv1AjbZytSLNxjS8DbEH7+dp1Zc=; b=Zg+0wmOvBo2HTuKYb+x+SeBTOmrQLCzCgxV7I1bi24CVpc6op68NriDt2+BWlEm31p Noso4eLGvuA/FEf6bR9mbPCsNJA3uhK9Px1ymmiPm+LxJcEmeK1PLONlBiwhym8BJ/R5 ULmAdUmKWx9Q2fFb1EDpXTDjE3KldOxDBmU4KhK53UVKOkxaZCwV/vjoZGXRZgXtCcju BD1jXMumWzZNznFCfF+2ajKc+DPy5i8mR6ufHpGcskujonuD3BACjr7oz6gqLBEuW0uy L8InMTK8Tq4mDsg7ox1TMtDsL4BOzrrjq2/QinYWb2SDwgAUMkK6V5miL2UL3VN00TY3 WgBw== X-Gm-Message-State: APjAAAVJkQcnQ9L5GdbErSiImJwbC4MT8BMJH99aEJGQAmvbz+NNtjfW ElJMAzQ6dJJ6cv8lFhrmV+jWbbg= X-Google-Smtp-Source: APXvYqxlNEAG5I7KKJ7ufFrlI2pnYFsdH1Ibwj5x0YeALbVAvKVlhTWoquaJdd2hrBWkR7/OhUbnCQ== X-Received: by 2002:a9d:64c1:: with SMTP id n1mr20244296otl.187.1552313312300; Mon, 11 Mar 2019 07:08:32 -0700 (PDT) Received: from serve.minyard.net (serve.minyard.net. [2001:470:b8f6:1b::1]) by smtp.gmail.com with ESMTPSA id h138sm2449428oic.8.2019.03.11.07.08.31 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Mon, 11 Mar 2019 07:08:31 -0700 (PDT) Received: from minyard.net (unknown [IPv6:2001:470:b8f6:1b:dd3c:1d89:d3d4:f9e3]) by serve.minyard.net (Postfix) with ESMTPSA id 2462E180001; Mon, 11 Mar 2019 14:08:31 +0000 (UTC) Date: Mon, 11 Mar 2019 09:08:29 -0500 From: Corey Minyard To: "Enrico Weigelt, metux IT consult" Cc: linux-kernel@vger.kernel.org Subject: Re: [PATCH 079/114] drivers: char: Kconfig: pedantic formatting Message-ID: <20190311140829.GE3498@minyard.net> Reply-To: minyard@acm.org References: <1552310346-7629-1-git-send-email-info@metux.net> <1552310346-7629-80-git-send-email-info@metux.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1552310346-7629-80-git-send-email-info@metux.net> User-Agent: Mutt/1.9.4 (2018-02-28) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Mar 11, 2019 at 02:18:31PM +0100, Enrico Weigelt, metux IT consult wrote: > Formatting of Kconfig files doesn't look so pretty, so let the > Great White Handkerchief come around and clean it up. > > Signed-off-by: Enrico Weigelt, metux IT consult IPMI portions look ok to me. Acked-by: Corey Minyard > --- > drivers/char/Kconfig | 24 +++++----- > drivers/char/agp/Kconfig | 13 +++--- > drivers/char/hw_random/Kconfig | 24 +++++----- > drivers/char/ipmi/Kconfig | 99 +++++++++++++++++++++--------------------- > 4 files changed, 80 insertions(+), 80 deletions(-) > > diff --git a/drivers/char/Kconfig b/drivers/char/Kconfig > index 72866a0..ab33583 100644 > --- a/drivers/char/Kconfig > +++ b/drivers/char/Kconfig > @@ -35,18 +35,18 @@ config SGI_SNSC > say Y. Otherwise, say N. > > config SGI_TIOCX > - bool "SGI TIO CX driver support" > - depends on (IA64_SGI_SN2 || IA64_GENERIC) > - help > - If you have an SGI Altix and you have fpga devices attached > - to your TIO, say Y here, otherwise say N. > + bool "SGI TIO CX driver support" > + depends on (IA64_SGI_SN2 || IA64_GENERIC) > + help > + If you have an SGI Altix and you have fpga devices attached > + to your TIO, say Y here, otherwise say N. > > config SGI_MBCS > - tristate "SGI FPGA Core Services driver support" > - depends on SGI_TIOCX > - help > - If you have an SGI Altix with an attached SABrick > - say Y or M here, otherwise say N. > + tristate "SGI FPGA Core Services driver support" > + depends on SGI_TIOCX > + help > + If you have an SGI Altix with an attached SABrick > + say Y or M here, otherwise say N. > > source "drivers/tty/serial/Kconfig" > source "drivers/tty/serdev/Kconfig" > @@ -461,8 +461,8 @@ config RAW_DRIVER > Once bound, I/O against /dev/raw/rawN uses efficient zero-copy I/O. > See the raw(8) manpage for more details. > > - Applications should preferably open the device (eg /dev/hda1) > - with the O_DIRECT flag. > + Applications should preferably open the device (eg /dev/hda1) > + with the O_DIRECT flag. > > config MAX_RAW_DEVS > int "Maximum number of RAW devices to support (1-65536)" > diff --git a/drivers/char/agp/Kconfig b/drivers/char/agp/Kconfig > index 6231714..6da28d5 100644 > --- a/drivers/char/agp/Kconfig > +++ b/drivers/char/agp/Kconfig > @@ -63,7 +63,7 @@ config AGP_AMD64 > This option gives you AGP support for the GLX component of > X using the on-CPU northbridge of the AMD Athlon64/Opteron CPUs. > You still need an external AGP bridge like the AMD 8151, VIA > - K8T400M, SiS755. It may also support other AGP bridges when loaded > + K8T400M, SiS755. It may also support other AGP bridges when loaded > with agp_try_unsupported=1. > > config AGP_INTEL > @@ -151,13 +151,12 @@ config AGP_EFFICEON > series processors with integrated northbridges. > > config AGP_SGI_TIOCA > - tristate "SGI TIO chipset AGP support" > - depends on AGP && (IA64_SGI_SN2 || IA64_GENERIC) > - help > - This option gives you AGP GART support for the SGI TIO chipset > - for IA64 processors. > + tristate "SGI TIO chipset AGP support" > + depends on AGP && (IA64_SGI_SN2 || IA64_GENERIC) > + help > + This option gives you AGP GART support for the SGI TIO chipset > + for IA64 processors. > > config INTEL_GTT > tristate > depends on X86 && PCI > - > diff --git a/drivers/char/hw_random/Kconfig b/drivers/char/hw_random/Kconfig > index 25a7d8f..6265931 100644 > --- a/drivers/char/hw_random/Kconfig > +++ b/drivers/char/hw_random/Kconfig > @@ -156,15 +156,15 @@ config HW_RANDOM_OMAP > tristate "OMAP Random Number Generator support" > depends on ARCH_OMAP16XX || ARCH_OMAP2PLUS || ARCH_MVEBU > default HW_RANDOM > - ---help--- > - This driver provides kernel-side support for the Random Number > + ---help--- > + This driver provides kernel-side support for the Random Number > Generator hardware found on OMAP16xx, OMAP2/3/4/5, AM33xx/AM43xx > multimedia processors, and Marvell Armada 7k/8k SoCs. > > To compile this driver as a module, choose M here: the > module will be called omap-rng. > > - If unsure, say Y. > + If unsure, say Y. > > config HW_RANDOM_OMAP3_ROM > tristate "OMAP3 ROM Random Number Generator support" > @@ -372,17 +372,17 @@ config HW_RANDOM_MESON > If unsure, say Y. > > config HW_RANDOM_CAVIUM > - tristate "Cavium ThunderX Random Number Generator support" > - depends on HW_RANDOM && PCI && (ARM64 || (COMPILE_TEST && 64BIT)) > - default HW_RANDOM > - ---help--- > - This driver provides kernel-side support for the Random Number > - Generator hardware found on Cavium SoCs. > + tristate "Cavium ThunderX Random Number Generator support" > + depends on HW_RANDOM && PCI && (ARM64 || (COMPILE_TEST && 64BIT)) > + default HW_RANDOM > + ---help--- > + This driver provides kernel-side support for the Random Number > + Generator hardware found on Cavium SoCs. > > - To compile this driver as a module, choose M here: the > - module will be called cavium_rng. > + To compile this driver as a module, choose M here: the > + module will be called cavium_rng. > > - If unsure, say Y. > + If unsure, say Y. > > config HW_RANDOM_MTK > tristate "Mediatek Random Number Generator support" > diff --git a/drivers/char/ipmi/Kconfig b/drivers/char/ipmi/Kconfig > index 94719fc..b6219c3 100644 > --- a/drivers/char/ipmi/Kconfig > +++ b/drivers/char/ipmi/Kconfig > @@ -1,40 +1,41 @@ > + > # > # IPMI device configuration > # > > menuconfig IPMI_HANDLER > - tristate 'IPMI top-level message handler' > - depends on HAS_IOMEM > - select IPMI_DMI_DECODE if DMI > - help > - This enables the central IPMI message handler, required for IPMI > - to work. > + tristate 'IPMI top-level message handler' > + depends on HAS_IOMEM > + select IPMI_DMI_DECODE if DMI > + help > + This enables the central IPMI message handler, required for IPMI > + to work. > > - IPMI is a standard for managing sensors (temperature, > - voltage, etc.) in a system. > + IPMI is a standard for managing sensors (temperature, > + voltage, etc.) in a system. > > - See for more details on the driver. > + See for more details on the driver. > > - If unsure, say N. > + If unsure, say N. > > config IPMI_DMI_DECODE > - select IPMI_PLAT_DATA > - bool > + select IPMI_PLAT_DATA > + bool > > config IPMI_PLAT_DATA > - bool > + bool > > if IPMI_HANDLER > > config IPMI_PANIC_EVENT > - bool 'Generate a panic event to all BMCs on a panic' > - help > - When a panic occurs, this will cause the IPMI message handler to, > - by default, generate an IPMI event describing the panic to each > - interface registered with the message handler. This is always > - available, the module parameter for ipmi_msghandler named > - panic_op can be set to "event" to chose this value, this config > - simply causes the default value to be set to "event". > + bool 'Generate a panic event to all BMCs on a panic' > + help > + When a panic occurs, this will cause the IPMI message handler to, > + by default, generate an IPMI event describing the panic to each > + interface registered with the message handler. This is always > + available, the module parameter for ipmi_msghandler named > + panic_op can be set to "event" to chose this value, this config > + simply causes the default value to be set to "event". > > config IPMI_PANIC_STRING > bool 'Generate OEM events containing the panic string' > @@ -53,43 +54,43 @@ config IPMI_PANIC_STRING > causes the default value to be set to "string". > > config IPMI_DEVICE_INTERFACE > - tristate 'Device interface for IPMI' > - help > - This provides an IOCTL interface to the IPMI message handler so > - userland processes may use IPMI. It supports poll() and select(). > + tristate 'Device interface for IPMI' > + help > + This provides an IOCTL interface to the IPMI message handler so > + userland processes may use IPMI. It supports poll() and select(). > > config IPMI_SI > - tristate 'IPMI System Interface handler' > - select IPMI_PLAT_DATA > - help > - Provides a driver for System Interfaces (KCS, SMIC, BT). > - Currently, only KCS and SMIC are supported. If > - you are using IPMI, you should probably say "y" here. > + tristate 'IPMI System Interface handler' > + select IPMI_PLAT_DATA > + help > + Provides a driver for System Interfaces (KCS, SMIC, BT). > + Currently, only KCS and SMIC are supported. If > + you are using IPMI, you should probably say "y" here. > > config IPMI_SSIF > - tristate 'IPMI SMBus handler (SSIF)' > - select I2C > - help > - Provides a driver for a SMBus interface to a BMC, meaning that you > - have a driver that must be accessed over an I2C bus instead of a > - standard interface. This module requires I2C support. > + tristate 'IPMI SMBus handler (SSIF)' > + select I2C > + help > + Provides a driver for a SMBus interface to a BMC, meaning that you > + have a driver that must be accessed over an I2C bus instead of a > + standard interface. This module requires I2C support. > > config IPMI_POWERNV > - depends on PPC_POWERNV > - tristate 'POWERNV (OPAL firmware) IPMI interface' > - help > - Provides a driver for OPAL firmware-based IPMI interfaces. > + depends on PPC_POWERNV > + tristate 'POWERNV (OPAL firmware) IPMI interface' > + help > + Provides a driver for OPAL firmware-based IPMI interfaces. > > config IPMI_WATCHDOG > - tristate 'IPMI Watchdog Timer' > - help > - This enables the IPMI watchdog timer. > + tristate 'IPMI Watchdog Timer' > + help > + This enables the IPMI watchdog timer. > > config IPMI_POWEROFF > - tristate 'IPMI Poweroff' > - help > - This enables a function to power off the system with IPMI if > - the IPMI management controller is capable of this. > + tristate 'IPMI Poweroff' > + help > + This enables a function to power off the system with IPMI if > + the IPMI management controller is capable of this. > > endif # IPMI_HANDLER > > @@ -125,7 +126,7 @@ config NPCM7XX_KCS_IPMI_BMC > > config ASPEED_BT_IPMI_BMC > depends on ARCH_ASPEED || COMPILE_TEST > - depends on REGMAP && REGMAP_MMIO && MFD_SYSCON > + depends on REGMAP && REGMAP_MMIO && MFD_SYSCON > tristate "BT IPMI bmc driver" > help > Provides a driver for the BT (Block Transfer) IPMI interface > -- > 1.9.1 >