From mboxrd@z Thu Jan 1 00:00:00 1970 From: Wolfgang Denk Date: Wed, 26 Jan 2011 08:50:29 +0100 Subject: [U-Boot] [PATCH V2 2/5] ARM: Add Support for Marvell Pantheon Familiy SoCs In-Reply-To: References: <1294632087-8025-1-git-send-email-leiwen@marvell.com> <1294632087-8025-3-git-send-email-leiwen@marvell.com> <20110125213200.2FB46B187@gemini.denx.de> Message-ID: <20110126075029.1218DBB0B9@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 Lei Wen, In message you wrote: > > >> + while (loop--); > > > > Please write: > > > > while (loop--) > > ; > > Fixed... > > > > But then - are you sure the compiler does not optimize this out? You > > probably want to use __udelay() instead. > > From the practice, we think this loop is enough to make timer stablize... There is nothing in this code to prevent the cmpiler from optimizing this out. You can as well delete these lines then. If you need a delay, then you must do better. > Involve the __udelay() may not suitable for the timer functions... Depends... 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 "The pathology is to want control, not that you ever get it, because of course you never do." - Gregory Bateson