On 10/24/2024 7:57 PM, Andrew Lunn wrote: >>> I'm just wondering if you have circular dependencies at runtime? >>> >>> Where you will need to be careful is probe time vs runtime. Since you >>> have circular phandles you need to first create all the clock >>> providers, and only then start the clock consumers. Otherwise you >>> might get into an endless EPROBE_DEFER loop. >>> >> >> The Rx/Tx clocks sourced from the SERDES are registered as provider >> clocks by the UNIPHY/PCS driver during probe time. There is no runtime >> operation needed for these clocks after this. > > So they are always ticking. You cannot turn them on/off? It is nice to > model them a fixed-clocks, since it describes the architecture, but i > have to question if it is worth the effort. > Yes, we cannot turn them off. However the rates of these clocks from SERDES to NSSCC, is not fixed. It will be either 312.5Mhz or 125Mhz, depending on the whether the SERDES mode is USXGMII or SGMII respectively. > Andrew
