On Thu, Jan 10, 2013 at 08:38:28AM -0600, Rob Herring wrote:

On 01/10/2013 05:02 AM, Russell King - ARM Linux wrote:

quoted

On Thu, Jan 03, 2013 at 11:54:16AM -0600, Rob Herring wrote:

quoted

-asmlinkage void __exception_irq_entry gic_handle_irq(struct pt_regs *regs)
+static asmlinkage void __exception_irq_entry gic_handle_irq(struct pt_regs *regs)

NAK.  No, not "static asmlinkage".

Okay will remove asmlinkage, but is there an issue with making this static?

No, just for the asmlinkage.  asmlinkage is used to mark functions which
are called from assembly.  Making them static generally means you can't
call them from assembly (unless you have asm() code in the .c file which
does, like our kernel_thread() implementation used to).

Ok, asmlinkage on ARM is a no-op, but it's useful cheap documentation to
mark which C functions get called from assembly.