[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <C3050A4DBA34F345975765E43127F10F1C061F3D@szxeml512-mbs.china.huawei.com>
Date: Tue, 6 Aug 2013 10:53:01 +0000
From: Caizhiyong <caizhiyong@...wei.com>
To: Andrew Morton <akpm@...ux-foundation.org>
CC: Karel Zak <kzak@...hat.com>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"Wanglin (Albert)" <albert.wanglin@...wei.com>,
Quyaxin <quyaxin@...wei.com>, Jens Axboe <axboe@...nel.dk>,
David Woodhouse <dwmw2@...radead.org>,
Marius Groeger <mag@...go.de>
Subject: RE: [PATCH] block: support embedded device command line partition
> -----Original Message-----
> From: Andrew Morton [mailto:akpm@...ux-foundation.org]
> Sent: Tuesday, August 06, 2013 6:22 AM
> To: Caizhiyong
> Cc: Karel Zak; linux-kernel@...r.kernel.org; Wanglin (Albert); Quyaxin; Jens Axboe;
> David Woodhouse; Marius Groeger
> Subject: Re: [PATCH] block: support embedded device command line partition
>
> On Sat, 3 Aug 2013 09:57:04 +0000 Caizhiyong <caizhiyong@...wei.com> wrote:
>
> > From: Cai Zhiyong <caizhiyong@...wei.com>
> >
> > Read block device partition table from command line.
> > The partition used for fixed block device (eMMC) embedded device.
> > It is no MBR, save storage space. Bootloader can be easily accessed by
> > absolute address of data on the block device.
> > Users can easily change the partition.
> >
> > This code reference MTD partition, source "drivers/mtd/cmdlinepart.c"
> > About the partition verbose reference
> "Documentation/block/cmdline-partition.txt"
> >
>
> Seems OK to me.
>
> The obvious question is: can we consolidate this with
> drivers/mtd/cmdlinepart.c in some fashion so the kernel doesn't have
> two parsers which do the same thing?
I will move the parser to "lib/cmdline.c".
Looking forward to your feedback.
Best regards,
Cai Zhiyong.
http://www.huawei.com
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists