[PATCH] ACPI: Test for ACPI_SUCCESS rather than !ACPI_FAILURE
Bjorn Helgaas
helgaas at kernel.org
Wed Jan 27 17:06:50 CET 2021
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 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.
No problem, it's all yours!
Bjorn
More information about the Alsa-devel
mailing list