Skip site navigation (1)Skip section navigation (2)
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>