diff mbox series

[1/2] ACPI / LPSS: Add alternative ACPI HIDs for Cherry Trail DMA controllers

Message ID 20180827074545.5009-2-hdegoede@redhat.com (mailing list archive)
State Not Applicable
Headers show
Series ACPI + dma: Add alternative ACPI HIDs for Cherry Trail DMA controllers | expand

Commit Message

Hans de Goede Aug. 27, 2018, 7:45 a.m. UTC
Bay and Cherry Trail DSTDs represent a different set of devices depending
on which OS the device think it is booting. One set of decices for Windows
and another set of devices for Android which targets the Android-x86 Linux
kernel fork (which e.g. used to have its own display driver instead of
using the i915 driver).

Which set of devices we are actually going to get is out of our control,
this is controlled by the ACPI OSID variable, which gets either set through
an EFI setup option, or sometimes is autodetected. So we need to support
both.

This commit adds support for the 80862286 and 808622C0 ACPI HIDs which we
get for the first resp. second DMA controller on Cherry Trail devices when
OSID is set to Android.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
 drivers/acpi/acpi_lpss.c | 2 ++
 1 file changed, 2 insertions(+)

Comments

Andy Shevchenko Aug. 29, 2018, 4:50 p.m. UTC | #1
On Mon, Aug 27, 2018 at 09:45:44AM +0200, Hans de Goede wrote:
> Bay and Cherry Trail DSTDs represent a different set of devices depending
> on which OS the device think it is booting. One set of decices for Windows
> and another set of devices for Android which targets the Android-x86 Linux
> kernel fork (which e.g. used to have its own display driver instead of
> using the i915 driver).
> 
> Which set of devices we are actually going to get is out of our control,
> this is controlled by the ACPI OSID variable, which gets either set through
> an EFI setup option, or sometimes is autodetected. So we need to support
> both.
> 
> This commit adds support for the 80862286 and 808622C0 ACPI HIDs which we
> get for the first resp. second DMA controller on Cherry Trail devices when
> OSID is set to Android.
> 

Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>

> Signed-off-by: Hans de Goede <hdegoede@redhat.com>
> ---
>  drivers/acpi/acpi_lpss.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/drivers/acpi/acpi_lpss.c b/drivers/acpi/acpi_lpss.c
> index 0569972b02e1..95d9c45e13cc 100644
> --- a/drivers/acpi/acpi_lpss.c
> +++ b/drivers/acpi/acpi_lpss.c
> @@ -327,9 +327,11 @@ static const struct acpi_device_id acpi_lpss_device_ids[] = {
>  	{ "INT33FC", },
>  
>  	/* Braswell LPSS devices */
> +	{ "80862286", LPSS_ADDR(lpss_dma_desc) },
>  	{ "80862288", LPSS_ADDR(bsw_pwm_dev_desc) },
>  	{ "8086228A", LPSS_ADDR(bsw_uart_dev_desc) },
>  	{ "8086228E", LPSS_ADDR(bsw_spi_dev_desc) },
> +	{ "808622C0", LPSS_ADDR(lpss_dma_desc) },
>  	{ "808622C1", LPSS_ADDR(bsw_i2c_dev_desc) },
>  
>  	/* Broadwell LPSS devices */
> -- 
> 2.18.0
>
Rafael J. Wysocki Sept. 14, 2018, 8:40 a.m. UTC | #2
On Wednesday, August 29, 2018 6:50:53 PM CEST Andy Shevchenko wrote:
> On Mon, Aug 27, 2018 at 09:45:44AM +0200, Hans de Goede wrote:
> > Bay and Cherry Trail DSTDs represent a different set of devices depending
> > on which OS the device think it is booting. One set of decices for Windows
> > and another set of devices for Android which targets the Android-x86 Linux
> > kernel fork (which e.g. used to have its own display driver instead of
> > using the i915 driver).
> > 
> > Which set of devices we are actually going to get is out of our control,
> > this is controlled by the ACPI OSID variable, which gets either set through
> > an EFI setup option, or sometimes is autodetected. So we need to support
> > both.
> > 
> > This commit adds support for the 80862286 and 808622C0 ACPI HIDs which we
> > get for the first resp. second DMA controller on Cherry Trail devices when
> > OSID is set to Android.
> > 
> 
> Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> 
> > Signed-off-by: Hans de Goede <hdegoede@redhat.com>
> > ---
> >  drivers/acpi/acpi_lpss.c | 2 ++
> >  1 file changed, 2 insertions(+)
> > 
> > diff --git a/drivers/acpi/acpi_lpss.c b/drivers/acpi/acpi_lpss.c
> > index 0569972b02e1..95d9c45e13cc 100644
> > --- a/drivers/acpi/acpi_lpss.c
> > +++ b/drivers/acpi/acpi_lpss.c
> > @@ -327,9 +327,11 @@ static const struct acpi_device_id acpi_lpss_device_ids[] = {
> >  	{ "INT33FC", },
> >  
> >  	/* Braswell LPSS devices */
> > +	{ "80862286", LPSS_ADDR(lpss_dma_desc) },
> >  	{ "80862288", LPSS_ADDR(bsw_pwm_dev_desc) },
> >  	{ "8086228A", LPSS_ADDR(bsw_uart_dev_desc) },
> >  	{ "8086228E", LPSS_ADDR(bsw_spi_dev_desc) },
> > +	{ "808622C0", LPSS_ADDR(lpss_dma_desc) },
> >  	{ "808622C1", LPSS_ADDR(bsw_i2c_dev_desc) },
> >  
> >  	/* Broadwell LPSS devices */
> 
> 

Patch applied, thanks!
diff mbox series

Patch

diff --git a/drivers/acpi/acpi_lpss.c b/drivers/acpi/acpi_lpss.c
index 0569972b02e1..95d9c45e13cc 100644
--- a/drivers/acpi/acpi_lpss.c
+++ b/drivers/acpi/acpi_lpss.c
@@ -327,9 +327,11 @@  static const struct acpi_device_id acpi_lpss_device_ids[] = {
 	{ "INT33FC", },
 
 	/* Braswell LPSS devices */
+	{ "80862286", LPSS_ADDR(lpss_dma_desc) },
 	{ "80862288", LPSS_ADDR(bsw_pwm_dev_desc) },
 	{ "8086228A", LPSS_ADDR(bsw_uart_dev_desc) },
 	{ "8086228E", LPSS_ADDR(bsw_spi_dev_desc) },
+	{ "808622C0", LPSS_ADDR(lpss_dma_desc) },
 	{ "808622C1", LPSS_ADDR(bsw_i2c_dev_desc) },
 
 	/* Broadwell LPSS devices */