[Top] [All Lists]

Re: [PATCH] Broadcom SWARM IDE in 2.6

To: "Steven J. Hill" <>
Subject: Re: [PATCH] Broadcom SWARM IDE in 2.6
From: Manish Lachwani <>
Date: Thu, 02 Dec 2004 12:18:35 -0800
In-reply-to: <>
Original-recipient: rfc822;
References: <> <>
User-agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.4.2) Gecko/20040308
Hi Steve,

"ide_init_default_irq" is defined in include/linux/ide.h and in

include/asm-mips/mach-generic/ide.h and include/asm-i386/ide.h are replicas. So, this compiler warning should appear on other platforms as well.

I have sent an email out to Jeff Garzik to understand why "ide_init_default_irq" is redefined

Manish Lachwani

Steven J. Hill wrote:
Manish Lachwani wrote:

I had sent an incomplete patch before. Please try out this new patch, attached.
Let me know if it works


This patch worked, however you need to fix a compiler warning before I am
willing to commit it. Please get rid of the warning shown below and submit
a new patch. Thanks!



CC      drivers/ide/ide-generic.o
In file included from drivers/ide/ide-generic.c:13:
include/linux/ide.h:277:1: warning: "ide_init_default_irq" redefined
In file included from include/asm/ide.h:11,
                 from include/linux/ide.h:271,
                 from drivers/ide/ide-generic.c:13:
include/asm-mips/mach-generic/ide.h:64:1: warning: this is the location of the previous definition

<Prev in Thread] Current Thread [Next in Thread>