Dear Jagan Teki,

In message <cad6g_rqtzzinzcx0halqr-jd4cjm8wyjzzggarz-+dwazjn...@mail.gmail.com> 
you wrote:
> 
> I thought it could be a good progress prints for sf read/write/erase
> commands as we didn't have
> these verbose prints before.
> 
> Do you think these are useful/required messages?, please take my above 
> concerns.

Some people will like such verbocity, others will dislike it.  Also,
some people will dislike th memory footprint that comes with the added
code.  So in any case this should be configurable.

But most of all: SF is just one storage device out of a much larger
list.  If we add such a feature, it should be done in common code and
in a generic way that all similar devices can use as well.

Please see again the other thread, and Simon's response.  I think it
makes sense to handle these things together, as a common set of
patches.


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: w...@denx.de
The only person who always got his work done by Friday
                                                 was Robinson Crusoe.
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to