Dear Jean-Christophe PLAGNIOL-VILLARD, In message <20090806202615.gh13...@game.jcrosoft.org> you wrote: > > as we are all supposed to have CONFIG_SYS_HZ at 1000 (mandtory) > to have cfi, tftp & co working perfectly I do not thing this is a good idea
Yes, this is the rule, and we would like to enforce it. > as you will need to fix each part of u-boot that use CONFIG_SYS_HZ > which make no sense > > the best will be to simply fix your timer However, the current situation is this: more than 60 boards (all of them ARM, it seems) use very different settings: include/configs/EB+MCF-EV123.h: 10000000 include/configs/EP1C20.h: (CONFIG_SYS_CLK_FREQ/(CONFIG_SYS_NIOS_TMRCNT + 1)) include/configs/EP1S10.h: (CONFIG_SYS_CLK_FREQ/(CONFIG_SYS_NIOS_TMRCNT + 1)) include/configs/EP1S40.h: (CONFIG_SYS_CLK_FREQ/(CONFIG_SYS_NIOS_TMRCNT + 1)) include/configs/KAREF.h: 100 include/configs/M5271EVB.h: 1000000 include/configs/METROBOX.h: 100 include/configs/MVBLUE.h: 10000 include/configs/PCI5441.h: (CONFIG_SYS_CLK_FREQ/(CONFIG_SYS_NIOS_TMRCNT + 1)) include/configs/PK1C20.h: (CONFIG_SYS_CLK_FREQ/(CONFIG_SYS_NIOS_TMRCNT + 1)) include/configs/SMN42.h: 2048 include/configs/VCMA9.h: 1562500 include/configs/actux1.h: 3333333 include/configs/actux2.h: 3333333 include/configs/actux3.h: 3333333 include/configs/actux4.h: 3333333 include/configs/apollon.h: ((CONFIG_SYS_CLK_FREQ)/(2 << CONFIG_SYS_PTV)) include/configs/armadillo.h: 2000 include/configs/assabet.h: 3686400 include/configs/at91rm9200dk.h: AT91C_MASTER_CLOCK/2 include/configs/at91rm9200ek.h: (AT91C_MASTER_CLOCK / 2) include/configs/cmc_pu2.h: (AT91C_MASTER_CLOCK/2) include/configs/csb637.h: AT91C_MASTER_CLOCK/2 include/configs/davinci_dm355evm.h: 24000000 include/configs/davinci_dvevm.h: 27000000 include/configs/davinci_schmoogie.h: 27000000 include/configs/davinci_sffsdr.h: 27000000 include/configs/davinci_sonata.h: 27000000 include/configs/dnp1110.h: 3686400 include/configs/eNET.h: 1024 include/configs/ep7312.h: 2000 include/configs/gcplus.h: 3686400 include/configs/idmr.h: (50000000 / 64) include/configs/impa7.h: 2000 include/configs/integratorap.h: 24000000 include/configs/integratorcp.h: 1000000 include/configs/ixdp425.h: 3333333 include/configs/kb9202.h: AT91C_MASTER_CLOCK/2 include/configs/lart.h: 3686400 include/configs/lpc2292sodimm.h: 2048 include/configs/lpd7a400-10.h: (508469) include/configs/lpd7a404-10.h: (508469) include/configs/m501sk.h: AT91C_MASTER_CLOCK/2 include/configs/modnet50.h: 900 include/configs/mp2usb.h: (AT91C_MASTER_CLOCK/2) include/configs/mx1ads.h: 3686400 include/configs/mx1fs2.h: 3686400 include/configs/ns9750dev.h: (CPU_CLK_FREQ/64) include/configs/omap1610h2.h: ((CONFIG_SYS_CLK_FREQ)/(2 << CONFIG_SYS_PTV)) include/configs/omap1610inn.h: ((CONFIG_SYS_CLK_FREQ)/(2 << CONFIG_SYS_PTV)) include/configs/omap2420h4.h: ((CONFIG_SYS_CLK_FREQ)/(2 << CONFIG_SYS_PTV)) include/configs/omap3_zoom2.h: ((V_SCLK) / (2 << CONFIG_SYS_PTV)) include/configs/omap5912osk.h: ((CONFIG_SYS_CLK_FREQ)/(2 << CONFIG_SYS_PTV)) include/configs/omap730p2.h: ((CONFIG_SYS_CLK_FREQ)/(2 << CONFIG_SYS_PTV)) include/configs/rsk7203.h: (CONFIG_SYS_CLK_FREQ / CMT_CLK_DIVIDER) include/configs/sbc2410x.h: 1562500 include/configs/sc520_cdp.h: 1024 include/configs/sc520_spunk.h: 1024 include/configs/scb9328.h: 3686400 include/configs/shannon.h: 3686400 include/configs/smdk2400.h: 1562500 include/configs/smdk2410.h: 1562500 include/configs/trab.h: 1562500 include/configs/versatile.h: (1000000 / 256) Until all these are fixed, it indeed makes sense to work around issues cause by such incorrect settings. 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 As usual, this being a 1.3.x release, I haven't even compiled this kernel yet. So if it works, you should be doubly impressed. - Linus Torvalds in <199506181536.saa10...@keos.cs.helsinki.fi> _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot