* Maciej W. Rozycki <macro@xxxxxxxxxxxxxx> [2006-02-20 13:33]: > > - If you want to compile this driver as a module ( = code which can be > > - inserted in and removed from the running kernel whenever you want), > > - say M here and read <file:Documentation/modules.txt>. The module will > > - be called ms02-nv.o. > > - > NAK. I disagree with you - it's obvious from the name of the config option (MTD_MS02NV) what the module will be called and people who compile their own kernels should know. More importantly, no other description in this Kconfig file mentions anything like this. However, since there seems to be no Kconfig "writing style", I guess it must as well be added. But I guess the MTD folks removed it for a reason. Still NAK, i.e. should I send the following patch to the MTD list, or ok to remove? [PATCH] Add information about modules to MTD_MS02NV - sync with linux-mips tree This synces drivers/mtd/devices/Kconfig with the linux-mips tree by adding some description about modules to MTD_MS02NV. Signed-off-by: Martin Michlmayr <tbm@xxxxxxxxxx> Acked-by: Maciej W. Rozycki <macro@xxxxxxxxxxxxxx> --- --- a/drivers/mtd/devices/Kconfig 2006-02-03 03:07:01.000000000 +0000 +++ b/drivers/mtd/devices/Kconfig 2006-02-19 23:50:11.000000000 +0000 @@ -47,6 +47,11 @@ accelerator. Say Y here if you have a DECstation 5000/2x0 or a DECsystem 5900 equipped with such a module. + If you want to compile this driver as a module ( = code which can be + inserted in and removed from the running kernel whenever you want), + say M here and read <file:Documentation/modules.txt>. The module will + be called ms02-nv.o. + config MTD_DATAFLASH tristate "Support for AT45xxx DataFlash" depends on MTD && SPI_MASTER && EXPERIMENTAL > > struct net_device *dev = root_lance_dev; > > struct lance_private *lp = netdev_priv(dev); > > - > > unregister_netdev(dev); > > #ifdef CONFIG_TC > > if (lp->slot >= 0) > > NAK. Upated patch sent to netdev. Thanks. -- Martin Michlmayr http://www.cyrius.com/
