All of lore.kernel.org
 help / color / mirror / Atom feed
From: Zhang Rui <rui.zhang@intel.com>
To: "Moore, Robert" <robert.moore@intel.com>
Cc: "linux-acpi@vger.kernel.org" <linux-acpi@vger.kernel.org>,
	"Lin, Ming M" <ming.m.lin@intel.com>,
	"Brown, Len" <lenb@kernel.org>
Subject: RE: Why is Scope(_TZ_) defined as ACPI_TYPE_THERMAL rather than ACPI_TYPE_LOCAL_SCOPE
Date: Tue, 21 Sep 2010 08:16:31 +0800	[thread overview]
Message-ID: <1285028191.2111.3472.camel@rui> (raw)
In-Reply-To: <4911F71203A09E4D9981D27F9D830858AF29BF79@orsmsx503.amr.corp.intel.com>

Hi, bob,

On Tue, 2010-09-21 at 04:29 +0800, Moore, Robert wrote:
> The comment appears to be correct as far as notifies on _TZ. I easily found a couple machines:
> 
> C:\ACPI_PREVIOUS\ASLFILES\DSDT DATABASE\Benq-Joybook_6000-017-original.asl(3652):                            Notify (\_TZ, 0x80)
> C:\ACPI_PREVIOUS\ASLFILES\DSDT DATABASE\Benq-Joybook_7000-R202-custom.asl(3247):                            Notify (\_TZ, 0x80)
> C:\ACPI_PREVIOUS\ASLFILES\DSDT DATABASE\Toshiba-Tecra_S1-2.30-custom.asl(708):                        Notify (\_TZ, 0x80)
> C:\ACPI_PREVIOUS\ASLFILES\DSDT DATABASE\Toshiba-Tecra_S1-2.30-custom.asl(723):                        Notify (\_TZ, 0x80)
> C:\ACPI_PREVIOUS\ASLFILES\DSDT DATABASE\Toshiba-Tecra_S1-2.30-custom.asl(3560):                        Notify (\_TZ, 0x80)
> 

what does Notify (\_TZ, 0x80) mean?
_TZ doesn't have _TMP method as it's not an thermal zone actually.
So what action should be taken when evaluating this ASL code?

Plus, if we really want "Notify(\_TZ, 0x80)", we can use
ACPI_TYPE_DEVICE rather than ACPI_TYPE_THERMAL, right?

thanks,
rui
> 
> 
> 
> >-----Original Message-----
> >From: Zhang, Rui
> >Sent: Monday, September 20, 2010 12:08 AM
> >To: Moore, Robert
> >Cc: linux-acpi@vger.kernel.org; Lin, Ming M; Brown, Len
> >Subject: Why is Scope(_TZ_) defined as ACPI_TYPE_THERMAL rather than
> >ACPI_TYPE_LOCAL_SCOPE
> >
> >Hi, Bob,
> >
> >this is the code in utglobal.c,
> >/*
> > * Predefined ACPI Names (Built-in to the Interpreter)
> > *
> > * NOTES:
> > * 1) _SB_ is defined to be a device to allow \_SB_._INI to be run
> > *    during the initialization sequence.
> > * 2) _TZ_ is defined to be a thermal zone in order to allow ASL code to
> > *    perform a Notify() operation on it.
> > */
> >const struct acpi_predefined_names acpi_gbl_pre_defined_names[] = {
> >        {"_GPE", ACPI_TYPE_LOCAL_SCOPE, NULL},
> >        {"_PR_", ACPI_TYPE_LOCAL_SCOPE, NULL},
> >        {"_SB_", ACPI_TYPE_DEVICE, NULL},
> >        {"_SI_", ACPI_TYPE_LOCAL_SCOPE, NULL},
> >        {"_TZ_", ACPI_TYPE_THERMAL, NULL},
> >        {"_REV", ACPI_TYPE_INTEGER, (char *)ACPI_CA_SUPPORT_LEVEL},
> >        {"_OS_", ACPI_TYPE_STRING, ACPI_OS_NAME},
> >        {"_GL_", ACPI_TYPE_MUTEX, (char *)1},
> >
> >I found a problem that Linux thermal driver tries to probe _TZ_ because
> >the _TZ_ ACPI namespace node type suggests it's a thermal device.
> >
> >I'm wondering why it's a device rather than a scope.
> >The comment above says ASL code may perform a Notify() operation on it,
> >but IMO, this notification is meaningless on its own, because _TZ_ can
> >neither detect the temperature, nor cool the system down, i.e. no action
> >should be taken on this notification.
> >
> >So I think we should change the type to ACPI_TYPE_LOCAL_SCOPE, just like
> >_PR_, _GPE.
> >
> >BTW: yes, we can do the tricks in Linux/ACPI code, but it's more likely
> >an ACPICA problem. what do you think?
> >
> >thanks,
> >rui
> 



  reply	other threads:[~2010-09-21  0:16 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-09-20  7:08 Why is Scope(_TZ_) defined as ACPI_TYPE_THERMAL rather than ACPI_TYPE_LOCAL_SCOPE Zhang Rui
2010-09-20 20:17 ` Moore, Robert
2010-09-20 20:29 ` Moore, Robert
2010-09-21  0:16   ` Zhang Rui [this message]
2010-09-21  0:56     ` Moore, Robert

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=1285028191.2111.3472.camel@rui \
    --to=rui.zhang@intel.com \
    --cc=lenb@kernel.org \
    --cc=linux-acpi@vger.kernel.org \
    --cc=ming.m.lin@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 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.