the following patch was just integrated into master: commit 109c08e05ac482c72c541ae1a3e408a686c9fb03 Author: Mike Loptien mike.loptien@se-eng.com Date: Fri Mar 15 13:40:13 2013 -0600
Lippert Fam14 DSDT: Remove INI method from AZHD device
I am removing the _INI method from the AZHD device because it does not seem to do anything and causes errors in the FWTS[1] (Firmware Test Suite) test 'method'. The INI method performs device specific initialization and is run when OSPM loads a description table. It must only access OperationRegions that have been indicated as available by the _REG (Region) method. We do not have a _REG method and during my testing, I added a REG method but it did not seem to make a difference in the PCI register space. The bit fields defined as NSDI (Disable No Snoop), NSDO (Disable No Snoop Override), and NSEN (Enable No Snoop Request) do not ever get written from their default values. And writing to these bit fields does not seem to be necessary because I did not notice any change in audio functionality.
In an effort to clean up as many FWTS errors as possible, I propose removing this method altogether. I have seen no change in operation (audio works with and without this method) and there does not seem to be any change in lspci or dmesg.
FWTS information can be found here: [1]: https://wiki.ubuntu.com/Kernel/Reference/fwts
This is the same change as made to Persimmon in Change-ID If8d86f: http://review.coreboot.org/#/c/2726/
Change-Id: Iff594d4a3493531561eb25d1cceeb97bcefde424 Signed-off-by: Mike Loptien mike.loptien@se-eng.com Reviewed-on: http://review.coreboot.org/2743 Tested-by: build bot (Jenkins) Reviewed-by: Stefan Reinauer stefan.reinauer@coreboot.org
Build-Tested: build bot (Jenkins) at Fri Mar 15 22:41:22 2013, giving +1 Reviewed-By: Stefan Reinauer stefan.reinauer@coreboot.org at Sun Mar 17 19:55:15 2013, giving +2 See http://review.coreboot.org/2743 for details.
-gerrit