On Tue, Dec 17, 2013 at 10:53:47AM +0100, Lothar Waßmann wrote: > > > + if (soc_opp_index >= soc_opp_count) { > > > > Can soc_opp_index be possibly greater than soc_opp_count? Otherwise, > > the condition check below is good enough? > > > > if (soc_opp_index == soc_opp_count) > > > it doen't harm to be on the safe side and use >= anyway! Well, it may confuse reader. At least, it took me some time understand how that ">" condition will happen. And it turns out never. Shawn -- 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