[PATCH] clk: tegra: fix return value check in tegra_clk_register_pllc()
Stephen Warren
swarren at wwwdotorg.org
Tue Oct 29 15:40:26 EDT 2013
On 10/28/2013 08:07 PM, Wei Yongjun wrote:
> From: Wei Yongjun <yongjun_wei at trendmicro.com.cn>
>
> In case of error, the function __clk_lookup() returns NULL pointer
> not ERR_PTR(). The IS_ERR() test in the return value check should
> be replaced with NULL test.
> parent = __clk_lookup(parent_name);
> - if (IS_ERR(parent)) {
> + if (!parent) {
clk_get() returns an error-value. It'd be nice if the internal clk APIs
all did the same for consistency... Otherwise, the clock subsystem is
essentially using both NULL and error-values as errors, which is rather
frowned upon.
More information about the linux-arm-kernel
mailing list