Hi Stephen > > From: Kuninori Morimoto <kuninori.morimoto.gx@xxxxxxxxxxx> > > > > Current simple-card is supporting this style for clocks > > > > sound { > > ... > > simple-audio-card,cpu { > > sound-dai = <&xxx>; > > clocks = <&cpu_clock>; > > }; > > simple-audio-card,codec { > > sound-dai = <&xxx>; > > clocks = <&codec_clock>; > > }; > > }; > > > > Now, it can support this style too, because we can use > > devm_get_clk_from_child() now. > > > > sound { > > ... > > clocks = <&cpu_clock>, <&codec_clock>; > > clock-names = "cpu", "codec"; > > clock-ranges; > > ... > > simple-audio-card,cpu { > > sound-dai = <&xxx>; > > }; > > simple-audio-card,codec { > > sound-dai = <&xxx>; > > }; > > }; > > > > Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@xxxxxxxxxxx> > > I don't see any reason why we need this patch though. The binding > works as is, so supporting different styles doesn't seem like a > good idea to me. Let's just keep what we have? Even if a sub-node > like cpu or codec gets more than one element in the clocks list > property, we can make that work by passing a clock-name then > based on some sort of other knowledge. OK, thanks. Let's skip this patch. But I believe this idea/method itself is not wrong (?) -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html