All of lore.kernel.org
 help / color / mirror / Atom feed
From: Greg Ungerer <gerg@snapgear.com>
To: Geert Uytterhoeven <geert@linux-m68k.org>
Cc: Greg Ungerer <gerg@uclinux.org>,
	uClinux list <uclinux-dev@uclinux.org>,
	Linux/m68k <linux-m68k@lists.linux-m68k.org>
Subject: Re: [PATCH] m68knommu: Move forward declaration of do_IRQ() from machdep.h to irq.h
Date: Mon, 4 Jul 2011 10:26:03 +1000	[thread overview]
Message-ID: <4E11089B.5070001@snapgear.com> (raw)
In-Reply-To: <alpine.DEB.2.00.1107031108390.6100@ayla.of.borg>

Hi Geert,

On 03/07/11 19:09, Geert Uytterhoeven wrote:
> Oops, there's also an asmlinkage missing. Corrected patch below.
>
> from 63dab1c663875083f095819431bbc6560c2a8dfb Mon Sep 17 00:00:00 2001
> From: Geert Uytterhoeven<geert@linux-m68k.org>
> Date: Sun, 3 Jul 2011 10:49:58 +0200
> Subject: [PATCH] m68knommu: Move forward declaration of do_IRQ() from machdep.h to irq.h
>
> It is not machine-specific, but common irq infrastructure.
> Also add the missing asmlinkage, to match its definition.
>
> Signed-off-by: Geert Uytterhoeven<geert@linux-m68k.org>

Acked-by: Greg Ungerer <gerg@uclinux.org>

Looks good. Are you going to push this one, or do you want
me too?

Regards
Greg


> ---
> I will need this for m68k genirq soon.
>
>   arch/m68k/include/asm/irq.h     |    2 ++
>   arch/m68k/include/asm/machdep.h |    1 -
>   2 files changed, 2 insertions(+), 1 deletions(-)
>
> diff --git a/arch/m68k/include/asm/irq.h b/arch/m68k/include/asm/irq.h
> index ddc5958..abde607 100644
> --- a/arch/m68k/include/asm/irq.h
> +++ b/arch/m68k/include/asm/irq.h
> @@ -160,4 +160,6 @@ extern unsigned int irq_canonicalize(unsigned int irq);
>   #define irq_canonicalize(irq)  (irq)
>   #endif /* CONFIG_MMU */
>
> +asmlinkage void do_IRQ(int irq, struct pt_regs *regs);
> +
>   #endif /* _M68K_IRQ_H_ */
> diff --git a/arch/m68k/include/asm/machdep.h b/arch/m68k/include/asm/machdep.h
> index 415d548..789f3b2 100644
> --- a/arch/m68k/include/asm/machdep.h
> +++ b/arch/m68k/include/asm/machdep.h
> @@ -40,6 +40,5 @@ extern unsigned long hw_timer_offset(void);
>   extern irqreturn_t arch_timer_interrupt(int irq, void *dummy);
>
>   extern void config_BSP(char *command, int len);
> -extern void do_IRQ(int irq, struct pt_regs *fp);
>
>   #endif /* _M68K_MACHDEP_H */


-- 
------------------------------------------------------------------------
Greg Ungerer  --  Principal Engineer        EMAIL:     gerg@snapgear.com
SnapGear Group, McAfee                      PHONE:       +61 7 3435 2888
8 Gardner Close                             FAX:         +61 7 3217 5323
Milton, QLD, 4064, Australia                WEB: http://www.SnapGear.com

  reply	other threads:[~2011-07-04  0:26 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-07-03  8:55 [PATCH] m68knommu: Move forward declaration of do_IRQ() from machdep.h to irq.h Geert Uytterhoeven
2011-07-03  9:09 ` Geert Uytterhoeven
2011-07-04  0:26   ` Greg Ungerer [this message]
2011-07-04 19:55     ` Geert Uytterhoeven

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4E11089B.5070001@snapgear.com \
    --to=gerg@snapgear.com \
    --cc=geert@linux-m68k.org \
    --cc=gerg@uclinux.org \
    --cc=linux-m68k@lists.linux-m68k.org \
    --cc=uclinux-dev@uclinux.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.