On Wed, Nov 30, 2011 at 06:06:43PM +0800, Josh Wu wrote: > + /* Get ISI_MCK, provided by programmable clock or external clock */ > + isi->mck = clk_get(dev, "isi_mck"); > + if (IS_ERR_OR_NULL(isi->mck)) { This should be IS_ERR() > + dev_err(dev, "Failed to get isi_mck\n"); > + ret = isi->mck ? PTR_ERR(isi->mck) : -EINVAL; ret = PTR_ERR(isi->mck); -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html