Re: [PATCH] PCI: endpoint: fix Kconfig indent style

From: Bjorn Helgaas <helgaas@kernel.org>
To: Shunsuke Mie <mie@igel.co.jp>
Cc: "Jon Mason" <jdmason@kudzu.us>,
	"Kishon Vijay Abraham I" <kishon@ti.com>,
	"Lorenzo Pieralisi" <lpieralisi@kernel.org>,
	"Krzysztof Wilczyński" <kw@linux.com>,
	"Bjorn Helgaas" <bhelgaas@google.com>,
	"Frank Li" <Frank.Li@nxp.com>,
	"Randy Dunlap" <rdunlap@infradead.org>,
	"Ren Zhijie" <renzhijie2@huawei.com>,
	linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] PCI: endpoint: fix Kconfig indent style
Date: Mon, 15 Aug 2022 13:41:42 -0500	[thread overview]
Message-ID: <20220815184142.GA1960338@bhelgaas> (raw)
In-Reply-To: <20220815025006.48167-1-mie@igel.co.jp>

On Mon, Aug 15, 2022 at 11:50:06AM +0900, Shunsuke Mie wrote:
> Change to follow the Kconfig style guide. This patch fixes to use tab
> rather than space to indent, while help text is indented an additional
> two spaces.
> 
> Fixes: e35f56bb0330 ("PCI: endpoint: Support NTB transfer between RC and EP")
> Signed-off-by: Shunsuke Mie <mie@igel.co.jp>

Thanks for doing this.  In the future make the subject line match the
history, e.g.,

  $ git log --oneline drivers/pci/endpoint/functions/Kconfig
  556a2c7dca33 ("PCI: endpoint: Fix Kconfig dependency")
  e35f56bb0330 ("PCI: endpoint: Support NTB transfer between RC and EP")
  8b821cf76150 ("PCI: endpoint: Add EP function driver to provide NTB functionality")
  98dbf5af4fdd ("PCI: endpoint: Select CRC32 to fix test build error")
  349e7a85b25f ("PCI: endpoint: functions: Add an EP function to test PCI")

Note that these are all capitalized ("Fix Kconfig indent style").

Lorenzo will likely fix this up when applying, so no need to repost
for this.

> ---
>  drivers/pci/endpoint/functions/Kconfig | 18 +++++++++---------
>  1 file changed, 9 insertions(+), 9 deletions(-)
> 
> diff --git a/drivers/pci/endpoint/functions/Kconfig b/drivers/pci/endpoint/functions/Kconfig
> index 295a033ee9a2..9fd560886871 100644
> --- a/drivers/pci/endpoint/functions/Kconfig
> +++ b/drivers/pci/endpoint/functions/Kconfig
> @@ -27,13 +27,13 @@ config PCI_EPF_NTB
>  	  If in doubt, say "N" to disable Endpoint NTB driver.
>  
>  config PCI_EPF_VNTB
> -        tristate "PCI Endpoint NTB driver"
> -        depends on PCI_ENDPOINT
> -        depends on NTB
> -        select CONFIGFS_FS
> -        help
> -          Select this configuration option to enable the Non-Transparent
> -          Bridge (NTB) driver for PCIe Endpoint. NTB driver implements NTB
> -          between PCI Root Port and PCIe Endpoint.
> +	tristate "PCI Endpoint NTB driver"
> +	depends on PCI_ENDPOINT
> +	depends on NTB
> +	select CONFIGFS_FS
> +	help
> +	  Select this configuration option to enable the Non-Transparent
> +	  Bridge (NTB) driver for PCIe Endpoint. NTB driver implements NTB
> +	  between PCI Root Port and PCIe Endpoint.
>  
> -          If in doubt, say "N" to disable Endpoint NTB driver.
> +	  If in doubt, say "N" to disable Endpoint NTB driver.
> -- 
> 2.17.1
> 