Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux

[S390] irq: simple coding style change

Use braces for if/else/list_for_each_entry bodies if the body consists
of more than a single line. Otherwise I get confused and check if there
is something broken whenever I see these code snippets.

Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>
Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>

authored by

Heiko Carstens and committed by
Martin Schwidefsky
7968ca81 ac2ac6e8

+6 -3
+6 -3
arch/s390/kernel/irq.c
··· 118 118 "a" (__do_softirq) 119 119 : "0", "1", "2", "3", "4", "5", "14", 120 120 "cc", "memory" ); 121 - } else 121 + } else { 122 122 /* We are already on the async stack. */ 123 123 __do_softirq(); 124 + } 124 125 } 125 126 126 127 local_irq_restore(flags); ··· 193 192 int index = ext_hash(code); 194 193 195 194 spin_lock_irqsave(&ext_int_hash_lock, flags); 196 - list_for_each_entry_rcu(p, &ext_int_hash[index], entry) 195 + list_for_each_entry_rcu(p, &ext_int_hash[index], entry) { 197 196 if (p->code == code && p->handler == handler) { 198 197 list_del_rcu(&p->entry); 199 198 kfree_rcu(p, rcu); 200 199 } 200 + } 201 201 spin_unlock_irqrestore(&ext_int_hash_lock, flags); 202 202 return 0; 203 203 } ··· 213 211 214 212 old_regs = set_irq_regs(regs); 215 213 irq_enter(); 216 - if (S390_lowcore.int_clock >= S390_lowcore.clock_comparator) 214 + if (S390_lowcore.int_clock >= S390_lowcore.clock_comparator) { 217 215 /* Serve timer interrupts first. */ 218 216 clock_comparator_work(); 217 + } 219 218 kstat_cpu(smp_processor_id()).irqs[EXTERNAL_INTERRUPT]++; 220 219 if (ext_code.code != 0x1004) 221 220 __get_cpu_var(s390_idle).nohz_delay = 1;