Dear Jason Liu, In message <[email protected]> you wrote: > > I set it according to the following reason, > > - Keep the same setting as the original when you commit the mx51 > support patch. Why you select 128KB? :)
Copying settings without reflecting if they make sense for your own system is not really a good idea. > - As I looked through other platform such as OMAP4 for MMC ENV > setting, it's also set for 128KB Ditto. Please keep in mind that the environment is CRC protected; that means the larger the environment area, the more time will be needed for the CRC calculation when the system boots. And quick booting is definitely a topic these days, so please reconsider. > - Leave much room for the user to store customer env. Come on. Do you know how much text can be saved in 128 KiB? I have seen some really kinky environment settings before, where a "printenv" would scroll trough thousands of lines (well, hundrets at least) so it was next to impossible to find anything, and they needed something around 10 KiB or so. 16 KiB is more than enough for 99.9% of the users. > > Can we save some time by saving or reading the environment reducing its > > size ? Or do you plan to save something more in this area ? > > It's always the trade-off, set to 512B or less will save some time > according to 128KB, but it will face much risk to change the code > frequently to meet the increasing env size requirement. > what's the size do you think is suitable? 16 KiB. 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: [email protected] "Consistency requires you to be as ignorant today as you were a year ago." - Bernard Berenson _______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
