Re: [PATCH] gpiolib-acpi: add _DEP support for Acer One 10

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 6/14/2016 12:01 PM, Mika Westerberg wrote:
On Tue, Jun 14, 2016 at 09:33:30AM +0200, Linus Walleij wrote:
Mika, Rafael: this came from Intel, can you review it? Should it be applied?
Looks reasonable so unless Rafael objects you can add my,

Acked-by: Mika Westerberg <mika.westerberg@xxxxxxxxxxxxxxx>

No objections.

Yours,
Linus Walleij

On Mon, May 16, 2016 at 5:18 AM, Zhang, Rui <rui.zhang@xxxxxxxxx> wrote:

 From 8590bdcacc94a1ab260b4ea2e6e747b0bc1d9bf8 Mon Sep 17 00:00:00 2001
From: Zhang Rui <rui.zhang@xxxxxxxxx>
Date: Wed, 11 May 2016 09:02:28 +0800
Subject: [PATCH] gpiolib-acpi: add _DEP support for Acer One 10

On Acer One 10, the ACPI battery driver can not be probed because
it depends on the GPIO controller as well as the I2C controller to work,
         Device (BATC)
         {
             Name (_HID, EisaId ("PNP0C0A") /* Control Method Battery */)
             ...
             Name (_DEP, Package (0x03)  // _DEP: Dependencies
             {
                 I2C1,
                 GPO2,
                 GPO0
             })
             ...
         }

The I2C dependency also exists on other platforms and has been fixed by commit
40e7fcb19293 ("ACPI: Add _DEP support to fix battery issue on Asus T100TA"),
this patch resolves the GPIO dependency for Acer One 10.

Link:https://bugzilla.kernel.org/show_bug.cgi?id=115191
CC: Mika Westerberg <mika.westerberg@xxxxxxxxxxxxxxx>
CC: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx>
Tested-by: Stace A. Zacharov <stace75@xxxxxxxxx>
Signed-off-by: Zhang Rui <rui.zhang@xxxxxxxxx>
---
  drivers/gpio/gpiolib-acpi.c | 1 +
  1 file changed, 1 insertion(+)

diff --git a/drivers/gpio/gpiolib-acpi.c b/drivers/gpio/gpiolib-acpi.c
index 682070d..4b809b0 100644
--- a/drivers/gpio/gpiolib-acpi.c
+++ b/drivers/gpio/gpiolib-acpi.c
@@ -836,6 +836,7 @@ void acpi_gpiochip_add(struct gpio_chip *chip)
         }

         acpi_gpiochip_request_regions(acpi_gpio);
+       acpi_walk_dep_device_list(handle);
  }

  void acpi_gpiochip_remove(struct gpio_chip *chip)
--
1.9.1



--
To unsubscribe from this list: send the line "unsubscribe linux-gpio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux SPI]     [Linux Kernel]     [Linux ARM (vger)]     [Linux ARM MSM]     [Linux Omap]     [Linux Arm]     [Linux Tegra]     [Fedora ARM]     [Linux for Samsung SOC]     [eCos]     [Linux Fastboot]     [Gcc Help]     [Git]     [DCCP]     [IETF Announce]     [Security]     [Linux MIPS]     [Yosemite Campsites]

  Powered by Linux