breakage with CONFIG_BLK_DEV_IDE=m

Robert Millan <[email protected]>
Newsgroups gmane.linux.ports.xbox.devel
Message-ID <[email protected]>
Hi!

Attempting to build Linux 2.4.x from xbox-linux CVS fails when
CONFIG_BLK_DEV_IDE=m (instead of =y):

xbox.c: In function `xbox_partition':
xbox.c:106: error: `ide_drive_t' undeclared (first use in this function)
xbox.c:106: error: (Each undeclared identifier is reported only once
xbox.c:106: error: for each function it appears in.)
xbox.c:106: error: `drive' undeclared (first use in this function)
xbox.c:116: warning: implicit declaration of function `ide_info_ptr'
xbox.c:108: warning: unused variable `last_lba28_sector'

At first glance it looks like <linux/ide.h> should be include unconditionaly
(since ide_drive_t is used unconditionaly), but the dependency is also in
ide-related symbols and linking would fail later if we fix that.

I don't know the configure magic to force build xbox.c as a module if ide has
been configured as module too.  Could someone please have a look?

A .config diff that produces the error (against the kernel.config in CVS)
follows:

--- /tmp/kernel.config	2004-09-15 13:40:08.000000000 +0200
+++ .config	2004-09-15 13:40:44.000000000 +0200
@@ -243,14 +243,14 @@
 #
 # IDE, ATA and ATAPI Block devices
 #
-CONFIG_BLK_DEV_IDE=y
+CONFIG_BLK_DEV_IDE=m
 # CONFIG_BLK_DEV_HD_IDE is not set
 # CONFIG_BLK_DEV_HD is not set
-CONFIG_BLK_DEV_IDEDISK=y
+CONFIG_BLK_DEV_IDEDISK=m
 CONFIG_IDEDISK_MULTI_MODE=y
 # CONFIG_IDEDISK_STROKE is not set
 # CONFIG_BLK_DEV_IDECS is not set
-CONFIG_BLK_DEV_IDECD=y
+CONFIG_BLK_DEV_IDECD=m
 # CONFIG_BLK_DEV_IDETAPE is not set
 # CONFIG_BLK_DEV_IDEFLOPPY is not set
 # CONFIG_BLK_DEV_IDESCSI is not set

-- 
 .''`.   Proudly running Debian GNU/kFreeBSD unstable/unreleased (on UFS2+S)
: :' :
`. `'    http://www.debian.org/ports/kfreebsd-gnu
  `-


-------------------------------------------------------
This SF.Net email is sponsored by: thawte's Crypto Challenge Vl
Crack the code and win a Sony DCRHC40 MiniDV Digital Handycam
Camcorder. More prizes in the weekly Lunch Hour Challenge.
Sign up NOW http://ad.doubleclick.net/clk;10740251;10262165;m
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.