public inbox for netdev@vger.kernel.org
 help / color / mirror / Atom feed
* [RESEND PATCH] arm64: defconfig: Enable DP83TG720 PHY driver
@ 2026-02-26 18:54 Meghana Malladi
  2026-02-27  6:42 ` Krzysztof Kozlowski
  0 siblings, 1 reply; 6+ messages in thread
From: Meghana Malladi @ 2026-02-26 18:54 UTC (permalink / raw)
  To: nm, vigneshr
  Cc: kristo, linux-arm-kernel, linux-kernel, netdev, andrew,
	hkallweit1, Meghana Malladi

Enable DP83TG720 PHY driver as a module to support TI's DP83TG720
1000BASE-T1 Automotive Ethernet PHY. This is required for the
DP83TG720-IND-SPE-EVM daughter card used with AM642 EVM ICSSG0
interface.

Signed-off-by: Meghana Malladi <m-malladi@ti.com>
---
 arch/arm64/configs/defconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
index b67d5b1fc45b..3b88df2ca5e1 100644
--- a/arch/arm64/configs/defconfig
+++ b/arch/arm64/configs/defconfig
@@ -420,6 +420,7 @@ CONFIG_REALTEK_PHY=y
 CONFIG_ROCKCHIP_PHY=y
 CONFIG_DP83867_PHY=y
 CONFIG_DP83869_PHY=m
+CONFIG_DP83TG720_PHY=m
 CONFIG_DP83TD510_PHY=y
 CONFIG_VITESSE_PHY=y
 CONFIG_XILINX_GMII2RGMII=m

base-commit: 4916f2e2f3fc9aef289fcd07949301e5c29094c2
prerequisite-patch-id: d9e4c1f3ec36250ae7ce49b720614eef25f9ffd3
-- 
2.43.0


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

* Re: [RESEND PATCH] arm64: defconfig: Enable DP83TG720 PHY driver
  2026-02-26 18:54 [RESEND PATCH] arm64: defconfig: Enable DP83TG720 PHY driver Meghana Malladi
@ 2026-02-27  6:42 ` Krzysztof Kozlowski
  2026-02-27  7:15   ` Meghana Malladi
  0 siblings, 1 reply; 6+ messages in thread
From: Krzysztof Kozlowski @ 2026-02-27  6:42 UTC (permalink / raw)
  To: Meghana Malladi, nm, vigneshr
  Cc: kristo, linux-arm-kernel, linux-kernel, netdev, andrew,
	hkallweit1

On 26/02/2026 19:54, Meghana Malladi wrote:
> Enable DP83TG720 PHY driver as a module to support TI's DP83TG720
> 1000BASE-T1 Automotive Ethernet PHY. This is required for the
> DP83TG720-IND-SPE-EVM daughter card used with AM642 EVM ICSSG0
> interface.

Grepping for DP83TG720 or IND-SPE-EVM gives me no results. Are you sure
this is the board supported upstream?

> 
> Signed-off-by: Meghana Malladi <m-malladi@ti.com>
> ---
>  arch/arm64/configs/defconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
> index b67d5b1fc45b..3b88df2ca5e1 100644
> --- a/arch/arm64/configs/defconfig
> +++ b/arch/arm64/configs/defconfig
> @@ -420,6 +420,7 @@ CONFIG_REALTEK_PHY=y
>  CONFIG_ROCKCHIP_PHY=y
>  CONFIG_DP83867_PHY=y
>  CONFIG_DP83869_PHY=m
> +CONFIG_DP83TG720_PHY=m
>  CONFIG_DP83TD510_PHY=y
>  CONFIG_VITESSE_PHY=y
>  CONFIG_XILINX_GMII2RGMII=m
> 
> base-commit: 4916f2e2f3fc9aef289fcd07949301e5c29094c2
> prerequisite-patch-id: d9e4c1f3ec36250ae7ce49b720614eef25f9ffd3

Why do you have dependencies in defconfig?


Best regards,
Krzysztof

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

* Re: [RESEND PATCH] arm64: defconfig: Enable DP83TG720 PHY driver
  2026-02-27  6:42 ` Krzysztof Kozlowski
@ 2026-02-27  7:15   ` Meghana Malladi
  2026-02-27  7:21     ` Krzysztof Kozlowski
  0 siblings, 1 reply; 6+ messages in thread
From: Meghana Malladi @ 2026-02-27  7:15 UTC (permalink / raw)
  To: Krzysztof Kozlowski, nm
  Cc: kristo, linux-arm-kernel, linux-kernel, netdev, andrew,
	hkallweit1, Raghavendra, Vignesh



On 2/27/26 12:12, Krzysztof Kozlowski wrote:
> On 26/02/2026 19:54, Meghana Malladi wrote:
>> Enable DP83TG720 PHY driver as a module to support TI's DP83TG720
>> 1000BASE-T1 Automotive Ethernet PHY. This is required for the
>> DP83TG720-IND-SPE-EVM daughter card used with AM642 EVM ICSSG0
>> interface.
> 
> Grepping for DP83TG720 or IND-SPE-EVM gives me no results. Are you sure
> this is the board supported upstream?
> 

Yes, this series adds support adds support for the board: 
https://lore.kernel.org/all/20231212054144.87527-1-o.rempel@pengutronix.de/

>>
>> Signed-off-by: Meghana Malladi <m-malladi@ti.com>
>> ---
>>   arch/arm64/configs/defconfig | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
>> index b67d5b1fc45b..3b88df2ca5e1 100644
>> --- a/arch/arm64/configs/defconfig
>> +++ b/arch/arm64/configs/defconfig
>> @@ -420,6 +420,7 @@ CONFIG_REALTEK_PHY=y
>>   CONFIG_ROCKCHIP_PHY=y
>>   CONFIG_DP83867_PHY=y
>>   CONFIG_DP83869_PHY=m
>> +CONFIG_DP83TG720_PHY=m
>>   CONFIG_DP83TD510_PHY=y
>>   CONFIG_VITESSE_PHY=y
>>   CONFIG_XILINX_GMII2RGMII=m
>>
>> base-commit: 4916f2e2f3fc9aef289fcd07949301e5c29094c2
>> prerequisite-patch-id: d9e4c1f3ec36250ae7ce49b720614eef25f9ffd3
> 
> Why do you have dependencies in defconfig?
> 

You're right - the prerequisite-patch-id line shouldn't be in the 
defconfig patch. The defconfig change itself has no code dependencies; 
it simply enables an already-upstream PHY driver as a module. Will fix 
this in v2. Thanks.

> 
> Best regards,
> Krzysztof


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

* Re: [RESEND PATCH] arm64: defconfig: Enable DP83TG720 PHY driver
  2026-02-27  7:15   ` Meghana Malladi
@ 2026-02-27  7:21     ` Krzysztof Kozlowski
  2026-02-27  7:24       ` Meghana Malladi
  0 siblings, 1 reply; 6+ messages in thread
From: Krzysztof Kozlowski @ 2026-02-27  7:21 UTC (permalink / raw)
  To: Meghana Malladi, nm
  Cc: kristo, linux-arm-kernel, linux-kernel, netdev, andrew,
	hkallweit1, Raghavendra, Vignesh

On 27/02/2026 08:15, Meghana Malladi wrote:
> 
> 
> On 2/27/26 12:12, Krzysztof Kozlowski wrote:
>> On 26/02/2026 19:54, Meghana Malladi wrote:
>>> Enable DP83TG720 PHY driver as a module to support TI's DP83TG720
>>> 1000BASE-T1 Automotive Ethernet PHY. This is required for the
>>> DP83TG720-IND-SPE-EVM daughter card used with AM642 EVM ICSSG0
>>> interface.
>>
>> Grepping for DP83TG720 or IND-SPE-EVM gives me no results. Are you sure
>> this is the board supported upstream?
>>
> 
> Yes, this series adds support adds support for the board: 
> https://lore.kernel.org/all/20231212054144.87527-1-o.rempel@pengutronix.de/

I still wonder why there is no DTS for this.

Best regards,
Krzysztof

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

* Re: [RESEND PATCH] arm64: defconfig: Enable DP83TG720 PHY driver
  2026-02-27  7:21     ` Krzysztof Kozlowski
@ 2026-02-27  7:24       ` Meghana Malladi
  2026-02-27  7:29         ` Krzysztof Kozlowski
  0 siblings, 1 reply; 6+ messages in thread
From: Meghana Malladi @ 2026-02-27  7:24 UTC (permalink / raw)
  To: Krzysztof Kozlowski, nm
  Cc: kristo, linux-arm-kernel, linux-kernel, netdev, andrew,
	hkallweit1, Raghavendra, Vignesh



On 2/27/26 12:51, Krzysztof Kozlowski wrote:
> On 27/02/2026 08:15, Meghana Malladi wrote:
>>
>>
>> On 2/27/26 12:12, Krzysztof Kozlowski wrote:
>>> On 26/02/2026 19:54, Meghana Malladi wrote:
>>>> Enable DP83TG720 PHY driver as a module to support TI's DP83TG720
>>>> 1000BASE-T1 Automotive Ethernet PHY. This is required for the
>>>> DP83TG720-IND-SPE-EVM daughter card used with AM642 EVM ICSSG0
>>>> interface.
>>>
>>> Grepping for DP83TG720 or IND-SPE-EVM gives me no results. Are you sure
>>> this is the board supported upstream?
>>>
>>
>> Yes, this series adds support adds support for the board:
>> https://lore.kernel.org/all/20231212054144.87527-1-o.rempel@pengutronix.de/
> 
> I still wonder why there is no DTS for this.
> 

I posted DTS patch separately for this: 
https://lore.kernel.org/all/20260226115454.1730618-1-m-malladi@ti.com/

Does it make more sense to post them together as a series ?

> Best regards,
> Krzysztof


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

* Re: [RESEND PATCH] arm64: defconfig: Enable DP83TG720 PHY driver
  2026-02-27  7:24       ` Meghana Malladi
@ 2026-02-27  7:29         ` Krzysztof Kozlowski
  0 siblings, 0 replies; 6+ messages in thread
From: Krzysztof Kozlowski @ 2026-02-27  7:29 UTC (permalink / raw)
  To: Meghana Malladi, nm
  Cc: kristo, linux-arm-kernel, linux-kernel, netdev, andrew,
	hkallweit1, Raghavendra, Vignesh

On 27/02/2026 08:24, Meghana Malladi wrote:
> 
> 
> On 2/27/26 12:51, Krzysztof Kozlowski wrote:
>> On 27/02/2026 08:15, Meghana Malladi wrote:
>>>
>>>
>>> On 2/27/26 12:12, Krzysztof Kozlowski wrote:
>>>> On 26/02/2026 19:54, Meghana Malladi wrote:
>>>>> Enable DP83TG720 PHY driver as a module to support TI's DP83TG720
>>>>> 1000BASE-T1 Automotive Ethernet PHY. This is required for the
>>>>> DP83TG720-IND-SPE-EVM daughter card used with AM642 EVM ICSSG0
>>>>> interface.
>>>>
>>>> Grepping for DP83TG720 or IND-SPE-EVM gives me no results. Are you sure
>>>> this is the board supported upstream?
>>>>
>>>
>>> Yes, this series adds support adds support for the board:
>>> https://lore.kernel.org/all/20231212054144.87527-1-o.rempel@pengutronix.de/
>>
>> I still wonder why there is no DTS for this.
>>
> 
> I posted DTS patch separately for this: 
> https://lore.kernel.org/all/20260226115454.1730618-1-m-malladi@ti.com/
> 
> Does it make more sense to post them together as a series ?

Of course, you target the same subsystem - soc. Why triggering all these
investigations and questions...

Best regards,
Krzysztof

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

end of thread, other threads:[~2026-02-27  7:29 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-02-26 18:54 [RESEND PATCH] arm64: defconfig: Enable DP83TG720 PHY driver Meghana Malladi
2026-02-27  6:42 ` Krzysztof Kozlowski
2026-02-27  7:15   ` Meghana Malladi
2026-02-27  7:21     ` Krzysztof Kozlowski
2026-02-27  7:24       ` Meghana Malladi
2026-02-27  7:29         ` Krzysztof Kozlowski

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox