Subject: Re: [patch] snapper(4) issues
To: Marco Trillo <marcotrillo@gmail.com>
From: Michael Lorenz <macallan@netbsd.org>
List: port-macppc
Date: 10/07/2007 12:49:48
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Hello,

On Oct 7, 2007, at 09:36, Marco Trillo wrote:

> The attached patch fixes a couple of issues with the snapper(4)  
> driver:
>
>  o In the bass, treble and gain controls, center the 0 dB point at the
> value 128. This is to make them easier to configure and set up
> correctly: the 128 value is the neutral value (so bass/treble/gain is
> disabled -- which is what is desired in many cases). Values higher
> than 128 emphasize bass/treble or add gain; values lesser than 128
> deemphasize bass/treble or add negative gain. Also make 128 the
> default value.

Hmm, I thought it already was that way. At least it was my intention  
when adding bass/treble controls for TAS3004.

> Apart from that, the patch also contains the following general  
> cleanup:
>
>  o Don't hardcode absolute addresses for the FCR registers. Use the
> "baseaddr" parameter passed from obio(4).

Yeah, that had to go away.

>  o Some mixer parameters should be disabled if a "software codec"  
> is used.

Indeed.

> I hope that this is useful!

I'll have a more detailed look later today, thanks for working on this!

have fun
Michael
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.7 (Darwin)

iQEVAwUBRwkOLMpnzkX8Yg2nAQKMmQgAi3tJEQuyT509VYuZzC5dSYMGO52BlL9e
AxnjhaR7uGW96rneL9DQLCjYi1BBKN7SXv4sTnyksMrcGIHlLxO92hOpshL2UOWS
NkntJX31f0jB9anS+VAEmPBN79iOLPRdlMylcqu2jn+sCrCZQhCRvSf65bjuvEos
LR8QWyiPtGO9eDbl+dGRS8xa/WOjg61NsvM0DA4MQTj+sU6hPTbuCKModSI5gbcu
kBrfQdrMeYwJyt6pZsYRCjmcVKQzccX70mB5vTb3j0H+ibDUPCcK9nsqB67Q8Edq
N6VywAW9IdeagWngKJxZ6xG4/LbiEAB8qTL1vJJXipzBus3XKrqkgg==
=Pant
-----END PGP SIGNATURE-----