Date: Sat, 27 Dec 2014 12:55:48 -0800 From: Rui Paulo <rpaulo@me.com> To: Luiz Otavio O Souza <loos@FreeBSD.org> Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r276297 - head/sys/arm/broadcom/bcm2835 Message-ID: <B8E275B2-A50A-4EA1-87B3-72C9F94E6B48@me.com> In-Reply-To: <201412271352.sBRDqY6M098448@svn.freebsd.org> References: <201412271352.sBRDqY6M098448@svn.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Dec 27, 2014, at 05:52, Luiz Otavio O Souza <loos@FreeBSD.org> wrote: >=20 > Author: loos > Date: Sat Dec 27 13:52:33 2014 > New Revision: 276297 > URL: https://svnweb.freebsd.org/changeset/base/276297 >=20 > Log: > On interrupt handler, save the actual data read from mbox. The = previous > macro wasn't needed and was being used with swapped arguments which = always > give the same result (0) defeating the overflow check. >=20 > On initialization, do not use bcm_mbox_intr() to read the pending = messages, > with the new semaphore based implementation this will lead to = semaphore > being incremented on the channels that contain pending data and will = make > the first read for that channel return stale data. >=20 > This fixes the hang that happens on boot while initializing the = cpufreq on > Raspberry Pi. Thanks! -- Rui Paulo
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?B8E275B2-A50A-4EA1-87B3-72C9F94E6B48>