Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 5 May 2014 14:39:46 -0600
From:      Warner Losh <imp@bsdimp.com>
To:        Hans Petter Selasky <hselasky@freebsd.org>
Cc:        svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers <src-committers@freebsd.org>
Subject:   Re: svn commit: r265359 - head/sys/modules/sound/sound
Message-ID:  <7745BD19-5BAE-47BD-8DE4-10CF7E23FA9D@gmail.com>
In-Reply-To: <201405051431.s45EVZ2q027729@svn.freebsd.org>
References:  <201405051431.s45EVZ2q027729@svn.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help

On May 5, 2014, at 8:31 AM, Hans Petter Selasky <hselasky@freebsd.org> =
wrote:

> Author: hselasky
> Date: Mon May  5 14:31:34 2014
> New Revision: 265359
> URL: http://svnweb.freebsd.org/changeset/base/265359
>=20
> Log:
>  Build the kernel sound module without ISA DMA support for ARM and =
MIPS
>  platforms, because these platforms do not implement the ISA DMA
>  API. Else the sound modules cannot be loaded when running these
>  platforms.
>=20
>  MFC after:	2 weeks
>=20
> Modified:
>  head/sys/modules/sound/sound/Makefile
>=20
> Modified: head/sys/modules/sound/sound/Makefile
> =
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D
> --- head/sys/modules/sound/sound/Makefile	Mon May  5 11:50:52 2014	=
(r265358)
> +++ head/sys/modules/sound/sound/Makefile	Mon May  5 14:31:34 2014	=
(r265359)
> @@ -44,7 +44,8 @@ CLEANFILES+=3D	feeder_eq_gen.h feeder_rate
>=20
> EXPORT_SYMS=3D	YES	# XXX evaluate
>=20
> -.if ${MACHINE_CPUARCH} =3D=3D "sparc64" || ${MACHINE_CPUARCH} =3D=3D =
"powerpc"
> +.if ${MACHINE_CPUARCH} =3D=3D "sparc64" || ${MACHINE_CPUARCH} =3D=3D =
"powerpc" || \
> +    ${MACHINE_CPUARCH} =3D=3D "arm" || ${MACHINE_CPUARCH} =3D=3D =
"mips"
> # Create an empty opt_isa.h in order to keep kmod.mk from linking in =
an
> # existing one from KERNBUILDDIR which possibly has DEV_ISA defined so
> # sound.ko is always built without isadma support.

Rather than an opt-in approach here, why not lust list the three that =
have it since
they are the only ones that will ever have it=85

Warner=



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?7745BD19-5BAE-47BD-8DE4-10CF7E23FA9D>