From mboxrd@z Thu Jan  1 00:00:00 1970
From: Stefan Roese <sr@denx.de>
Date: Fri, 24 Apr 2009 20:21:11 +0200
Subject: [U-Boot] [PATCH] ubi: Remove flash selection parameter
	(nor|nand|onenand) from "ubi part"
In-Reply-To: <20090424153441.GB27635@ld0162-tx32.am.freescale.net>
References: <1240581619-746-1-git-send-email-sr@denx.de>
	<20090424153441.GB27635@ld0162-tx32.am.freescale.net>
Message-ID: <200904242021.11701.sr@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

On Friday 24 April 2009, Scott Wood wrote:
> On Fri, Apr 24, 2009 at 04:00:19PM +0200, Stefan Roese wrote:
> > This patch removes the now unnecessary flash type parameter from the
> > "ubi part" command. Currently the user has to define the type of flash
> > he will be using UBI on. Example:
> >
> > => ubi part nor partition1
> >
> > With this patch this type parameter is not needed anymore. The user can
> > now select the partition directly without the flash type paramter.
> > Example:
> >
> > => ubi part partition1
> >
> > This breaks backward compatibility right now because of the change in the
> > command syntax. But UBI support is still quite fresh and the advantage of
> > this new command is syntax big enough for this change. Additionally the
> > code is much cleaner now.
>
> Yes, some polymorphism at last. :-)
>
> Now to do the same with cmd_nand/cmd_onenand...  and NOR flash for that
> matter.

That's a bigger task...

> > +		sprintf(mtd_dev, "%s%d", MTD_DEV_TYPE(dev->id->type), dev->id->num);
> > +		printf("%s: mtd_dev=%s!!!\n", __func__, mtd_dev); // test-only
> > +		ubi_dev.mtd_info = get_mtd_device_nm(mtd_dev);
> > +		printf("%s: mtd_info=%p!!!\n", __func__, ubi_dev.mtd_info); //
> > test-only
>
> Should these be debug()?

Ups. Thanks for catching.

> > +                if (IS_ERR(ubi_dev.mtd_info)) {
>
> s/spaces/tabs/

OK. Will fix and resubmit.

Thanks.

Best regards,
Stefan

=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: office at denx.de
=====================================================================