Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 30 Sep 2014 14:37:58 +0100
From:      Andrew Turner <andrew@fubar.geek.nz>
To:        Andreas Tobler <andreast@FreeBSD.org>
Cc:        Andrew Turner <andrew@FreeBSD.org>, svn-src-head@FreeBSD.org, svn-src-all@FreeBSD.org, src-committers@FreeBSD.org
Subject:   Re: svn commit: r272209 - in head/sys/arm: arm include
Message-ID:  <20140930143758.27d26ab9@bender.lan>
In-Reply-To: <5429A58E.2030508@FreeBSD.org>
References:  <201409270957.s8R9vYrw056987@svn.freebsd.org> <5429A58E.2030508@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, 29 Sep 2014 20:31:42 +0200
Andreas Tobler <andreast@FreeBSD.org> wrote:

> Hi Andrew,
> 
> On 27.09.14 11:57, Andrew Turner wrote:
> > Author: andrew
> > Date: Sat Sep 27 09:57:34 2014
> > New Revision: 272209
> > URL: http://svnweb.freebsd.org/changeset/base/272209
> >
> > Log:
> >    Add machine/sysreg.h to simplify accessing the system control
> > coprocessor registers and use it in the ARMv7 CPU functions.
> >
> >    The sysreg.h file has been checked by hand, however it may
> > contain errors with the comments on when a register was first
> > introduced. The ARMv7 cpu functions have been checked by compiling
> > both the previous and this version and comparing the md5 of the
> > object files.
> >
> >    Submitted by:	Svatopluk Kraus <onwahe at gmail.com>
> >    Submitted by:	Michal Meloun <meloun at miracle.cz>
> >    Reviewed by:	ian, rpaulo
> >    Differential Revision: https://reviews.freebsd.org/D795
> >
> > Added:
> >    head/sys/arm/include/sysreg.h   (contents, props changed)
> 
> This one breaks kernel build with gcc-4.2.1.
> 
> __ARM_ARCH not defined. On gcc-4.2.1 there is no __ARM_ARCH builtin.
> Later gcc do have it.
> 
> The include below fixes the build.

Fixed in r272300.

Andrew



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20140930143758.27d26ab9>