On Thursday 21 July 2011 11:13 AM, padma venkat wrote:
Hi Tushar,

On Thu, Jul 21, 2011 at 10:58 AM, Tushar Behera
<[email protected]>  wrote:
On Thursday 21 July 2011 04:42 PM, Padmavathi Venna wrote:


[snip]

+       gc = irq_alloc_generic_chip("s5p64x0-eint", 1, S5P_IRQ_EINT_BASE,
+                                   S5P_VA_GPIO, handle_level_irq);

                               ^^^^
Added spaces purposefully to make the parameters aligned properly in
the next line

But spaces should never be used for indentation.

Quoting from http://www.kernel.org/doc/Documentation/CodingStyle

"Outside of comments, documentation and except in Kconfig, spaces are never used for indentation"

nitpick: Should be TABS

Thanks&Regards
Padma

[snip]

--
Tushar Behera
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to