Re: [U-Boot-Users] [PATCH] Fill in remaining MTD driver data for OneNAND (take #2)

Don't cast the return of malloc.

ok.

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.

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.

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

cheers,

Fathi