[U-Boot] [PATCH V4 1/2] ext4fs ls load support

[Wolfgang Denk]

Dear UMA SHANKAR,

please do not top post / full quote.

In message <27542660.71361332746650945.JavaMail.weblogic at epml04> you wrote:
>
> Currently, Our ext4 implementation is capable of listing and reading (ls and load) ext2 partitions as well. 
> But, we wanted the ext4 code to be separate from ext2 implementation. 

This does not make much sense to me.

> So, I propose, we can let both these implementations be there in
> uboot under different CONFIG Options (as is the case currently).

What would be the rationale here?

I think more and more people are going to use ext4.  It makes no sense
to have different commands and different options to support both ext2
(and ext3, to some degree) and ext4 (and ext3?) file systems.  And
especially, it makes no sense to dusplicate large amounts of code.

Please use common code whererver possible.

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: wd at denx.de
"I knew then (in 1970) that a 4-kbyte minicomputer would cost as much
as a house. So I reasoned  that  after  college,  I'd  have  to  live
cheaply in an apartment and put all my money into owning a computer."
      - Apple co-founder Steve Wozniak, EE Times, June 6, 1988, pg 45