netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH net-next] net: ti: fix CPTS driver build on arm
@ 2014-04-02  1:26 Alexei Starovoitov
  2014-04-02  6:22 ` Daniel Borkmann
  2014-04-03 15:25 ` David Miller
  0 siblings, 2 replies; 3+ messages in thread
From: Alexei Starovoitov @ 2014-04-02  1:26 UTC (permalink / raw)
  To: David S. Miller; +Cc: Daniel Borkmann, Richard Cochran, netdev

fix build errors:
drivers/net/ethernet/ti/cpts.c:266:12: error: 'ETH_HLEN' undeclared (first use in this function)
drivers/net/ethernet/ti/cpts.c:276:23: error: 'VLAN_HLEN' undeclared (first use in this function)

Fixes: 408eccce3204 ("net: ptp: move PTP classifier in its own file")
Reported-by: Fengguang Wu <fengguang.wu@intel.com>
Signed-off-by: Alexei Starovoitov <ast@plumgrid.com>
Suggested-by: Daniel Borkmann <dborkman@redhat.com>
Cc: Richard Cochran <richardcochran@gmail.com>
---

arm build looks clean now.
Sorry for the prior breakage.

 drivers/net/ethernet/ti/cpts.c |    2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/net/ethernet/ti/cpts.c b/drivers/net/ethernet/ti/cpts.c
index a3bbf59eaafd..243513980b51 100644
--- a/drivers/net/ethernet/ti/cpts.c
+++ b/drivers/net/ethernet/ti/cpts.c
@@ -26,6 +26,8 @@
 #include <linux/time.h>
 #include <linux/uaccess.h>
 #include <linux/workqueue.h>
+#include <linux/if_ether.h>
+#include <linux/if_vlan.h>
 
 #include "cpts.h"
 
-- 
1.7.9.5

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH net-next] net: ti: fix CPTS driver build on arm
  2014-04-02  1:26 [PATCH net-next] net: ti: fix CPTS driver build on arm Alexei Starovoitov
@ 2014-04-02  6:22 ` Daniel Borkmann
  2014-04-03 15:25 ` David Miller
  1 sibling, 0 replies; 3+ messages in thread
From: Daniel Borkmann @ 2014-04-02  6:22 UTC (permalink / raw)
  To: Alexei Starovoitov; +Cc: David S. Miller, Richard Cochran, netdev

On 04/02/2014 03:26 AM, Alexei Starovoitov wrote:
> fix build errors:
> drivers/net/ethernet/ti/cpts.c:266:12: error: 'ETH_HLEN' undeclared (first use in this function)
> drivers/net/ethernet/ti/cpts.c:276:23: error: 'VLAN_HLEN' undeclared (first use in this function)
>
> Fixes: 408eccce3204 ("net: ptp: move PTP classifier in its own file")
> Reported-by: Fengguang Wu <fengguang.wu@intel.com>
> Signed-off-by: Alexei Starovoitov <ast@plumgrid.com>
> Suggested-by: Daniel Borkmann <dborkman@redhat.com>
> Cc: Richard Cochran <richardcochran@gmail.com>

Acked-by: Daniel Borkmann <dborkman@redhat.com>

Thanks, should have set up an ARM toolchain :/ (allmodconfig on x86_64 was fine).

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH net-next] net: ti: fix CPTS driver build on arm
  2014-04-02  1:26 [PATCH net-next] net: ti: fix CPTS driver build on arm Alexei Starovoitov
  2014-04-02  6:22 ` Daniel Borkmann
@ 2014-04-03 15:25 ` David Miller
  1 sibling, 0 replies; 3+ messages in thread
From: David Miller @ 2014-04-03 15:25 UTC (permalink / raw)
  To: ast; +Cc: dborkman, richardcochran, netdev

From: Alexei Starovoitov <ast@plumgrid.com>
Date: Tue,  1 Apr 2014 18:26:48 -0700

> fix build errors:
> drivers/net/ethernet/ti/cpts.c:266:12: error: 'ETH_HLEN' undeclared (first use in this function)
> drivers/net/ethernet/ti/cpts.c:276:23: error: 'VLAN_HLEN' undeclared (first use in this function)
> 
> Fixes: 408eccce3204 ("net: ptp: move PTP classifier in its own file")
> Reported-by: Fengguang Wu <fengguang.wu@intel.com>
> Signed-off-by: Alexei Starovoitov <ast@plumgrid.com>
> Suggested-by: Daniel Borkmann <dborkman@redhat.com>

Applied, thanks.

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2014-04-03 15:23 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-04-02  1:26 [PATCH net-next] net: ti: fix CPTS driver build on arm Alexei Starovoitov
2014-04-02  6:22 ` Daniel Borkmann
2014-04-03 15:25 ` David Miller

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).