From mboxrd@z Thu Jan 1 00:00:00 1970 From: grant.likely@secretlab.ca (Grant Likely) Date: Tue, 20 Sep 2011 11:17:06 -0600 Subject: [PATCH V2 1/6] spi/spi-pl022: Resolve formatting issues In-Reply-To: References: Message-ID: <20110920171706.GA7781@ponder.secretlab.ca> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Wed, Aug 10, 2011 at 02:20:54PM +0530, Viresh Kumar wrote: > There were few formatting related issues in code. This patch fixes them. > Fixes include: > - Remove extra blank lines > - align code to 80 cols > - combine several lines to one line > - Replace multiple spaces with tabs > - Remove spaces before labels > > Signed-off-by: Viresh Kumar > Tested-by: Linus Walleij > @@ -2243,23 +2233,23 @@ pl022_probe(struct amba_device *adev, const struct amba_id *id) > amba_vcore_disable(adev); > return 0; > > - err_spi_register: > - err_start_queue: > - err_init_queue: > +err_spi_register: > +err_start_queue: > +err_init_queue: > destroy_queue(pl022); > pl022_dma_remove(pl022); > free_irq(adev->irq[0], pl022); > pm_runtime_disable(&adev->dev); > - err_no_irq: > +err_no_irq: > clk_put(pl022->clk); > - err_no_clk: > +err_no_clk: > iounmap(pl022->virtbase); > - err_no_ioremap: > +err_no_ioremap: > amba_release_regions(adev); > - err_no_ioregion: > +err_no_ioregion: > spi_master_put(master); > - err_no_master: > - err_no_pdata: > +err_no_master: > +err_no_pdata: These label indentations are actually on purpose. I'll apply the rest of the patch, but I'm going to remove this hunk. Indenting error labels by 1 space makes 'diff' show the function name instead of the label in the context line. g.