linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
From: Dan Malek <dan@mvista.com>
To: Ralph Blach <rcblach@raleigh.ibm.com>
Cc: Ralph Blach <rblach@intrex.net>, linuxppc-dev@lists.linuxppc.org
Subject: Re: es1371.o sound module on a IBM 405 walnut platform
Date: Mon, 19 Mar 2001 14:36:45 -0500	[thread overview]
Message-ID: <3AB65FCD.44826199@mvista.com> (raw)
In-Reply-To: 3AB65B72.F77A0760@raleigh.ibm.com


Ralph Blach wrote:

> Well it does.  And I recomend a new kernel interface, and Dan, I KNOW
> YOU WILL not like THE VERY IDEA,
> by here it is.

You are right, I won't :-).  Here is why.  First of all, all Linux
ports understand how to manage the little-endian PCI interface.  We
don't want help from hardware swapping bytes in the lanes because it
just complicates software with special case functions and actually
introduces more overhead instead of removing it like the design you
thought would work.  Second, I have worked with hardware like this
in the past, and have painfully learned you simply can't arbitrarily
swap the bytes in the lanes.  If you have data structures you are
trying to share, the data has to be swapped based upon the size of
the object.  The hardware has no clue what you are trying to access
and will screw it up.  Just believe me, I've been there.

> copy_from_user and copy_to_use would then do the bit to little endian
> conversions.

Nope, the sound drivers already know how to do this if necessary.

> By the same token, an interface to allocate noncached memory would be
> spiffy.

That's already been done for the 8xx and 4xx ports in the kernel
trees.


	-- Dan

** Sent via the linuxppc-dev mail list. See http://lists.linuxppc.org/

  reply	other threads:[~2001-03-19 19:36 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-03-18 16:35 es1371.o sound module on a IBM 405 walnut platform Ralph Blach
2001-03-18 21:02 ` Dan Malek
2001-03-19 19:18   ` Ralph Blach
2001-03-19 19:36     ` Dan Malek [this message]
2001-03-21  1:10       ` Ralph Blach
2001-03-21  1:30         ` Dan Malek

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=3AB65FCD.44826199@mvista.com \
    --to=dan@mvista.com \
    --cc=linuxppc-dev@lists.linuxppc.org \
    --cc=rblach@intrex.net \
    --cc=rcblach@raleigh.ibm.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).