On 2 September 2013 01:57, Rafael J. Wysocki <rjw@xxxxxxx> wrote: > The second tab is one too many, I usually write such things like this: > > if (policy->governor_busy > || (policy->governor_enabled && event == CPUFREQ_GOV_START) > || ... > > Then it is much easier to distinguish the conditional code from the condition > itself. I see... I tried to do it this way but got confused again :) You fix it this time and I will understand it from that :) -- viresh -- 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