* [PATCH 0/3] Enable EHCI-platform driver for use with ACPI
@ 2015-08-12 21:51 ` Jeremy Linton
0 siblings, 0 replies; 38+ messages in thread
From: Jeremy Linton @ 2015-08-12 21:51 UTC (permalink / raw
To: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
Cc: linux-acpi-u79uwXL29TY76Z2rM5mHXA,
linux-usb-u79uwXL29TY76Z2rM5mHXA,
stern-nwvwT67g6+6dFdvTe/nMLpVzexx5G7lz,
linux-ci5G2KO2hbZ+pU9mqzGVBQ, suravee.suthikulpanit-5C7GfCeVMHo,
rafael.j.wysocki-ral2JQCrhuEAvxtiuMwx3w,
Catalin.Marinas-5wv7dgnIgG8, Jeremy Linton
This patch set enables the USB on the ARM Juno development platform
when using an ACPI kernel. This patch is in two primary parts. First
It tweaks the code in acpi_check_dma() to support the recently added
CCA attribute which describes a device which is not cache coherent.
It then adds an error message if the device cannot be configured with
the current DMA settings, finally it adds the ACPI PNP id to the
acpi_match_table entry.
Jeremy Linton (3):
Honor ACPI _CCA attribute setting
Display a DMA error message
Add ACPI bindings for the EHCI platform driver.
drivers/usb/host/ehci-platform.c | 13 ++++++++++++-
include/acpi/acpi_bus.h | 5 +++--
2 files changed, 15 insertions(+), 3 deletions(-)
--
2.4.3
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH 0/3] Enable EHCI-platform driver for use with ACPI
@ 2015-08-12 21:51 ` Jeremy Linton
0 siblings, 0 replies; 38+ messages in thread
From: Jeremy Linton @ 2015-08-12 21:51 UTC (permalink / raw
To: linux-arm-kernel
This patch set enables the USB on the ARM Juno development platform
when using an ACPI kernel. This patch is in two primary parts. First
It tweaks the code in acpi_check_dma() to support the recently added
CCA attribute which describes a device which is not cache coherent.
It then adds an error message if the device cannot be configured with
the current DMA settings, finally it adds the ACPI PNP id to the
acpi_match_table entry.
Jeremy Linton (3):
Honor ACPI _CCA attribute setting
Display a DMA error message
Add ACPI bindings for the EHCI platform driver.
drivers/usb/host/ehci-platform.c | 13 ++++++++++++-
include/acpi/acpi_bus.h | 5 +++--
2 files changed, 15 insertions(+), 3 deletions(-)
--
2.4.3
^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH 1/3] Honor ACPI _CCA attribute setting
2015-08-12 21:51 ` Jeremy Linton
@ 2015-08-12 21:51 ` Jeremy Linton
-1 siblings, 0 replies; 38+ messages in thread
From: Jeremy Linton @ 2015-08-12 21:51 UTC (permalink / raw
To: linux-arm-kernel
Cc: linux-acpi, linux-usb, stern, linux, suravee.suthikulpanit,
rafael.j.wysocki, Catalin.Marinas, Jeremy Linton
ACPI configurations can now mark devices as noncoherent,
support that choice.
Signed-off-by: Jeremy Linton <jeremy.linton@arm.com>
---
include/acpi/acpi_bus.h | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/include/acpi/acpi_bus.h b/include/acpi/acpi_bus.h
index 83061ca..7ecb8e4 100644
--- a/include/acpi/acpi_bus.h
+++ b/include/acpi/acpi_bus.h
@@ -399,7 +399,7 @@ static inline bool acpi_check_dma(struct acpi_device *adev, bool *coherent)
* case 1. Do not support and disable DMA.
* case 2. Support but rely on arch-specific cache maintenance for
* non-coherence DMA operations.
- * Currently, we implement case 1 above.
+ * Currently, we implement case 2 above.
*
* For the case when _CCA is missing (i.e. cca_seen=0) and
* platform specifies ACPI_CCA_REQUIRED, we do not support DMA,
@@ -407,7 +407,8 @@ static inline bool acpi_check_dma(struct acpi_device *adev, bool *coherent)
*
* See acpi_init_coherency() for more info.
*/
- if (adev->flags.coherent_dma) {
+ if (adev->flags.coherent_dma ||
+ (adev->flags.cca_seen && IS_ENABLED(CONFIG_ARM64))) {
ret = true;
if (coherent)
*coherent = adev->flags.coherent_dma;
--
2.4.3
^ permalink raw reply related [flat|nested] 38+ messages in thread
* [PATCH 1/3] Honor ACPI _CCA attribute setting
@ 2015-08-12 21:51 ` Jeremy Linton
0 siblings, 0 replies; 38+ messages in thread
From: Jeremy Linton @ 2015-08-12 21:51 UTC (permalink / raw
To: linux-arm-kernel
ACPI configurations can now mark devices as noncoherent,
support that choice.
Signed-off-by: Jeremy Linton <jeremy.linton@arm.com>
---
include/acpi/acpi_bus.h | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/include/acpi/acpi_bus.h b/include/acpi/acpi_bus.h
index 83061ca..7ecb8e4 100644
--- a/include/acpi/acpi_bus.h
+++ b/include/acpi/acpi_bus.h
@@ -399,7 +399,7 @@ static inline bool acpi_check_dma(struct acpi_device *adev, bool *coherent)
* case 1. Do not support and disable DMA.
* case 2. Support but rely on arch-specific cache maintenance for
* non-coherence DMA operations.
- * Currently, we implement case 1 above.
+ * Currently, we implement case 2 above.
*
* For the case when _CCA is missing (i.e. cca_seen=0) and
* platform specifies ACPI_CCA_REQUIRED, we do not support DMA,
@@ -407,7 +407,8 @@ static inline bool acpi_check_dma(struct acpi_device *adev, bool *coherent)
*
* See acpi_init_coherency() for more info.
*/
- if (adev->flags.coherent_dma) {
+ if (adev->flags.coherent_dma ||
+ (adev->flags.cca_seen && IS_ENABLED(CONFIG_ARM64))) {
ret = true;
if (coherent)
*coherent = adev->flags.coherent_dma;
--
2.4.3
^ permalink raw reply related [flat|nested] 38+ messages in thread
* [PATCH 2/3] Display a DMA error message
2015-08-12 21:51 ` Jeremy Linton
@ 2015-08-12 21:51 ` Jeremy Linton
-1 siblings, 0 replies; 38+ messages in thread
From: Jeremy Linton @ 2015-08-12 21:51 UTC (permalink / raw
To: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
Cc: linux-acpi-u79uwXL29TY76Z2rM5mHXA,
linux-usb-u79uwXL29TY76Z2rM5mHXA,
stern-nwvwT67g6+6dFdvTe/nMLpVzexx5G7lz,
linux-ci5G2KO2hbZ+pU9mqzGVBQ, suravee.suthikulpanit-5C7GfCeVMHo,
rafael.j.wysocki-ral2JQCrhuEAvxtiuMwx3w,
Catalin.Marinas-5wv7dgnIgG8, Jeremy Linton
If the ehci driver fails to configure the dma settings then display
a dev error instead of simply failing. This is triggered in an
ACPI world if the user fails to set the _CCA on the device.
Signed-off-by: Jeremy Linton <jeremy.linton-5wv7dgnIgG8@public.gmane.org>
---
drivers/usb/host/ehci-platform.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/usb/host/ehci-platform.c b/drivers/usb/host/ehci-platform.c
index 2593def..82e396f 100644
--- a/drivers/usb/host/ehci-platform.c
+++ b/drivers/usb/host/ehci-platform.c
@@ -162,8 +162,10 @@ static int ehci_platform_probe(struct platform_device *dev)
err = dma_coerce_mask_and_coherent(&dev->dev,
pdata->dma_mask_64 ? DMA_BIT_MASK(64) : DMA_BIT_MASK(32));
- if (err)
+ if (err) {
+ dev_err(&dev->dev, "Error: DMA setup failed\n");
return err;
+ }
irq = platform_get_irq(dev, 0);
if (irq < 0) {
--
2.4.3
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply related [flat|nested] 38+ messages in thread
* [PATCH 2/3] Display a DMA error message
@ 2015-08-12 21:51 ` Jeremy Linton
0 siblings, 0 replies; 38+ messages in thread
From: Jeremy Linton @ 2015-08-12 21:51 UTC (permalink / raw
To: linux-arm-kernel
If the ehci driver fails to configure the dma settings then display
a dev error instead of simply failing. This is triggered in an
ACPI world if the user fails to set the _CCA on the device.
Signed-off-by: Jeremy Linton <jeremy.linton@arm.com>
---
drivers/usb/host/ehci-platform.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/usb/host/ehci-platform.c b/drivers/usb/host/ehci-platform.c
index 2593def..82e396f 100644
--- a/drivers/usb/host/ehci-platform.c
+++ b/drivers/usb/host/ehci-platform.c
@@ -162,8 +162,10 @@ static int ehci_platform_probe(struct platform_device *dev)
err = dma_coerce_mask_and_coherent(&dev->dev,
pdata->dma_mask_64 ? DMA_BIT_MASK(64) : DMA_BIT_MASK(32));
- if (err)
+ if (err) {
+ dev_err(&dev->dev, "Error: DMA setup failed\n");
return err;
+ }
irq = platform_get_irq(dev, 0);
if (irq < 0) {
--
2.4.3
^ permalink raw reply related [flat|nested] 38+ messages in thread
* [PATCH 3/3] Add ACPI bindings for the EHCI platform driver.
2015-08-12 21:51 ` Jeremy Linton
@ 2015-08-12 21:51 ` Jeremy Linton
-1 siblings, 0 replies; 38+ messages in thread
From: Jeremy Linton @ 2015-08-12 21:51 UTC (permalink / raw
To: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
Cc: linux-acpi-u79uwXL29TY76Z2rM5mHXA,
linux-usb-u79uwXL29TY76Z2rM5mHXA,
stern-nwvwT67g6+6dFdvTe/nMLpVzexx5G7lz,
linux-ci5G2KO2hbZ+pU9mqzGVBQ, suravee.suthikulpanit-5C7GfCeVMHo,
rafael.j.wysocki-ral2JQCrhuEAvxtiuMwx3w,
Catalin.Marinas-5wv7dgnIgG8, Jeremy Linton
This enables USB on the ARM juno board when booted with
an ACPI kernel. The PNP id comes from the PNP/ACPI registry
and describes a EHCI controller without debug.
Signed-off-by: Jeremy Linton <jeremy.linton-5wv7dgnIgG8@public.gmane.org>
---
drivers/usb/host/ehci-platform.c | 9 +++++++++
1 file changed, 9 insertions(+)
diff --git a/drivers/usb/host/ehci-platform.c b/drivers/usb/host/ehci-platform.c
index 82e396f..1807baa 100644
--- a/drivers/usb/host/ehci-platform.c
+++ b/drivers/usb/host/ehci-platform.c
@@ -27,6 +27,7 @@
#include <linux/io.h>
#include <linux/module.h>
#include <linux/of.h>
+#include <linux/acpi.h>
#include <linux/phy/phy.h>
#include <linux/platform_device.h>
#include <linux/reset.h>
@@ -382,6 +383,13 @@ static const struct of_device_id vt8500_ehci_ids[] = {
};
MODULE_DEVICE_TABLE(of, vt8500_ehci_ids);
+static const struct acpi_device_id ehci_acpi_match[] = {
+ { "PNP0D20", 0 },
+ { }
+};
+MODULE_DEVICE_TABLE(acpi, ehci_acpi_match);
+
+
static const struct platform_device_id ehci_platform_table[] = {
{ "ehci-platform", 0 },
{ }
@@ -400,6 +408,7 @@ static struct platform_driver ehci_platform_driver = {
.name = "ehci-platform",
.pm = &ehci_platform_pm_ops,
.of_match_table = vt8500_ehci_ids,
+ .acpi_match_table = ACPI_PTR(ehci_acpi_match),
}
};
--
2.4.3
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply related [flat|nested] 38+ messages in thread
* [PATCH 3/3] Add ACPI bindings for the EHCI platform driver.
@ 2015-08-12 21:51 ` Jeremy Linton
0 siblings, 0 replies; 38+ messages in thread
From: Jeremy Linton @ 2015-08-12 21:51 UTC (permalink / raw
To: linux-arm-kernel
This enables USB on the ARM juno board when booted with
an ACPI kernel. The PNP id comes from the PNP/ACPI registry
and describes a EHCI controller without debug.
Signed-off-by: Jeremy Linton <jeremy.linton@arm.com>
---
drivers/usb/host/ehci-platform.c | 9 +++++++++
1 file changed, 9 insertions(+)
diff --git a/drivers/usb/host/ehci-platform.c b/drivers/usb/host/ehci-platform.c
index 82e396f..1807baa 100644
--- a/drivers/usb/host/ehci-platform.c
+++ b/drivers/usb/host/ehci-platform.c
@@ -27,6 +27,7 @@
#include <linux/io.h>
#include <linux/module.h>
#include <linux/of.h>
+#include <linux/acpi.h>
#include <linux/phy/phy.h>
#include <linux/platform_device.h>
#include <linux/reset.h>
@@ -382,6 +383,13 @@ static const struct of_device_id vt8500_ehci_ids[] = {
};
MODULE_DEVICE_TABLE(of, vt8500_ehci_ids);
+static const struct acpi_device_id ehci_acpi_match[] = {
+ { "PNP0D20", 0 },
+ { }
+};
+MODULE_DEVICE_TABLE(acpi, ehci_acpi_match);
+
+
static const struct platform_device_id ehci_platform_table[] = {
{ "ehci-platform", 0 },
{ }
@@ -400,6 +408,7 @@ static struct platform_driver ehci_platform_driver = {
.name = "ehci-platform",
.pm = &ehci_platform_pm_ops,
.of_match_table = vt8500_ehci_ids,
+ .acpi_match_table = ACPI_PTR(ehci_acpi_match),
}
};
--
2.4.3
^ permalink raw reply related [flat|nested] 38+ messages in thread
* Re: [PATCH 0/3] Enable EHCI-platform driver for use with ACPI
2015-08-12 21:51 ` Jeremy Linton
@ 2015-08-13 9:44 ` Graeme Gregory
-1 siblings, 0 replies; 38+ messages in thread
From: Graeme Gregory @ 2015-08-13 9:44 UTC (permalink / raw
To: Jeremy Linton
Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-acpi-u79uwXL29TY76Z2rM5mHXA,
linux-usb-u79uwXL29TY76Z2rM5mHXA,
stern-nwvwT67g6+6dFdvTe/nMLpVzexx5G7lz,
linux-ci5G2KO2hbZ+pU9mqzGVBQ, suravee.suthikulpanit-5C7GfCeVMHo,
rafael.j.wysocki-ral2JQCrhuEAvxtiuMwx3w,
Catalin.Marinas-5wv7dgnIgG8
On Wed, Aug 12, 2015 at 04:51:27PM -0500, Jeremy Linton wrote:
> This patch set enables the USB on the ARM Juno development platform
> when using an ACPI kernel. This patch is in two primary parts. First
> It tweaks the code in acpi_check_dma() to support the recently added
> CCA attribute which describes a device which is not cache coherent.
>
> It then adds an error message if the device cannot be configured with
> the current DMA settings, finally it adds the ACPI PNP id to the
> acpi_match_table entry.
>
Patch 1 I am unsure of (I just don't know enough about DMA), but for the other
two it looks fine to me.
Reviewed-by: Graeme Gregory <graeme.gregory-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
> Jeremy Linton (3):
> Honor ACPI _CCA attribute setting
> Display a DMA error message
> Add ACPI bindings for the EHCI platform driver.
>
> drivers/usb/host/ehci-platform.c | 13 ++++++++++++-
> include/acpi/acpi_bus.h | 5 +++--
> 2 files changed, 15 insertions(+), 3 deletions(-)
>
> --
> 2.4.3
>
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
> the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH 0/3] Enable EHCI-platform driver for use with ACPI
@ 2015-08-13 9:44 ` Graeme Gregory
0 siblings, 0 replies; 38+ messages in thread
From: Graeme Gregory @ 2015-08-13 9:44 UTC (permalink / raw
To: linux-arm-kernel
On Wed, Aug 12, 2015 at 04:51:27PM -0500, Jeremy Linton wrote:
> This patch set enables the USB on the ARM Juno development platform
> when using an ACPI kernel. This patch is in two primary parts. First
> It tweaks the code in acpi_check_dma() to support the recently added
> CCA attribute which describes a device which is not cache coherent.
>
> It then adds an error message if the device cannot be configured with
> the current DMA settings, finally it adds the ACPI PNP id to the
> acpi_match_table entry.
>
Patch 1 I am unsure of (I just don't know enough about DMA), but for the other
two it looks fine to me.
Reviewed-by: Graeme Gregory <graeme.gregory@linaro.org>
> Jeremy Linton (3):
> Honor ACPI _CCA attribute setting
> Display a DMA error message
> Add ACPI bindings for the EHCI platform driver.
>
> drivers/usb/host/ehci-platform.c | 13 ++++++++++++-
> include/acpi/acpi_bus.h | 5 +++--
> 2 files changed, 15 insertions(+), 3 deletions(-)
>
> --
> 2.4.3
>
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
> the body of a message to majordomo at vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 2/3] Display a DMA error message
2015-08-12 21:51 ` Jeremy Linton
@ 2015-08-13 11:02 ` Hanjun Guo
-1 siblings, 0 replies; 38+ messages in thread
From: Hanjun Guo @ 2015-08-13 11:02 UTC (permalink / raw
To: Jeremy Linton, linux-arm-kernel
Cc: linux-acpi, linux-usb, stern, linux, suravee.suthikulpanit,
rafael.j.wysocki, Catalin.Marinas
On 2015/8/13 5:51, Jeremy Linton wrote:
> If the ehci driver fails to configure the dma settings then display
> a dev error instead of simply failing. This is triggered in an
> ACPI world if the user fails to set the _CCA on the device.
>
> Signed-off-by: Jeremy Linton <jeremy.linton@arm.com>
> ---
> drivers/usb/host/ehci-platform.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/usb/host/ehci-platform.c b/drivers/usb/host/ehci-platform.c
> index 2593def..82e396f 100644
> --- a/drivers/usb/host/ehci-platform.c
> +++ b/drivers/usb/host/ehci-platform.c
> @@ -162,8 +162,10 @@ static int ehci_platform_probe(struct platform_device *dev)
>
> err = dma_coerce_mask_and_coherent(&dev->dev,
> pdata->dma_mask_64 ? DMA_BIT_MASK(64) : DMA_BIT_MASK(32));
> - if (err)
> + if (err) {
> + dev_err(&dev->dev, "Error: DMA setup failed\n");
> return err;
> + }
>
> irq = platform_get_irq(dev, 0);
> if (irq < 0) {
Reviewed-by: Hanjun Guo <hanjun.guo@linaro.org>
Thanks
Hanjun
^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH 2/3] Display a DMA error message
@ 2015-08-13 11:02 ` Hanjun Guo
0 siblings, 0 replies; 38+ messages in thread
From: Hanjun Guo @ 2015-08-13 11:02 UTC (permalink / raw
To: linux-arm-kernel
On 2015/8/13 5:51, Jeremy Linton wrote:
> If the ehci driver fails to configure the dma settings then display
> a dev error instead of simply failing. This is triggered in an
> ACPI world if the user fails to set the _CCA on the device.
>
> Signed-off-by: Jeremy Linton <jeremy.linton@arm.com>
> ---
> drivers/usb/host/ehci-platform.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/usb/host/ehci-platform.c b/drivers/usb/host/ehci-platform.c
> index 2593def..82e396f 100644
> --- a/drivers/usb/host/ehci-platform.c
> +++ b/drivers/usb/host/ehci-platform.c
> @@ -162,8 +162,10 @@ static int ehci_platform_probe(struct platform_device *dev)
>
> err = dma_coerce_mask_and_coherent(&dev->dev,
> pdata->dma_mask_64 ? DMA_BIT_MASK(64) : DMA_BIT_MASK(32));
> - if (err)
> + if (err) {
> + dev_err(&dev->dev, "Error: DMA setup failed\n");
> return err;
> + }
>
> irq = platform_get_irq(dev, 0);
> if (irq < 0) {
Reviewed-by: Hanjun Guo <hanjun.guo@linaro.org>
Thanks
Hanjun
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 3/3] Add ACPI bindings for the EHCI platform driver.
2015-08-12 21:51 ` Jeremy Linton
@ 2015-08-13 11:50 ` Hanjun Guo
-1 siblings, 0 replies; 38+ messages in thread
From: Hanjun Guo @ 2015-08-13 11:50 UTC (permalink / raw
To: Jeremy Linton, linux-arm-kernel
Cc: linux-acpi, linux-usb, stern, linux, suravee.suthikulpanit,
rafael.j.wysocki, Catalin.Marinas
Hi Jeremy,
Just minor comments below. (resend as it's blocked by mailist)
On 2015/8/13 5:51, Jeremy Linton wrote:
> This enables USB on the ARM juno board when booted with
> an ACPI kernel. The PNP id comes from the PNP/ACPI registry
> and describes a EHCI controller without debug.
>
> Signed-off-by: Jeremy Linton <jeremy.linton@arm.com>
> ---
> drivers/usb/host/ehci-platform.c | 9 +++++++++
> 1 file changed, 9 insertions(+)
>
> diff --git a/drivers/usb/host/ehci-platform.c b/drivers/usb/host/ehci-platform.c
> index 82e396f..1807baa 100644
> --- a/drivers/usb/host/ehci-platform.c
> +++ b/drivers/usb/host/ehci-platform.c
> @@ -27,6 +27,7 @@
> #include <linux/io.h>
> #include <linux/module.h>
> #include <linux/of.h>
> +#include <linux/acpi.h>
this file list the head file in alphabetical order, I think it's
better to follow that rule.
> #include <linux/phy/phy.h>
> #include <linux/platform_device.h>
> #include <linux/reset.h>
> @@ -382,6 +383,13 @@ static const struct of_device_id vt8500_ehci_ids[] = {
> };
> MODULE_DEVICE_TABLE(of, vt8500_ehci_ids);
>
> +static const struct acpi_device_id ehci_acpi_match[] = {
> + { "PNP0D20", 0 },
> + { }
> +};
> +MODULE_DEVICE_TABLE(acpi, ehci_acpi_match);
> +
> +
Single empty line will be fine :)
> static const struct platform_device_id ehci_platform_table[] = {
> { "ehci-platform", 0 },
> { }
> @@ -400,6 +408,7 @@ static struct platform_driver ehci_platform_driver = {
> .name = "ehci-platform",
> .pm = &ehci_platform_pm_ops,
> .of_match_table = vt8500_ehci_ids,
> + .acpi_match_table = ACPI_PTR(ehci_acpi_match),
> }
> };
>
With that minor fixes,
Reviewed-by: Hanjun Guo <hanjun.guo@linaro.org>
Thanks
Hanjun
^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH 3/3] Add ACPI bindings for the EHCI platform driver.
@ 2015-08-13 11:50 ` Hanjun Guo
0 siblings, 0 replies; 38+ messages in thread
From: Hanjun Guo @ 2015-08-13 11:50 UTC (permalink / raw
To: linux-arm-kernel
Hi Jeremy,
Just minor comments below. (resend as it's blocked by mailist)
On 2015/8/13 5:51, Jeremy Linton wrote:
> This enables USB on the ARM juno board when booted with
> an ACPI kernel. The PNP id comes from the PNP/ACPI registry
> and describes a EHCI controller without debug.
>
> Signed-off-by: Jeremy Linton <jeremy.linton@arm.com>
> ---
> drivers/usb/host/ehci-platform.c | 9 +++++++++
> 1 file changed, 9 insertions(+)
>
> diff --git a/drivers/usb/host/ehci-platform.c b/drivers/usb/host/ehci-platform.c
> index 82e396f..1807baa 100644
> --- a/drivers/usb/host/ehci-platform.c
> +++ b/drivers/usb/host/ehci-platform.c
> @@ -27,6 +27,7 @@
> #include <linux/io.h>
> #include <linux/module.h>
> #include <linux/of.h>
> +#include <linux/acpi.h>
this file list the head file in alphabetical order, I think it's
better to follow that rule.
> #include <linux/phy/phy.h>
> #include <linux/platform_device.h>
> #include <linux/reset.h>
> @@ -382,6 +383,13 @@ static const struct of_device_id vt8500_ehci_ids[] = {
> };
> MODULE_DEVICE_TABLE(of, vt8500_ehci_ids);
>
> +static const struct acpi_device_id ehci_acpi_match[] = {
> + { "PNP0D20", 0 },
> + { }
> +};
> +MODULE_DEVICE_TABLE(acpi, ehci_acpi_match);
> +
> +
Single empty line will be fine :)
> static const struct platform_device_id ehci_platform_table[] = {
> { "ehci-platform", 0 },
> { }
> @@ -400,6 +408,7 @@ static struct platform_driver ehci_platform_driver = {
> .name = "ehci-platform",
> .pm = &ehci_platform_pm_ops,
> .of_match_table = vt8500_ehci_ids,
> + .acpi_match_table = ACPI_PTR(ehci_acpi_match),
> }
> };
>
With that minor fixes,
Reviewed-by: Hanjun Guo <hanjun.guo@linaro.org>
Thanks
Hanjun
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 3/3] Add ACPI bindings for the EHCI platform driver.
2015-08-13 11:50 ` Hanjun Guo
@ 2015-08-13 14:45 ` Jeremy Linton
-1 siblings, 0 replies; 38+ messages in thread
From: Jeremy Linton @ 2015-08-13 14:45 UTC (permalink / raw
To: Hanjun Guo, linux-arm-kernel@lists.infradead.org
Cc: linux-acpi@vger.kernel.org, linux-usb@vger.kernel.org,
stern@rowland.harvard.edu, linux@prisktech.co.nz,
suravee.suthikulpanit@amd.com, rafael.j.wysocki@intel.com,
Catalin Marinas
Hello Hanjun,
On 08/13/2015 06:50 AM, Hanjun Guo wrote:
> With that minor fixes,
>
> Reviewed-by: Hanjun Guo <hanjun.guo@linaro.org>
Fixes applied, thanks.
^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH 3/3] Add ACPI bindings for the EHCI platform driver.
@ 2015-08-13 14:45 ` Jeremy Linton
0 siblings, 0 replies; 38+ messages in thread
From: Jeremy Linton @ 2015-08-13 14:45 UTC (permalink / raw
To: linux-arm-kernel
Hello Hanjun,
On 08/13/2015 06:50 AM, Hanjun Guo wrote:
> With that minor fixes,
>
> Reviewed-by: Hanjun Guo <hanjun.guo@linaro.org>
Fixes applied, thanks.
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 2/3] Display a DMA error message
2015-08-12 21:51 ` Jeremy Linton
@ 2015-08-13 14:52 ` Alan Stern
-1 siblings, 0 replies; 38+ messages in thread
From: Alan Stern @ 2015-08-13 14:52 UTC (permalink / raw
To: Jeremy Linton
Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-acpi-u79uwXL29TY76Z2rM5mHXA,
linux-usb-u79uwXL29TY76Z2rM5mHXA, linux-ci5G2KO2hbZ+pU9mqzGVBQ,
suravee.suthikulpanit-5C7GfCeVMHo,
rafael.j.wysocki-ral2JQCrhuEAvxtiuMwx3w,
Catalin.Marinas-5wv7dgnIgG8
On Wed, 12 Aug 2015, Jeremy Linton wrote:
> If the ehci driver fails to configure the dma settings then display
> a dev error instead of simply failing. This is triggered in an
> ACPI world if the user fails to set the _CCA on the device.
>
> Signed-off-by: Jeremy Linton <jeremy.linton-5wv7dgnIgG8@public.gmane.org>
> ---
> drivers/usb/host/ehci-platform.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/usb/host/ehci-platform.c b/drivers/usb/host/ehci-platform.c
> index 2593def..82e396f 100644
> --- a/drivers/usb/host/ehci-platform.c
> +++ b/drivers/usb/host/ehci-platform.c
> @@ -162,8 +162,10 @@ static int ehci_platform_probe(struct platform_device *dev)
>
> err = dma_coerce_mask_and_coherent(&dev->dev,
> pdata->dma_mask_64 ? DMA_BIT_MASK(64) : DMA_BIT_MASK(32));
> - if (err)
> + if (err) {
> + dev_err(&dev->dev, "Error: DMA setup failed\n");
> return err;
> + }
>
> irq = platform_get_irq(dev, 0);
> if (irq < 0) {
>
Too bad the ohci-platform driver doesn't get the same update, but
it clearly wouldn't belong in this patch set. Anyway,
Acked-by: Alan Stern <stern-nwvwT67g6+6dFdvTe/nMLpVzexx5G7lz@public.gmane.org>
Alan Stern
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH 2/3] Display a DMA error message
@ 2015-08-13 14:52 ` Alan Stern
0 siblings, 0 replies; 38+ messages in thread
From: Alan Stern @ 2015-08-13 14:52 UTC (permalink / raw
To: linux-arm-kernel
On Wed, 12 Aug 2015, Jeremy Linton wrote:
> If the ehci driver fails to configure the dma settings then display
> a dev error instead of simply failing. This is triggered in an
> ACPI world if the user fails to set the _CCA on the device.
>
> Signed-off-by: Jeremy Linton <jeremy.linton@arm.com>
> ---
> drivers/usb/host/ehci-platform.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/usb/host/ehci-platform.c b/drivers/usb/host/ehci-platform.c
> index 2593def..82e396f 100644
> --- a/drivers/usb/host/ehci-platform.c
> +++ b/drivers/usb/host/ehci-platform.c
> @@ -162,8 +162,10 @@ static int ehci_platform_probe(struct platform_device *dev)
>
> err = dma_coerce_mask_and_coherent(&dev->dev,
> pdata->dma_mask_64 ? DMA_BIT_MASK(64) : DMA_BIT_MASK(32));
> - if (err)
> + if (err) {
> + dev_err(&dev->dev, "Error: DMA setup failed\n");
> return err;
> + }
>
> irq = platform_get_irq(dev, 0);
> if (irq < 0) {
>
Too bad the ohci-platform driver doesn't get the same update, but
it clearly wouldn't belong in this patch set. Anyway,
Acked-by: Alan Stern <stern@rowland.harvard.edu>
Alan Stern
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 3/3] Add ACPI bindings for the EHCI platform driver.
2015-08-12 21:51 ` Jeremy Linton
@ 2015-08-13 14:53 ` Alan Stern
-1 siblings, 0 replies; 38+ messages in thread
From: Alan Stern @ 2015-08-13 14:53 UTC (permalink / raw
To: Jeremy Linton
Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-acpi-u79uwXL29TY76Z2rM5mHXA,
linux-usb-u79uwXL29TY76Z2rM5mHXA, linux-ci5G2KO2hbZ+pU9mqzGVBQ,
suravee.suthikulpanit-5C7GfCeVMHo,
rafael.j.wysocki-ral2JQCrhuEAvxtiuMwx3w,
Catalin.Marinas-5wv7dgnIgG8
On Wed, 12 Aug 2015, Jeremy Linton wrote:
> This enables USB on the ARM juno board when booted with
> an ACPI kernel. The PNP id comes from the PNP/ACPI registry
> and describes a EHCI controller without debug.
>
> Signed-off-by: Jeremy Linton <jeremy.linton-5wv7dgnIgG8@public.gmane.org>
Looks okay, but...
> ---
> drivers/usb/host/ehci-platform.c | 9 +++++++++
> 1 file changed, 9 insertions(+)
>
> diff --git a/drivers/usb/host/ehci-platform.c b/drivers/usb/host/ehci-platform.c
> index 82e396f..1807baa 100644
> --- a/drivers/usb/host/ehci-platform.c
> +++ b/drivers/usb/host/ehci-platform.c
> @@ -27,6 +27,7 @@
> #include <linux/io.h>
> #include <linux/module.h>
> #include <linux/of.h>
> +#include <linux/acpi.h>
> #include <linux/phy/phy.h>
> #include <linux/platform_device.h>
> #include <linux/reset.h>
> @@ -382,6 +383,13 @@ static const struct of_device_id vt8500_ehci_ids[] = {
> };
> MODULE_DEVICE_TABLE(of, vt8500_ehci_ids);
>
> +static const struct acpi_device_id ehci_acpi_match[] = {
> + { "PNP0D20", 0 },
How about a comment on this line? Something like:
+ { "PNP0D20", 0 }, /* EHCI controller without debug */
I realize the of_device_id table doesn't include similar comments, but
there the entries are more or less self-documenting. The same cannot
be said for "PNP0D20".
> + { }
> +};
> +MODULE_DEVICE_TABLE(acpi, ehci_acpi_match);
> +
> +
Undesired extra blank line.
> static const struct platform_device_id ehci_platform_table[] = {
> { "ehci-platform", 0 },
> { }
> @@ -400,6 +408,7 @@ static struct platform_driver ehci_platform_driver = {
> .name = "ehci-platform",
> .pm = &ehci_platform_pm_ops,
> .of_match_table = vt8500_ehci_ids,
> + .acpi_match_table = ACPI_PTR(ehci_acpi_match),
> }
> };
With these changes:
Acked-by: Alan Stern <stern-nwvwT67g6+6dFdvTe/nMLpVzexx5G7lz@public.gmane.org>
Alan Stern
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH 3/3] Add ACPI bindings for the EHCI platform driver.
@ 2015-08-13 14:53 ` Alan Stern
0 siblings, 0 replies; 38+ messages in thread
From: Alan Stern @ 2015-08-13 14:53 UTC (permalink / raw
To: linux-arm-kernel
On Wed, 12 Aug 2015, Jeremy Linton wrote:
> This enables USB on the ARM juno board when booted with
> an ACPI kernel. The PNP id comes from the PNP/ACPI registry
> and describes a EHCI controller without debug.
>
> Signed-off-by: Jeremy Linton <jeremy.linton@arm.com>
Looks okay, but...
> ---
> drivers/usb/host/ehci-platform.c | 9 +++++++++
> 1 file changed, 9 insertions(+)
>
> diff --git a/drivers/usb/host/ehci-platform.c b/drivers/usb/host/ehci-platform.c
> index 82e396f..1807baa 100644
> --- a/drivers/usb/host/ehci-platform.c
> +++ b/drivers/usb/host/ehci-platform.c
> @@ -27,6 +27,7 @@
> #include <linux/io.h>
> #include <linux/module.h>
> #include <linux/of.h>
> +#include <linux/acpi.h>
> #include <linux/phy/phy.h>
> #include <linux/platform_device.h>
> #include <linux/reset.h>
> @@ -382,6 +383,13 @@ static const struct of_device_id vt8500_ehci_ids[] = {
> };
> MODULE_DEVICE_TABLE(of, vt8500_ehci_ids);
>
> +static const struct acpi_device_id ehci_acpi_match[] = {
> + { "PNP0D20", 0 },
How about a comment on this line? Something like:
+ { "PNP0D20", 0 }, /* EHCI controller without debug */
I realize the of_device_id table doesn't include similar comments, but
there the entries are more or less self-documenting. The same cannot
be said for "PNP0D20".
> + { }
> +};
> +MODULE_DEVICE_TABLE(acpi, ehci_acpi_match);
> +
> +
Undesired extra blank line.
> static const struct platform_device_id ehci_platform_table[] = {
> { "ehci-platform", 0 },
> { }
> @@ -400,6 +408,7 @@ static struct platform_driver ehci_platform_driver = {
> .name = "ehci-platform",
> .pm = &ehci_platform_pm_ops,
> .of_match_table = vt8500_ehci_ids,
> + .acpi_match_table = ACPI_PTR(ehci_acpi_match),
> }
> };
With these changes:
Acked-by: Alan Stern <stern@rowland.harvard.edu>
Alan Stern
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 3/3] Add ACPI bindings for the EHCI platform driver.
2015-08-13 14:53 ` Alan Stern
@ 2015-08-13 15:13 ` Jeremy Linton
-1 siblings, 0 replies; 38+ messages in thread
From: Jeremy Linton @ 2015-08-13 15:13 UTC (permalink / raw
To: Alan Stern
Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
linux-acpi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-usb-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-ci5G2KO2hbZ+pU9mqzGVBQ@public.gmane.org,
suravee.suthikulpanit-5C7GfCeVMHo@public.gmane.org,
rafael.j.wysocki-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org,
Catalin Marinas
Alan,
On 08/13/2015 09:53 AM, Alan Stern wrote:
> With these changes:
Applied..
> Acked-by: Alan Stern <stern-nwvwT67g6+6dFdvTe/nMLpVzexx5G7lz@public.gmane.org>
Thanks,
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH 3/3] Add ACPI bindings for the EHCI platform driver.
@ 2015-08-13 15:13 ` Jeremy Linton
0 siblings, 0 replies; 38+ messages in thread
From: Jeremy Linton @ 2015-08-13 15:13 UTC (permalink / raw
To: linux-arm-kernel
Alan,
On 08/13/2015 09:53 AM, Alan Stern wrote:
> With these changes:
Applied..
> Acked-by: Alan Stern <stern@rowland.harvard.edu>
Thanks,
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 1/3] Honor ACPI _CCA attribute setting
2015-08-12 21:51 ` Jeremy Linton
@ 2015-08-14 1:45 ` Suravee Suthikulpanit
-1 siblings, 0 replies; 38+ messages in thread
From: Suravee Suthikulpanit @ 2015-08-14 1:45 UTC (permalink / raw
To: Jeremy Linton, linux-arm-kernel
Cc: linux-acpi, linux-usb, stern, linux, rafael.j.wysocki,
Catalin.Marinas, Arnd Bergmann
Hi Jeremy,
On 8/13/15 04:51, Jeremy Linton wrote:
> ACPI configurations can now mark devices as noncoherent,
> support that choice.
>
> Signed-off-by: Jeremy Linton <jeremy.linton@arm.com>
> ---
> include/acpi/acpi_bus.h | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/include/acpi/acpi_bus.h b/include/acpi/acpi_bus.h
> index 83061ca..7ecb8e4 100644
> --- a/include/acpi/acpi_bus.h
> +++ b/include/acpi/acpi_bus.h
> @@ -399,7 +399,7 @@ static inline bool acpi_check_dma(struct acpi_device *adev, bool *coherent)
> * case 1. Do not support and disable DMA.
> * case 2. Support but rely on arch-specific cache maintenance for
> * non-coherence DMA operations.
> - * Currently, we implement case 1 above.
> + * Currently, we implement case 2 above.
> *
> * For the case when _CCA is missing (i.e. cca_seen=0) and
> * platform specifies ACPI_CCA_REQUIRED, we do not support DMA,
> @@ -407,7 +407,8 @@ static inline bool acpi_check_dma(struct acpi_device *adev, bool *coherent)
> *
> * See acpi_init_coherency() for more info.
> */
> - if (adev->flags.coherent_dma) {
> + if (adev->flags.coherent_dma ||
> + (adev->flags.cca_seen && IS_ENABLED(CONFIG_ARM64))) {
> ret = true;
> if (coherent)
> *coherent = adev->flags.coherent_dma;
>
This change was in my earlier revisions for the original patch series to
add ACPI CCA support. At the time, this was pushed back since we were
not sure whether this would be a useful case, and whether such hardware
exists.
Would it be useful to document somewhere (may be in the GIT commit
message) about which hardware might need this?
Arnd/Catalin, any feedback on this?
Thanks,
Suravee
^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH 1/3] Honor ACPI _CCA attribute setting
@ 2015-08-14 1:45 ` Suravee Suthikulpanit
0 siblings, 0 replies; 38+ messages in thread
From: Suravee Suthikulpanit @ 2015-08-14 1:45 UTC (permalink / raw
To: linux-arm-kernel
Hi Jeremy,
On 8/13/15 04:51, Jeremy Linton wrote:
> ACPI configurations can now mark devices as noncoherent,
> support that choice.
>
> Signed-off-by: Jeremy Linton <jeremy.linton@arm.com>
> ---
> include/acpi/acpi_bus.h | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/include/acpi/acpi_bus.h b/include/acpi/acpi_bus.h
> index 83061ca..7ecb8e4 100644
> --- a/include/acpi/acpi_bus.h
> +++ b/include/acpi/acpi_bus.h
> @@ -399,7 +399,7 @@ static inline bool acpi_check_dma(struct acpi_device *adev, bool *coherent)
> * case 1. Do not support and disable DMA.
> * case 2. Support but rely on arch-specific cache maintenance for
> * non-coherence DMA operations.
> - * Currently, we implement case 1 above.
> + * Currently, we implement case 2 above.
> *
> * For the case when _CCA is missing (i.e. cca_seen=0) and
> * platform specifies ACPI_CCA_REQUIRED, we do not support DMA,
> @@ -407,7 +407,8 @@ static inline bool acpi_check_dma(struct acpi_device *adev, bool *coherent)
> *
> * See acpi_init_coherency() for more info.
> */
> - if (adev->flags.coherent_dma) {
> + if (adev->flags.coherent_dma ||
> + (adev->flags.cca_seen && IS_ENABLED(CONFIG_ARM64))) {
> ret = true;
> if (coherent)
> *coherent = adev->flags.coherent_dma;
>
This change was in my earlier revisions for the original patch series to
add ACPI CCA support. At the time, this was pushed back since we were
not sure whether this would be a useful case, and whether such hardware
exists.
Would it be useful to document somewhere (may be in the GIT commit
message) about which hardware might need this?
Arnd/Catalin, any feedback on this?
Thanks,
Suravee
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 1/3] Honor ACPI _CCA attribute setting
2015-08-14 1:45 ` Suravee Suthikulpanit
@ 2015-08-14 13:14 ` Catalin Marinas
-1 siblings, 0 replies; 38+ messages in thread
From: Catalin Marinas @ 2015-08-14 13:14 UTC (permalink / raw
To: Suravee Suthikulpanit
Cc: Jeremy Linton, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
rafael.j.wysocki-ral2JQCrhuEAvxtiuMwx3w, Arnd Bergmann,
linux-usb-u79uwXL29TY76Z2rM5mHXA, linux-ci5G2KO2hbZ+pU9mqzGVBQ,
linux-acpi-u79uwXL29TY76Z2rM5mHXA,
stern-nwvwT67g6+6dFdvTe/nMLpVzexx5G7lz
On Fri, Aug 14, 2015 at 08:45:22AM +0700, Suravee Suthikulpanit wrote:
> On 8/13/15 04:51, Jeremy Linton wrote:
> >ACPI configurations can now mark devices as noncoherent,
> >support that choice.
> >
> >Signed-off-by: Jeremy Linton <jeremy.linton-5wv7dgnIgG8@public.gmane.org>
> >---
> > include/acpi/acpi_bus.h | 5 +++--
> > 1 file changed, 3 insertions(+), 2 deletions(-)
> >
> >diff --git a/include/acpi/acpi_bus.h b/include/acpi/acpi_bus.h
> >index 83061ca..7ecb8e4 100644
> >--- a/include/acpi/acpi_bus.h
> >+++ b/include/acpi/acpi_bus.h
> >@@ -399,7 +399,7 @@ static inline bool acpi_check_dma(struct acpi_device *adev, bool *coherent)
> > * case 1. Do not support and disable DMA.
> > * case 2. Support but rely on arch-specific cache maintenance for
> > * non-coherence DMA operations.
> >- * Currently, we implement case 1 above.
> >+ * Currently, we implement case 2 above.
> > *
> > * For the case when _CCA is missing (i.e. cca_seen=0) and
> > * platform specifies ACPI_CCA_REQUIRED, we do not support DMA,
> >@@ -407,7 +407,8 @@ static inline bool acpi_check_dma(struct acpi_device *adev, bool *coherent)
> > *
> > * See acpi_init_coherency() for more info.
> > */
> >- if (adev->flags.coherent_dma) {
> >+ if (adev->flags.coherent_dma ||
> >+ (adev->flags.cca_seen && IS_ENABLED(CONFIG_ARM64))) {
> > ret = true;
> > if (coherent)
> > *coherent = adev->flags.coherent_dma;
> >
>
> This change was in my earlier revisions for the original patch series to add
> ACPI CCA support. At the time, this was pushed back since we were not sure
> whether this would be a useful case, and whether such hardware exists.
>
> Would it be useful to document somewhere (may be in the GIT commit message)
> about which hardware might need this?
So far, it's the ARM Juno development board (the emphasis here is on
being able to use it for development, not a production system).
I think the commit log should also give you credit for the original
implementation.
> Arnd/Catalin, any feedback on this?
That's where it was left in the previous thread:
https://lkml.org/lkml/2015/5/21/376
(and I'll refrain from further comments ;))
--
Catalin
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH 1/3] Honor ACPI _CCA attribute setting
@ 2015-08-14 13:14 ` Catalin Marinas
0 siblings, 0 replies; 38+ messages in thread
From: Catalin Marinas @ 2015-08-14 13:14 UTC (permalink / raw
To: linux-arm-kernel
On Fri, Aug 14, 2015 at 08:45:22AM +0700, Suravee Suthikulpanit wrote:
> On 8/13/15 04:51, Jeremy Linton wrote:
> >ACPI configurations can now mark devices as noncoherent,
> >support that choice.
> >
> >Signed-off-by: Jeremy Linton <jeremy.linton@arm.com>
> >---
> > include/acpi/acpi_bus.h | 5 +++--
> > 1 file changed, 3 insertions(+), 2 deletions(-)
> >
> >diff --git a/include/acpi/acpi_bus.h b/include/acpi/acpi_bus.h
> >index 83061ca..7ecb8e4 100644
> >--- a/include/acpi/acpi_bus.h
> >+++ b/include/acpi/acpi_bus.h
> >@@ -399,7 +399,7 @@ static inline bool acpi_check_dma(struct acpi_device *adev, bool *coherent)
> > * case 1. Do not support and disable DMA.
> > * case 2. Support but rely on arch-specific cache maintenance for
> > * non-coherence DMA operations.
> >- * Currently, we implement case 1 above.
> >+ * Currently, we implement case 2 above.
> > *
> > * For the case when _CCA is missing (i.e. cca_seen=0) and
> > * platform specifies ACPI_CCA_REQUIRED, we do not support DMA,
> >@@ -407,7 +407,8 @@ static inline bool acpi_check_dma(struct acpi_device *adev, bool *coherent)
> > *
> > * See acpi_init_coherency() for more info.
> > */
> >- if (adev->flags.coherent_dma) {
> >+ if (adev->flags.coherent_dma ||
> >+ (adev->flags.cca_seen && IS_ENABLED(CONFIG_ARM64))) {
> > ret = true;
> > if (coherent)
> > *coherent = adev->flags.coherent_dma;
> >
>
> This change was in my earlier revisions for the original patch series to add
> ACPI CCA support. At the time, this was pushed back since we were not sure
> whether this would be a useful case, and whether such hardware exists.
>
> Would it be useful to document somewhere (may be in the GIT commit message)
> about which hardware might need this?
So far, it's the ARM Juno development board (the emphasis here is on
being able to use it for development, not a production system).
I think the commit log should also give you credit for the original
implementation.
> Arnd/Catalin, any feedback on this?
That's where it was left in the previous thread:
https://lkml.org/lkml/2015/5/21/376
(and I'll refrain from further comments ;))
--
Catalin
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 1/3] Honor ACPI _CCA attribute setting
2015-08-14 1:45 ` Suravee Suthikulpanit
@ 2015-08-14 14:12 ` Jeremy Linton
-1 siblings, 0 replies; 38+ messages in thread
From: Jeremy Linton @ 2015-08-14 14:12 UTC (permalink / raw
To: Suravee Suthikulpanit,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org
Cc: linux-acpi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-usb-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
stern-nwvwT67g6+6dFdvTe/nMLpVzexx5G7lz@public.gmane.org,
linux-ci5G2KO2hbZ+pU9mqzGVBQ@public.gmane.org,
rafael.j.wysocki-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org,
Catalin Marinas, Arnd Bergmann
Hello Suravee,
On 08/13/2015 08:45 PM, Suravee Suthikulpanit wrote:
> On 8/13/15 04:51, Jeremy Linton wrote:
>> ACPI configurations can now mark devices as noncoherent,
>> support that choice.
>
> This change was in my earlier revisions for the original patch series to
> add ACPI CCA support. At the time, this was pushed back since we were
> not sure whether this would be a useful case, and whether such hardware
> exists.
Yes, I was made aware of that conversation during the review here.
Basically, the ARM Juno development platform has a mix of coherent and
noncoherent devices. The EHCI being one of the primary noncoherent
peripherals. Without which, the usefulness of the platform for
developing with an ACPI kernel is limited.
Thanks,
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH 1/3] Honor ACPI _CCA attribute setting
@ 2015-08-14 14:12 ` Jeremy Linton
0 siblings, 0 replies; 38+ messages in thread
From: Jeremy Linton @ 2015-08-14 14:12 UTC (permalink / raw
To: linux-arm-kernel
Hello Suravee,
On 08/13/2015 08:45 PM, Suravee Suthikulpanit wrote:
> On 8/13/15 04:51, Jeremy Linton wrote:
>> ACPI configurations can now mark devices as noncoherent,
>> support that choice.
>
> This change was in my earlier revisions for the original patch series to
> add ACPI CCA support. At the time, this was pushed back since we were
> not sure whether this would be a useful case, and whether such hardware
> exists.
Yes, I was made aware of that conversation during the review here.
Basically, the ARM Juno development platform has a mix of coherent and
noncoherent devices. The EHCI being one of the primary noncoherent
peripherals. Without which, the usefulness of the platform for
developing with an ACPI kernel is limited.
Thanks,
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 2/3] Display a DMA error message
2015-08-12 21:51 ` Jeremy Linton
@ 2015-08-14 21:19 ` Arnd Bergmann
-1 siblings, 0 replies; 38+ messages in thread
From: Arnd Bergmann @ 2015-08-14 21:19 UTC (permalink / raw
To: linux-arm-kernel
Cc: Jeremy Linton, rafael.j.wysocki, Catalin.Marinas, linux-usb,
linux, linux-acpi, stern, suravee.suthikulpanit
On Wednesday 12 August 2015 16:51:29 Jeremy Linton wrote:
> diff --git a/drivers/usb/host/ehci-platform.c b/drivers/usb/host/ehci-platform.c
> index 2593def..82e396f 100644
> --- a/drivers/usb/host/ehci-platform.c
> +++ b/drivers/usb/host/ehci-platform.c
> @@ -162,8 +162,10 @@ static int ehci_platform_probe(struct platform_device *dev)
>
> err = dma_coerce_mask_and_coherent(&dev->dev,
> pdata->dma_mask_64 ? DMA_BIT_MASK(64) : DMA_BIT_MASK(32));
> - if (err)
> + if (err) {
> + dev_err(&dev->dev, "Error: DMA setup failed\n");
> return err;
> + }
We should really stop doing this: the platform should provide the
correct dma mask when creating the device, instead of setting a
bogus pdata field. Do not duplicate this bug for ACPI.
Arnd
^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH 2/3] Display a DMA error message
@ 2015-08-14 21:19 ` Arnd Bergmann
0 siblings, 0 replies; 38+ messages in thread
From: Arnd Bergmann @ 2015-08-14 21:19 UTC (permalink / raw
To: linux-arm-kernel
On Wednesday 12 August 2015 16:51:29 Jeremy Linton wrote:
> diff --git a/drivers/usb/host/ehci-platform.c b/drivers/usb/host/ehci-platform.c
> index 2593def..82e396f 100644
> --- a/drivers/usb/host/ehci-platform.c
> +++ b/drivers/usb/host/ehci-platform.c
> @@ -162,8 +162,10 @@ static int ehci_platform_probe(struct platform_device *dev)
>
> err = dma_coerce_mask_and_coherent(&dev->dev,
> pdata->dma_mask_64 ? DMA_BIT_MASK(64) : DMA_BIT_MASK(32));
> - if (err)
> + if (err) {
> + dev_err(&dev->dev, "Error: DMA setup failed\n");
> return err;
> + }
We should really stop doing this: the platform should provide the
correct dma mask when creating the device, instead of setting a
bogus pdata field. Do not duplicate this bug for ACPI.
Arnd
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 2/3] Display a DMA error message
2015-08-14 21:19 ` Arnd Bergmann
@ 2015-08-14 21:44 ` Jeremy Linton
-1 siblings, 0 replies; 38+ messages in thread
From: Jeremy Linton @ 2015-08-14 21:44 UTC (permalink / raw
To: Arnd Bergmann, linux-arm-kernel@lists.infradead.org
Cc: rafael.j.wysocki@intel.com, Catalin Marinas,
linux-usb@vger.kernel.org, linux@prisktech.co.nz,
linux-acpi@vger.kernel.org, stern@rowland.harvard.edu,
suravee.suthikulpanit@amd.com
Arnd,
On 08/14/2015 04:19 PM, Arnd Bergmann wrote:
> On Wednesday 12 August 2015 16:51:29 Jeremy Linton wrote:
>> diff --git a/drivers/usb/host/ehci-platform.c b/drivers/usb/host/ehci-platform.c
>> index 2593def..82e396f 100644
>> --- a/drivers/usb/host/ehci-platform.c
>> +++ b/drivers/usb/host/ehci-platform.c
>> @@ -162,8 +162,10 @@ static int ehci_platform_probe(struct platform_device *dev)
>>
>> err = dma_coerce_mask_and_coherent(&dev->dev,
>> pdata->dma_mask_64 ? DMA_BIT_MASK(64) : DMA_BIT_MASK(32));
>> - if (err)
>> + if (err) {
>> + dev_err(&dev->dev, "Error: DMA setup failed\n");
>> return err;
>> + }
>
> We should really stop doing this: the platform should provide the
> correct dma mask when creating the device, instead of setting a
> bogus pdata field. Do not duplicate this bug for ACPI.
I'm not sure I understand, I'm not changing the mask anywhere, all I'm
doing is printing a message if the mask cannot be set.
AKA if CCA is missing or set to 0 then, this line is triggered because
the dma_coerce_mask_and_coherent() routine returns with a failure
indicating there aren't any DMA ops to set a mask on. Right now what
happens is the device detection displays that there is an EHCI
controller then silently disappears with telling the user why it didn't
configure.
Thanks,
^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH 2/3] Display a DMA error message
@ 2015-08-14 21:44 ` Jeremy Linton
0 siblings, 0 replies; 38+ messages in thread
From: Jeremy Linton @ 2015-08-14 21:44 UTC (permalink / raw
To: linux-arm-kernel
Arnd,
On 08/14/2015 04:19 PM, Arnd Bergmann wrote:
> On Wednesday 12 August 2015 16:51:29 Jeremy Linton wrote:
>> diff --git a/drivers/usb/host/ehci-platform.c b/drivers/usb/host/ehci-platform.c
>> index 2593def..82e396f 100644
>> --- a/drivers/usb/host/ehci-platform.c
>> +++ b/drivers/usb/host/ehci-platform.c
>> @@ -162,8 +162,10 @@ static int ehci_platform_probe(struct platform_device *dev)
>>
>> err = dma_coerce_mask_and_coherent(&dev->dev,
>> pdata->dma_mask_64 ? DMA_BIT_MASK(64) : DMA_BIT_MASK(32));
>> - if (err)
>> + if (err) {
>> + dev_err(&dev->dev, "Error: DMA setup failed\n");
>> return err;
>> + }
>
> We should really stop doing this: the platform should provide the
> correct dma mask when creating the device, instead of setting a
> bogus pdata field. Do not duplicate this bug for ACPI.
I'm not sure I understand, I'm not changing the mask anywhere, all I'm
doing is printing a message if the mask cannot be set.
AKA if CCA is missing or set to 0 then, this line is triggered because
the dma_coerce_mask_and_coherent() routine returns with a failure
indicating there aren't any DMA ops to set a mask on. Right now what
happens is the device detection displays that there is an EHCI
controller then silently disappears with telling the user why it didn't
configure.
Thanks,
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 2/3] Display a DMA error message
2015-08-14 21:44 ` Jeremy Linton
@ 2015-08-14 21:56 ` Arnd Bergmann
-1 siblings, 0 replies; 38+ messages in thread
From: Arnd Bergmann @ 2015-08-14 21:56 UTC (permalink / raw
To: linux-arm-kernel
Cc: Jeremy Linton, linux-usb@vger.kernel.org, Catalin Marinas,
rafael.j.wysocki@intel.com, linux@prisktech.co.nz,
linux-acpi@vger.kernel.org, stern@rowland.harvard.edu,
suravee.suthikulpanit@amd.com
On Friday 14 August 2015 16:44:12 Jeremy Linton wrote:
> On 08/14/2015 04:19 PM, Arnd Bergmann wrote:
> > On Wednesday 12 August 2015 16:51:29 Jeremy Linton wrote:
> >> diff --git a/drivers/usb/host/ehci-platform.c b/drivers/usb/host/ehci-platform.c
> >> index 2593def..82e396f 100644
> >> --- a/drivers/usb/host/ehci-platform.c
> >> +++ b/drivers/usb/host/ehci-platform.c
> >> @@ -162,8 +162,10 @@ static int ehci_platform_probe(struct platform_device *dev)
> >>
> >> err = dma_coerce_mask_and_coherent(&dev->dev,
> >> pdata->dma_mask_64 ? DMA_BIT_MASK(64) : DMA_BIT_MASK(32));
> >> - if (err)
> >> + if (err) {
> >> + dev_err(&dev->dev, "Error: DMA setup failed\n");
> >> return err;
> >> + }
> >
> > We should really stop doing this: the platform should provide the
> > correct dma mask when creating the device, instead of setting a
> > bogus pdata field. Do not duplicate this bug for ACPI.
>
>
> I'm not sure I understand, I'm not changing the mask anywhere, all I'm
> doing is printing a message if the mask cannot be set.
The problem is that dma_coerce_mask_and_coherent() is not a proper
interface for a driver to use, it's an annotation that tells us
that the driver is trying to do something horrible by overriding
the dma mask that was set by the platform.
> AKA if CCA is missing or set to 0 then, this line is triggered because
> the dma_coerce_mask_and_coherent() routine returns with a failure
> indicating there aren't any DMA ops to set a mask on. Right now what
> happens is the device detection displays that there is an EHCI
> controller then silently disappears with telling the user why it didn't
> configure.
Adding a warning is fine, but the text is not right here (the driver is
not setting up the dma mask, but trying to override the one that has been
set up by the platform), and the solution should really be to not
call dma_coerce_mask_and_coherent() at all for ACPI. The main reason
that we have not removed this call here yet is that there is still a
MIPS platform that relies on it. We should probably remove it for DT
already and only do it if platform data was provided, but that requires
careful testing.
Arnd
^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH 2/3] Display a DMA error message
@ 2015-08-14 21:56 ` Arnd Bergmann
0 siblings, 0 replies; 38+ messages in thread
From: Arnd Bergmann @ 2015-08-14 21:56 UTC (permalink / raw
To: linux-arm-kernel
On Friday 14 August 2015 16:44:12 Jeremy Linton wrote:
> On 08/14/2015 04:19 PM, Arnd Bergmann wrote:
> > On Wednesday 12 August 2015 16:51:29 Jeremy Linton wrote:
> >> diff --git a/drivers/usb/host/ehci-platform.c b/drivers/usb/host/ehci-platform.c
> >> index 2593def..82e396f 100644
> >> --- a/drivers/usb/host/ehci-platform.c
> >> +++ b/drivers/usb/host/ehci-platform.c
> >> @@ -162,8 +162,10 @@ static int ehci_platform_probe(struct platform_device *dev)
> >>
> >> err = dma_coerce_mask_and_coherent(&dev->dev,
> >> pdata->dma_mask_64 ? DMA_BIT_MASK(64) : DMA_BIT_MASK(32));
> >> - if (err)
> >> + if (err) {
> >> + dev_err(&dev->dev, "Error: DMA setup failed\n");
> >> return err;
> >> + }
> >
> > We should really stop doing this: the platform should provide the
> > correct dma mask when creating the device, instead of setting a
> > bogus pdata field. Do not duplicate this bug for ACPI.
>
>
> I'm not sure I understand, I'm not changing the mask anywhere, all I'm
> doing is printing a message if the mask cannot be set.
The problem is that dma_coerce_mask_and_coherent() is not a proper
interface for a driver to use, it's an annotation that tells us
that the driver is trying to do something horrible by overriding
the dma mask that was set by the platform.
> AKA if CCA is missing or set to 0 then, this line is triggered because
> the dma_coerce_mask_and_coherent() routine returns with a failure
> indicating there aren't any DMA ops to set a mask on. Right now what
> happens is the device detection displays that there is an EHCI
> controller then silently disappears with telling the user why it didn't
> configure.
Adding a warning is fine, but the text is not right here (the driver is
not setting up the dma mask, but trying to override the one that has been
set up by the platform), and the solution should really be to not
call dma_coerce_mask_and_coherent() at all for ACPI. The main reason
that we have not removed this call here yet is that there is still a
MIPS platform that relies on it. We should probably remove it for DT
already and only do it if platform data was provided, but that requires
careful testing.
Arnd
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 3/3] Add ACPI bindings for the EHCI platform driver.
2015-08-13 15:13 ` Jeremy Linton
@ 2015-08-18 3:35 ` Huang Shijie
-1 siblings, 0 replies; 38+ messages in thread
From: Huang Shijie @ 2015-08-18 3:35 UTC (permalink / raw
To: Jeremy Linton
Cc: Alan Stern,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
linux-acpi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-usb-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-ci5G2KO2hbZ+pU9mqzGVBQ@public.gmane.org,
suravee.suthikulpanit-5C7GfCeVMHo@public.gmane.org,
rafael.j.wysocki-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org,
Catalin Marinas
On Thu, Aug 13, 2015 at 10:13:44AM -0500, Jeremy Linton wrote:
Hi Jeremy,
Please add my tested-by for your next version:
Tested-by: Huang Shijie <shijie.huang-5wv7dgnIgG8@public.gmane.org>
thanks
Huang Shijie
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH 3/3] Add ACPI bindings for the EHCI platform driver.
@ 2015-08-18 3:35 ` Huang Shijie
0 siblings, 0 replies; 38+ messages in thread
From: Huang Shijie @ 2015-08-18 3:35 UTC (permalink / raw
To: linux-arm-kernel
On Thu, Aug 13, 2015 at 10:13:44AM -0500, Jeremy Linton wrote:
Hi Jeremy,
Please add my tested-by for your next version:
Tested-by: Huang Shijie <shijie.huang@arm.com>
thanks
Huang Shijie
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 1/3] Honor ACPI _CCA attribute setting
2015-08-14 14:12 ` Jeremy Linton
@ 2015-08-18 3:36 ` Huang Shijie
-1 siblings, 0 replies; 38+ messages in thread
From: Huang Shijie @ 2015-08-18 3:36 UTC (permalink / raw
To: Jeremy Linton
Cc: Suravee Suthikulpanit,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
linux-acpi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-usb-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
stern-nwvwT67g6+6dFdvTe/nMLpVzexx5G7lz@public.gmane.org,
linux-ci5G2KO2hbZ+pU9mqzGVBQ@public.gmane.org,
rafael.j.wysocki-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org,
Catalin Marinas, Arnd Bergmann
On Fri, Aug 14, 2015 at 09:12:28AM -0500, Jeremy Linton wrote:
> Hello Suravee,
>
> On 08/13/2015 08:45 PM, Suravee Suthikulpanit wrote:
> > On 8/13/15 04:51, Jeremy Linton wrote:
> >> ACPI configurations can now mark devices as noncoherent,
> >> support that choice.
> >
> > This change was in my earlier revisions for the original patch series to
> > add ACPI CCA support. At the time, this was pushed back since we were
> > not sure whether this would be a useful case, and whether such hardware
> > exists.
>
> Yes, I was made aware of that conversation during the review here.
>
> Basically, the ARM Juno development platform has a mix of coherent and
> noncoherent devices. The EHCI being one of the primary noncoherent
> peripherals. Without which, the usefulness of the platform for
> developing with an ACPI kernel is limited.
>
Tesed-by: Huang Shijie <shijie.huang-5wv7dgnIgG8@public.gmane.org>
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH 1/3] Honor ACPI _CCA attribute setting
@ 2015-08-18 3:36 ` Huang Shijie
0 siblings, 0 replies; 38+ messages in thread
From: Huang Shijie @ 2015-08-18 3:36 UTC (permalink / raw
To: linux-arm-kernel
On Fri, Aug 14, 2015 at 09:12:28AM -0500, Jeremy Linton wrote:
> Hello Suravee,
>
> On 08/13/2015 08:45 PM, Suravee Suthikulpanit wrote:
> > On 8/13/15 04:51, Jeremy Linton wrote:
> >> ACPI configurations can now mark devices as noncoherent,
> >> support that choice.
> >
> > This change was in my earlier revisions for the original patch series to
> > add ACPI CCA support. At the time, this was pushed back since we were
> > not sure whether this would be a useful case, and whether such hardware
> > exists.
>
> Yes, I was made aware of that conversation during the review here.
>
> Basically, the ARM Juno development platform has a mix of coherent and
> noncoherent devices. The EHCI being one of the primary noncoherent
> peripherals. Without which, the usefulness of the platform for
> developing with an ACPI kernel is limited.
>
Tesed-by: Huang Shijie <shijie.huang@arm.com>
^ permalink raw reply [flat|nested] 38+ messages in thread
end of thread, other threads:[~2015-08-18 3:36 UTC | newest]
Thread overview: 38+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-08-12 21:51 [PATCH 0/3] Enable EHCI-platform driver for use with ACPI Jeremy Linton
2015-08-12 21:51 ` Jeremy Linton
2015-08-12 21:51 ` [PATCH 1/3] Honor ACPI _CCA attribute setting Jeremy Linton
2015-08-12 21:51 ` Jeremy Linton
2015-08-14 1:45 ` Suravee Suthikulpanit
2015-08-14 1:45 ` Suravee Suthikulpanit
[not found] ` <55CD4832.7070301-5C7GfCeVMHo@public.gmane.org>
2015-08-14 13:14 ` Catalin Marinas
2015-08-14 13:14 ` Catalin Marinas
2015-08-14 14:12 ` Jeremy Linton
2015-08-14 14:12 ` Jeremy Linton
[not found] ` <55CDF74C.3050703-5wv7dgnIgG8@public.gmane.org>
2015-08-18 3:36 ` Huang Shijie
2015-08-18 3:36 ` Huang Shijie
[not found] ` <1439416290-21228-1-git-send-email-jeremy.linton-5wv7dgnIgG8@public.gmane.org>
2015-08-12 21:51 ` [PATCH 2/3] Display a DMA error message Jeremy Linton
2015-08-12 21:51 ` Jeremy Linton
2015-08-13 11:02 ` Hanjun Guo
2015-08-13 11:02 ` Hanjun Guo
[not found] ` <1439416290-21228-3-git-send-email-jeremy.linton-5wv7dgnIgG8@public.gmane.org>
2015-08-13 14:52 ` Alan Stern
2015-08-13 14:52 ` Alan Stern
2015-08-14 21:19 ` Arnd Bergmann
2015-08-14 21:19 ` Arnd Bergmann
2015-08-14 21:44 ` Jeremy Linton
2015-08-14 21:44 ` Jeremy Linton
2015-08-14 21:56 ` Arnd Bergmann
2015-08-14 21:56 ` Arnd Bergmann
2015-08-12 21:51 ` [PATCH 3/3] Add ACPI bindings for the EHCI platform driver Jeremy Linton
2015-08-12 21:51 ` Jeremy Linton
2015-08-13 11:50 ` Hanjun Guo
2015-08-13 11:50 ` Hanjun Guo
2015-08-13 14:45 ` Jeremy Linton
2015-08-13 14:45 ` Jeremy Linton
[not found] ` <1439416290-21228-4-git-send-email-jeremy.linton-5wv7dgnIgG8@public.gmane.org>
2015-08-13 14:53 ` Alan Stern
2015-08-13 14:53 ` Alan Stern
[not found] ` <Pine.LNX.4.44L0.1508131049001.1277-100000-IYeN2dnnYyZXsRXLowluHWD2FQJk+8+b@public.gmane.org>
2015-08-13 15:13 ` Jeremy Linton
2015-08-13 15:13 ` Jeremy Linton
[not found] ` <55CCB428.1000404-5wv7dgnIgG8@public.gmane.org>
2015-08-18 3:35 ` Huang Shijie
2015-08-18 3:35 ` Huang Shijie
2015-08-13 9:44 ` [PATCH 0/3] Enable EHCI-platform driver for use with ACPI Graeme Gregory
2015-08-13 9:44 ` Graeme Gregory
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.