[MPlayer-dev-eng] PowerPC multichannel playback with cmpci

[C.L. Tien - 田承禮]

I don't know what the winmail is, however, the ad_hw3d.c.diff is only 875 bytes:

=========================================================
--- MPlayer-20040331/libmpcodecs/org/ad_hwac3.c	2003-09-26 15:21:08.000000000 -0
400
+++ MPlayer-20040331/libmpcodecs/new/ad_hwac3.c	2004-03-31 15:09:31.000000000 -0
500
@@ -165,11 +165,7 @@
     buf[5] = 0x00;
     buf[6] = (len << 3) & 0xFF;
     buf[7] = (len >> 5) & 0xFF;
-#ifdef WORDS_BIGENDIAN
-    memcpy(buf + 8, sh_audio->a_in_buffer, len);  // untested
-#else
     swab(sh_audio->a_in_buffer, buf + 8, len);
-#endif
     memset(buf + 8 + len, 0, 6144 - 8 - len);
 
     return 6144;
@@ -359,12 +355,8 @@
   {
     mp_msg(MSGT_DECAUDIO, MSGL_ERR, "DTS: more data than fits\n");
   }
-#ifdef WORDS_BIGENDIAN
-  memcpy(&buf[8], indata_ptr, fsize);  // untested
-#else
   //TODO if fzise is odd, swab doesn't copy the last byte
   swab(indata_ptr, &buf[8], fsize);
-#endif
   memset(&buf[fsize + 8], 0, nr_samples * 2 * 2 - (fsize + 8));
 
   return nr_samples * 2 * 2;
======================================================

In short, I simply use the same copy way as lttle-endian systems for both AC3 and DTS. Verify on other big-endian platforms is still necessary.

Sincerely,
ChenLi Tien

-----Original Message-----
From:	Joey Parrish [mailto:joey at nicewarrior.org]
Sent:	2004/4/2 [星期五] 上午 11:36
To:	mplayer-dev-eng at mplayerhq.hu
Cc:	
Subject:	Re: [MPlayer-dev-eng] PowerPC multichannel playback with cmpci
On Thu, Apr 01, 2004 at 11:35:35AM +0800, C.L. Tien - ?????? wrote:
> Hi,
> 
> I use MPlayer (20040331) with my OSS driver (cmpci), I found it always asks for AFMT_S16_BE, even the driver said only AFMT_S16_LE supported in DSP_GETFMT ioctl.
> 
> I added the AFMT_S16_BE support by simply swapping the hi-byte and lo-byte, now it can play 1/2 channel audio (but with high-pitch noise), but when I tried 4-channel audio with a DVD, there are lots of noise, the same version works well with PC.
> 
> BTW, the MPlayer can play AC3 stream well with PowerPC after patch I posted last time.

I didn't see a patch in your last mail.  It had [PATCH] in the subject,
but the attachment was "winmail.dat".

> > [-- Attachment #2: winmail.dat --]
> > [-- Type: application/ms-tnef, Encoding: base64, Size: 5.1K --]

Please resend.

--Joey

-- 
"Living in the complex world of the future is somewhat
like having bees live in your head.  But, there they are."

_______________________________________________
MPlayer-dev-eng mailing list
MPlayer-dev-eng at mplayerhq.hu
http://mplayerhq.hu/mailman/listinfo/mplayer-dev-eng