[PATCH] flash_eraseall: Remove warning

[Boris Brezillon]

On Wed, 27 Apr 2016 10:27:33 +0200
Richard Weinberger <richard at nod.at> wrote:

> Am 27.04.2016 um 10:21 schrieb Boris Brezillon:
> >> Well, flash_erase <mtddev> 0 0 is the same as flash_eraseall <mtddev>.
> >> But flash_eraseall is still more handy to use.
> >> Let's keep flash_eraseall as short cut and don't force users to
> >> use a less handy user interface.  
> > 
> > Hm, I'm not convinced by this change. Aren't we trying to make people
> > switch to flash_erase instead of using flash_eraseall.
> > If you want to make flash_erase more easy to use, you can always erase
> > the whole partition if only <mtddev> is passed in arguments, which IMO
> > would be clearer than having to pass 0 0.  
> 
> So, you suggest fixing flash_erase such that a "flash_erase /dev/mtd0"
> works and acts like "flash_erase /dev/mtd0 0 0"?

Yep.

> 
> Would work too. :-)
> 
> Thanks,
> //richard



-- 
Boris Brezillon, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com