netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH net v2] net: pch_gbe: Allow build on MIPS_GENERIC kernel
@ 2023-06-07  5:59 Jiaxun Yang
  2023-06-07 20:41 ` Jakub Kicinski
                   ` (2 more replies)
  0 siblings, 3 replies; 5+ messages in thread
From: Jiaxun Yang @ 2023-06-07  5:59 UTC (permalink / raw)
  To: netdev; +Cc: davem, edumazet, kuba, pabeni, linux-kernel, Jiaxun Yang

MIPS Boston board, which is using MIPS_GENERIC kernel is using
EG20T PCH and thus need this driver.

Dependency of PCH_GBE, PTP_1588_CLOCK_PCH is also fixed for
MIPS_GENERIC.

Note that CONFIG_PCH_GBE is selected in arch/mips/configs/generic/
board-boston.config for a while, some how it's never wired up
in Kconfig.

Signed-off-by: Jiaxun Yang <jiaxun.yang@flygoat.com>
---
v2: Add PTP_1588_CLOCK_PCH dependency.

Netdev maintainers, is it possible to squeeze this tiny patch into
fixes tree?
---
 drivers/net/ethernet/oki-semi/pch_gbe/Kconfig | 2 +-
 drivers/ptp/Kconfig                           | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/net/ethernet/oki-semi/pch_gbe/Kconfig b/drivers/net/ethernet/oki-semi/pch_gbe/Kconfig
index 4e18b64dceb9..9651cc714ef2 100644
--- a/drivers/net/ethernet/oki-semi/pch_gbe/Kconfig
+++ b/drivers/net/ethernet/oki-semi/pch_gbe/Kconfig
@@ -5,7 +5,7 @@
 
 config PCH_GBE
 	tristate "OKI SEMICONDUCTOR IOH(ML7223/ML7831) GbE"
-	depends on PCI && (X86_32 || COMPILE_TEST)
+	depends on PCI && (MIPS_GENERIC || X86_32 || COMPILE_TEST)
 	depends on PTP_1588_CLOCK
 	select MII
 	select PTP_1588_CLOCK_PCH
diff --git a/drivers/ptp/Kconfig b/drivers/ptp/Kconfig
index b00201d81313..32dff1b4f891 100644
--- a/drivers/ptp/Kconfig
+++ b/drivers/ptp/Kconfig
@@ -102,7 +102,7 @@ config PTP_1588_CLOCK_INES
 
 config PTP_1588_CLOCK_PCH
 	tristate "Intel PCH EG20T as PTP clock"
-	depends on X86_32 || COMPILE_TEST
+	depends on MIPS_GENERIC || X86_32 || COMPILE_TEST
 	depends on HAS_IOMEM && PCI
 	depends on NET
 	depends on PTP_1588_CLOCK
-- 
2.39.2 (Apple Git-143)


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH net v2] net: pch_gbe: Allow build on MIPS_GENERIC kernel
  2023-06-07  5:59 [PATCH net v2] net: pch_gbe: Allow build on MIPS_GENERIC kernel Jiaxun Yang
@ 2023-06-07 20:41 ` Jakub Kicinski
  2023-06-08  4:44   ` Jiaxun Yang
  2023-06-08 12:08 ` Simon Horman
  2023-06-09  2:30 ` patchwork-bot+netdevbpf
  2 siblings, 1 reply; 5+ messages in thread
From: Jakub Kicinski @ 2023-06-07 20:41 UTC (permalink / raw)
  To: Jiaxun Yang; +Cc: netdev, davem, edumazet, pabeni, linux-kernel

On Wed,  7 Jun 2023 13:59:53 +0800 Jiaxun Yang wrote:
> v2: Add PTP_1588_CLOCK_PCH dependency.
> 
> Netdev maintainers, is it possible to squeeze this tiny patch into
> fixes tree?

Hm.. probably a little late for that. The first version didn't build,
feels too risky. We don't want to break the build of Linus's tree at
rc6/rc7. The merge window is two weeks out, is that really too long
of a wait?

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH net v2] net: pch_gbe: Allow build on MIPS_GENERIC kernel
  2023-06-07 20:41 ` Jakub Kicinski
@ 2023-06-08  4:44   ` Jiaxun Yang
  0 siblings, 0 replies; 5+ messages in thread
From: Jiaxun Yang @ 2023-06-08  4:44 UTC (permalink / raw)
  To: Jakub Kicinski; +Cc: netdev, David S . Miller, edumazet, pabeni, linux-kernel



> 2023年6月8日 04:41,Jakub Kicinski <kuba@kernel.org> 写道:
> 
> On Wed,  7 Jun 2023 13:59:53 +0800 Jiaxun Yang wrote:
>> v2: Add PTP_1588_CLOCK_PCH dependency.
>> 
>> Netdev maintainers, is it possible to squeeze this tiny patch into
>> fixes tree?
> 
> Hm.. probably a little late for that. The first version didn't build,
> feels too risky. We don't want to break the build of Linus's tree at
> rc6/rc7. The merge window is two weeks out, is that really too long
> of a wait?

Ah if that’s the case waiting until next merge window is fine for me.

Thanks
Jiaxun


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH net v2] net: pch_gbe: Allow build on MIPS_GENERIC kernel
  2023-06-07  5:59 [PATCH net v2] net: pch_gbe: Allow build on MIPS_GENERIC kernel Jiaxun Yang
  2023-06-07 20:41 ` Jakub Kicinski
@ 2023-06-08 12:08 ` Simon Horman
  2023-06-09  2:30 ` patchwork-bot+netdevbpf
  2 siblings, 0 replies; 5+ messages in thread
From: Simon Horman @ 2023-06-08 12:08 UTC (permalink / raw)
  To: Jiaxun Yang; +Cc: netdev, davem, edumazet, kuba, pabeni, linux-kernel

On Wed, Jun 07, 2023 at 01:59:53PM +0800, Jiaxun Yang wrote:
> MIPS Boston board, which is using MIPS_GENERIC kernel is using
> EG20T PCH and thus need this driver.
> 
> Dependency of PCH_GBE, PTP_1588_CLOCK_PCH is also fixed for
> MIPS_GENERIC.
> 
> Note that CONFIG_PCH_GBE is selected in arch/mips/configs/generic/
> board-boston.config for a while, some how it's never wired up
> in Kconfig.
> 
> Signed-off-by: Jiaxun Yang <jiaxun.yang@flygoat.com>
> ---
> v2: Add PTP_1588_CLOCK_PCH dependency.
> 
> Netdev maintainers, is it possible to squeeze this tiny patch into
> fixes tree?

That question aside, this looks good to me.

Reviewed-by: Simon Horman <simon.horman@corigine.com>

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH net v2] net: pch_gbe: Allow build on MIPS_GENERIC kernel
  2023-06-07  5:59 [PATCH net v2] net: pch_gbe: Allow build on MIPS_GENERIC kernel Jiaxun Yang
  2023-06-07 20:41 ` Jakub Kicinski
  2023-06-08 12:08 ` Simon Horman
@ 2023-06-09  2:30 ` patchwork-bot+netdevbpf
  2 siblings, 0 replies; 5+ messages in thread
From: patchwork-bot+netdevbpf @ 2023-06-09  2:30 UTC (permalink / raw)
  To: Jiaxun Yang; +Cc: netdev, davem, edumazet, kuba, pabeni, linux-kernel

Hello:

This patch was applied to netdev/net-next.git (main)
by Jakub Kicinski <kuba@kernel.org>:

On Wed,  7 Jun 2023 13:59:53 +0800 you wrote:
> MIPS Boston board, which is using MIPS_GENERIC kernel is using
> EG20T PCH and thus need this driver.
> 
> Dependency of PCH_GBE, PTP_1588_CLOCK_PCH is also fixed for
> MIPS_GENERIC.
> 
> Note that CONFIG_PCH_GBE is selected in arch/mips/configs/generic/
> board-boston.config for a while, some how it's never wired up
> in Kconfig.
> 
> [...]

Here is the summary with links:
  - [net,v2] net: pch_gbe: Allow build on MIPS_GENERIC kernel
    https://git.kernel.org/netdev/net-next/c/c8cc2ae229ff

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] 5+ messages in thread

end of thread, other threads:[~2023-06-09  2:30 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-06-07  5:59 [PATCH net v2] net: pch_gbe: Allow build on MIPS_GENERIC kernel Jiaxun Yang
2023-06-07 20:41 ` Jakub Kicinski
2023-06-08  4:44   ` Jiaxun Yang
2023-06-08 12:08 ` Simon Horman
2023-06-09  2:30 ` patchwork-bot+netdevbpf

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).