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

quoted

quoted

+           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