[RFC PATCH 4/8] xilinx: tsn: Add Ethernet MAC (EMAC) and MDIO support to the TSN driver

[Neeli, Srinivas]

[Public]

Hi,

> -----Original Message-----
> From: Andrew Lunn <andrew at lunn.ch>
> Sent: Friday, February 20, 2026 8:33 PM
> To: Neeli, Srinivas <srinivas.neeli at amd.com>
> Cc: andrew+netdev at lunn.ch; davem at davemloft.net;
> edumazet at google.com; kuba at kernel.org; pabeni at redhat.com; Simek,
> Michal <michal.simek at amd.com>; robh at kernel.org; krzk+dt at kernel.org;
> conor+dt at kernel.org; richardcochran at gmail.com; netdev at vger.kernel.org;
> linux-kernel at vger.kernel.org; devicetree at vger.kernel.org; linux-arm-
> kernel at lists.infradead.org; git (AMD-Xilinx) <git at amd.com>
> Subject: Re: [RFC PATCH 4/8] xilinx: tsn: Add Ethernet MAC (EMAC) and MDIO
> support to the TSN driver
>
> > > > +           tsn_mdio_mdc_enable(emac);
> > >
> > > It is unusual to stop MDC. I suspect some PHYs will not like this.
> > > What is your reason for doing this.
> > >
> > The intention is to reduce power consumption by gating  the clock when it is
> not required.
>
> There is only one other MDIO bus driver that i know of which disables the
> clock between transactions. And it does it because the board has ageing
> problems, and if the clock it left ticking all the time, the magic smoke eventually
> escapes.
>
> How many different PHYs have you tested this with? How good a feeling do
> you have this is not going to cause problems? How much power do you
> actually save?
>
>        Andrew
>
This change was expected to provide some power savings when the clock is disabled.
Since I have not validated this through practical measurements and have tested it with only one PHY, I have decided to drop the change.
Thanks for the guidance.

Thanks
Neeli Srinivas