Content-Disposition: inline
Content-Transfer-Encoding: 8bit
Content-Type: text/plain; charset="US-ASCII"
MIME-Version: 1.0
Date: Wed, 15 Apr 2020 15:25:33 +0000
From: "Nia Alarie" <nia@netbsd.org>
Subject: CVS commit: src
To: source-changes@NetBSD.org
Message-Id: <20200415152533.C37BBFB27@cvs.NetBSD.org>
Sender: source-changes-owner@NetBSD.org
Precedence: bulk
Reply-To: source-changes-d@NetBSD.org
Mail-Reply-To: "Nia Alarie" <nia@netbsd.org>
Mail-Followup-To: source-changes-d@NetBSD.org

Module Name:	src
Committed By:	nia
Date:		Wed Apr 15 15:25:33 UTC 2020

Modified Files:
	src/lib/libossaudio: ossaudio.c
	src/sys/compat/ossaudio: ossaudio.c

Log Message:
ossaudio: Make SNDCTL_DSP_SETFMT conform with OSSv4.

The OSSv4 spec says we shouldn't really error if an invalid format is
chosen by an application. Things are especially likely to be confused
if we return MULAW, since in OSSv4 terms that means that's the native
hardware format. Instead, set and return the current hardware format
if an invalid format is chosen.

For the 24-bit sample formats, note that the NetBSD kernel currently
can't handle them in its default configuration, and will return an error
code if you attempt to use them. So, if an applicaton requests 24-bit PCM,
promote it to 32-bit PCM. According to the spec, this is valid and
applications should be checking the return value anyway.

In the Linux compat layer, we just use S16LE as a fallback. The OSSv3
headers that are still being shipped with Linux don't contain definitions
for fancier formats and we can reasonably expect all applications to
support S16LE.


To generate a diff of this commit:
cvs rdiff -u -r1.39 -r1.40 src/lib/libossaudio/ossaudio.c
cvs rdiff -u -r1.79 -r1.80 src/sys/compat/ossaudio/ossaudio.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.