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@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. No problem, it's all yours! Bjorn
