From mboxrd@z Thu Jan  1 00:00:00 1970
From: Scott Wood <scottwood@freescale.com>
Date: Wed, 06 Aug 2008 11:37:54 -0500
Subject: [U-Boot-Users] [PATCH] Fill in remaining MTD driver data for
	OneNAND (take #2)
In-Reply-To: <200808061832.04718.fabo@debian.org>
References: <200808061832.04718.fabo@debian.org>
Message-ID: <4899D362.4090802@freescale.com>
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

Fathi BOUDRA wrote:
>> Why not just declare a static array?
> 
> I tried with a static array but it doesn't give the expected result (a quick 
> test with onenand info command returns an empty mtd name), so I used a 
> pointer.

Odd... Maybe a relocation issue?

>> It'd be better to use snprintf, even if you're pretty sure it won't
>> overflow.
> 
> I tried it too :) U-Boot doesn't have snprintf or asprintf or I missed 
> something.

Hmm, so it doesn't.  That sucks.

> Do I need to re-submit the patch only for cast ?

No, I'll fix it and apply to nand-flash/testing.

-Scott