From mboxrd@z Thu Jan  1 00:00:00 1970
From: Stephen Warren <swarren@wwwdotorg.org>
Date: Fri, 22 Mar 2013 15:18:38 -0600
Subject: [U-Boot] [PATCH] README: document the requirements for
	CONFIG_SYS_HZ
In-Reply-To: <20130322211203.BA4752014CE@gemini.denx.de>
References: <1363923692-12748-1-git-send-email-swarren@wwwdotorg.org>
	<20130322211203.BA4752014CE@gemini.denx.de>
Message-ID: <514CCAAE.6020909@wwwdotorg.org>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On 03/22/2013 03:12 PM, Wolfgang Denk wrote:
> 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 ...

It seems that's true.

>> +- 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.

That is not what that sentence means.