Earlier we were checking on !dev_opp where as find_device_opp returns a error pointer in case of error. So correcting the check as in the earlier code even if find_device_opp returns an error opp_init_cpufreq_table was not exiting. Signed-off-by: Thara Gopinath <thara@xxxxxx> --- arch/arm/plat-omap/opp.c | 4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/arch/arm/plat-omap/opp.c b/arch/arm/plat-omap/opp.c index a06b88d..d88a2e0 100644 --- a/arch/arm/plat-omap/opp.c +++ b/arch/arm/plat-omap/opp.c @@ -457,8 +457,10 @@ void opp_init_cpufreq_table(struct device *dev, int i = 0; dev_opp = find_device_opp(dev); - if (WARN_ON(!dev_opp)) + if (IS_ERR(dev_opp)) { + WARN_ON(1); return; + } freq_table = kzalloc(sizeof(struct cpufreq_frequency_table) * (dev_opp->enabled_opp_count + 1), GFP_ATOMIC); -- 1.7.0.rc1.33.g07cf0f -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html