* [PATCH net-next] ptp: ocp: Simplify Kconfig.
@ 2021-08-25 21:17 Jonathan Lemon
2021-08-25 21:32 ` Arnd Bergmann
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Jonathan Lemon @ 2021-08-25 21:17 UTC (permalink / raw)
To: netdev; +Cc: davem, kuba, arnd, rdunlap, richardcochran, kernel-team
Remove the 'imply' statements, these apparently are not doing
what I expected. Platform modules which are used by the driver
still need to be enabled in the overall config for them to be
used, but there isn't a hard dependency on them.
Use 'depend' for selectable modules which provide functions
used directly by the driver.
Signed-off-by: Jonathan Lemon <jonathan.lemon@gmail.com>
---
drivers/ptp/Kconfig | 10 ++--------
1 file changed, 2 insertions(+), 8 deletions(-)
diff --git a/drivers/ptp/Kconfig b/drivers/ptp/Kconfig
index 32660dc11354..f02bedf41264 100644
--- a/drivers/ptp/Kconfig
+++ b/drivers/ptp/Kconfig
@@ -171,16 +171,10 @@ config PTP_1588_CLOCK_OCP
tristate "OpenCompute TimeCard as PTP clock"
depends on PTP_1588_CLOCK
depends on HAS_IOMEM && PCI
- depends on SPI && I2C && MTD
+ depends on I2C && MTD
+ depends on SERIAL_8250
depends on !S390
- imply SPI_MEM
- imply SPI_XILINX
- imply MTD_SPI_NOR
- imply I2C_XILINX
- select SERIAL_8250
select NET_DEVLINK
-
- default n
help
This driver adds support for an OpenCompute time card.
--
2.31.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH net-next] ptp: ocp: Simplify Kconfig.
2021-08-25 21:17 [PATCH net-next] ptp: ocp: Simplify Kconfig Jonathan Lemon
@ 2021-08-25 21:32 ` Arnd Bergmann
2021-08-25 23:22 ` Randy Dunlap
2021-08-26 11:10 ` patchwork-bot+netdevbpf
2 siblings, 0 replies; 4+ messages in thread
From: Arnd Bergmann @ 2021-08-25 21:32 UTC (permalink / raw)
To: Jonathan Lemon
Cc: Networking, David Miller, Jakub Kicinski, Randy Dunlap,
Richard Cochran, kernel-team
On Wed, Aug 25, 2021 at 11:17 PM Jonathan Lemon
<jonathan.lemon@gmail.com> wrote:
>
> Remove the 'imply' statements, these apparently are not doing
> what I expected. Platform modules which are used by the driver
> still need to be enabled in the overall config for them to be
> used, but there isn't a hard dependency on them.
>
> Use 'depend' for selectable modules which provide functions
> used directly by the driver.
>
> Signed-off-by: Jonathan Lemon <jonathan.lemon@gmail.com>
> ---
> drivers/ptp/Kconfig | 10 ++--------
> 1 file changed, 2 insertions(+), 8 deletions(-)
Reviewed-by: Arnd Bergmann <arnd@arndb.de>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH net-next] ptp: ocp: Simplify Kconfig.
2021-08-25 21:17 [PATCH net-next] ptp: ocp: Simplify Kconfig Jonathan Lemon
2021-08-25 21:32 ` Arnd Bergmann
@ 2021-08-25 23:22 ` Randy Dunlap
2021-08-26 11:10 ` patchwork-bot+netdevbpf
2 siblings, 0 replies; 4+ messages in thread
From: Randy Dunlap @ 2021-08-25 23:22 UTC (permalink / raw)
To: Jonathan Lemon, netdev; +Cc: davem, kuba, arnd, richardcochran, kernel-team
On 8/25/21 2:17 PM, Jonathan Lemon wrote:
> Remove the 'imply' statements, these apparently are not doing
> what I expected. Platform modules which are used by the driver
> still need to be enabled in the overall config for them to be
> used, but there isn't a hard dependency on them.
>
> Use 'depend' for selectable modules which provide functions
> used directly by the driver.
>
> Signed-off-by: Jonathan Lemon <jonathan.lemon@gmail.com>
> ---
> drivers/ptp/Kconfig | 10 ++--------
> 1 file changed, 2 insertions(+), 8 deletions(-)
>
> diff --git a/drivers/ptp/Kconfig b/drivers/ptp/Kconfig
> index 32660dc11354..f02bedf41264 100644
> --- a/drivers/ptp/Kconfig
> +++ b/drivers/ptp/Kconfig
> @@ -171,16 +171,10 @@ config PTP_1588_CLOCK_OCP
> tristate "OpenCompute TimeCard as PTP clock"
> depends on PTP_1588_CLOCK
> depends on HAS_IOMEM && PCI
> - depends on SPI && I2C && MTD
> + depends on I2C && MTD
> + depends on SERIAL_8250
> depends on !S390
> - imply SPI_MEM
> - imply SPI_XILINX
> - imply MTD_SPI_NOR
> - imply I2C_XILINX
> - select SERIAL_8250
> select NET_DEVLINK
> -
> - default n
> help
> This driver adds support for an OpenCompute time card.
>
>
Reviewed-by: Randy Dunlap <rdunlap@infradead.org>
Thanks.
--
~Randy
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH net-next] ptp: ocp: Simplify Kconfig.
2021-08-25 21:17 [PATCH net-next] ptp: ocp: Simplify Kconfig Jonathan Lemon
2021-08-25 21:32 ` Arnd Bergmann
2021-08-25 23:22 ` Randy Dunlap
@ 2021-08-26 11:10 ` patchwork-bot+netdevbpf
2 siblings, 0 replies; 4+ messages in thread
From: patchwork-bot+netdevbpf @ 2021-08-26 11:10 UTC (permalink / raw)
To: Jonathan Lemon
Cc: netdev, davem, kuba, arnd, rdunlap, richardcochran, kernel-team
Hello:
This patch was applied to netdev/net-next.git (refs/heads/master):
On Wed, 25 Aug 2021 14:17:33 -0700 you wrote:
> Remove the 'imply' statements, these apparently are not doing
> what I expected. Platform modules which are used by the driver
> still need to be enabled in the overall config for them to be
> used, but there isn't a hard dependency on them.
>
> Use 'depend' for selectable modules which provide functions
> used directly by the driver.
>
> [...]
Here is the summary with links:
- [net-next] ptp: ocp: Simplify Kconfig.
https://git.kernel.org/netdev/net-next/c/bc8e05d6b965
You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2021-08-26 11:10 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-25 21:17 [PATCH net-next] ptp: ocp: Simplify Kconfig Jonathan Lemon
2021-08-25 21:32 ` Arnd Bergmann
2021-08-25 23:22 ` Randy Dunlap
2021-08-26 11:10 ` patchwork-bot+netdevbpf
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.