[PATCH] Minor coding style fix

Kernel generally follow the style

if (func()) {
/* failed case */
} else {
/* success */
}


-aneesh

diff --git a/kernel/sys.c b/kernel/sys.c
index 98489d8..55cb77c 100644
--- a/kernel/sys.c
+++ b/kernel/sys.c
@@ -517,7 +517,7 @@ EXPORT_SYMBOL_GPL(srcu_notifier_call_cha
 void srcu_init_notifier_head(struct srcu_notifier_head *nh)
 {
 	mutex_init(&nh->mutex);
-	if (init_srcu_struct(&nh->srcu) < 0)
+	if (init_srcu_struct(&nh->srcu))
 		BUG();
 	nh->head = NULL;
 }

---

• Follow-Ups:
  • Re: [PATCH] Minor coding style fix
    • From: "Jesper Juhl" <[email protected]>

• Prev by Date: [PATCH] user struct irq_chip instead of struct hw_interrupt_type
• Next by Date: Re: + clocksource-initialize-list-value.patch added to -mm tree
• Previous by thread: [PATCH] user struct irq_chip instead of struct hw_interrupt_type
• Next by thread: Re: [PATCH] Minor coding style fix
• Index(es):
  • Date
  • Thread

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]