On Tue, Dec 20, 2022 at 11:14:44PM +0000, Conor Dooley wrote: > On Tue, Dec 20, 2022 at 08:50:50AM +0800, Hal Feng wrote: > > + clock-controller@13020000 { > > + compatible = "starfive,jh7110-syscrg"; > > + reg = <0x13020000 0x10000>; > > + clocks = <&osc>, <&gmac1_rmii_refin>, > > + <&gmac1_rgmii_rxin>, > > + <&i2stx_bclk_ext>, <&i2stx_lrck_ext>, > > + <&i2srx_bclk_ext>, <&i2srx_lrck_ext>, > > + <&tdm_ext>, <&mclk_ext>; > > + clock-names = "osc", "gmac1_rmii_refin", > > + "gmac1_rgmii_rxin", > > + "i2stx_bclk_ext", "i2stx_lrck_ext", > > + "i2srx_bclk_ext", "i2srx_lrck_ext", > > + "tdm_ext", "mclk_ext"; > > + #clock-cells = <1>; > > + #reset-cells = <1>; > > + }; > > Also, whatever happened to GMAC0? It has fixed-clocks defined in the DTS > but doesn't appear in the binding or driver? Ah, I should have looked at the next patch... D'oh.
Attachment:
signature.asc
Description: PGP signature