[acpi-jp 2125] New ACPI diffs ready for testing

[Shizuka Kudo]

--- Nate Lawson <nate at root.org> wrote:
> I have prepared two versions of ACPI for testing, one from the 0228 dist
> and one from 0328.  Both patches are against the current version in the
> tree so you should be able to just "cd /usr/src; zcat *diff.gz | patch".  
> I have tested both pretty thoroughly on my laptop but need more testing
> from others to decide which to import.  Neither crash, but the 0328 dist
> adds a lot more console messages complaining about my battery (see
> separate message for more info on this.)
> 
>     http://www.root.org/~nate/freebsd/acpi-0228.diff.gz
>     http://www.root.org/~nate/freebsd/acpi-0328.diff.gz
> 

Nate,

The new patch, 0328, works fine for both my ASUS TUSI-M (SiS 630ET) and EPOX 3PTA (i815EP) except
that an old problem on the 3PTA still exists.  Here's the question I posted before, i.e. testing
of ACPI_STA_PRESENT and ACPI_STA_FUNCTIONAL doesn't work for my 3PTA and causes trouble with the
PS/2 mouse & printer.

> I just replaced my -current with an I815EP mother board and found that the mouse and printer not
> working.  When I was moving the mouse, the cursor on the console just disappeared and reappeared
> after I stop moving the mouse.  For the printer port, when I sent some data, the printer's data
> LED was flashing, indicating data was received, but it never printed anything.
> 
> I stopped loading ACPI at boot and everything become normal.  I did have some good 
> experience with
> this motherboard and ACPI several months ago so I searched back and found that the following
> commit caused the problem
> 
> > Revision 1.3 / (download) - annotate - [select for diffs], Thu Oct 31 20:43:02 2002 UTC (4
months, 2 weeks ago) by iwasaki 
> > Branch: MAIN 
> > Changes since 1.2: +4 -5 lines
> > Diff to previous 1.2 (colored) 
> > 
> > Don't examine ACPI_STA_ENABLE bit of device's_STA object on initial
> > state.  Instead, use ACPI_STA_PRESENT and ACPI_STA_FUNCTIONAL for it.
> > In some ACPI BIOS implementations, boot disabled devices don't have
> > ACPI_STA_ENABLE bit in _STA object.
> > Also it is not fatal if getting current IRQ of boot disabled devices
> > is failed in initial state.
> 
> In particular this patch causes my problem:
> 
> @@ -461,9 +460,9 @@ acpi_pci_link_add_prt(device_t pcidev, A
>  		return_ACPI_STATUS (error);
>  	}
>  
> -	if (!(sta & ACPI_STA_ENABLE)) {
> +	if (!(sta & (ACPI_STA_PRESENT | ACPI_STA_FUNCTIONAL))) {
>  		ACPI_DEBUG_PRINT((ACPI_DB_ERROR,
> -		    "PCI interrupt link is disabled - %s\n",
> +		    "PCI interrupt link is not functional - %s\n",
>  		    acpi_name(handle)));
>  		return_ACPI_STATUS (AE_ERROR);
>  	}
> 


__________________________________________________
Do you Yahoo!?
The New Yahoo! Search - Faster. Easier. Bingo
http://search.yahoo.com