|
From: Bill F. <bil...@mi...> - 2001-11-04 17:55:23
|
Could someone please commit these patches to CVS, unless of course
there's some problem with either the idea behind the patch or the
patch itself. I've updated them to xine 0.9.3 and also checked that
they still apply cleanly to the latest CVS. This is one of the last
items that I would like to address for PPC support before 1.0, although
the patch is also generally useful for anyone with braindead audio
drivers that result in 48 KHz audio being used even though the audio
hardware only supports 44.1 KHz.
-Thanks
-Bill
---------- Forwarded message ----------
Date: Mon, 22 Oct 2001 23:05:49 -0400 (EDT)
From: Bill Fink <bil...@mi...>
To: xin...@li...
Cc: Bill Fink <bil...@mi...>
Subject: Patch to Force Audio Output Rate to Specified Value (e.g. 44100)
Hi,
The attached patches implement a new .xinerc configuration parameter
audio_force_rate that, if non-zero, forces the audio output rate to
the specified value for systems with braindead audio drivers that don't
properly communicate back what audio rates they actually do support.
The default value of 0 attempts to automatically detect the optimum
audio output rate.
The patch also removes the cruft of the FORCE_44K_MAX compilation
parameter that was used to support the PPC platform. Newer PPC Linux
kernel OSS audio drivers no longer need this kludge, and newer PPC
systems support 48 KHz audio. Older PPC systems running not so recent
kernels will need to specify audio_force_rate:44100 in their .xinerc
for properly functioning audio.
This patch should also be useful to others who have audio hardware
that can only support 44.1 KHz audio, but because of braindead audio
drivers, xine believes that they can handle 48 KHz audio.
The patch is against xine 0.9.2, but should also apply fine to CVS
(at least it did to a fresh CVS I downloaded earlier today).
-Cheers
-Bill
P.S. I'm including the two patches as attachments rather than in-line
because pine (version 4.33-7) was stripping lines that contained
only blanks down to just a newline, which was causing problems
to patch.
|