On Wed, Feb 19, 2014 at 12:32:02PM +0200, Ivan Khoronzhuk wrote:

quoted

quoted

+	aemif->clk = devm_clk_get(dev, NULL);
+	if (IS_ERR(aemif->clk)) {
+		dev_err(dev, "cannot get clock 'aemif'\n");
+		return PTR_ERR(aemif->clk);

No freeing memory?

There is no need to free memory explicitly.
devm_kzalloc is used instead of kzalloc.

Yes, but where does the device on the error path get removed?