Received-SPF: pass (google.com: domain of linux-kernel+bounces-125270-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
From: David McFarland <corngood@gmail.com>
To: "Linux regression tracking (Thorsten Leemhuis)" <regressions@leemhuis.info>
Cc: Chris Feng <chris.feng@mediatek.com>,  Linux regressions mailing list
 <regressions@lists.linux.dev>,  "Rafael J . Wysocki"
 <rafael.j.wysocki@intel.com>,  Alex Hung <alexhung@gmail.com>,  Hans de
 Goede <hdegoede@redhat.com>,  Ilpo =?utf-8?Q?J=C3=A4rvinen?=
 <ilpo.jarvinen@linux.intel.com>,  "platform-driver-x86@vger.kernel.org"
 <platform-driver-x86@vger.kernel.org>,  LKML
 <linux-kernel@vger.kernel.org>,  linux-pm@vger.kernel.org
Subject: Re: [PATCH 1/1] platform/x86/intel/hid: Don't wake on 5-button
 releases
In-Reply-To: <1198933e-bf89-4237-a6e8-f7daeeebf885@leemhuis.info> (Linux
	regression tracking's message of "Fri, 29 Mar 2024 14:51:47 +0100")
References: <20240318191153.6978-1-corngood@gmail.com>
	<20240318191153.6978-2-corngood@gmail.com>
	<1198933e-bf89-4237-a6e8-f7daeeebf885@leemhuis.info>
Date: Fri, 29 Mar 2024 15:06:10 -0300
Message-ID: <87bk6wlxpp.fsf@gmail.com>
User-Agent: Gnus/5.13 (Gnus v5.13)
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain

"Linux regression tracking (Thorsten Leemhuis)"
<regressions@leemhuis.info> writes:

> David, from here is looks like this is stalled for ten days now. Or was
> there some progress and I just missed it?

No, I've not seen any emails since your last.

> From the cover letter[1] is sounds a lot like a "Fixes: 0c4cae1bc00d31
> ("PM: hibernate: Avoid missing wakeup events during hibernation")" would
> be appropriate here.

The specific behaviour I encountered (failure to hibernate) started with
that commit, but I think it just exposed the underlying behaviour (wake
on button release), which probably dates to when the driver was
introduced.

I believe it would have been possible to reproduce the other behaviour I
mentioned (long hold of button to suspend causes the machine to wake on
release), even before 0c4cae1bc00d31.

That's why I left it off, but I'm happy to revise.