On 7 March 2013 08:49, Harvey Harrison <harvey.harrison@xxxxxxxxx> wrote: > On Wed, Mar 6, 2013 at 4:32 PM, Viresh Kumar <viresh.kumar@xxxxxxxxxx> wrote: > >> clk[cluster] = clk_get_sys(name, NULL); >> - if (!IS_ERR(clk[cluster])) { >> + if (!IS_ERR_OR_NULL(clk[cluster])) { >> pr_debug("%s: clk: %p & freq table: %p, cluster: %d\n", >> __func__, clk[cluster], freq_table[cluster], >> cluster); > > > You seem pretty attached to IS_ERR_OR_NULL here. Haha. Not really. I just wanted to get more logical conclusion out. Please check the other mail (where i would reply to Russell), which might have more discussion around this. -- To unsubscribe from this list: send the line "unsubscribe cpufreq" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html