--=-=-=


Love <lha@stacken.kth.se> writes:

> TAMURA Kent <kent@NetBSD.org> writes:
>
>> In message "Re: ac97 and suspend/resume on laptops"
>>     on 04/07/15, Love <lha@stacken.kth.se> writes:
>>> I think the codec isn't ready.  With this patch, my IBM T30 (with same
>>> chipset as you have) plays sounds just fine after resume. I also used
>>> include the dont restore reset reg part of your ac97 patch, unclear if that
>>> is really needed.
>>
>> Your patch is not acceptable.  Some boards never set ICH_PR.
>
> But some boards (my IBM T30) set ICH_SCR or ICH_S2CR before primary codec
> is finished starting.
>
> Your boards don't have a primary codec, they are just broken, or is the
> auich_codec_reset function broken ?

So reading the spec it seems ICH_{,S,S2}PR isn't avaible after busmastering
have been turned on and and assuming they are readable after busmastering
any of them is a mistake.

Love


--=-=-=
Content-Type: application/pgp-signature

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.4 (NetBSD)

iQEVAwUAQPePqnW+NPVfDpmCAQIqVgf8Cqag3AJRfjONw+gKUUsqVvAvkaw+96UL
/Yuq+9ogkRuYE6gtjENPrfahr8g6Fo2UlTpxPqhHX+jWLh/DgF3TrTe4wQLbP0Mi
2mlBE6mit+EP1h0Lhjc8AsE2KzfmcSYT8QcI0khSonFzqy16YLRqR8r/bO+OPj2K
M34jQlIpWs3gNAbjAvRtPllTqdJcqhmrhq3rHMKa9OGQaK5o9cTVbvLvpreHNjP9
HEmvUDFpL+A7mtMmBEzTA8tUPx5EeXvstlerv2P55v+oNKK4zzWcLwS3UvrKcDih
MLng5XLhGsea8bDO/wr4jBHWLd1RY5IvMD4W2mMGHz6nSoPPn0BqLg==
=QTVo
-----END PGP SIGNATURE-----
--=-=-=--