Re: [2.4 PATCH] problems with modular and nonmodular ide mix

[Marcelo Tosatti]

On Mon, Jul 19, 2004 at 02:01:16PM +0300, Meelis Roos wrote:
> I was trying to figure out an unresolved modules symbol on PPC
> (2.4.27-BK): init_cmd640_vlb(). This is used in ide.c and defined in
> pci/cmd640.c. It appears that nothing from drivers/ide/pci is compiled
> at all - make traverses the directory but nothing gets compiled.
> Relevant part of .config is below.
> 
> So at least sl82c105 and cmd640 should get compiled but don't.
> 
> I suspected that since IDE is modular, only drivers configured as
> modules are compiled but not static ones. I set sl82c105 to M and it was
> compiled.
> 
> So the problem is that cmd640 can only be compiled in statically but not
> into modular IDE. What about the patch below to fix this? It builds
> and loads fine here, but it only fixes an obscure configuration.

Hi Meelis,

The thing is cmd640 can't be compiled as a module - just dont
use IDE modular if you need cmd640. 

> ===== drivers/ide/Config.in 1.41 vs edited =====
> --- 1.41/drivers/ide/Config.in	2004-05-22 23:30:37 +03:00
> +++ edited/drivers/ide/Config.in	2004-07-19 13:58:33 +03:00
> @@ -27,8 +27,10 @@
> 
>     comment 'IDE chipset support/bugfixes'
>     if [ "$CONFIG_BLK_DEV_IDE" != "n" ]; then
> -      dep_bool '  CMD640 chipset bugfix/support' CONFIG_BLK_DEV_CMD640 $CONFIG_X86
> -      dep_bool '    CMD640 enhanced support' CONFIG_BLK_DEV_CMD640_ENHANCED $CONFIG_BLK_DEV_CMD640
> +      dep_tristate '  CMD640 chipset bugfix/support' CONFIG_BLK_DEV_CMD640 $CONFIG_X86
> +      if [ "$CONFIG_BLK_DEV_CMD640" != "n" ]; then
> +         bool '    CMD640 enhanced support' CONFIG_BLK_DEV_CMD640_ENHANCED
> +      fi
>        dep_bool '  ISA-PNP EIDE support' CONFIG_BLK_DEV_ISAPNP $CONFIG_ISAPNP
>        if [ "$CONFIG_PCI" = "y" ]; then
>  	 bool '  PCI IDE chipset support' CONFIG_BLK_DEV_IDEPCI
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/