Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 1 Mar 2017 18:06:26 -0800 (PST)
From:      "Rodney W. Grimes" <freebsd-rwg@pdx.rh.CN85.dnsmgr.net>
To:        Ian Lepore <ian@freebsd.org>
Cc:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-11@freebsd.org
Subject:   Re: svn commit: r314525 - in stable/11/sys: arm/arm arm/include conf
Message-ID:  <201703020206.v2226QGL060130@pdx.rh.CN85.dnsmgr.net>
In-Reply-To: <201703012245.v21MjCkC054978@repo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
> Author: ian
> Date: Wed Mar  1 22:45:12 2017
> New Revision: 314525
> URL: https://svnweb.freebsd.org/changeset/base/314525
...

> 
> Modified: stable/11/sys/conf/files.arm
> ==============================================================================
> --- stable/11/sys/conf/files.arm	Wed Mar  1 21:58:26 2017	(r314524)
> +++ stable/11/sys/conf/files.arm	Wed Mar  1 22:45:12 2017	(r314525)
> @@ -53,7 +53,8 @@ arm/arm/gdb_machdep.c		optional	gdb
>  arm/arm/generic_timer.c		optional	generic_timer
>  arm/arm/gic.c			optional	gic
>  arm/arm/hdmi_if.m		optional	hdmi
> -arm/arm/identcpu.c		standard
> +arm/arm/identcpu-v4.c		optional	!armv6
> +arm/arm/identcpu-v6.c		optional	armv6
>  arm/arm/in_cksum.c		optional	inet | inet6
>  arm/arm/in_cksum_arm.S		optional	inet | inet6
>  arm/arm/intr.c			optional	!intrng
> _______________________________________________
> svn-src-stable-11@freebsd.org mailing list
> https://lists.freebsd.org/mailman/listinfo/svn-src-stable-11
> To unsubscribe, send any mail to "svn-src-stable-11-unsubscribe@freebsd.org"

If we are adding armv4 as an optional supported cpu type (I didnt think
we could run on anything less than an armv6, this patch clearly seems to
make that false.) I think we should just do the above as

> +arm/arm/identcpu-v4.c		optional	!armv6 | armv4
> +arm/arm/identcpu-v6.c		optional	armv6

As your above code would make the config for an armv6 machine be
expressed by the lack of a CPU token in the CONFIG file, which is
non intutive.

-- 
Rod Grimes                                                 rgrimes@freebsd.org



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