Martin Fuzzey wrote:

+If a clock is provided, clock-frequency is optional

+If no clock is provided clock-frequency is required (this represents the codec +being clocked by an external signal not present in the clock tree)

+If both a clock and clock-frequency are provided the clock's rate will be set.

• Example:

  codec: sgtl5000@0a { compatible = "fsl,sgtl5000"; reg = <0x0a>;

• clock-frequency = <20000000>;

• clocks = <&clks 162>; /* cko1 */

The code looks okay, but I'm not sure this example is right. The binding says that clock-frequency is optional if 'clocks' is provided, but here you list both. You should probably have two examples.