ACPI / LPSS: override SDIO device's private register space size
PATCH 1/1 ACPI / LPSS: override SDIO device's private register space size
The SDIO device in Lynxpoint has its LTR registers allocated for a WiFi device (a child of the SDIO device) instead in ACPI namespace even though they physically belong to the SDIO device. In order to be able to access the LTR registers from the ACPI LPSS driver for diagnostic purposes we need to enlarge size of this private register space.
Work around this by adding a possibility to override size of the private register space in the ACPI LPSS driver and set correct size for the SDIO device.
---
Applies on top of https://patchwork.kernel.org/patch/2742521/ (ACPI / LPSS:
Power up LPSS devices during enumeration)
drivers/acpi/acpi_lpss.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-)
drivers/acpi/acpi_lpss.c +6 -1
@@ -44,6 +44,7 @@ struct lpss_device_desc {const char *clkdev_name;bool ltr_required;unsigned int prv_offset; +size_t prv_size_override;bool clk_gate;struct lpss_shared_clock *shared_clock; };
@@ -69,6 +70,7 @@ static struct lpss_device_desc lpt_dev_desc = {static struct lpss_device_desc lpt_sdio_dev_desc = { .prv_offset =0x1000 , + .prv_size_override =0x1018 , .ltr_required = true, };
@@ -226,7 +228,10 @@ static int acpi_lpss_create_device(struct acpi_device *adev,list_for_each_entry (rentry, &resource_list, node)if (resource_type (&rentry->res) ==IORESOURCE_MEM ) { - pdata->mmio_size =resource_size (&rentry->res); +if (dev_desc->prv_size_override) + pdata->mmio_size = dev_desc->prv_size_override; +else + pdata->mmio_size =resource_size (&rentry->res); pdata->mmio_base =ioremap (rentry->res.start, pdata->mmio_size); pdata->dev_desc = dev_desc;
Signature · 1 line
1.8.3.1
Quoted 6 lines · Mika Westerberg
The SDIO device in Lynxpoint has its LTR registers allocated for a WiFi device (a child of the SDIO device) instead in ACPI namespace even though they physically belong to the SDIO device. In order to be able to access the LTR registers from the ACPI LPSS driver for diagnostic purposes we need to enlarge size of this private register space.
Quoted 8 lines
Work around this by adding a possibility to override size of the private register space in the ACPI LPSS driver and set correct size for the SDIO device.
Signed-off-by Mika Westerberg <[email protected]>---
Applies on top of https://patchwork.kernel.org/patch/2742521/ (ACPI / LPSS:
Power up LPSS devices during enumeration)
Queued up for 3.11.
Thanks,
Rafael
Quoted 36 lines
drivers/acpi/acpi_lpss.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-)drivers/acpi/acpi_lpss.c +6
-1@@ -44,6 +44,7 @@ struct lpss_device_desc {const char *clkdev_name;bool ltr_required;unsigned int prv_offset; +size_t prv_size_override;bool clk_gate;struct lpss_shared_clock *shared_clock; };@@ -69,6 +70,7 @@ static struct lpss_device_desc lpt_dev_desc = {static struct lpss_device_desc lpt_sdio_dev_desc = { .prv_offset =0x1000 , + .prv_size_override =0x1018 , .ltr_required = true, };@@ -226,7 +228,10 @@ static int acpi_lpss_create_device(struct acpi_device *adev,list_for_each_entry (rentry, &resource_list, node)if (resource_type (&rentry->res) ==IORESOURCE_MEM ) { - pdata->mmio_size =resource_size (&rentry->res); +if (dev_desc->prv_size_override) + pdata->mmio_size = dev_desc->prv_size_override; +else + pdata->mmio_size =resource_size (&rentry->res); pdata->mmio_base =ioremap (rentry->res.start, pdata->mmio_size); pdata->dev_desc = dev_desc;
Signature · 2 lines
I speak only for myself. Rafael J. Wysocki, Intel Open Source Technology Center.