From mboxrd@z Thu Jan 1 00:00:00 1970 From: Wolfgang Denk Date: Fri, 22 Mar 2013 22:12:03 +0100 Subject: [U-Boot] [PATCH] README: document the requirements for CONFIG_SYS_HZ In-Reply-To: <1363923692-12748-1-git-send-email-swarren@wwwdotorg.org> References: <1363923692-12748-1-git-send-email-swarren@wwwdotorg.org> Message-ID: <20130322211203.BA4752014CE@gemini.denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Dear Stephen, In message <1363923692-12748-1-git-send-email-swarren@wwwdotorg.org> you wrote: > CONFIG_SYS_HZ must be 1000, and get_timer() must therefore return mS. > Document this. "must return mS" ? "mS" means Milli-Siemens, which is a unit of electric conductance and electric admittance. You mean "ms" = milliseconds ... > +- CPU timer options: > + CONFIG_SYS_HZ > + > + The frequency of the timer returned by get_timer(). This value > + MUST be 1000. This is misleading, or even incorrect. If I read this, I would expect that get_timer() returns 1000. Obviously, this is incorrect. Instead, get_timer() returns timestamp information in millisecond resolution. 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 Crash programs fail because they are based on the theory that, with nine women pregnant, you can get a baby a month. - Wernher von Braun