* [patch] dont force uclinux mtd map to be root dev
@ 2007-04-13 12:19 Mike Frysinger
2007-04-13 12:28 ` Christoph Hellwig
0 siblings, 1 reply; 2+ messages in thread
From: Mike Frysinger @ 2007-04-13 12:19 UTC (permalink / raw)
To: greg; +Cc: michele.damico, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 1177 bytes --]
the cheesy uclinux mtd maps can be used for more than just the root device, so
i think we should drop the forcing. also, i feel like this is a policy
decision that shouldnt be in the kernel in the first place. people who have
been lazy and boot with uclinux mtd maps and dont put root= into their
commandline can simply add the appropriate root= line either into their
bootloader or into the compiled in bootargs
Signed-off-by: Mike Frysinger <vapier@gentoo.org>
---
diff --git a/drivers/mtd/maps/uclinux.c b/drivers/mtd/maps/uclinux.c
index 389fea2..14ffb1a 100644
--- a/drivers/mtd/maps/uclinux.c
+++ b/drivers/mtd/maps/uclinux.c
@@ -16,7 +16,6 @@
#include <linux/kernel.h>
#include <linux/fs.h>
#include <linux/major.h>
-#include <linux/root_dev.h>
#include <linux/mtd/mtd.h>
#include <linux/mtd/map.h>
#include <linux/mtd/partitions.h>
@@ -89,10 +88,6 @@ int __init uclinux_mtd_init(void)
uclinux_ram_mtdinfo = mtd;
add_mtd_partitions(mtd, uclinux_romfs, NUM_PARTITIONS);
- printk("uclinux[mtd]: set %s to be root filesystem\n",
- uclinux_romfs[0].name);
- ROOT_DEV = MKDEV(MTD_BLOCK_MAJOR, 0);
-
return(0);
}
[-- Attachment #2: Type: application/pgp-signature, Size: 827 bytes --]
^ permalink raw reply [flat|nested] 2+ messages in thread* Re: [patch] dont force uclinux mtd map to be root dev
2007-04-13 12:19 [patch] dont force uclinux mtd map to be root dev Mike Frysinger
@ 2007-04-13 12:28 ` Christoph Hellwig
0 siblings, 0 replies; 2+ messages in thread
From: Christoph Hellwig @ 2007-04-13 12:28 UTC (permalink / raw)
To: Mike Frysinger; +Cc: greg, michele.damico, linux-kernel
On Fri, Apr 13, 2007 at 08:19:31AM -0400, Mike Frysinger wrote:
> the cheesy uclinux mtd maps can be used for more than just the root device, so
> i think we should drop the forcing. also, i feel like this is a policy
> decision that shouldnt be in the kernel in the first place. people who have
> been lazy and boot with uclinux mtd maps and dont put root= into their
> commandline can simply add the appropriate root= line either into their
> bootloader or into the compiled in bootargs
Agreed. I remember trying to get rid of this a while ago, but I can't
remember what happened to it.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2007-04-13 12:28 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2007-04-13 12:19 [patch] dont force uclinux mtd map to be root dev Mike Frysinger
2007-04-13 12:28 ` Christoph Hellwig
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®