* [PATCH] spi: Remove CONFIG_ prefix from Kconfig select
@ 2020-03-05 15:15 Joe Perches
[not found] ` <f8ac6b32a29b9a05b58a7e58ffe8b780642abbf1.camel-6d6DIl74uiNBDgjK7y7TUQ@public.gmane.org>
0 siblings, 1 reply; 7+ messages in thread
From: Joe Perches @ 2020-03-05 15:15 UTC (permalink / raw)
To: John Garry; +Cc: Mark Brown, linux-spi, LKML
commit a2ca53b52e00 ("spi: Add HiSilicon v3xx SPI NOR flash
controller driver") likely inadvertently used a select statement
with a CONFIG_ prefix, remove the prefix.
Signed-off-by: Joe Perches <joe@perches.com>
---
drivers/spi/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/spi/Kconfig b/drivers/spi/Kconfig
index 82177d..2dc7bd 100644
--- a/drivers/spi/Kconfig
+++ b/drivers/spi/Kconfig
@@ -292,7 +292,7 @@ config SPI_HISI_SFC_V3XX
tristate "HiSilicon SPI-NOR Flash Controller for Hi16XX chipsets"
depends on (ARM64 && ACPI) || COMPILE_TEST
depends on HAS_IOMEM
- select CONFIG_MTD_SPI_NOR
+ select MTD_SPI_NOR
help
This enables support for HiSilicon v3xx SPI-NOR flash controller
found in hi16xx chipsets.
^ permalink raw reply [flat|nested] 7+ messages in thread[parent not found: <f8ac6b32a29b9a05b58a7e58ffe8b780642abbf1.camel-6d6DIl74uiNBDgjK7y7TUQ@public.gmane.org>]
* Re: [PATCH] spi: Remove CONFIG_ prefix from Kconfig select [not found] ` <f8ac6b32a29b9a05b58a7e58ffe8b780642abbf1.camel-6d6DIl74uiNBDgjK7y7TUQ@public.gmane.org> @ 2020-03-05 15:53 ` John Garry [not found] ` <e265e7df-fd8c-691f-389a-c0e0e377ed9e-hv44wF8Li93QT0dZR+AlfA@public.gmane.org> 2020-03-06 15:19 ` Applied "spi: Remove CONFIG_ prefix from Kconfig select" to the spi tree Mark Brown 1 sibling, 1 reply; 7+ messages in thread From: John Garry @ 2020-03-05 15:53 UTC (permalink / raw) To: Joe Perches; +Cc: Mark Brown, linux-spi, LKML On 05/03/2020 15:15, Joe Perches wrote: > commit a2ca53b52e00 ("spi: Add HiSilicon v3xx SPI NOR flash > controller driver") likely inadvertently used a select statement > with a CONFIG_ prefix, remove the prefix. > > Signed-off-by: Joe Perches <joe-6d6DIl74uiNBDgjK7y7TUQ@public.gmane.org> It's a pity checkpatch can't pick this stuff up... The select is just for enabling the driver of a slave driver, so should not cause any build issues when not enabled. thanks FWIW, Acked-by: John Garry <john.garry-hv44wF8Li93QT0dZR+AlfA@public.gmane.org> > --- > drivers/spi/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/spi/Kconfig b/drivers/spi/Kconfig > index 82177d..2dc7bd 100644 > --- a/drivers/spi/Kconfig > +++ b/drivers/spi/Kconfig > @@ -292,7 +292,7 @@ config SPI_HISI_SFC_V3XX > tristate "HiSilicon SPI-NOR Flash Controller for Hi16XX chipsets" > depends on (ARM64 && ACPI) || COMPILE_TEST > depends on HAS_IOMEM > - select CONFIG_MTD_SPI_NOR > + select MTD_SPI_NOR > help > This enables support for HiSilicon v3xx SPI-NOR flash controller > found in hi16xx chipsets. > > > . > ^ permalink raw reply [flat|nested] 7+ messages in thread
[parent not found: <e265e7df-fd8c-691f-389a-c0e0e377ed9e-hv44wF8Li93QT0dZR+AlfA@public.gmane.org>]
* Re: [PATCH] spi: Remove CONFIG_ prefix from Kconfig select [not found] ` <e265e7df-fd8c-691f-389a-c0e0e377ed9e-hv44wF8Li93QT0dZR+AlfA@public.gmane.org> @ 2020-03-05 16:01 ` Joe Perches [not found] ` <84f8504924380fc6380216d03d28a2285e3d106e.camel-6d6DIl74uiNBDgjK7y7TUQ@public.gmane.org> 0 siblings, 1 reply; 7+ messages in thread From: Joe Perches @ 2020-03-05 16:01 UTC (permalink / raw) To: John Garry; +Cc: Mark Brown, linux-spi, LKML On Thu, 2020-03-05 at 15:53 +0000, John Garry wrote: > On 05/03/2020 15:15, Joe Perches wrote: > > commit a2ca53b52e00 ("spi: Add HiSilicon v3xx SPI NOR flash > > controller driver") likely inadvertently used a select statement > > with a CONFIG_ prefix, remove the prefix. > > > > Signed-off-by: Joe Perches <joe-6d6DIl74uiNBDgjK7y7TUQ@public.gmane.org> > > It's a pity checkpatch can't pick this stuff up... You are free to write a rule and submit a patch. The grammar is pretty hard to verify from a patch fragment though. There are 2 existing nominal false positives. $ git grep 'select\b.*\bCONFIG_' -- '*/Kconfig*' drivers/edac/Kconfig: select CONFIG_ACPI_NFIT. drivers/edac/Kconfig: select CONFIG_ACPI_NFIT. ^ permalink raw reply [flat|nested] 7+ messages in thread
[parent not found: <84f8504924380fc6380216d03d28a2285e3d106e.camel-6d6DIl74uiNBDgjK7y7TUQ@public.gmane.org>]
* Re: [PATCH] spi: Remove CONFIG_ prefix from Kconfig select [not found] ` <84f8504924380fc6380216d03d28a2285e3d106e.camel-6d6DIl74uiNBDgjK7y7TUQ@public.gmane.org> @ 2020-03-05 16:20 ` John Garry [not found] ` <55edba5c-78d6-12e2-eff6-f16c204badee-hv44wF8Li93QT0dZR+AlfA@public.gmane.org> 0 siblings, 1 reply; 7+ messages in thread From: John Garry @ 2020-03-05 16:20 UTC (permalink / raw) To: Joe Perches; +Cc: Mark Brown, linux-spi, LKML, Linuxarm On 05/03/2020 16:01, Joe Perches wrote: > On Thu, 2020-03-05 at 15:53 +0000, John Garry wrote: >> On 05/03/2020 15:15, Joe Perches wrote: >>> commit a2ca53b52e00 ("spi: Add HiSilicon v3xx SPI NOR flash >>> controller driver") likely inadvertently used a select statement >>> with a CONFIG_ prefix, remove the prefix. >>> >>> Signed-off-by: Joe Perches <joe-6d6DIl74uiNBDgjK7y7TUQ@public.gmane.org> >> >> It's a pity checkpatch can't pick this stuff up... > > You are free to write a rule and submit a patch. > I'm not opposed to the idea... > The grammar is pretty hard to verify from a patch > fragment though. > > There are 2 existing nominal false positives. > > $ git grep 'select\b.*\bCONFIG_' -- '*/Kconfig*' > drivers/edac/Kconfig: select CONFIG_ACPI_NFIT. > drivers/edac/Kconfig: select CONFIG_ACPI_NFIT. Ah, that's in the help text. I guess that those can be ignored simply based on the indentation. I also see these: drivers/i2c/busses/Kconfig: to also select CONFIG_TYPEC_FUSB302=m. drivers/platform/chrome/Kconfig: select CONFIG_MFD_CROS_EC_DEV The 2nd looks like it's incorrect. And then also: $ git grep 'depends on\b.*\bCONFIG_' -- '*/Kconfig*' samples/Kconfig: depends on CONFIG_ANDROID_BINDERFS Thanks, John ^ permalink raw reply [flat|nested] 7+ messages in thread
[parent not found: <55edba5c-78d6-12e2-eff6-f16c204badee-hv44wF8Li93QT0dZR+AlfA@public.gmane.org>]
* Re: [PATCH] spi: Remove CONFIG_ prefix from Kconfig select [not found] ` <55edba5c-78d6-12e2-eff6-f16c204badee-hv44wF8Li93QT0dZR+AlfA@public.gmane.org> @ 2020-03-05 16:21 ` Randy Dunlap [not found] ` <15ad3c68-4e67-0c92-a0e1-c62861673561-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org> 0 siblings, 1 reply; 7+ messages in thread From: Randy Dunlap @ 2020-03-05 16:21 UTC (permalink / raw) To: John Garry, Joe Perches; +Cc: Mark Brown, linux-spi, LKML, Linuxarm On 3/5/20 8:20 AM, John Garry wrote: > On 05/03/2020 16:01, Joe Perches wrote: >> On Thu, 2020-03-05 at 15:53 +0000, John Garry wrote: >>> On 05/03/2020 15:15, Joe Perches wrote: >>>> commit a2ca53b52e00 ("spi: Add HiSilicon v3xx SPI NOR flash >>>> controller driver") likely inadvertently used a select statement >>>> with a CONFIG_ prefix, remove the prefix. >>>> >>>> Signed-off-by: Joe Perches <joe-6d6DIl74uiNBDgjK7y7TUQ@public.gmane.org> >>> >>> It's a pity checkpatch can't pick this stuff up... >> >> You are free to write a rule and submit a patch. >> > > I'm not opposed to the idea... > >> The grammar is pretty hard to verify from a patch >> fragment though. >> >> There are 2 existing nominal false positives. >> >> $ git grep 'select\b.*\bCONFIG_' -- '*/Kconfig*' >> drivers/edac/Kconfig: select CONFIG_ACPI_NFIT. >> drivers/edac/Kconfig: select CONFIG_ACPI_NFIT. > > Ah, that's in the help text. I guess that those can be ignored simply based on the indentation. > > I also see these: > > drivers/i2c/busses/Kconfig: to also select CONFIG_TYPEC_FUSB302=m. > drivers/platform/chrome/Kconfig: select CONFIG_MFD_CROS_EC_DEV > > The 2nd looks like it's incorrect. Yes, we just got a patch for that one today. > > And then also: > $ git grep 'depends on\b.*\bCONFIG_' -- '*/Kconfig*' > samples/Kconfig: depends on CONFIG_ANDROID_BINDERFS > > Thanks, > John -- ~Randy Reported-by: Randy Dunlap <rdunlap-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org> ^ permalink raw reply [flat|nested] 7+ messages in thread
[parent not found: <15ad3c68-4e67-0c92-a0e1-c62861673561-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org>]
* Re: [PATCH] spi: Remove CONFIG_ prefix from Kconfig select [not found] ` <15ad3c68-4e67-0c92-a0e1-c62861673561-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org> @ 2020-03-05 16:38 ` Joe Perches 0 siblings, 0 replies; 7+ messages in thread From: Joe Perches @ 2020-03-05 16:38 UTC (permalink / raw) To: Randy Dunlap, John Garry; +Cc: Mark Brown, linux-spi, LKML, Linuxarm On Thu, 2020-03-05 at 08:21 -0800, Randy Dunlap wrote: > On 3/5/20 8:20 AM, John Garry wrote: > > On 05/03/2020 16:01, Joe Perches wrote: > > > On Thu, 2020-03-05 at 15:53 +0000, John Garry wrote: > > > > On 05/03/2020 15:15, Joe Perches wrote: > > > > > commit a2ca53b52e00 ("spi: Add HiSilicon v3xx SPI NOR flash > > > > > controller driver") likely inadvertently used a select statement > > > > > with a CONFIG_ prefix, remove the prefix. > > > > > > > > > > Signed-off-by: Joe Perches <joe-6d6DIl74uiNBDgjK7y7TUQ@public.gmane.org> > > > > > > > > It's a pity checkpatch can't pick this stuff up... > > > > > > You are free to write a rule and submit a patch. > > > > > > > I'm not opposed to the idea... It'd be better I think to update the grammar verification in the tool that parses these files. see: scripts/kconfig/... > > > The grammar is pretty hard to verify from a patch > > > fragment though. > > > > > > There are 2 existing nominal false positives. > > > > > > $ git grep 'select\b.*\bCONFIG_' -- '*/Kconfig*' > > > drivers/edac/Kconfig: select CONFIG_ACPI_NFIT. > > > drivers/edac/Kconfig: select CONFIG_ACPI_NFIT. > > > > Ah, that's in the help text. I guess that those can be ignored simply based on the indentation. The grammar is per line whitespace independent, so not necessarily. > > drivers/platform/chrome/Kconfig: select CONFIG_MFD_CROS_EC_DEV > > > > The 2nd looks like it's incorrect. > > Yes, we just got a patch for that one today. That's the reason I submitted this one. ^ permalink raw reply [flat|nested] 7+ messages in thread
* Applied "spi: Remove CONFIG_ prefix from Kconfig select" to the spi tree [not found] ` <f8ac6b32a29b9a05b58a7e58ffe8b780642abbf1.camel-6d6DIl74uiNBDgjK7y7TUQ@public.gmane.org> 2020-03-05 15:53 ` John Garry @ 2020-03-06 15:19 ` Mark Brown 1 sibling, 0 replies; 7+ messages in thread From: Mark Brown @ 2020-03-06 15:19 UTC (permalink / raw) To: Joe Perches Cc: John Garry, linux-spi, linux-spi-u79uwXL29TY76Z2rM5mHXA, LKML, Mark Brown, Randy Dunlap The patch spi: Remove CONFIG_ prefix from Kconfig select has been applied to the spi tree at https://git.kernel.org/pub/scm/linux/kernel/git/broonie/spi.git All being well this means that it will be integrated into the linux-next tree (usually sometime in the next 24 hours) and sent to Linus during the next merge window (or sooner if it is a bug fix), however if problems are discovered then the patch may be dropped or reverted. You may get further e-mails resulting from automated or manual testing and review of the tree, please engage with people reporting problems and send followup patches addressing any issues that are reported if needed. If any updates are required or you are submitting further changes they should be sent as incremental updates against current git, existing patches will not be replaced. Please add any relevant lists and maintainers to the CCs when replying to this mail. Thanks, Mark >From e14572c52546c16e159c4c1814984843a119e823 Mon Sep 17 00:00:00 2001 From: Joe Perches <joe-6d6DIl74uiNBDgjK7y7TUQ@public.gmane.org> Date: Thu, 5 Mar 2020 07:15:53 -0800 Subject: [PATCH] spi: Remove CONFIG_ prefix from Kconfig select commit a2ca53b52e00 ("spi: Add HiSilicon v3xx SPI NOR flash controller driver") likely inadvertently used a select statement with a CONFIG_ prefix, remove the prefix. Reported-by: Randy Dunlap <rdunlap-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org> Signed-off-by: Joe Perches <joe-6d6DIl74uiNBDgjK7y7TUQ@public.gmane.org> Acked-by: John Garry <john.garry-hv44wF8Li93QT0dZR+AlfA@public.gmane.org> Link: https://lore.kernel.org/r/f8ac6b32a29b9a05b58a7e58ffe8b780642abbf1.camel-6d6DIl74uiNBDgjK7y7TUQ@public.gmane.org Signed-off-by: Mark Brown <broonie-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> --- drivers/spi/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/spi/Kconfig b/drivers/spi/Kconfig index 887fefe87fd5..9c5dadb238dc 100644 --- a/drivers/spi/Kconfig +++ b/drivers/spi/Kconfig @@ -292,7 +292,7 @@ config SPI_HISI_SFC_V3XX tristate "HiSilicon SPI-NOR Flash Controller for Hi16XX chipsets" depends on (ARM64 && ACPI) || COMPILE_TEST depends on HAS_IOMEM - select CONFIG_MTD_SPI_NOR + select MTD_SPI_NOR help This enables support for HiSilicon v3xx SPI-NOR flash controller found in hi16xx chipsets. -- 2.20.1 ^ permalink raw reply related [flat|nested] 7+ messages in thread
end of thread, other threads:[~2020-03-06 15:19 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-03-05 15:15 [PATCH] spi: Remove CONFIG_ prefix from Kconfig select Joe Perches
[not found] ` <f8ac6b32a29b9a05b58a7e58ffe8b780642abbf1.camel-6d6DIl74uiNBDgjK7y7TUQ@public.gmane.org>
2020-03-05 15:53 ` John Garry
[not found] ` <e265e7df-fd8c-691f-389a-c0e0e377ed9e-hv44wF8Li93QT0dZR+AlfA@public.gmane.org>
2020-03-05 16:01 ` Joe Perches
[not found] ` <84f8504924380fc6380216d03d28a2285e3d106e.camel-6d6DIl74uiNBDgjK7y7TUQ@public.gmane.org>
2020-03-05 16:20 ` John Garry
[not found] ` <55edba5c-78d6-12e2-eff6-f16c204badee-hv44wF8Li93QT0dZR+AlfA@public.gmane.org>
2020-03-05 16:21 ` Randy Dunlap
[not found] ` <15ad3c68-4e67-0c92-a0e1-c62861673561-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org>
2020-03-05 16:38 ` Joe Perches
2020-03-06 15:19 ` Applied "spi: Remove CONFIG_ prefix from Kconfig select" to the spi tree Mark Brown
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).