On Fri, Mar 8, 2019 at 12:32 PM Fabio Estevam <festevam@xxxxxxxxx> wrote: > > Hi Daniel, > > On Fri, Mar 1, 2019 at 12:53 PM Daniel Baluta <daniel.baluta@xxxxxxxxx> wrote: > > > They are used by simple-card.c machine driver. > > > > asoc_simple_card_parse_clk > > -> /* Parse dai->sysclk come from "clocks = <&xxx>" */ > > clk = devm_get_clk_from_child(dev, node, NULL); > > The simple-card looks for the "clocks" properties inside the > simple-card node, not inside the codec node, right? I see your point now. Indeed, "clocks" properties is the one inside simple-card. sound-wm8524 { compatible = "simple-audio-card"; link_codec: simple-audio-card,codec { sound-dai = <&wm8524>; clocks = <&clk IMX8MQ_CLK_SAI2_ROOT>; Will remove them from wm8524: audio-codec node because they are useless there. thanks, Daniel.