Re: [PATCH] 2.4.1p4 : dmasound ed21- Pismo byte-swap

quoted

this should (now I fixed a typo in the machine name) also solve the LE
byte-swapping problem on Pismo (by soft-swapping).

Can't apps cope with it if it doesn't do the byte-swapping in H/W?  If it
can, I think the code should be removed.  No software translation in kernel
if we can avoid it at all.

I agree 100% - the kernel should _never_ have been doing this stuff in the
first place (it can't even make a particularly good job of some of it
without fp).... and, AFAIK, that is the way of things for ALSA.

However, in OSS-land, it seems that quite a few apps rely on it (or don't
treat endian-ness sensibly) - OK, so they are, in some sense, buggy ...

... by comparison with the large amount of other code already in there to do
rate expansion and compression/decompression it is a relatively small price
to pay for having all pmac machines behave the same...

... at the next revision of the driver we could try issuing it with "16bit
44k1 Signed BE" only - and see how much of the desktop world works ;-)

iain.

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

On 1/18/01 7:47 PM, Iain Sandoe wrote:

[...]

... by comparison with the large amount of other code already in there to do
rate expansion and compression/decompression it is a relatively small price
to pay for having all pmac machines behave the same...

I know it's a small price, and some of those translations shouldn't be there
either, I think.

... at the next revision of the driver we could try issuing it with "16bit
44k1 Signed BE" only - and see how much of the desktop world works ;-)

Let's fix apps and keep the kernel lean.


Takashi Oe


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