[PATCH] ACPI: Test for ACPI_SUCCESS rather than !ACPI_FAILURE
Rafael J. Wysocki
rafael at kernel.org
Wed Jan 27 16:44:02 CET 2021
On Wed, Jan 27, 2021 at 4:16 PM Bjorn Helgaas <helgaas at kernel.org> wrote:
>
> On Tue, Jan 26, 2021 at 02:23:17PM -0600, Bjorn Helgaas wrote:
> > From: Bjorn Helgaas <bhelgaas at 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 at 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.
Cheers!
More information about the Alsa-devel
mailing list