Sparse was reporting quite a few warnings for the driver.
Those get fixed by this patch.
Signed-off-by: Dariusz Marcinkiewicz
---
drivers/net/ethernet/ec_bhf.c | 20 ++--
1 file changed, 10 insertions(+), 10 deletions(-)
diff --git a/drivers/net/ethernet/ec_bhf.c
Sparse was reporting quite a few warnings for the driver.
Those get fixed by this patch.
Signed-off-by: Dariusz Marcinkiewicz rek...@newterm.pl
---
drivers/net/ethernet/ec_bhf.c | 20 ++--
1 file changed, 10 insertions(+), 10 deletions(-)
diff --git
Sparse was reporting quite a few warnings for the driver.
Those get fixed by this patch.
---
drivers/net/ethernet/ec_bhf.c | 20 ++--
1 file changed, 10 insertions(+), 10 deletions(-)
diff --git a/drivers/net/ethernet/ec_bhf.c b/drivers/net/ethernet/ec_bhf.c
index
Sparse was reporting quite a few warnings for the driver.
Those get fixed by this patch.
---
drivers/net/ethernet/ec_bhf.c | 20 ++--
1 file changed, 10 insertions(+), 10 deletions(-)
diff --git a/drivers/net/ethernet/ec_bhf.c b/drivers/net/ethernet/ec_bhf.c
index
ONFIG_DM9000) += davicom/
obj-$(CONFIG_DNET) += dnet.o
obj-$(CONFIG_NET_VENDOR_DEC) += dec/
diff --git a/drivers/net/ethernet/ec_bhf.c b/drivers/net/ethernet/ec_bhf.c
new file mode 100644
index 000..4884205
--- /dev/null
+++ b/drivers/net/ethernet/ec_bhf.c
@@ -0,0 +1,706 @@
+ /*
+ * drivers/net
On Tue, May 06, 2014 at 04:56:06PM +0200, Tobias Klauser wrote:
> > + if (skb) {
> > + memcpy(skb_put(skb, pkt_size), data, pkt_size);
> > + skb->protocol = eth_type_trans(skb, priv->net_dev);
> > + dev_dbg(dev, "Protocol type: %x\n",
On Tue, May 06, 2014 at 04:56:06PM +0200, Tobias Klauser wrote:
+ if (skb) {
+ memcpy(skb_put(skb, pkt_size), data, pkt_size);
+ skb-protocol = eth_type_trans(skb, priv-net_dev);
+ dev_dbg(dev, Protocol type: %x\n,
-$(CONFIG_NET_VENDOR_DEC) += dec/
diff --git a/drivers/net/ethernet/ec_bhf.c b/drivers/net/ethernet/ec_bhf.c
new file mode 100644
index 000..4884205
--- /dev/null
+++ b/drivers/net/ethernet/ec_bhf.c
@@ -0,0 +1,706 @@
+ /*
+ * drivers/net/ethernet/beckhoff/ec_bhf.c
+ *
+ * Copyright (C) 2014 Darek
ex 000..9763cd6
--- /dev/null
+++ b/drivers/net/ethernet/ec_bhf.c
@@ -0,0 +1,705 @@
+ /*
+ * drivers/net/ethernet/beckhoff/ec_bhf.c
+ *
+ * Copyright (C) 2014 Darek Marcinkiewicz
+ *
+ * This software is licensed under the terms of the GNU General Public
+ * License version 2, as published by the
On Mon, May 05, 2014 at 04:32:49PM +0200, Bartlomiej Zolnierkiewicz wrote:
>
> Hi,
>
Hi.
>
> There needs to be at least
>
> depends on PCI
>
> here or the driver will break build on platforms that doesn't have PCI
> (i.e. ARM EXYNOS).
>
Thank you. I'll update the patch.
--
DM
--
To
On Mon, May 05, 2014 at 04:32:49PM +0200, Bartlomiej Zolnierkiewicz wrote:
Hi,
Hi.
There needs to be at least
depends on PCI
here or the driver will break build on platforms that doesn't have PCI
(i.e. ARM EXYNOS).
Thank you. I'll update the patch.
--
DM
--
To unsubscribe
/ethernet/ec_bhf.c
@@ -0,0 +1,705 @@
+ /*
+ * drivers/net/ethernet/beckhoff/ec_bhf.c
+ *
+ * Copyright (C) 2014 Darek Marcinkiewicz rek...@newterm.pl
+ *
+ * This software is licensed under the terms of the GNU General Public
+ * License version 2, as published by the Free Software Foundation, and
+ * may
On Sun, May 04, 2014 at 11:19:28PM +0200, Francois Romieu wrote:
> Darek Marcinkiewicz :
> > On Sun, May 04, 2014 at 08:43:51PM +0200, Francois Romieu wrote:
> [...]
> > > Regarding tx_dnext updates, you may add a short notice in
> > > ec_bhf_start_xmit
> >
drivers/net/ethernet/beckhoff/ec_bhf.c
+ *
+ * Copyright (C) 2014 Darek Marcinkiewicz
+ *
+ * This software is licensed under the terms of the GNU General Public
+ * License version 2, as published by the Free Software Foundation, and
+ * may be copied, distributed, and modified under
On Sun, May 04, 2014 at 08:43:51PM +0200, Francois Romieu wrote:
> Darek Marcinkiewicz :
> > On Sat, May 03, 2014 at 01:40:29PM +0200, Francois Romieu wrote:
> [...]
> > Thank you. I am attaching 2 of thse to this repsonse - the other two
> > are no longer relevant d
On Sun, May 04, 2014 at 01:04:13PM +0200, Darek Marcinkiewicz wrote:
> >
...
> attached to the EtherCAT bus. As for the locking on the tx path,
> I have removed that completely on receiving path. I simply didn't know
...
Erm. I meant - I have removed it on the tx path. It was gon
On Sat, May 03, 2014 at 01:40:29PM +0200, Francois Romieu wrote:
> Darek Marcinkiewicz :
> [...]
> > This driver adds support for EtherCAT master module located on CCAT
> > FPGA found on Beckhoff CX series industrial PCs. The driver exposes
> > EtherCAT master
thernet/ec_bhf.c
new file mode 100644
index 000..feb40cd
--- /dev/null
+++ b/drivers/net/ethernet/ec_bhf.c
@@ -0,0 +1,701 @@
+ /*
+ * drivers/net/ethernet/beckhoff/ec_bhf.c
+ *
+ * Copyright (C) 2014 Darek Marcinkiewicz
+ *
+ * This software is licensed under the terms of the GNU General Pub
..feb40cd
--- /dev/null
+++ b/drivers/net/ethernet/ec_bhf.c
@@ -0,0 +1,701 @@
+ /*
+ * drivers/net/ethernet/beckhoff/ec_bhf.c
+ *
+ * Copyright (C) 2014 Darek Marcinkiewicz rek...@newterm.pl
+ *
+ * This software is licensed under the terms of the GNU General Public
+ * License version 2, as published
On Sat, May 03, 2014 at 01:40:29PM +0200, Francois Romieu wrote:
Darek Marcinkiewicz rek...@newterm.pl :
[...]
This driver adds support for EtherCAT master module located on CCAT
FPGA found on Beckhoff CX series industrial PCs. The driver exposes
EtherCAT master as an ethernet interface
On Sun, May 04, 2014 at 01:04:13PM +0200, Darek Marcinkiewicz wrote:
...
attached to the EtherCAT bus. As for the locking on the tx path,
I have removed that completely on receiving path. I simply didn't know
...
Erm. I meant - I have removed it on the tx path. It was gone on rx
path already
On Sun, May 04, 2014 at 08:43:51PM +0200, Francois Romieu wrote:
Darek Marcinkiewicz rek...@newterm.pl :
On Sat, May 03, 2014 at 01:40:29PM +0200, Francois Romieu wrote:
[...]
Thank you. I am attaching 2 of thse to this repsonse - the other two
are no longer relevant due to the changes I
) 2014 Darek Marcinkiewicz rek...@newterm.pl
+ *
+ * This software is licensed under the terms of the GNU General Public
+ * License version 2, as published by the Free Software Foundation, and
+ * may be copied, distributed, and modified under those terms.
+ *
+ * This program is distributed
On Sun, May 04, 2014 at 11:19:28PM +0200, Francois Romieu wrote:
Darek Marcinkiewicz rek...@newterm.pl :
On Sun, May 04, 2014 at 08:43:51PM +0200, Francois Romieu wrote:
[...]
Regarding tx_dnext updates, you may add a short notice in
ec_bhf_start_xmit
and ec_bhf_process_tx explaining
Hello. Thank you for your comments.
On Thu, May 01, 2014 at 05:53:31PM +0200, Francois Romieu wrote:
> Darek Marcinkiewicz :
> [changes]
>
[...]
>
> > +
> > +struct bh_priv {
>
> Nit: it would be nice to avoid "_bh_" as it is already used in a set of
&g
ONFIG_DM9000) += davicom/
obj-$(CONFIG_DNET) += dnet.o
obj-$(CONFIG_NET_VENDOR_DEC) += dec/
diff --git a/drivers/net/ethernet/ec_bhf.c b/drivers/net/ethernet/ec_bhf.c
new file mode 100644
index 000..8e78b8d
--- /dev/null
+++ b/drivers/net/ethernet/ec_bhf.c
@@ -0,0 +1,740 @@
+ /*
+ * drivers/net/ethernet/beck
-$(CONFIG_NET_VENDOR_DEC) += dec/
diff --git a/drivers/net/ethernet/ec_bhf.c b/drivers/net/ethernet/ec_bhf.c
new file mode 100644
index 000..8e78b8d
--- /dev/null
+++ b/drivers/net/ethernet/ec_bhf.c
@@ -0,0 +1,740 @@
+ /*
+ * drivers/net/ethernet/beckhoff/ec_bhf.c
+ *
+ * Copyright (C) 2014 Darek
Hello. Thank you for your comments.
On Thu, May 01, 2014 at 05:53:31PM +0200, Francois Romieu wrote:
Darek Marcinkiewicz rek...@newterm.pl :
[changes]
[...]
+
+struct bh_priv {
Nit: it would be nice to avoid _bh_ as it is already used in a set of
common kernel functions.
I renamed
a/drivers/net/ethernet/ec_bh.c b/drivers/net/ethernet/ec_bh.c
new file mode 100644
index 000..2ed2cee
--- /dev/null
+++ b/drivers/net/ethernet/ec_bh.c
@@ -0,0 +1,785 @@
+/*
+ * drivers/net/ethernet/beckhoff/ec_bh.c
+ *
+ * Copyright (C) 2014 Darek Marcinkiewicz
+ *
+ * This software is licens
100644
index 000..2ed2cee
--- /dev/null
+++ b/drivers/net/ethernet/ec_bh.c
@@ -0,0 +1,785 @@
+/*
+ * drivers/net/ethernet/beckhoff/ec_bh.c
+ *
+ * Copyright (C) 2014 Darek Marcinkiewicz rek...@newterm.pl
+ *
+ * This software is licensed under the terms of the GNU General Public
+ * License version 2
On Tue, Apr 29, 2014 at 03:45:08PM -0400, David Miller wrote:
> From: Darek Marcinkiewicz
> Date: Mon, 28 Apr 2014 07:36:58 +0200
>
> > +void *ec_bh_alloc_dma_mem(struct bh_priv *priv,
> > +int channel,
> > +
On Sun, Apr 27, 2014 at 06:21:41PM -0400, David Miller wrote:
>
> Please do not post new versions of patches as replies to existing
> email threads.
>
> Instead, post the new version as a fresh list posting.
>
Ok, thank you for the info. I've sent it in a new thread. I've also have
just sent
b/drivers/net/ethernet/ec_bh.c
new file mode 100644
index 000..5c11c73
--- /dev/null
+++ b/drivers/net/ethernet/ec_bh.c
@@ -0,0 +1,773 @@
+/*
+ * drivers/net/ethernet/beckhoff/ec_bh.c
+ *
+ * Copyright (C) 2014 Darek Marcinkiewicz
+ *
+ * This software is licensed under the terms of the
000..5c11c73
--- /dev/null
+++ b/drivers/net/ethernet/ec_bh.c
@@ -0,0 +1,773 @@
+/*
+ * drivers/net/ethernet/beckhoff/ec_bh.c
+ *
+ * Copyright (C) 2014 Darek Marcinkiewicz rek...@newterm.pl
+ *
+ * This software is licensed under the terms of the GNU General Public
+ * License version 2
On Sun, Apr 27, 2014 at 06:21:41PM -0400, David Miller wrote:
Please do not post new versions of patches as replies to existing
email threads.
Instead, post the new version as a fresh list posting.
Ok, thank you for the info. I've sent it in a new thread. I've also have
just sent updated
On Tue, Apr 29, 2014 at 03:45:08PM -0400, David Miller wrote:
From: Darek Marcinkiewicz rek...@newterm.pl
Date: Mon, 28 Apr 2014 07:36:58 +0200
+void *ec_bh_alloc_dma_mem(struct bh_priv *priv,
+int channel,
+u8 **buf
+ b/drivers/net/ethernet/ec_bh.c
@@ -0,0 +1,768 @@
+/*
+ * drivers/net/ethernet/beckhoff/ec_bh.c
+ *
+ * Copyright (C) 2014 Darek Marcinkiewicz
+ *
+ * This software is licensed under the terms of the GNU General Public
+ * License version 2, as published by the Free Software Foundation,
On Sat, Apr 26, 2014 at 12:24:00PM -0400, David Miller wrote:
> From: Darek Marcinkiewicz
> Date: Thu, 24 Apr 2014 22:46:15 +0200
>
> All of these descriptor structures need to be properly annotated using
> endianness types, and their fields accessed using proper endia
On Sat, Apr 26, 2014 at 12:24:00PM -0400, David Miller wrote:
From: Darek Marcinkiewicz rek...@newterm.pl
Date: Thu, 24 Apr 2014 22:46:15 +0200
All of these descriptor structures need to be properly annotated using
endianness types, and their fields accessed using proper endian
@@
+/*
+ * drivers/net/ethernet/beckhoff/ec_bh.c
+ *
+ * Copyright (C) 2014 Darek Marcinkiewicz rek...@newterm.pl
+ *
+ * This software is licensed under the terms of the GNU General Public
+ * License version 2, as published by the Free Software Foundation, and
+ * may be copied, distributed, and modified under
ethernet/ec_bh.c
new file mode 100644
index 000..51892bf
--- /dev/null
+++ b/drivers/net/ethernet/ec_bh.c
@@ -0,0 +1,769 @@
+/*
+ * drivers/net/ethernet/beckhoff/ec_bh.c
+ *
+ * Copyright (C) 2014 Darek Marcinkiewicz
+ *
+ * This software is licensed under the terms of the GNU General Public
+ * Lic
--- /dev/null
+++ b/drivers/net/ethernet/ec_bh.c
@@ -0,0 +1,769 @@
+/*
+ * drivers/net/ethernet/beckhoff/ec_bh.c
+ *
+ * Copyright (C) 2014 Darek Marcinkiewicz rek...@newterm.pl
+ *
+ * This software is licensed under the terms of the GNU General Public
+ * License version 2, as published by the Free
42 matches
Mail list logo