From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1750830AbWFXPed (ORCPT ); Sat, 24 Jun 2006 11:34:33 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1750833AbWFXPed (ORCPT ); Sat, 24 Jun 2006 11:34:33 -0400 Received: from scrub.xs4all.nl ([194.109.195.176]:7896 "EHLO scrub.xs4all.nl") by vger.kernel.org with ESMTP id S1750830AbWFXPec (ORCPT ); Sat, 24 Jun 2006 11:34:32 -0400 Date: Sat, 24 Jun 2006 17:34:21 +0200 (CEST) From: Roman Zippel X-X-Sender: roman@scrub.home To: Roland Dreier cc: Andrew Morton , linux-kernel@vger.kernel.org, jbeulich@novell.com Subject: Re: Subject: [PATCH] reintegrate irqreturn.h into hardirq.h In-Reply-To: Message-ID: References: MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Hi, On Sat, 24 Jun 2006, Roland Dreier wrote: > > -extern fastcall irqreturn_t handle_IRQ_event(unsigned int irq, struct pt_regs *regs, > > +extern fastcall int handle_IRQ_event(unsigned int irq, struct pt_regs *regs, > > struct irqaction *action); > > This seems like a step backwards: this changes the declaration of > handle_IRQ_event() so it no longer matches the real definition. Then it's better to change the definition back as well, we generally want to avoid typedefs and this one is only a compatibility typedef meant for drivers. bye, Roman