From: "Rafael J. Wysocki" <rjw@sisk.pl>
To: "Adachi, Kenichi" <aileenja@s6.dion.ne.jp>
Cc: Lv Zheng <lv.zheng@intel.com>,
"Rafael J. Wysocki" <rafael.j.wysocki@intel.com>,
Robert Moore <robert.moore@intel.com>,
Len Brown <len.brown@intel.com>,
linux-acpi@vger.kernel.org
Subject: Re: [PATCH 6/9] ACPICA: Make ACPI Power Management Timer (PM Timer) optional.
Date: Sat, 17 Aug 2013 02:22:20 +0200 [thread overview]
Message-ID: <1434762.cis3jMuHy8@vostro.rjw.lan> (raw)
In-Reply-To: <CAEYoeW6oXnZtmBEquu8MZ9LyBKTibUtn4Pie+28v5MWCrJSyNQ@mail.gmail.com>
On Friday, August 09, 2013 12:17:21 PM Adachi, Kenichi wrote:
> Hi,
>
> On Thu, Aug 8, 2013 at 4:29 PM, Lv Zheng <lv.zheng@intel.com> wrote:
> >
> > From: Bob Moore <robert.moore@intel.com>
> >
> > PM Timer is now optional.
> ...
> > The change doesn't affect Linux directly, because it has never relied
> > on the presence of the PM timer.
> >
> Has it been so?
>
> IIRC, there're a couple of accesses to PM Timer via:
> - inl(acpi_gbl_FADT.xpm_timer_block.address);
> typically in processor driver code: ex) drivers/acpi/processor_idle.c
> which seems to rely on the presence of the PM timer.
That's correct, so I modified the changelog as it was not accurate. However,
we don't rely on the PM timer presence any more, because we found systems
in the field where the PM timer was not present.
Thanks,
Rafael
next prev parent reply other threads:[~2013-08-17 0:11 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-08-08 7:29 [PATCH 0/9] ACPICA 20130725 Release Lv Zheng
2013-08-08 7:29 ` [PATCH 1/9] ACPICA: Export acpi_tb_validate_rsdp() Lv Zheng
2013-08-08 7:29 ` [PATCH 2/9] ACPICA: Emit all unresolved method externals in a text block Lv Zheng
2013-08-08 7:29 ` [PATCH 3/9] ACPICA: DeRefOf operator: Update to fully resolve FieldUnit and BufferField refs Lv Zheng
2013-08-08 7:29 ` [PATCH 4/9] ACPICA: Fix possible fault for methods that optionally have no return value Lv Zheng
2013-08-08 7:29 ` [PATCH 5/9] ACPICA: Fix divergences of the commit - ACPICA: Expose OSI version Lv Zheng
2013-08-08 7:29 ` [PATCH 6/9] ACPICA: Make ACPI Power Management Timer (PM Timer) optional Lv Zheng
2013-08-09 3:17 ` Adachi, Kenichi
2013-08-17 0:22 ` Rafael J. Wysocki [this message]
2013-08-08 7:29 ` [PATCH 7/9] ACPICA: Return error if DerefOf resolves to a null package element Lv Zheng
2013-08-08 7:30 ` [PATCH 8/9] ACPICA: Update names for walk_namespace callbacks to clarify usage Lv Zheng
2013-08-08 7:30 ` [PATCH 9/9] ACPICA: Update version to 20130725 Lv Zheng
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=1434762.cis3jMuHy8@vostro.rjw.lan \
--to=rjw@sisk.pl \
--cc=aileenja@s6.dion.ne.jp \
--cc=len.brown@intel.com \
--cc=linux-acpi@vger.kernel.org \
--cc=lv.zheng@intel.com \
--cc=rafael.j.wysocki@intel.com \
--cc=robert.moore@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;
as well as URLs for NNTP newsgroup(s).