Re: [PATCH] ARM: beaglebone: add delay in lowlevel.c

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello Sascha,

On Wed, Jan 08, 2025 at 03:32:04PM +0100, Sascha Hauer wrote:

> Calling udelay(1000) and adding a comment saying it delays 1.8ms looks
> inconsistent. Maybe better count up to 2 in __udelay() above which makes

I completely agree somehow since the time is not even constant and depends on
the PLL settings just done before.

I suggest the following:

Removing the "* 3" fancy thingy in the function's loop, calling the
function with 3000 instead of 1000, changing comment to "needed on some
Beaglebone Black for warm start after reset".

That would be as simple as possible.

Regards
Konsti


-- 
INSIDE M2M GmbH
Konstantin Kletschke
Berenbosteler Straße 76 B
30823 Garbsen

Telefon: +49 (0) 5137 90950136
Mobil: +49 (0) 151 15256238
Fax: +49 (0) 5137 9095010

konstantin.kletschke@xxxxxxxxxxxxx
http://www.inside-m2m.de 

Geschäftsführung: Michael Emmert, Derek Uhlig
HRB: 111204, AG Hannover





[Index of Archives]     [Linux Embedded]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux