From owner-svn-src-all@FreeBSD.ORG Tue May 6 23:27:51 2014 Return-Path: Delivered-To: svn-src-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 540A4C09 for ; Tue, 6 May 2014 23:27:51 +0000 (UTC) Received: from mail-pd0-f171.google.com (mail-pd0-f171.google.com [209.85.192.171]) (using TLSv1 with cipher ECDHE-RSA-RC4-SHA (128/128 bits)) (Client CN "smtp.gmail.com", Issuer "Google Internet Authority G2" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 1FF2D124 for ; Tue, 6 May 2014 23:27:50 +0000 (UTC) Received: by mail-pd0-f171.google.com with SMTP id r10so182619pdi.2 for ; Tue, 06 May 2014 16:27:49 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:sender:content-type:mime-version:subject:from :in-reply-to:date:cc:message-id:references:to; bh=h7SEroYrcr63xQsIglHWB0lTbMU4urGyK7b9xhwgktA=; b=ieWth3eTWctqXtE7UKX9nnL8XGRSwHXHHvMuhsMD9fsqaByZmpQN0mYekf8BKVWrix /gq4Tia93tlYF7RadT1E9gCB2ydAIfw8rL7j2AaNjgaeNJeCtsaRLFRHMLCGmL1fIXMj OgmSPQBGWldfEJw26FD2a6N2ONWKxrKTLkh1gGKe4jvhAv80GbAoTKVTR9UTBnkW6cZf mX/E0kzOW63Om9cYROdYgnKwzpQ3+Elv8rCFJhX+tnM4UHjBQ+tn0bzXnZlFHt5Wacma gvmsA+VhBN+6f7EhBxCAFAUrHxOSsqa37mfNWTA33V2GDMKHXK6Q0BSKyf3IKhY/2bY5 8WkQ== X-Gm-Message-State: ALoCoQlSA/2BDFt7iNEP9FofVbUi4NFe4eLD2xdKcdoQ0Y1bJczV5ItxpaR85YeOI7uxRaqlCJLJ X-Received: by 10.66.149.37 with SMTP id tx5mr11687322pab.81.1399418869625; Tue, 06 May 2014 16:27:49 -0700 (PDT) Received: from [10.64.26.239] (dc1-prod.netflix.com. [69.53.236.251]) by mx.google.com with ESMTPSA id xr9sm104085292pab.5.2014.05.06.16.27.48 for (version=TLSv1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Tue, 06 May 2014 16:27:48 -0700 (PDT) Sender: Warner Losh Content-Type: multipart/signed; boundary="Apple-Mail=_386D76C3-88F7-4AAD-ACF1-504F12C1A0F9"; protocol="application/pgp-signature"; micalg=pgp-sha512 Mime-Version: 1.0 (Mac OS X Mail 7.2 \(1874\)) Subject: Re: svn commit: r265359 - head/sys/modules/sound/sound From: Warner Losh In-Reply-To: <20140506225551.GD43976@funkthat.com> Date: Tue, 6 May 2014 17:27:45 -0600 Message-Id: References: <201405051431.s45EVZ2q027729@svn.freebsd.org> <7745BD19-5BAE-47BD-8DE4-10CF7E23FA9D@gmail.com> <53690381.9040909@selasky.org> <20140506160154.GA679@alchemy.franken.de> <20140506225551.GD43976@funkthat.com> To: John-Mark Gurney X-Mailer: Apple Mail (2.1874) Cc: Hans Petter Selasky , svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers , Marius Strobl X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 06 May 2014 23:27:51 -0000 --Apple-Mail=_386D76C3-88F7-4AAD-ACF1-504F12C1A0F9 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=windows-1252 On May 6, 2014, at 4:55 PM, John-Mark Gurney wrote: > Marius Strobl wrote this message on Tue, May 06, 2014 at 18:01 +0200: >> On Tue, May 06, 2014 at 05:45:05PM +0200, Hans Petter Selasky wrote: >>> On 05/05/14 22:39, Warner Losh wrote: >>>>=20 >>>> On May 5, 2014, at 8:31 AM, Hans Petter Selasky = wrote: >>>>=20 >>>>> 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. >>>>=20 >>>> 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? >>>>=20 >>>> Warner >>>>=20 >>>=20 >>> Hi, >>>=20 >>> Maybe the sound DMA file belongs in its own module? >>>=20 >>=20 >> It's not just sndbuf_dma.c but also the #ifdef'ed DEV_ISA code in >> channel.c on architectures that have ISA but not ISA DMA support, >> hence the current kludge. >=20 > I thought busdma was suppose to abstract all of this away? Is it > just that this was never complete? or? Never completed for ISA DMA=85=20 Warner --Apple-Mail=_386D76C3-88F7-4AAD-ACF1-504F12C1A0F9 Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename=signature.asc Content-Type: application/pgp-signature; name=signature.asc Content-Description: Message signed with OpenPGP using GPGMail -----BEGIN PGP SIGNATURE----- Comment: GPGTools - https://gpgtools.org iQIcBAEBCgAGBQJTaW/xAAoJEGwc0Sh9sBEAIrIQAIOz6uEAmEuAdannwagb0w9O jwYuKq0eRb6kVnsONXkTUgCgMe3uWpuhvEjI/J15VUomy8Ax1G+fin9MP6OWIYSe Xq+0BpK+BT5NlKDvYtCyK7Vl/EOj/1EaOhgVc1gTuHy+nIIbj+XYBZcQeqqDnfh3 F0wGLwS7kaCmTR8ZeW9Ub/a0xm+70LjSa+kcuJcM9e9xVyYVMlGh+JroorL4rDuu hPE4N/2Bbny3wKrqm+PV4yu5XXag2zQSkD11ejo4/F5ojfl2FU/c75ZEgOpu0bD8 g52dyplhNJTClY01T9DjtkwIajcr7iwBvRELZEXxuwPhURnYmI9QkdO4icivU11W SdXJiA7ahUbkXP3Te/4N2MYw7FA2JvVueqfo00FUJhC3BJj0J6RR5OYJlfr+RfRM JwXgrlhFdJLTb5lcTDjXccHBZF2zni9gepgV0X5Uq2hKBhJXDcbJdcTa9DuTAevU 7SxRnzX5UfanypB8Urxm1xFBIRIHOf1WmRVxVwOA2F9QLayisBfAzGQkWjl10/yn iMtYybnNQLRnMxw0eaYkxZH8gJmrcbLTx0zE9pwrzV/AXzqh5EwxBPVuxNsSlaos 7JZv28gaayxo6DUc76fY7h/F3C/aw2LVsALffK3O2a9wGUmHIwgoOOfNHJhwNqGJ XVdkJvz88spkGYRweAON =TyIs -----END PGP SIGNATURE----- --Apple-Mail=_386D76C3-88F7-4AAD-ACF1-504F12C1A0F9--