[PATCH] Add idle wait support for 44x platforms

Josh Boyer jwboyer at linux.vnet.ibm.com
Fri Apr 4 12:59:27 EST 2008


On Fri, 4 Apr 2008 10:03:56 +1100
tony at bakeyournoodle.com (Tony Breeds) wrote:

> > +int __init ppc44x_idle_init(void)
> > +{
> > +	if(of_find_node_by_path("/hypervisor") != NULL) {
>           ^ space
> > +		/* if we find /hypervisor node is in device tree,
> > +		   set idle mode to wait */
> > +		current_mode = 1; /* wait mode */
> > +	}
> 
> You don't really need the braces {} here.

They are technically not required, but I like them there none the
less because of the multi-line comment.  I find it to be better style
personally.

josh



More information about the Linuxppc-dev mailing list