mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* Configure/compile error with 2.5.38
@ 2002-09-23 15:41 Matthew Wilcox
  2002-09-23 17:27 ` Andre Hedrick
  0 siblings, 1 reply; 2+ messages in thread
From: Matthew Wilcox @ 2002-09-23 15:41 UTC (permalink / raw)
  To: andre; +Cc: linux-kernel


Got an interesting one here; this combination fails to link:

CONFIG_BLK_DEV_CMD640=y
# CONFIG_BLK_DEV_IDEPCI is not set

Why not?  Well.. cmd640.c is in drivers/ide/pci/ which is only
entered if CONFIG_BLK_DEV_IDEPCI, and if CONFIG_BLK_DEV_CMD640 is set,
drivers/ide/ide.c contains a reference to cmd640_vlb which is defined
in cmd640.c.

The "obvious" solution to make CMD640 depend on IDEPCI is not correct
because of the aforementioned VLB controllers.  Maybe we could just add
the line:
obj-$(CONFIG_BLK_DEV_CMD640)            += pci/
to drivers/ide/Makefile which is a bit of a kludge..

-- 
Revolutions do not require corporate support.

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: Configure/compile error with 2.5.38
  2002-09-23 15:41 Configure/compile error with 2.5.38 Matthew Wilcox
@ 2002-09-23 17:27 ` Andre Hedrick
  0 siblings, 0 replies; 2+ messages in thread
From: Andre Hedrick @ 2002-09-23 17:27 UTC (permalink / raw)
  To: Matthew Wilcox; +Cc: linux-kernel, Alan Cox, Jens Axboe


Correct, that is why I had it scheduled to be moved into legacy.
However until many issuse are settle down minor fixes like this are on
hold.

On Mon, 23 Sep 2002, Matthew Wilcox wrote:

> 
> Got an interesting one here; this combination fails to link:
> 
> CONFIG_BLK_DEV_CMD640=y
> # CONFIG_BLK_DEV_IDEPCI is not set
> 
> Why not?  Well.. cmd640.c is in drivers/ide/pci/ which is only
> entered if CONFIG_BLK_DEV_IDEPCI, and if CONFIG_BLK_DEV_CMD640 is set,
> drivers/ide/ide.c contains a reference to cmd640_vlb which is defined
> in cmd640.c.
> 
> The "obvious" solution to make CMD640 depend on IDEPCI is not correct
> because of the aforementioned VLB controllers.  Maybe we could just add
> the line:
> obj-$(CONFIG_BLK_DEV_CMD640)            += pci/
> to drivers/ide/Makefile which is a bit of a kludge..
> 
> -- 
> Revolutions do not require corporate support.
> 

Andre Hedrick
LAD Storage Consulting Group


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2002-09-23 19:32 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2002-09-23 15:41 Configure/compile error with 2.5.38 Matthew Wilcox
2002-09-23 17:27 ` Andre Hedrick

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®