public inbox for linux-acpi@vger.kernel.org
 help / color / mirror / Atom feed
From: Bruno Ducrot <ducrot@poupinou.org>
To: Ben B <kernel@bb.cactii.net>
Cc: Thomas Renninger <trenn@suse.de>, linux-acpi@vger.kernel.org
Subject: Re: DSDT/ASL compiler error
Date: Fri, 18 Aug 2006 22:17:59 +0200	[thread overview]
Message-ID: <20060818201759.GC4945@poupinou.org> (raw)
In-Reply-To: <20060818121048.GA1579@cactii.net>

On Fri, Aug 18, 2006 at 02:10:48PM +0200, Ben B wrote:
> Bruno Ducrot <ducrot@poupinou.org> uttered the following thing:
> > On Fri, Aug 18, 2006 at 12:33:30PM +0200, Thomas Renninger wrote:
> > > > 
> > > > dsdt.dsl  3275:                                         And (Local1, 0xFFFF)
> > > > Warning  1104 -        Result is not used, operator has no effect ^ 
> > > > 
> > I'm not sure if that's one is harmless:
> > > > dsdt.dsl  3275:                                         And (Local1, 0xFFFF)
> > It's as if the asl writter wanted to clear the upper bits of Local1, but forgot to
> > assign it back to Local1.  It might be something like this:
> > 	And(Local1, 0xFFFF, Local1)
> > 
> > It's only a rough guess though, and wihtout knowing what was the
> > original intention, and without the source of the ASL, it's hard
> > to tell for sure.
> 
> This is the context around there. I can send the whole asl if you'd like
> also:
> 
>                                 Store (C15B, Local1)
>                                 If (Local1)
>                                 {
>                                     If (And (C165, 0x40))
>                                     {
>                                         Add (Not (Local1), 0x01, Local1)
>                                         And (Local1, 0xFFFF)
>                                     }
>                                 }
> 
>                                 Store (Local1, Index (DerefOf (Index (Local0, 0x02)), 0x0E))

Sound like it really should be And(Local1, 0xFFFF, Local1).

> accelerometer, which I only discovered browsing the ASL:
> 
>             Device (ACEL)
>             {
>                 Name (_HID, EisaId ("HPQ0004"))
> 
> "HP Mobile Data Protection Sensor"....but that's for another project. :)

Indeed.

-- 
Bruno Ducrot

--  Which is worse:  ignorance or apathy?
--  Don't know.  Don't care.

  reply	other threads:[~2006-08-18 20:18 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-08-17 17:03 DSDT/ASL compiler error Ben B
2006-08-18 10:33 ` Thomas Renninger
2006-08-18 11:25   ` Bruno Ducrot
2006-08-18 12:10     ` Ben B
2006-08-18 20:17       ` Bruno Ducrot [this message]
2006-08-18 13:09     ` Thomas Renninger
2006-08-18 20:18       ` Bruno Ducrot
2006-08-19 12:50         ` Thomas Renninger
2006-08-20 10:09           ` Bruno Ducrot
2006-08-20 10:22           ` Bruno Ducrot
2006-08-20 14:34             ` Thomas Renninger

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=20060818201759.GC4945@poupinou.org \
    --to=ducrot@poupinou.org \
    --cc=kernel@bb.cactii.net \
    --cc=linux-acpi@vger.kernel.org \
    --cc=trenn@suse.de \
    /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