[PATCH] USB: ohci-omap - avoid including mach/irqs.h

Subsystems: omap usb support, the rest, usb ohci driver, usb subsystem

STALE3664d

6 messages, 4 authors, 2016-08-27 · open the first message on its own page

[PATCH] USB: ohci-omap - avoid including mach/irqs.h

From: Russell King <hidden>
Date: 2016-08-19 16:31:31

ohci-omap doesn't need to include mach/irqs.h - nothing within this
driver needs anything from this header file.  Remove this include.

Signed-off-by: Russell King <redacted>
---
 drivers/usb/host/ohci-omap.c | 1 -
 1 file changed, 1 deletion(-)
diff --git a/drivers/usb/host/ohci-omap.c b/drivers/usb/host/ohci-omap.c
index de7c68602a7e..495c1454b9e8 100644
--- a/drivers/usb/host/ohci-omap.c
+++ b/drivers/usb/host/ohci-omap.c
@@ -36,7 +36,6 @@
 #include <mach/mux.h>
 
 #include <mach/hardware.h>
-#include <mach/irqs.h>
 #include <mach/usb.h>
 
 
-- 
2.1.0

[PATCH] USB: ohci-omap - avoid including mach/irqs.h

From: stern@rowland.harvard.edu (Alan Stern)
Date: 2016-08-22 15:28:21

On Fri, 19 Aug 2016, Russell King wrote:
quoted hunk
ohci-omap doesn't need to include mach/irqs.h - nothing within this
driver needs anything from this header file.  Remove this include.

Signed-off-by: Russell King <redacted>
---
 drivers/usb/host/ohci-omap.c | 1 -
 1 file changed, 1 deletion(-)
diff --git a/drivers/usb/host/ohci-omap.c b/drivers/usb/host/ohci-omap.c
index de7c68602a7e..495c1454b9e8 100644
--- a/drivers/usb/host/ohci-omap.c
+++ b/drivers/usb/host/ohci-omap.c
@@ -36,7 +36,6 @@
 #include <mach/mux.h>
 
 #include <mach/hardware.h>
-#include <mach/irqs.h>
 #include <mach/usb.h>
Acked-by: Alan Stern <stern@rowland.harvard.edu>

[PATCH] USB: ohci-omap - avoid including mach/irqs.h

From: linux@armlinux.org.uk (Russell King - ARM Linux)
Date: 2016-08-26 16:43:51

On Mon, Aug 22, 2016 at 11:28:21AM -0400, Alan Stern wrote:
On Fri, 19 Aug 2016, Russell King wrote:
quoted
ohci-omap doesn't need to include mach/irqs.h - nothing within this
driver needs anything from this header file.  Remove this include.

Signed-off-by: Russell King <redacted>
---
 drivers/usb/host/ohci-omap.c | 1 -
 1 file changed, 1 deletion(-)
diff --git a/drivers/usb/host/ohci-omap.c b/drivers/usb/host/ohci-omap.c
index de7c68602a7e..495c1454b9e8 100644
--- a/drivers/usb/host/ohci-omap.c
+++ b/drivers/usb/host/ohci-omap.c
@@ -36,7 +36,6 @@
 #include <mach/mux.h>
 
 #include <mach/hardware.h>
-#include <mach/irqs.h>
 #include <mach/usb.h>
Acked-by: Alan Stern <stern@rowland.harvard.edu>
Thanks.  Will Greg be picking this up?

-- 
RMK's Patch system: http://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up
according to speedtest.net.

[PATCH] USB: ohci-omap - avoid including mach/irqs.h

From: stern@rowland.harvard.edu (Alan Stern)
Date: 2016-08-26 17:20:53

On Fri, 26 Aug 2016, Russell King - ARM Linux wrote:
On Mon, Aug 22, 2016 at 11:28:21AM -0400, Alan Stern wrote:
quoted
On Fri, 19 Aug 2016, Russell King wrote:
quoted
ohci-omap doesn't need to include mach/irqs.h - nothing within this
driver needs anything from this header file.  Remove this include.

Signed-off-by: Russell King <redacted>
---
 drivers/usb/host/ohci-omap.c | 1 -
 1 file changed, 1 deletion(-)
diff --git a/drivers/usb/host/ohci-omap.c b/drivers/usb/host/ohci-omap.c
index de7c68602a7e..495c1454b9e8 100644
--- a/drivers/usb/host/ohci-omap.c
+++ b/drivers/usb/host/ohci-omap.c
@@ -36,7 +36,6 @@
 #include <mach/mux.h>
 
 #include <mach/hardware.h>
-#include <mach/irqs.h>
 #include <mach/usb.h>
Acked-by: Alan Stern <stern@rowland.harvard.edu>
Thanks.  Will Greg be picking this up?
Yes, he should.  Unless you want it to go via a different tree...

Alan Stern

[PATCH] USB: ohci-omap - avoid including mach/irqs.h

From: linux@armlinux.org.uk (Russell King - ARM Linux)
Date: 2016-08-26 18:05:19

On Fri, Aug 26, 2016 at 01:20:53PM -0400, Alan Stern wrote:
On Fri, 26 Aug 2016, Russell King - ARM Linux wrote:
quoted
On Mon, Aug 22, 2016 at 11:28:21AM -0400, Alan Stern wrote:
quoted
On Fri, 19 Aug 2016, Russell King wrote:
quoted
ohci-omap doesn't need to include mach/irqs.h - nothing within this
driver needs anything from this header file.  Remove this include.

Signed-off-by: Russell King <redacted>
---
 drivers/usb/host/ohci-omap.c | 1 -
 1 file changed, 1 deletion(-)
diff --git a/drivers/usb/host/ohci-omap.c b/drivers/usb/host/ohci-omap.c
index de7c68602a7e..495c1454b9e8 100644
--- a/drivers/usb/host/ohci-omap.c
+++ b/drivers/usb/host/ohci-omap.c
@@ -36,7 +36,6 @@
 #include <mach/mux.h>
 
 #include <mach/hardware.h>
-#include <mach/irqs.h>
 #include <mach/usb.h>
Acked-by: Alan Stern <stern@rowland.harvard.edu>
Thanks.  Will Greg be picking this up?
Yes, he should.  Unless you want it to go via a different tree...
I'm fine with Greg picking this (and the other two) up as none
of my (increasing) patch stack depends on these patches.

-- 
RMK's Patch system: http://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up
according to speedtest.net.

[PATCH] USB: ohci-omap - avoid including mach/irqs.h

From: gregkh@linuxfoundation.org (Greg Kroah-Hartman)
Date: 2016-08-27 12:51:19

On Fri, Aug 26, 2016 at 07:05:19PM +0100, Russell King - ARM Linux wrote:
On Fri, Aug 26, 2016 at 01:20:53PM -0400, Alan Stern wrote:
quoted
On Fri, 26 Aug 2016, Russell King - ARM Linux wrote:
quoted
On Mon, Aug 22, 2016 at 11:28:21AM -0400, Alan Stern wrote:
quoted
On Fri, 19 Aug 2016, Russell King wrote:
quoted
ohci-omap doesn't need to include mach/irqs.h - nothing within this
driver needs anything from this header file.  Remove this include.

Signed-off-by: Russell King <redacted>
---
 drivers/usb/host/ohci-omap.c | 1 -
 1 file changed, 1 deletion(-)
diff --git a/drivers/usb/host/ohci-omap.c b/drivers/usb/host/ohci-omap.c
index de7c68602a7e..495c1454b9e8 100644
--- a/drivers/usb/host/ohci-omap.c
+++ b/drivers/usb/host/ohci-omap.c
@@ -36,7 +36,6 @@
 #include <mach/mux.h>
 
 #include <mach/hardware.h>
-#include <mach/irqs.h>
 #include <mach/usb.h>
Acked-by: Alan Stern <stern@rowland.harvard.edu>
Thanks.  Will Greg be picking this up?
Yes, he should.  Unless you want it to go via a different tree...
I'm fine with Greg picking this (and the other two) up as none
of my (increasing) patch stack depends on these patches.
Ok, I'll take it in my next round, thanks.

greg k-h
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help