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 <viresh.ku...@st.com>
> Tested-by: Linus Walleij <linus.wall...@linaro.org>
> @@ -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.


------------------------------------------------------------------------------
All the data continuously generated in your IT infrastructure contains a
definitive record of customers, application performance, security
threats, fraudulent activity and more. Splunk takes this data and makes
sense of it. Business sense. IT sense. Common sense.
http://p.sf.net/sfu/splunk-d2dcopy1
_______________________________________________
spi-devel-general mailing list
spi-devel-general@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/spi-devel-general

Reply via email to