Dear Stephen, In message <56167db6.3000...@wwwdotorg.org> you wrote: > > >> What's the expected correct behavior here? > > > > After successful loading the data to memory, load_addr should be set > > correctly, for all commands. In the error case, the value of > > load_addr is undefined. > > Is this documented anywhere? If not, I'm not convinced that there's a > contract to be followed; it "just happens" that some filesystem-related > commands work(ed) that way (and as Benoît pointed out, apparently some > don't irrespective of the mentioned patch).
I'm afraid it's not documented, but it is what I would consider a sane and consistent behaviour. If we intend to implement POLA [1] (and I very much think we should), this is how U-Boot should behave. [1] https://en.wikipedia.org/wiki/Principle_of_least_astonishment Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de What we hope ever to do with ease, we must learn first to do with diligence. - Samuel Johnson _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
