From: Matt Sealey <matt@genesi-usa.com>
To: Sven Luther <sven@powerlinux.fr>
Cc: linuxppc-dev@ozlabs.org, Olaf Hering <olaf@aepfle.de>
Subject: Re: add phy-handle property for fec_mpc52xx
Date: Wed, 09 Jan 2008 15:34:15 +0000 [thread overview]
Message-ID: <4784E977.3090006@genesi-usa.com> (raw)
In-Reply-To: <20080109145850.GA4413@powerlinux.fr>
I'd also say that the sound driver isn't ready for production use and it
misuses the BestComm task for generic bidirectional buffer copies by
forcing it to 32-bit transfers (this obviously involves some software
upsampling at some point as nearly all common sound data is 16-bit,
also twice the bandwidth to achieve). The Freescale docs for this task
(and the one provided by the firmware, no doubt, is identical in
operation) clearly state that the transfer size is configurable, so
saying it only accepts 32-bit sample data is simply inefficient and
lazy.
Someone needs to sit down and look at these, after a year of simple
stagnation since Sylvain lost the time to maintain them. I don't think
it is at all a good idea to think "oh it has been in the wild for a
year and nobody maintains it so we will mainline it now". That's a
ridiculous development philosophy. And, since nobody seems to give a
shit, it will stay as bad as it is right now (a quick hack) whether
in the wild or mainlined..
--
Matt Sealey <matt@genesi-usa.com>
Genesi, Manager, Developer Relations
Sven Luther wrote:
> On Wed, Jan 09, 2008 at 07:50:14AM -0700, Grant Likely wrote:
>> On 1/9/08, Sven Luther <sven@powerlinux.fr> wrote:
>>> On Wed, Jan 09, 2008 at 07:44:58AM -0700, Grant Likely wrote:
>>>> Woo! Thanks Olaf. I was just about to sit down and write something
>>>> like this myself. Looks good to me. I'll pick this up (but I'm going
>>>> to move it to the fixup_device_tree_efika() function)
>>> Indeed, thanks, this makes the efika kernel again work out of the box.
>>> Would it be possible to merge this upstream asap ?
>> I'll see if paulus will pick it up for 2.6.24
>
> Cool, this would mean the only thing missing the patchset i have been
> carrying is the sound driver. Do you see something else that has been
> added since then ?
>
> Friendly,
>
> Sven Luther
> _______________________________________________
> Linuxppc-dev mailing list
> Linuxppc-dev@ozlabs.org
> https://ozlabs.org/mailman/listinfo/linuxppc-dev
next prev parent reply other threads:[~2008-01-09 15:31 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-01-09 14:06 add phy-handle property for fec_mpc52xx Olaf Hering
2008-01-09 14:44 ` Grant Likely
2008-01-09 14:49 ` Sven Luther
2008-01-09 14:50 ` Grant Likely
2008-01-09 14:58 ` Sven Luther
2008-01-09 15:34 ` Matt Sealey [this message]
2008-01-09 15:21 ` Matt Sealey
2008-01-09 15:20 ` Grant Likely
2008-01-09 15:21 ` Sven Luther
2008-01-09 15:10 ` Grant Likely
2008-01-09 15:26 ` Matt Sealey
2008-01-09 15:35 ` Sven Luther
2008-01-09 16:30 ` Matt Sealey
2008-01-09 16:29 ` Sven Luther
2008-01-09 16:36 ` Matt Sealey
2008-01-09 17:05 ` Sven Luther
2008-01-09 17:17 ` Matt Sealey
2008-01-09 17:22 ` Sven Luther
2008-01-09 15:46 ` Grant Likely
2008-01-09 16:35 ` Matt Sealey
2008-01-09 16:44 ` Grant Likely
2008-01-09 15:49 ` David Woodhouse
2008-01-09 16:48 ` Matt Sealey
2008-01-10 2:21 ` Paul Mackerras
2008-01-10 5:04 ` Benjamin Herrenschmidt
2008-01-10 9:14 ` David Woodhouse
2008-01-09 16:02 ` Olof Johansson
2008-01-09 16:26 ` Grant Likely
2008-01-10 2:20 ` Paul Mackerras
2008-01-10 11:47 ` Olaf Hering
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=4784E977.3090006@genesi-usa.com \
--to=matt@genesi-usa.com \
--cc=linuxppc-dev@ozlabs.org \
--cc=olaf@aepfle.de \
--cc=sven@powerlinux.fr \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.