From mboxrd@z Thu Jan  1 00:00:00 1970
From: Wolfgang Denk <wd@denx.de>
Date: Sun, 08 Mar 2009 23:12:46 +0100
Subject: [U-Boot] [PATCH] Seperate mtdpart command from jffs2
In-Reply-To: <200903061413.50906.sr@denx.de>
References: <20090303050321.GA29890@july> <200903061413.50906.sr@denx.de>
Message-ID: <20090308221246.B91A3832E8B8@gemini.denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Dear Stefan Roese,

In message <200903061413.50906.sr@denx.de> you wrote:
> 
> >  COBJS-$(CONFIG_CMD_IRQ) += cmd_irq.o
> >  COBJS-$(CONFIG_CMD_ITEST) += cmd_itest.o
> >  COBJS-$(CONFIG_CMD_JFFS2) += cmd_jffs2.o
> > +COBJS-$(CONFIG_JFFS2_CMDLINE) += cmd_mtdparts.o
> 
> Shouldn't we change this define (CONFIG_JFFS2_CMDLINE) to something else now, 
> since it's not really JFFS2 specific anymore? I suggest to change it to 
> CONFIG_CMD_MTDPARTS. This seems more logical to me.
> 
> What do you think? 

I agree - CONFIG_CMD_MTDPARTS is much better - CONFIG_CMD_MTDPARTS is
much better. Thanks.


Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
"Spock, did you see the looks on their faces?"
"Yes, Captain, a sort of vacant contentment."