Hi Thomas and Gregory,
On Mon, Jan 21, 2013 at 11:07:10PM +0100, Gregory CLEMENT wrote:
> On 01/21/2013 07:17 PM, Thomas Petazzoni wrote:
> >> + irq_set_percpu_devid(virq);
> >> + irq_set_chip_and_handler(virq, &armada_370_xp_irq_chip,
> >> + handle_percpu_devid_irq);
> >> +
> >> + } else {
> >> + irq_set_chip_and_handler(virq, &armada_370_xp_irq_chip,
> >> + handle_level_irq);
> >> + }
> >
> > Braces useless since there is only one statement in the else.
> >
IMHO, this is an exception to the rule.
Since the first block is more than one line,
we usually put braces on the single line block too.
(or at least that's what Documentation/CodingStyle says).
Regards,
--
Ezequiel García, Free Electrons
Embedded Linux, Kernel and Android Engineering
http://free-electrons.com
_______________________________________________
devicetree-discuss mailing list
[email protected]
https://lists.ozlabs.org/listinfo/devicetree-discuss
