public inbox for linuxppc-dev@ozlabs.org 
 help / color / mirror / Atom feed
From: Sylvain Munaut <tnt@246tnt•com>
To: Marian Balakowicz <m8@semihalf•com>
Cc: spitzauer_77@gmx•de, linuxppc-dev@ozlabs•org
Subject: Re: State of the MPC5200 PSC AC97 driver
Date: Fri, 11 Apr 2008 09:29:34 +0200	[thread overview]
Message-ID: <47FF135E.2060003@246tnt.com> (raw)
In-Reply-To: <47FDEF9B.5000506@semihalf.com>


> Last year you have posted a MPC5200 PSC AC97 driver patch
> "[PATCH 9/9] sound: Add support for Freescale MPC5200 AC97 interface."
> with the following comment:
>
>   
>> Not quite a clean driver, but it get things done (well, mostly).
>> Only included to be able to test functionalityi/usage of
>> the BestComm driver.
>>     
Yes ... and it still applies.
>
> There are various FIXMEs and commented out code here and there.
> Could you elaborate a bit on the overall state of the driver's
> functionality,
> which areas need improvement and attention?
>   
It's a minimum boiler plate.
I filled the function at the minimum to get some sound output and being
able to hear music coming out of it :)
I also completely skipped the 5200 (not B) support ...

At first there was no DMA at all (full software copy). I added some
simple DMA later just to prove it could work and how to use the API.

I just wrote it to get some sound, prove the interface could work under
linux and to show how to use DMA. I had hoped someone else would finish
it ... (yeah, I know ... big mistake).
> Seems that you mainly tested BestComm with this driver, what was the
> overall
> BestComm performance, any issues here? Did you use any specific test setup
> involving some dedicated application, etc.?
>   
My test was listening to Gorrillaz "Feel Good inc" using mplayer ...

> Did anyone else tried it and/or has a updated version or can share
> experience?
>   
At the time several other people tried it and it worked ... unless you
did lots of harddrive IO and then it crumbled ...



Sylvain

  parent reply	other threads:[~2008-04-11  7:29 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-04-10 10:44 State of the MPC5200 PSC AC97 driver Marian Balakowicz
2008-04-10 13:50 ` Robert Schwebel
2008-04-10 15:25 ` Matt Sealey
2008-04-11  6:51   ` Robert Schwebel
2008-04-11  7:34     ` Sven Luther
2008-04-11  7:29 ` Sylvain Munaut [this message]
2008-04-15  8:03   ` Juergen Beisert
2008-04-16 11:24     ` Juergen Beisert
2008-04-17 14:19       ` RFC: " Juergen Beisert
2008-04-17 14:23         ` Matt Sealey
2008-04-17 14:41           ` Juergen Beisert
2008-04-17 14:54             ` Matt Sealey
2008-04-17 15:05         ` Matt Sealey
2008-04-17 15:23           ` Juergen Beisert
2008-04-17 15:10         ` Matt Sealey
2008-04-17 15:23           ` Juergen Beisert
2008-04-17 15:43             ` Matt Sealey
2008-04-17 15:46               ` Sergei Shtylyov
2008-04-18 15:43         ` Peter Czanik
2008-04-18 16:02           ` Juergen Beisert
2008-04-18 18:11             ` Matt Sealey
2008-04-18 22:53               ` Grant Likely
2008-04-19 12:02                 ` Matt Sealey
2008-04-19 15:59                   ` Grant Likely
2008-04-21  8:02                   ` Juergen Beisert
2008-04-21 17:04                     ` Matt Sealey
2008-04-19 16:03           ` Grant Likely
2008-04-22 14:20         ` Olaf Hering
2008-04-11  9:23 ` State of the " Marian Balakowicz
2008-04-11 13:50   ` Grant Likely
2008-04-11 14:53     ` Marian Balakowicz

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=47FF135E.2060003@246tnt.com \
    --to=tnt@246tnt$(echo .)com \
    --cc=linuxppc-dev@ozlabs$(echo .)org \
    --cc=m8@semihalf$(echo .)com \
    --cc=spitzauer_77@gmx$(echo .)de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox