Hi Stephen
From: Kuninori Morimoto kuninori.morimoto.gx@renesas.com
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@renesas.com
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 itself is not wrong (?)