2023-06-07 06:12:23

by Jiaxun Yang

[permalink] [raw]
Subject: [PATCH net v2] net: pch_gbe: Allow build on MIPS_GENERIC kernel

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 <[email protected]>
---
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)



2023-06-07 21:44:47

by Jakub Kicinski

[permalink] [raw]
Subject: Re: [PATCH net v2] net: pch_gbe: Allow build on MIPS_GENERIC 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?

2023-06-08 05:05:37

by Jiaxun Yang

[permalink] [raw]
Subject: Re: [PATCH net v2] net: pch_gbe: Allow build on MIPS_GENERIC kernel



> 2023年6月8日 04:41,Jakub Kicinski <[email protected]> 写道:
>
> 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


2023-06-08 12:24:43

by Simon Horman

[permalink] [raw]
Subject: Re: [PATCH net v2] net: pch_gbe: Allow build on MIPS_GENERIC 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 <[email protected]>
> ---
> 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 <[email protected]>

2023-06-09 03:04:46

by patchwork-bot+netdevbpf

[permalink] [raw]
Subject: Re: [PATCH net v2] net: pch_gbe: Allow build on MIPS_GENERIC kernel

Hello:

This patch was applied to netdev/net-next.git (main)
by Jakub Kicinski <[email protected]>:

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