From: Alexey Starikovskiy <astarikovskiy@suse.de>
To: Zhao Yakui <yakui.zhao@intel.com>
Cc: Carlos Corbacho <carlos@strangeworlds.co.uk>,
linux-acpi@vger.kernel.org, Len Brown <len.brown@intel.com>,
Len Brown <lenb@kernel.org>
Subject: Re: [PATCH 091/171] ACPI: GPE enabling should happen after EC installation
Date: Sun, 04 May 2008 10:40:42 +0400 [thread overview]
Message-ID: <481D5A6A.4060603@suse.de> (raw)
In-Reply-To: <1209896900.28197.65.camel@yakui_zhao.sh.intel.com>
Zhao Yakui wrote:
> But after the patch is applied, it seems that we make a lot of changes
> about the GPE initialization flowchart in ACPICA.
And this flowchart is cast in stone?
> Will these be easily accepted by ACPICA author?
You should probably ask him, not me. We are talking Bob Moore here, yes?
Why don't you add CC to him if you are worried?
> At the same time some RUNTIME GPEs will be enabled twice. When
> acpi_enable_gpe is called in the course of EC installation, some RUNTIME
> GPEs will also be enabled. In fact when the acpi_ev_install_fadt_gpes is
> called, all the RUNTIME GPEs will be enabled. In such case it brings
> that some RUNTIME GPEs are enabled twice. Of course the patch should
> work fine if it is safe to enable RUNTTME GPEs twice.
Please advise me, how we are not enabling them twice if EC init happens after?
The same acpi_enable_gpe will be called from it.
Regards,
Alex.
next prev parent reply other threads:[~2008-05-04 6:40 UTC|newest]
Thread overview: 54+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-04-29 8:43 ACPI patches for 2.6.26 Len Brown
2008-04-29 8:43 ` [PATCH 001/171] ACPI: crosslink ACPI and "real" device nodes Len Brown
2008-04-29 8:43 ` [PATCH 002/171] ACPI: EC: Restore udelay in poll mode Len Brown
2008-04-29 8:43 ` [PATCH 003/171] ACPI: EC: Add poll timer Len Brown
2008-04-29 8:43 ` [PATCH 004/171] ACPI: EC: Improve debug output Len Brown
2008-04-29 8:43 ` [PATCH 005/171] ACPI: EC: Replace broken controller workarounds with poll mode Len Brown
2008-04-29 8:43 ` [PATCH 006/171] ACPI: EC: Switch off GPE mode during suspend/resume Len Brown
2008-04-29 8:43 ` [PATCH 007/171] ACPI: EC: Detect irq storm Len Brown
2008-04-29 8:43 ` [PATCH 008/171] ACPI: EC: Use default setup handler Len Brown
2008-04-29 8:44 ` [PATCH 009/171] ACPI: EC: Don't delete boot EC Len Brown
2008-04-29 8:44 ` [PATCH 010/171] ACPI : Disable the device's ability to wake the sleeping system in the boot phase Len Brown
2008-04-29 8:44 ` [PATCH 011/171] #if 0 acpi/bay.c:eject_removable_drive() Len Brown
2008-04-29 8:44 ` [PATCH 012/171] PM: Remove legacy PM Len Brown
2008-04-29 8:44 ` [PATCH 013/171] PM: arch/x86/kernel/apm_32.c: fix build warning Len Brown
2008-04-29 8:44 ` [PATCH 014/171] MIPS Alchemy: Crapectomy after removal of pm_send_all calls Len Brown
2008-04-29 8:44 ` [PATCH 085/171] eeepc-laptop: add base driver Len Brown
2008-04-29 8:44 ` [PATCH 086/171] eeepc-laptop: add backlight Len Brown
2008-04-29 8:44 ` [PATCH 087/171] eeepc-laptop: add hwmon fan control Len Brown
2008-04-29 8:44 ` [PATCH 088/171] eeepc-laptop: MAINTAINERS entry Len Brown
2008-04-29 8:44 ` [PATCH 089/171] ACPI: re-name acpi_pm_ops to acpi_suspend_ops Len Brown
2008-04-29 8:44 ` [PATCH 091/171] ACPI: GPE enabling should happen after EC installation Len Brown
2008-04-30 10:25 ` Zhao Yakui
2008-04-30 6:58 ` Carlos Corbacho
2008-04-30 7:17 ` Carlos Corbacho
2008-04-30 16:43 ` Zhao Yakui
2008-04-30 10:04 ` Alexey Starikovskiy
2008-05-04 10:28 ` Zhao Yakui
2008-05-04 6:40 ` Alexey Starikovskiy [this message]
2008-05-04 16:22 ` Zhao Yakui
2008-04-30 14:53 ` Henrique de Moraes Holschuh
2008-04-29 8:44 ` [PATCH 092/171] 2.6.25 regression: powertop says 120K wakeups/sec Len Brown
2008-04-29 8:44 ` [PATCH 093/171] flush kacpi_notify_wq before removing notify handler Len Brown
2008-04-29 8:44 ` [PATCH 094/171] thermal: add the support for building the generic thermal as a module Len Brown
2008-04-29 8:44 ` [PATCH 095/171] thermal: add new get_crit_temp callback Len Brown
2008-04-29 8:44 ` [PATCH 096/171] thermal: add hwmon sysfs I/F Len Brown
2008-04-29 8:44 ` [PATCH 097/171] thermal: update the documentation Len Brown
2008-04-29 8:44 ` [PATCH 098/171] ACPI: elide a non-zero test on a result that is never 0 Len Brown
2008-04-29 8:44 ` [PATCH 099/171] ACPI: update thermal temperature Len Brown
2008-04-29 8:44 ` [PATCH 100/171] thermal: re-name thermal.c to thermal_sys.c Len Brown
2008-04-29 8:44 ` [PATCH 156/171] ACPI: thinkpad-acpi: BIOS backlight mode helper (v2) Len Brown
2008-04-29 8:44 ` [PATCH 157/171] ACPI: thinkpad-acpi: warn once about weird hotkey masks Len Brown
2008-04-29 8:44 ` [PATCH 158/171] ACPI: thinkpad-acpi: enhance box identification output Len Brown
2008-04-29 8:44 ` [PATCH 159/171] ACPI: thinkpad-acpi: rate-limit CMOS/EC unsynced error messages Len Brown
2008-04-29 8:44 ` [PATCH 160/171] ACPI: thinkpad-acpi: fix brightness dimming control bug Len Brown
2008-04-29 8:44 ` [PATCH 161/171] ACPI: thinkpad-acpi: claim tpacpi as an official short handle Len Brown
2008-04-29 8:44 ` [PATCH 162/171] ACPI: thinkpad-acpi: prepare light and LED for sysfs support Len Brown
2008-04-29 8:44 ` [PATCH 163/171] ACPI: thinkpad-acpi: add sysfs led class support for thinklight (v3.1) Len Brown
2008-04-29 8:44 ` [PATCH 164/171] ACPI: thinkpad-acpi: add sysfs led class support to thinkpad leds (v3.1) Len Brown
2008-04-29 8:44 ` [PATCH 165/171] ACPI: thinkpad-acpi: fluff really minor fix Len Brown
2008-04-29 8:44 ` [PATCH 166/171] ACPI: thinkpad-acpi: use a private workqueue Len Brown
2008-04-29 8:44 ` [PATCH 167/171] ACPI: thinkpad-acpi: fix selects in Kconfig Len Brown
2008-04-29 8:44 ` [PATCH 168/171] ACPI: thinkpad-acpi: bump up version to 0.20 Len Brown
2008-04-29 8:44 ` [PATCH 170/171] ACPI: check a return value correctly in acpi_power_get_context() Len Brown
2008-04-29 8:44 ` [PATCH 171/171] thinkpad_acpi: fix possible NULL pointer dereference if kstrdup failed Len Brown
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=481D5A6A.4060603@suse.de \
--to=astarikovskiy@suse.de \
--cc=carlos@strangeworlds.co.uk \
--cc=len.brown@intel.com \
--cc=lenb@kernel.org \
--cc=linux-acpi@vger.kernel.org \
--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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.