On 18 March 2014 18:09, Mark Brown <broo...@kernel.org> wrote: > On Tue, Mar 18, 2014 at 03:53:02PM +0530, Sachin Kamat wrote: >> Silences the following type of error/warnings: >> ERROR: that open brace { should be on the previous line >> WARNING: Missing a blank line after declarations > > Applied, but... > >> - static const u8 max8660_addresses[MAX8660_N_REGS] = >> - { 0x10, 0x12, 0x20, 0x23, 0x24, 0x29, 0x2a, 0x32, 0x33, 0x39, 0x80 }; >> + static const u8 max8660_addresses[MAX8660_N_REGS] = { >> + 0x10, 0x12, 0x20, 0x23, 0x24, 0x29, 0x2a, 0x32, 0x33, 0x39, 0x80 }; > > Think about what you're doing here - you want the }; on a new line too.
Sorry about it. I was kind of in two minds about shifting it. I will send a followup for completeness. -- With warm regards, Sachin -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/