Date: Mon, 22 Apr 2013 10:30:56 +0200 From: Damjan Marion <damjan.marion@gmail.com> To: Ruslan Bukin <br@bsdpad.com> Cc: freebsd-arm@freebsd.org Subject: Re: gic.c and interrupt priority mask register (GICC_PMR) Message-ID: <A7E29012-7650-4F82-80E8-F01EFDD2B21B@gmail.com> In-Reply-To: <20130422075054.GA6831@jail.io> References: <5171C5C6.7080907@sbcglobal.net> <17C7A0BE-43E4-422C-8E0F-11897DBB188E@gmail.com> <20130422075054.GA6831@jail.io>
next in thread | previous in thread | raw e-mail | index | archive | help
On Apr 22, 2013, at 9:50 AM, Ruslan Bukin <br@bsdpad.com> wrote: > On Sat, Apr 20, 2013 at 11:50:53PM +0200, Damjan Marion wrote: >>=20 >> On Apr 20, 2013, at 12:31 AM, Thomas Skibo = <ThomasSkibo@sbcglobal.net> wrote: >>=20 >>>=20 >>> Hello. >>>=20 >>> I mentioned this as an aside in another email but I'd like to = revisit it. >>>=20 >>> My Zynq port doesn't work unless I initialize the GIC interrupt = priority mask register (GICC_PMR) which I do in a hack in zy7_machdep.c. = The GICC_PMR register is never touched in gic.c and I wonder how other = ARM ports work without having it initialized. I figure either they use = a different interrupt controller, their GIC implementation has a = different reset value for the PMR, or a boot-loader sets up the register = before the kernel is entered. >>>=20 >>> The ARM Generic Interrupt Controller Architecture Specification = (version 2.0) states that the reset value of GICC_PMR is 0 which masks = all interrupts. So shouldn't gic.c initialize it to 0xff if the PMR = functionality isn't used? >>>=20 >>> --Thomas >>=20 >> Hi Thomas, >>=20 >> Makes sense. GIC is used on several platforms so i guess those = implementations have different reset value. >>=20 >> I can commit this if nobody objects. >>=20 >=20 > Exynos4,5 have also GICC_PMR =3D=3D 0 masking all interrupts by = default.=20 > So it is very important to commit it. Done in r249762 Damjan
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?A7E29012-7650-4F82-80E8-F01EFDD2B21B>