lkml.org 
[lkml]   [2018]   [Apr]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 4.14 131/183] ACPI / bus: Do not call _STA on battery devices with unmet dependencies
    Date
    4.14-stable review patch.  If anyone has any objections, please let me know.

    ------------------

    From: Hans de Goede <hdegoede@redhat.com>


    [ Upstream commit 54ddce7062242036402242242c07c60c0b505f84 ]

    The battery code uses acpi_device->dep_unmet to check for unmet deps and
    if there are unmet deps it does not bind to the device to avoid errors
    about missing OpRegions when calling ACPI methods on the device.

    The missing OpRegions when there are unmet deps problem also applies to
    the _STA method of some battery devices and calling it too early results
    in errors like these:

    [ 0.123579] ACPI Error: No handler for Region [ECRM] (00000000ba9edc4c)
    [GenericSerialBus] (20170831/evregion-166)
    [ 0.123601] ACPI Error: Region GenericSerialBus (ID=9) has no handler
    (20170831/exfldio-299)
    [ 0.123618] ACPI Error: Method parse/execution failed
    \_SB.I2C1.BAT1._STA, AE_NOT_EXIST (20170831/psparse-550)

    This commit fixes these errors happening when acpi_get_bus_status gets
    called by checking dep_unmet for battery devices and reporting a status
    of 0 until all dependencies are met.

    Signed-off-by: Hans de Goede <hdegoede@redhat.com>
    Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
    Signed-off-by: Sasha Levin <alexander.levin@microsoft.com>
    Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
    ---
    drivers/acpi/bus.c | 6 ++++++
    1 file changed, 6 insertions(+)

    --- a/drivers/acpi/bus.c
    +++ b/drivers/acpi/bus.c
    @@ -146,6 +146,12 @@ int acpi_bus_get_status(struct acpi_devi
    return 0;
    }

    + /* Battery devices must have their deps met before calling _STA */
    + if (acpi_device_is_battery(device) && device->dep_unmet) {
    + acpi_set_device_status(device, 0);
    + return 0;
    + }
    +
    status = acpi_bus_get_status_handle(device->handle, &sta);
    if (ACPI_FAILURE(status))
    return -ENODEV;

    \
     
     \ /
      Last update: 2018-04-25 13:03    [W:4.691 / U:0.548 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site