On Wed, Jan 27, 2021 at 04:44:02PM +0100, Rafael J. Wysocki wrote:
On Wed, Jan 27, 2021 at 4:16 PM Bjorn Helgaas helgaas@kernel.org wrote:
On Tue, Jan 26, 2021 at 02:23:17PM -0600, Bjorn Helgaas wrote:
From: Bjorn Helgaas bhelgaas@google.com
The double negative makes it hard to read "if (!ACPI_FAILURE(status))". Replace it with "if (ACPI_SUCCESS(status))".
Signed-off-by: Bjorn Helgaas bhelgaas@google.com
This isn't really an ACPI patch, but I'm sending it to you, Rafael, since it seems easier to just apply these all at once. But I'd be happy to split them up into individual patches if you'd rather.
Thanks, everybody. Rafael, I'll just merge this via my tree to avoid burdening you.
It may conflict with some janitorial stuff I'm doing, though, so unless you've already applied it, I'd prefer to take it via the ACPI tree.
No problem, it's all yours!
Bjorn