Thread (178 messages) flat view 178 messages, 24 authors, 2014-03-26
STALE4522d REVIEWED: 1 (0M)

1 review trailer.

[PATCH 06/62] ARM: davinci: export da8xx_syscfg0_base

From: Sekhar Nori <hidden>
Date: 2014-03-20 12:22:39

On Thursday 20 March 2014 05:27 PM, Arnd Bergmann wrote:
On Thursday 20 March 2014, Sekhar Nori wrote:
quoted
diff --git a/drivers/usb/host/ohci-hcd.c b/drivers/usb/host/ohci-hcd.c
index 3586460..c807d3f 100644
--- a/drivers/usb/host/ohci-hcd.c
+++ b/drivers/usb/host/ohci-hcd.c
@@ -1178,7 +1178,8 @@ MODULE_LICENSE ("GPL");
 #define SA1111_DRIVER          ohci_hcd_sa1111_driver
 #endif
 
-#ifdef CONFIG_ARCH_DAVINCI_DA8XX
+/* DA8XX uses platform internal symbols. Cannot be built as module. */
+#if defined(CONFIG_ARCH_DAVINCI_DA8XX) && !defined(CONFIG_USB_OHCI_HCD_MODULE)
 #include "ohci-da8xx.c"
 #define DAVINCI_PLATFORM_DRIVER        ohci_hcd_da8xx_driver
 #endif
I wouldn't want to submit that patch to GregKH ;-)

How about doing the same thing in a somewhat less sneaky way?

Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Much better! Please feel free to add

Acked-by: Sekhar Nori <redacted>

if it helps.

Regards,
Sekhar
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help