From: Alexey Starikovskiy <astarikovskiy@suse.de>
To: Zhao Yakui <yakui.zhao@intel.com>
Cc: Lin Ming <ming.m.lin@intel.com>, linux-acpi <linux-acpi@vger.kernel.org>
Subject: Re: Bug 11884 - _REG and _INI
Date: Mon, 15 Dec 2008 12:26:56 +0300 [thread overview]
Message-ID: <494622E0.4020600@suse.de> (raw)
In-Reply-To: <1229326070.3991.7.camel@yakui_zhao.sh.intel.com>
Zhao Yakui wrote:
> On Mon, 2008-12-15 at 11:51 +0800, Lin Ming wrote:
>> http://bugzilla.kernel.org/show_bug.cgi?id=11884
>>
>> Method(\_SB._INI)
>> {
>> //initialize package CUZO
>> }
>>
>> Method(\_SB_.PCI0.LPCB.EC0_._REG)
>> {
>> //access package CUZO
>> }
>>
>> For this machine, \_SB._INI must be run before \_SB_.PCI0.LPCB.EC0_._REG
>>
>> But currently early EC _REG method is run before _INI as below acpi_bus_init shows.
>>
>> void __init acpi_bus_init(void)
>> .....
>> status = acpi_ec_ecdt_probe();
>> /* Ignore result. Not having an ECDT is not fatal. */
>>
>> status = acpi_initialize_objects(ACPI_FULL_INITIALIZATION);
>> .....
>> }
>>
>> acpi_ec_ecdt_probe will install ec handler although no ECDT found in this machine.
>>
>> Alexey,
>> How about that if there's no ECDT found, we do not install ec handler before acpi_initialize_objects?
>
> I do some test with the help of KVM. In my test the ECDT table is
> defined. And there also exists the EC device in DSDT table.Of course
> there exists the _INI/_REG object under the scope of EC device. Under
> the scope of \_SB.PCI0 there exists the _INI/_REG object.
>
>>From the test result it seems that all the _INI objects are evaluated
> before _REG object even when there exists the ECDT table.
>
> Hi, Alexey
> Can the Lin Ming's proposal be considered based on the above test?
Hi Yakui,
His proposal needs to be considered even without your test (but the test itself is quite helpful),
just because HP does not violate the ACPI spec, but ASUS notebooks without ECDT do violate it
(and fake ECDT workaround is there for them).
Thanks,
Alex.
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2008-12-15 9:27 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-12-15 3:51 Bug 11884 - _REG and _INI Lin Ming
2008-12-15 7:27 ` Zhao Yakui
2008-12-15 9:26 ` Alexey Starikovskiy [this message]
2008-12-15 10:47 ` Alexey Starikovskiy
2008-12-16 1:07 ` Zhao Yakui
2008-12-16 10:17 ` Alexey Starikovskiy
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=494622E0.4020600@suse.de \
--to=astarikovskiy@suse.de \
--cc=linux-acpi@vger.kernel.org \
--cc=ming.m.lin@intel.com \
--cc=yakui.zhao@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox