On Wed, May 25, 2011 at 18:09, Todd Poynor <toddpoynor@xxxxxxxxxx> wrote:
>> + if (result || !freq_table) {
>> + dev_err(mpu_dev, "%s: cpu%d: unable to get freq table [%d]\n",
>> + __func__, policy->cpu, result);
>> + return result;
>
> The "|| !freq_table" isn't needed, and technically allows the code to
> return zero for an error return if the subexpression does evaluate
> true.
>
Ack..
Regards,
Nishanth Menon
--
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
