Date: Sun, 12 Aug 2012 12:18:10 +1200 From: Andrew Turner <andrew@fubar.geek.nz> To: Ian Lepore <freebsd@damnhippie.dyndns.org> Cc: "freebsd-arm@FreeBSD.org" <freebsd-arm@freebsd.org> Subject: Re: projects/armv6 merge Message-ID: <20120812121810.0f8e301a@fubar.geek.nz> In-Reply-To: <1344717708.1186.20.camel@revolution.hippie.lan> References: <50256231.3030008@bluezbox.com> <CD231BAC-93E9-4EA1-89C1-8D299CDA581A@bsdimp.com> <D36543E2-2AC1-495A-8514-73F43607F2C4@bluezbox.com> <1344717708.1186.20.camel@revolution.hippie.lan>
next in thread | previous in thread | raw e-mail | index | archive | help
On Sat, 11 Aug 2012 14:41:48 -0600 Ian Lepore <freebsd@damnhippie.dyndns.org> wrote: > files: sys/arm/lpc/lpc_machdep.c sys/arm/mv/mv_machdep.c > description: > Add missing brace. Comment out definition of proc0_tf, because it's > already defined in arm/machdep.c. > > Maybe it's the definition in arm/machdep.c that has to go, but if so > then a whole lot of other <soc>_machdep.c files need to have the > definition added. They were moved from <soc>_machdep.c to arm/marchdep.c as part of cleaning up duplicate code in initarm. You can remove them from <soc>_machdep.c. initarm() should then call init_proc0(). The mv version appears correct, however neither the lpc and tegra versions call init_proc0() and should be fixed. Andrew
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20120812121810.0f8e301a>