* [PATCH] soc: aspeed: Improve kconfig
@ 2020-09-16 12:57 Joel Stanley
2020-09-16 16:34 ` Cédric Le Goater
2020-09-16 23:18 ` Andrew Jeffery
0 siblings, 2 replies; 3+ messages in thread
From: Joel Stanley @ 2020-09-16 12:57 UTC (permalink / raw)
To: Cédric Le Goater, Arnd Bergmann, Andrew Jeffery
Cc: linux-aspeed, linux-arm-kernel
Reword the kconfig text to be consistent and reflect that most drivers
are available for all supported ASPEED chips (2400, 2500 and 2600).
Rearrange the symbols the SoC drivers depend on so the menu doesn't
appear unless you are building for ASPEED for compile testing.
The SYSCON_MFD and REGMAP options are usually selected by drivers that
need them, so do this.
Signed-off-by: Joel Stanley <joel@jms.id.au>
---
drivers/soc/aspeed/Kconfig | 36 ++++++++++++++++++++----------------
1 file changed, 20 insertions(+), 16 deletions(-)
diff --git a/drivers/soc/aspeed/Kconfig b/drivers/soc/aspeed/Kconfig
index c95fa30f1a76..9ce252445605 100644
--- a/drivers/soc/aspeed/Kconfig
+++ b/drivers/soc/aspeed/Kconfig
@@ -1,32 +1,36 @@
# SPDX-License-Identifier: GPL-2.0-only
-menu "Aspeed SoC drivers"
-config SOC_ASPEED
- def_bool y
- depends on ARCH_ASPEED || COMPILE_TEST
+if ARCH_ASPEED || COMPILE_TEST
+
+menu "ASPEED SoC drivers"
config ASPEED_LPC_CTRL
- depends on SOC_ASPEED && REGMAP && MFD_SYSCON
- tristate "Aspeed ast2400/2500 HOST LPC to BMC bridge control"
+ tristate "ASPEED LPC firmware cycle control"
+ select REGMAP
+ select MFD_SYSCON
help
- Control Aspeed ast2400/2500 HOST LPC to BMC mappings through
- ioctl()s, the driver also provides a read/write interface to a BMC ram
- region where the host LPC read/write region can be buffered.
+ Control LPC firmware cycle mappings through ioctl()s. The driver
+ also provides a read/write interface to a BMC ram region where the
+ host LPC read/write region can be buffered.
config ASPEED_LPC_SNOOP
- tristate "Aspeed ast2500 HOST LPC snoop support"
- depends on SOC_ASPEED && REGMAP && MFD_SYSCON
+ tristate "ASPEED LPC snoop support"
+ select REGMAP
+ select MFD_SYSCON
help
Provides a driver to control the LPC snoop interface which
allows the BMC to listen on and save the data written by
the host to an arbitrary LPC I/O port.
config ASPEED_P2A_CTRL
- depends on SOC_ASPEED && REGMAP && MFD_SYSCON
- tristate "Aspeed ast2400/2500 HOST P2A VGA MMIO to BMC bridge control"
+ tristate "ASPEED P2A (VGA MMIO to BMC) bridge control"
+ select REGMAP
+ select MFD_SYSCON
help
- Control Aspeed ast2400/2500 HOST P2A VGA MMIO to BMC mappings through
- ioctl()s, the driver also provides an interface for userspace mappings to
- a pre-defined region.
+ Control ASPEED P2A VGA MMIO to BMC mappings through ioctl()s. The
+ driver also provides an interface for userspace mappings to a
+ pre-defined region.
endmenu
+
+endif
--
2.28.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] soc: aspeed: Improve kconfig
2020-09-16 12:57 [PATCH] soc: aspeed: Improve kconfig Joel Stanley
@ 2020-09-16 16:34 ` Cédric Le Goater
2020-09-16 23:18 ` Andrew Jeffery
1 sibling, 0 replies; 3+ messages in thread
From: Cédric Le Goater @ 2020-09-16 16:34 UTC (permalink / raw)
To: Joel Stanley, Arnd Bergmann, Andrew Jeffery
Cc: linux-aspeed, linux-arm-kernel
On 9/16/20 2:57 PM, Joel Stanley wrote:
> Reword the kconfig text to be consistent and reflect that most drivers
> are available for all supported ASPEED chips (2400, 2500 and 2600).
>
> Rearrange the symbols the SoC drivers depend on so the menu doesn't
> appear unless you are building for ASPEED for compile testing.
>
> The SYSCON_MFD and REGMAP options are usually selected by drivers that
> need them, so do this.
>
> Signed-off-by: Joel Stanley <joel@jms.id.au>
Reviewed-by: Cédric Le Goater <clg@kaod.org>
> ---
> drivers/soc/aspeed/Kconfig | 36 ++++++++++++++++++++----------------
> 1 file changed, 20 insertions(+), 16 deletions(-)
>
> diff --git a/drivers/soc/aspeed/Kconfig b/drivers/soc/aspeed/Kconfig
> index c95fa30f1a76..9ce252445605 100644
> --- a/drivers/soc/aspeed/Kconfig
> +++ b/drivers/soc/aspeed/Kconfig
> @@ -1,32 +1,36 @@
> # SPDX-License-Identifier: GPL-2.0-only
> -menu "Aspeed SoC drivers"
>
> -config SOC_ASPEED
> - def_bool y
> - depends on ARCH_ASPEED || COMPILE_TEST
> +if ARCH_ASPEED || COMPILE_TEST
> +
> +menu "ASPEED SoC drivers"
>
> config ASPEED_LPC_CTRL
> - depends on SOC_ASPEED && REGMAP && MFD_SYSCON
> - tristate "Aspeed ast2400/2500 HOST LPC to BMC bridge control"
> + tristate "ASPEED LPC firmware cycle control"
> + select REGMAP
> + select MFD_SYSCON
> help
> - Control Aspeed ast2400/2500 HOST LPC to BMC mappings through
> - ioctl()s, the driver also provides a read/write interface to a BMC ram
> - region where the host LPC read/write region can be buffered.
> + Control LPC firmware cycle mappings through ioctl()s. The driver
> + also provides a read/write interface to a BMC ram region where the
> + host LPC read/write region can be buffered.
>
> config ASPEED_LPC_SNOOP
> - tristate "Aspeed ast2500 HOST LPC snoop support"
> - depends on SOC_ASPEED && REGMAP && MFD_SYSCON
> + tristate "ASPEED LPC snoop support"
> + select REGMAP
> + select MFD_SYSCON
> help
> Provides a driver to control the LPC snoop interface which
> allows the BMC to listen on and save the data written by
> the host to an arbitrary LPC I/O port.
>
> config ASPEED_P2A_CTRL
> - depends on SOC_ASPEED && REGMAP && MFD_SYSCON
> - tristate "Aspeed ast2400/2500 HOST P2A VGA MMIO to BMC bridge control"
> + tristate "ASPEED P2A (VGA MMIO to BMC) bridge control"
> + select REGMAP
> + select MFD_SYSCON
> help
> - Control Aspeed ast2400/2500 HOST P2A VGA MMIO to BMC mappings through
> - ioctl()s, the driver also provides an interface for userspace mappings to
> - a pre-defined region.
> + Control ASPEED P2A VGA MMIO to BMC mappings through ioctl()s. The
> + driver also provides an interface for userspace mappings to a
> + pre-defined region.
>
> endmenu
> +
> +endif
>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] soc: aspeed: Improve kconfig
2020-09-16 12:57 [PATCH] soc: aspeed: Improve kconfig Joel Stanley
2020-09-16 16:34 ` Cédric Le Goater
@ 2020-09-16 23:18 ` Andrew Jeffery
1 sibling, 0 replies; 3+ messages in thread
From: Andrew Jeffery @ 2020-09-16 23:18 UTC (permalink / raw)
To: Joel Stanley, Cédric Le Goater, Arnd Bergmann
Cc: linux-aspeed, linux-arm-kernel
On Wed, 16 Sep 2020, at 22:27, Joel Stanley wrote:
> Reword the kconfig text to be consistent and reflect that most drivers
> are available for all supported ASPEED chips (2400, 2500 and 2600).
>
> Rearrange the symbols the SoC drivers depend on so the menu doesn't
> appear unless you are building for ASPEED for compile testing.
>
> The SYSCON_MFD and REGMAP options are usually selected by drivers that
> need them, so do this.
>
> Signed-off-by: Joel Stanley <joel@jms.id.au>
Reviewed-by: Andrew Jeffery <andrew@aj.id.au>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-09-16 23:20 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-09-16 12:57 [PATCH] soc: aspeed: Improve kconfig Joel Stanley
2020-09-16 16:34 ` Cédric Le Goater
2020-09-16 23:18 ` Andrew Jeffery
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).