On Thu, Oct 20, 2022 at 12:39:46PM +0200, Michal Simek wrote: > > > On 10/20/22 11:41, Andy Chiu wrote: > > CAUTION: This message has originated from an External Source. Please use proper judgment and caution when opening attachments, clicking links, or responding to this email. > > > > > > Add a property to set mdio bus frequency at runtime by DT. > > > > Signed-off-by: Andy Chiu <andy.chiu@xxxxxxxxxx> > > Reviewed-by: Greentime Hu <greentime.hu@xxxxxxxxxx> > > --- > > Documentation/devicetree/bindings/net/xilinx_axienet.txt | 3 +++ > > 1 file changed, 3 insertions(+) > > > > diff --git a/Documentation/devicetree/bindings/net/xilinx_axienet.txt b/Documentation/devicetree/bindings/net/xilinx_axienet.txt > > index 1aa4c6006cd0..d78cf402aa2a 100644 > > --- a/Documentation/devicetree/bindings/net/xilinx_axienet.txt > > +++ b/Documentation/devicetree/bindings/net/xilinx_axienet.txt > > @@ -43,6 +43,9 @@ Optional properties: > > support both 1000BaseX and SGMII modes. If set, the phy-mode > > should be set to match the mode selected on core reset (i.e. > > by the basex_or_sgmii core input line). > > +- xlnx,mdio-freq: Define the clock frequency of the MDIO bus. If the property > > + does not pressent on the DT, then the mdio driver would use > > + the default 2.5 MHz clock, as mentioned on 802.3 spc. > > Isn't it better to specify it based on ccf description. It means &clk and > used clock framework to find value? Or use 'bus-frequency' which IIRC is defined for MDIO. Rob