From: Henrique de Moraes Holschuh <hmh@hmh.eng.br>
To: Karol Lewandowski <karol.k.lewandowski@gmail.com>
Cc: linux-kernel@vger.kernel.org, astarikovskiy@suse.de,
lenb@kernel.org, linux-acpi@vger.kernel.org, rjwysocki@sisk.pl
Subject: Re: [PATCH] Fix ACPI's EC regression from 2.6.29: use usleep() instead of msleep()
Date: Mon, 7 Sep 2009 19:43:59 -0300 [thread overview]
Message-ID: <20090907224359.GI16533@khazad-dum.debian.net> (raw)
In-Reply-To: <20090907220413.GA2593@bizet.domek.prywatny>
On Tue, 08 Sep 2009, Karol Lewandowski wrote:
> > On Mon, 07 Sep 2009, Karol Lewandowski wrote:
> > > Following patch reverts commit that changed usleep() to mleep()
> > > in ACPI's EC driver. That change made hotkeys on my Thinkpad T21
> > > unusable after resume.
> >
> > Karol, can you send us the dmidecode information for your thinkpad?
>
> Sure, here goes dmidecode output with UUID and SN grayed out (I guess
> that isn't needed).
Thank you, and yes, it is a good idea to gray out the UUID and SN, we don't
need or want them.
Can you please send the output of recent acpidump to me by private email?
You are using the latest T21 BIOS, and a full ACPI dump can be quite
valuable for my work in thinkpad-acpi.
I think the best way to go about this, as long as we don't get reports from
many other boxes failing to resume or hitting ACPI EC problems, is to add a
"slow_thinkpad_ec" quirk, and use dmi matches to enable it on the T21 and
other thinkpads of that vintage (definately all of the T2x and A2x, maybe a
few others. I have a candidate list here that I assembled for other
reasons.
--
"One disk to rule them all, One disk to find them. One disk to bring
them all and in the darkness grind them. In the Land of Redmond
where the shadows lie." -- The Silicon Valley Tarot
Henrique Holschuh
prev parent reply other threads:[~2009-09-07 22:44 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-09-07 21:14 Karol Lewandowski
2009-09-07 21:57 ` Henrique de Moraes Holschuh
2009-09-07 22:04 ` Karol Lewandowski
2009-09-07 22:43 ` Henrique de Moraes Holschuh [this message]
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=20090907224359.GI16533@khazad-dum.debian.net \
--to=hmh@hmh.eng.br \
--cc=astarikovskiy@suse.de \
--cc=karol.k.lewandowski@gmail.com \
--cc=lenb@kernel.org \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=rjwysocki@sisk.pl \
/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
all inboxes | Powered by JetHome®