[PATCH] config: armv7hl/default: Disable DMA for MUSB

[Matwey V. Kornilov]

From: "Matwey V. Kornilov" <matwey.kornilov-***@public.gmane.org>

Hi, this is both for stable and master.

An issue is the following. We can not have multiple DMA modes in the same kernel due to current musb design.
So, it will work only for specific hardware (OMAP DMA). Current configuration leads to OOPS at BeaglBone Black (CPI41 DMA which is disabled), because it tries to use wrong DMA functions (from OMAP DMA).

The fairest way to fix it is to disable DMA for all hardware until fix in upstream.

Signed-off-by: Matwey V. Kornilov <matwey.kornilov-***@public.gmane.org>
---
config/armv7hl/default | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/config/armv7hl/default b/config/armv7hl/default
index 71ae044..423b2cb 100644
--- a/config/armv7hl/default
+++ b/config/armv7hl/default
@@ -5534,8 +5534,8 @@ CONFIG_USB_MUSB_AM335X_CHILD=m
# CONFIG_USB_UX500_DMA is not set
# CONFIG_USB_INVENTRA_DMA is not set
# CONFIG_USB_TI_CPPI41_DMA is not set
-CONFIG_USB_TUSB_OMAP_DMA=y
-# CONFIG_MUSB_PIO_ONLY is not set
+# CONFIG_USB_TUSB_OMAP_DMA is not set
+CONFIG_MUSB_PIO_ONLY=y
CONFIG_USB_DWC3=m
# CONFIG_USB_DWC3_HOST is not set
# CONFIG_USB_DWC3_GADGET is not set

[Alexander Graf]

Post by Matwey V. Kornilov
Hi, this is both for stable and master.
An issue is the following. We can not have multiple DMA modes in the same kernel due to current musb design.
So, it will work only for specific hardware (OMAP DMA). Current configuration leads to OOPS at BeaglBone Black (CPI41 DMA which is disabled), because it tries to use wrong DMA functions (from OMAP DMA).
The fairest way to fix it is to disable DMA for all hardware until fix in upstream.

Thanks, applied to master. I've also reflected the changes on the lpae
flavor. The "lpae" flavor should be identical to the "default" flavor,
but with LPAE turned on.


Alex

[Andreas Färber]

Hi Matwey,

Post by Matwey V. Kornilov
Hi, this is both for stable and master.

Please put comments like this below --- so that they don't end up in the
Git commit message.

Thanks,
Andreas