Source-Changes-D archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: src/sys/dev/acpi



On Thu, Feb 17, 2011 at 07:36:49PM +0000, Jukka Ruohonen wrote:
> Log Message:
> As explained in the new ACPICA documentation, as of ACPICA 20101207, the
> _PRW methods are no longer automatically executed as part of the ACPICA
> initialization. Refactor and rewrite the wake-device code to account this.

Despite of testing before the import, a regression still slipped through.
The following change that affects acpiec(4) is the most plausible culprit:

Re-introduced the support to enable multi-byte transfers for Embedded
Controller (EC) operation regions. A reported problem was found to be a bug
in the host OS, not in the multi-byte support. Previously, the maximum data 
size passed to the EC operation region handler was a single byte. There are
often EC Fields larger than one byte that need to be transferred, and it is 
useful for the EC driver to lock these as a single transaction. This change
enables single transfers larger than 8 bits. This effectively changes the
access to the EC space from ByteAcc to AnyAcc, and will probably require   
changes to the host OS Embedded Controller driver to enable 16/32/64/256-bit
transfers in addition to 8-bit transfers. Alexey Starikovskiy, Lin Ming.

(ACPICA 20100806.)

I don't have cycles to debug this any more today, but I will continue
tomorrow. In the meanwhile, if someone has an idea about this, please feel
free to join.

- Jukka.


Home | Main Index | Thread Index | Old Index