-----Original Message-----
From: Hans de Goede [mailto:hdegoede@xxxxxxxxxx]
Sent: Wednesday, February 14, 2018 2:33 AM
To: Moore, Robert <robert.moore@xxxxxxxxx>; Rafael J. Wysocki
<rjw@xxxxxxxxxxxxx>; linux-acpi@xxxxxxxxxxxxxxx; Schmauss, Erik
<erik.schmauss@xxxxxxxxx>; Box, David E <david.e.box@xxxxxxxxx>
Cc: Len Brown <lenb@xxxxxxxxxx>; Bjorn Helgaas <bhelgaas@xxxxxxxxxx>;
linux-pci@xxxxxxxxxxxxxxx; devel@xxxxxxxxxx
Subject: Re: [PATCH v2 0/5] ACPI: Do not call _STA on battery devices
with unmet dependencies
Hi,
On 08-02-18 18:34, Moore, Robert wrote:
The last one needs to go in via ACPICA anyway or we'll need to carry
a Linux-specific replacement for acpi_get_object_info().
Ack, I'm discussing how to proceed with this with the ACPICA people.
Regards,
Hans
I normally don't like to change external interfaces (the biggest
reason being the sheer number of different OS vendors that use ACPICA),
but in this case, I feel it may be OK. This interface is probably not
heavily used, and there is precedent for this when the invocation of
_SUB was removed.
On the other hand, I think I saw it mentioned that Linux calls this
interface only once, is this correct?
More or less, Linux calls it once when enumerating all the devices and a
few ACPI drivers call it again to get some extra info they need.
If so, perhaps it would be easiest to simply remove this call and use
direct control method invocations to obtain the desired data.
If we go this route we would probably end up copying the function, since
as said several drivers use it too.
If we go ahead and make this change to ACPICA: I have not seen the
patch, but please make sure that all vestiges of this are removed. This
includes all invocations within ACPICA that examine the result of the
_STA invocation, the CurrentStatus field of ACPI_DEVICE_INFO, the
ACPI_VALID_STA flag, etc.
The patch already does that:
https://patchwork.kernel.org/patch/10186195/
