All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Renninger <trenn@suse.de>
To: Thomas Lindroth <thomas.lindroth@gmail.com>
Cc: Alexey Starikovskiy <aystarik@gmail.com>, linux-acpi@vger.kernel.org
Subject: Re: Problem: ACPI related soft lockup on Fujitsu-Siemens AMILO Si 2636
Date: Wed, 8 Oct 2008 13:08:51 +0200	[thread overview]
Message-ID: <200810081308.52221.trenn@suse.de> (raw)
In-Reply-To: <48EC9004.6010105@gmail.com>

On Wednesday 08 October 2008 12:48:36 Thomas Lindroth wrote:
> Alexey Starikovskiy wrote:
> > Alexey Starikovskiy wrote:
> >> This is AML implementation of EC driver... it relies on certain speed
> >> of AML interpreter to work properly (look for Noop inserted as delay).
> >>
> >> It is possible to replace this driver with the native Linux one, which
> >> might be
> >> more robust, but this involves heavy DSDT hacking.
> >>
> >> Overall, it is heavily mis-designed BIOS, so don't expect it to be
> >> fixed on Linux side.
> >>
> >> Regards,
> >> Alex.
> >
> > Please check if making Noop a small delay helps
>
> I've tried the patch with an udelay of 100 but it did not make any
> difference. I've also tried raising it to 200 but that created an
> unbootable system because one of the ACPI init methods failed.
>
> You are right, this is a really broken BIOS.
>
> I find it odd that only I experience this problem. According to one
> site this laptop is the 7:th most popular laptop right now so someone
> else should have reported something.

Could you provide a short explanation what should be fixed in which way in the 
BIOS tables. Best in a way that I can just forward this.
It's hard in the laptop area to convince fsc to fix it, but I at least like to 
try and to tell them.
Hmm, I think the last two mails should already be enough, I think  I got it.

Is there a general hint I could add into the ACPI Linux guideline, like:
Don't implement EC data/index register reads/writes in AML yourself.
Define a EC variable and e.g. use Store(ECXY, Local0) to read and
Store(Local0, ECXY) to write to specific EC bytes.

Did I get this right?

Thanks,

     Thomas

  reply	other threads:[~2008-10-08 11:08 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-10-06 11:23 Problem: ACPI related soft lockup on Fujitsu-Siemens AMILO Si 2636 Thomas Lindroth
2008-10-08  7:20 ` Thomas Lindroth
2008-10-08  7:42   ` Alexey Starikovskiy
2008-10-08  8:49     ` Alexey Starikovskiy
2008-10-08 10:48       ` Thomas Lindroth
2008-10-08 11:08         ` Thomas Renninger [this message]
2008-10-08 11:52           ` Alexey Starikovskiy
2008-10-08 22:19             ` Thomas Renninger
2008-10-09  1:50   ` Len Brown
2008-10-09  7:46     ` Thomas Lindroth

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=200810081308.52221.trenn@suse.de \
    --to=trenn@suse.de \
    --cc=aystarik@gmail.com \
    --cc=linux-acpi@vger.kernel.org \
    --cc=thomas.lindroth@gmail.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.