| Message ID | 1400167727-20233-1-git-send-email-mika.westerberg@linux.intel.com (mailing list archive) |
|---|---|
| State | Not Applicable, archived |
| Headers | show |
On Fri, May 16, 2014 at 02:10:47PM +0200, Rafael J. Wysocki wrote: > On Thursday, May 15, 2014 06:28:46 PM Mika Westerberg wrote: > > From: Jin Yao <yao.jin@intel.com> > > > > Now that the x86 dynamic IRQ allocation problem has been resolved with > > commmit 62a08ae2a576 (genirq: x86: Ensure that dynamic irq allocation does > > not conflict), we can add back Baytrail-T ACPI ID to the pinctrl driver. > > > > This makes the driver to work on Asus T100 where it is needed for several > > things like ACPI GPIO events and SD card detection. > > > > References: https://bugzilla.kernel.org/show_bug.cgi?id=68291 > > Signed-off-by: Jin Yao <yao.jin@intel.com> > > Signed-off-by: Mika Westerberg <mika.westerberg@linux.intel.com> > > Acked-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com> Thanks! > And I think this is pinctrl material really. Indeed, only one change touches drivers/acpi/ but wanted to keep you in the loop. I'm hoping that Linus W. takes these. -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Thursday, May 15, 2014 06:28:46 PM Mika Westerberg wrote: > From: Jin Yao <yao.jin@intel.com> > > Now that the x86 dynamic IRQ allocation problem has been resolved with > commmit 62a08ae2a576 (genirq: x86: Ensure that dynamic irq allocation does > not conflict), we can add back Baytrail-T ACPI ID to the pinctrl driver. > > This makes the driver to work on Asus T100 where it is needed for several > things like ACPI GPIO events and SD card detection. > > References: https://bugzilla.kernel.org/show_bug.cgi?id=68291 > Signed-off-by: Jin Yao <yao.jin@intel.com> > Signed-off-by: Mika Westerberg <mika.westerberg@linux.intel.com> Acked-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com> And I think this is pinctrl material really. > --- > drivers/acpi/acpi_lpss.c | 1 + > drivers/pinctrl/pinctrl-baytrail.c | 1 + > 2 files changed, 2 insertions(+) > > diff --git a/drivers/acpi/acpi_lpss.c b/drivers/acpi/acpi_lpss.c > index 24e49a52cefb..c314d709e5a6 100644 > --- a/drivers/acpi/acpi_lpss.c > +++ b/drivers/acpi/acpi_lpss.c > @@ -189,6 +189,7 @@ static const struct acpi_device_id acpi_lpss_device_ids[] = { > { "80860F14", (unsigned long)&byt_sdio_dev_desc }, > { "80860F41", (unsigned long)&byt_i2c_dev_desc }, > { "INT33B2", }, > + { "INT33FC", }, > > { "INT3430", (unsigned long)&lpt_dev_desc }, > { "INT3431", (unsigned long)&lpt_dev_desc }, > diff --git a/drivers/pinctrl/pinctrl-baytrail.c b/drivers/pinctrl/pinctrl-baytrail.c > index 6e8301f77187..447f1dc25ea6 100644 > --- a/drivers/pinctrl/pinctrl-baytrail.c > +++ b/drivers/pinctrl/pinctrl-baytrail.c > @@ -572,6 +572,7 @@ static const struct dev_pm_ops byt_gpio_pm_ops = { > > static const struct acpi_device_id byt_gpio_acpi_match[] = { > { "INT33B2", 0 }, > + { "INT33FC", 0 }, > { } > }; > MODULE_DEVICE_TABLE(acpi, byt_gpio_acpi_match); >
On Thu, May 15, 2014 at 5:28 PM, Mika Westerberg <mika.westerberg@linux.intel.com> wrote: > From: Jin Yao <yao.jin@intel.com> > > Now that the x86 dynamic IRQ allocation problem has been resolved with > commmit 62a08ae2a576 (genirq: x86: Ensure that dynamic irq allocation does > not conflict), we can add back Baytrail-T ACPI ID to the pinctrl driver. > > This makes the driver to work on Asus T100 where it is needed for several > things like ACPI GPIO events and SD card detection. > > References: https://bugzilla.kernel.org/show_bug.cgi?id=68291 > Signed-off-by: Jin Yao <yao.jin@intel.com> > Signed-off-by: Mika Westerberg <mika.westerberg@linux.intel.com> Patch applied with Rafael's ACK. Applying this and 2/2 to v3.16 though, protest if there is a problem with this. Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Thu, May 22, 2014 at 11:50:00PM +0200, Linus Walleij wrote: > On Thu, May 15, 2014 at 5:28 PM, Mika Westerberg > <mika.westerberg@linux.intel.com> wrote: > > > From: Jin Yao <yao.jin@intel.com> > > > > Now that the x86 dynamic IRQ allocation problem has been resolved with > > commmit 62a08ae2a576 (genirq: x86: Ensure that dynamic irq allocation does > > not conflict), we can add back Baytrail-T ACPI ID to the pinctrl driver. > > > > This makes the driver to work on Asus T100 where it is needed for several > > things like ACPI GPIO events and SD card detection. > > > > References: https://bugzilla.kernel.org/show_bug.cgi?id=68291 > > Signed-off-by: Jin Yao <yao.jin@intel.com> > > Signed-off-by: Mika Westerberg <mika.westerberg@linux.intel.com> > > Patch applied with Rafael's ACK. > > Applying this and 2/2 to v3.16 though, protest if there is a problem with this. v3.16 is fine. Thanks! -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/acpi/acpi_lpss.c b/drivers/acpi/acpi_lpss.c index 24e49a52cefb..c314d709e5a6 100644 --- a/drivers/acpi/acpi_lpss.c +++ b/drivers/acpi/acpi_lpss.c @@ -189,6 +189,7 @@ static const struct acpi_device_id acpi_lpss_device_ids[] = { { "80860F14", (unsigned long)&byt_sdio_dev_desc }, { "80860F41", (unsigned long)&byt_i2c_dev_desc }, { "INT33B2", }, + { "INT33FC", }, { "INT3430", (unsigned long)&lpt_dev_desc }, { "INT3431", (unsigned long)&lpt_dev_desc }, diff --git a/drivers/pinctrl/pinctrl-baytrail.c b/drivers/pinctrl/pinctrl-baytrail.c index 6e8301f77187..447f1dc25ea6 100644 --- a/drivers/pinctrl/pinctrl-baytrail.c +++ b/drivers/pinctrl/pinctrl-baytrail.c @@ -572,6 +572,7 @@ static const struct dev_pm_ops byt_gpio_pm_ops = { static const struct acpi_device_id byt_gpio_acpi_match[] = { { "INT33B2", 0 }, + { "INT33FC", 0 }, { } }; MODULE_DEVICE_TABLE(acpi, byt_gpio_acpi_match);