On 09/09/2014 10:21 AM, Mark Brown wrote:>> if (ssi_private->clk)>> > clk_prepare_enable(ssi_private->clk);> Should be a !IS_ERR() - NULL is a valid clock.In that case, ssi_private->clk needs to be initialized to -EINVAL or something, so that the check works on systems that don't have any clocks.