]> git.neil.brown.name Git - gta04.git/commit
clk: fix clk_get on of_clk_get_by_name return check
authorShawn Guo <shawn.guo@linaro.org>
Wed, 18 Jul 2012 03:52:22 +0000 (11:52 +0800)
committerMike Turquette <mturquette@linaro.org>
Thu, 19 Jul 2012 21:07:45 +0000 (14:07 -0700)
commit9f1612d351a8e57d3d694e828641d3e4eeb224f8
tree454f1194686043575369eb66fa02075036f17b59
parentc782c384d289a382baf4a1ab544f6f4b9e72405b
clk: fix clk_get on of_clk_get_by_name return check

The commit 766e6a4 (clk: add DT clock binding support) plugs device
tree clk lookup of_clk_get_by_name into clk_get, and fall on non-DT
lookup clk_get_sys if DT lookup fails.

The return check on of_clk_get_by_name takes (clk != NULL) as a
successful DT lookup.  But it's not the case.  For any system that
does not define clk lookup in device tree, ERR_PTR(-ENOENT) will be
returned, and consequently, all the client drivers calling clk_get
in their probe functions will fail to probe with error code -ENOENT
returned.

Fix the issue by checking of_clk_get_by_name return with !IS_ERR(clk),
and update of_clk_get and of_clk_get_by_name for !CONFIG_OF build
correspondingly.

Signed-off-by: Shawn Guo <shawn.guo@linaro.org>
Acked-by: Rob Herring <rob.herring@calxeda.com>
Tested-by: Marek Vasut <marex@denx.de>
Tested-by: Lauri Hintsala <lauri.hintsala@bluegiga.com>
Signed-off-by: Mike Turquette <mturquette@linaro.org>
drivers/clk/clkdev.c
include/linux/clk.h