[2.6 patch] let BLK_DEV_AMD74XX depend on X86

[2.6 patch] let BLK_DEV_AMD74XX depend on X86

[Adrian Bunk]

It's unlikely that this driver will ever be of any use on other 
architectures.

This fixes the following compile error on ia64:

<--  snip  -->

...
  CC      drivers/ide/pci/amd74xx.o
/home/bunk/linux/kernel-2.6/linux-2.6.20-rc3-mm1/drivers/ide/pci/amd74xx.c: In function 'init_hwif_amd74xx':
/home/bunk/linux/kernel-2.6/linux-2.6.20-rc3-mm1/drivers/ide/pci/amd74xx.c:421: warning: implicit declaration of function 'pci_get_legacy_ide_irq'
  CC      drivers/ide/pci/cmd64x.o
...
  LD      .tmp_vmlinux1
drivers/built-in.o: In function `init_hwif_amd74xx':
/home/bunk/linux/kernel-2.6/linux-2.6.20-rc3-mm1/drivers/ide/pci/amd74xx.c:421: undefined reference to `pci_get_legacy_ide_irq'
make[1]: *** [.tmp_vmlinux1] Error 1

<--  snip  -->


This fixes kernel Bugzilla #6644.


Signed-off-by: Adrian Bunk <bunk@stusta.de>

--- linux-2.6.20-rc3-mm1/drivers/ide/Kconfig.old	2007-01-11 11:15:24.000000000 +0100
+++ linux-2.6.20-rc3-mm1/drivers/ide/Kconfig	2007-01-11 11:15:37.000000000 +0100
@@ -478,6 +478,7 @@
 
 config BLK_DEV_AMD74XX
 	tristate "AMD and nVidia IDE support"
+	depends on X86
 	help
 	  This driver adds explicit support for AMD-7xx and AMD-8111 chips
 	  and also for the nVidia nForce chip.  This allows the kernel to

Re: [2.6 patch] let BLK_DEV_AMD74XX depend on X86

[Alan]

On Thu, 11 Jan 2007 14:49:17 +0100
Adrian Bunk <bunk@stusta.de> wrote:

> It's unlikely that this driver will ever be of any use on other 
> architectures.
> 
> This fixes the following compile error on ia64:

NAK

pci_get_legacy_ide_irq() is a required method for all platforms and is
usually filled in by asm-generic/pci.h

Please fix the IA64 tree to do the right thing.

Alan

Re: [2.6 patch] let BLK_DEV_AMD74XX depend on X86

[Bartlomiej Zolnierkiewicz]

On 1/11/07, Alan <alan@lxorguk.ukuu.org.uk> wrote:
> On Thu, 11 Jan 2007 14:49:17 +0100
> Adrian Bunk <bunk@stusta.de> wrote:
>
> > It's unlikely that this driver will ever be of any use on other
> > architectures.
> >
> > This fixes the following compile error on ia64:
>
> NAK
>
> pci_get_legacy_ide_irq() is a required method for all platforms and is
> usually filled in by asm-generic/pci.h
>
> Please fix the IA64 tree to do the right thing.

I've already sent a patch do do this...

http://lkml.org/lkml/2007/1/11/118

Re: [2.6 patch] let BLK_DEV_AMD74XX depend on X86

[Segher Boessenkool]

> It's unlikely that this driver will ever be of any use on other
> architectures.

It's already being used, for example, the AMD8111 is used
one some PowerPC systems (some with IDE connected even).


Segher