Re: CVS commit: src/sys/arch

[Matt Thomas <matt%3am-software.com@localhost>]

On Jun 19, 2010, at 1:34 AM, KIYOHARA Takashi wrote:

> Hi! Jared,
> 
> 
> From: "Jared D. McNeill" <jmcneill%netbsd.org@localhost>
> Date: Wed, 16 Jun 2010 22:06:55 +0000
> 
>> Module Name: src
>> Committed By:        jmcneill
>> Date:                Wed Jun 16 22:06:54 UTC 2010
>> 
>> Modified Files:
>>      src/sys/arch/arm/arm: cpufunc.c
>>      src/sys/arch/arm/arm32: cpu.c db_interface.c pmap.c
>>      src/sys/arch/arm/conf: files.arm
>>      src/sys/arch/arm/include: armreg.h cpuconf.h cpufunc.h
>>      src/sys/arch/arm/include/arm32: pmap.h pte.h
>>      src/sys/arch/arm/omap: omap2_mputmr.c omap2_obio.c omap2_reg.h
>>      src/sys/arch/evbarm/beagle: beagle.h beagle_machdep.c beagle_start.S
>> Added Files:
>>      src/sys/arch/arm/arm: cpufunc_asm_armv7.S
>>      src/sys/arch/arm/arm32: cortexa8_pmc.c
>>      src/sys/arch/evbarm/conf: IGEPV2 std.igepv2
>> 
>> Log Message:
>> PR port-arm/43299: Support added for igepv2/cortexa8/omap3530
>> 
>> Apply patch from PR, with build fixes. ok skrll, matt
> 
> Why CPU_CORTEXA8 is ARM_ARCH_6?
> 
> http://cvsweb.netbsd.org/bsdweb.cgi/src/sys/arch/arm/include/cpuconf.h.diff?r1=1.16&r2=1.17&only_with_tag=MAIN&f=h
> 
> Index: cpuconf.h
> ===================================================================
> RCS file: /cvsroot/src/sys/arch/arm/include/cpuconf.h,v
> retrieving revision 1.16
> retrieving revision 1.17
> diff -u -r1.16 -r1.17
> --- cpuconf.h   27 Feb 2009 03:57:41 -0000      1.16
> +++ cpuconf.h   16 Jun 2010 22:06:53 -0000      1.17
> @@ -119,7 +119,7 @@
> #define        ARM_ARCH_5      0
> #endif
> 
> -#if defined(CPU_ARM11)
> +#if defined(CPU_ARM11) || defined(CPU_CORTEXA8)
> #define ARM_ARCH_6     1
> #else
> #define ARM_ARCH_6     0
> 
> 
> 
> Cortex-A8 is ARMv7 architecture.
> 
>  http://infocenter.arm.com/help/index.jsp
>    - Cortex-A series processors
>      - Cortex-A8
>        - Revision: r3p2
>          - Cortex-A8 Technical Reference Manual
>            - 1.2. ARMv7-A architecture
> 
> There is a possibility that the patch for Gumstix Overo that I made in
> old times helps your understanding though I am not well informed about
> the ARM architecture.
> 
>  ftp://ftp.netbsd.org/pub/NetBSD/misc/kiyohara/gumstix/armv7.diff

Because the patch was a minimum effort to get running.  I had changes to
it in my tree to use ARM_ARCH_7 and other fixes but my kernels were
dying.  So rather than commit my stuff, I had Jared commit his, then I
went through merge hell, and have now committed the better stuff.