On Tue, Sep 13, 2011 at 10:59:42AM -0700, K. Y. Srinivasan wrote: > @@ -403,31 +384,13 @@ static void vmbus_process_offer(struct work_struct *work) > spin_unlock_irqrestore(&vmbus_connection.channel_lock, flags); > > free_channel(newchannel); > - } else { > + } else > /* > * This state is used to indicate a successful open > * so that when we do close the channel normally, we > * can cleanup properly > */ > newchannel->state = CHANNEL_OPEN_STATE; It's not worth resending, but this should have kept the curly braces for two style reasons. 1) If one side of the if else pair has braces the other side gets braces too. 2) If it's more than one line, it gets braces even if it's not needed for semantic reasons. Sorry to email you these pedantic things, but if I don't then my OCD acts up and I feel all itchy for hours. Also you have a similar thing below as well. regards, dan carpenter _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel