On Wed, Jan 27, 2021 at 4:16 PM Bjorn Helgaas <helgaas@xxxxxxxxxx> wrote: > > On Tue, Jan 26, 2021 at 02:23:17PM -0600, Bjorn Helgaas wrote: > > From: Bjorn Helgaas <bhelgaas@xxxxxxxxxx> > > > > 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@xxxxxxxxxx> > > --- > > > > 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!