Skip to contentSkip to conversation

net/faraday: Explicitly include linux/of.h and linux/property.h

PATCH Applied Mark Brown 3 participants 3 messages Mar 30, 2017 – Apr 1, 2017

Subsystems (2)
lore
Share & tools
lore

Mark Brown

author
#0 lore raw

PATCH 1/1 net/faraday: Explicitly include linux/of.h and linux/property.h

This driver uses interfaces from linux/of.h and linux/property.h but relies on implict inclusion of those headers which means that changes in other headers could break the build, as happened in -next for arm today. Add a explicit includes.

---
 drivers/net/ethernet/faraday/ftgmac100.c | 2 ++
 1 file changed, 2 insertions(+)
drivers/net/ethernet/faraday/ftgmac100.c +2 -0
index 928b0df2b8e0..ade6b3e4ed13 100644
@@ -28,8 +28,10 @@
 #include <linux/io.h>
 #include <linux/module.h>
 #include <linux/netdevice.h>
+#include <linux/of.h>
 #include <linux/phy.h>
 #include <linux/platform_device.h>
+#include <linux/property.h>
 #include <net/ip.h>
 #include <net/ncsi.h>
 
Signature · 1 line
2.11.0
Acked-by on v1 1/1
#1 lore raw
Quoted 6 lines · Mark Brown

This driver uses interfaces from linux/of.h and linux/property.h but relies on implict inclusion of those headers which means that changes in other headers could break the build, as happened in -next for arm today. Add a explicit includes.

Thank you for fixing this Mark.

Cheers,

Joel

Quoted 22 lines
---
 drivers/net/ethernet/faraday/ftgmac100.c | 2 ++
 1 file changed, 2 insertions(+)
drivers/net/ethernet/faraday/ftgmac100.c +2 -1
index 928b0df2b8e0..ade6b3e4ed13 100644
@@ -28,8 +28,10 @@
 #include <linux/io.h>
 #include <linux/module.h>
 #include <linux/netdevice.h>
+#include <linux/of.h>
 #include <linux/phy.h>
 #include <linux/platform_device.h>
+#include <linux/property.h>
 #include <net/ip.h>
 #include <net/ncsi.h>
 
--

2.11.0

maintainer Applied
#2 lore raw
In reply to Mark Brown

From: Mark Brown <[email protected]>
Date: Thu, 30 Mar 2017 17:00:12 +0100

Quoted 6 lines

This driver uses interfaces from linux/of.h and linux/property.h but relies on implict inclusion of those headers which means that changes in other headers could break the build, as happened in -next for arm today. Add a explicit includes.

Applied, thank you.