Re: [PATCH] ethernet: select CONFIG_CRC32 as needed

2020-12-04 Thread Mark Einon
et/ethernet/freescale/fman/Kconfig | 1 + >  drivers/net/ethernet/mellanox/mlx5/core/Kconfig | 1 + >  drivers/net/ethernet/microchip/Kconfig  | 1 + >  drivers/net/ethernet/netronome/Kconfig  | 1 + >  drivers/net/ethernet/nxp/Kconfig    | 1 + >  drivers/net/ethernet/rocker/Kconfig | 1 + >  10 files changed, 10 insertions(+) For the Agere et131x driver: Acked-by: Mark Einon

Re: [PATCH] net: ethernet: et131x: Remove redundant register read

2020-07-17 Thread Mark Einon
On Fri, 2020-07-17 at 15:40 +0200, Andrew Lunn wrote: > On Fri, Jul 17, 2020 at 02:21:35PM +0100, Mark Einon wrote: > > Following the removal of an unused variable assignment (remove > > unused variable 'pm_csr') the associated register read can also go, > > as

[PATCH] net: ethernet: et131x: Remove redundant register read

2020-07-17 Thread Mark Einon
Following the removal of an unused variable assignment (remove unused variable 'pm_csr') the associated register read can also go, as the read also occurs in the subsequent et1310_in_phy_coma() call. Signed-off-by: Mark Einon --- drivers/net/ethernet/agere/et131x.c | 3 --- 1 file changed, 3

Re: [PATCH net-next] net: ethernet: et131x: Remove unused variable 'pm_csr'

2020-07-17 Thread Mark Einon
> Signed-off-by: Zhang Changzhong > --- > drivers/net/ethernet/agere/et131x.c | 10 +++--- > 1 file changed, 3 insertions(+), 7 deletions(-) > This makes sense, thanks. The readl() line modified here is also redundant as it gets called from the subsequent et1310_in_phy_coma() ca

Re: [PATCH net-next] net: agere: use true,false for bool variable

2020-05-05 Thread Mark Einon
riable > > Signed-off-by: Jason Yan Acked-by: Mark Einon

Re: [PATCH 2/6] net: ethernet: et131x: constify ethtool_ops structures

2016-08-31 Thread Mark Einon
{r.p,ok1.p,ok2.p}; > identifier r.i; > @@ > i@p > > @depends on !bad disable optional_qualifier@ > identifier r.i; > @@ > static > +const > struct ethtool_ops i = { ... }; > // > > Signed-off-by: Julia Lawall <julia.law...@lip6.fr> Looks good, thanks. Acked-by: Mark Einon <mark.ei...@gmail.com>

Re: [PATCH 2/6] net: ethernet: et131x: constify ethtool_ops structures

2016-08-31 Thread Mark Einon
{r.p,ok1.p,ok2.p}; > identifier r.i; > @@ > i@p > > @depends on !bad disable optional_qualifier@ > identifier r.i; > @@ > static > +const > struct ethtool_ops i = { ... }; > // > > Signed-off-by: Julia Lawall Looks good, thanks. Acked-by: Mark Einon

[tip:x86/cleanups] x86/earlyprintk: Put CONFIG_PCI-only functions under the #ifdef

2015-04-03 Thread tip-bot for Mark Einon
Commit-ID: 7f99f8b94c9355acdb14f1be28cb19aac741da68 Gitweb: http://git.kernel.org/tip/7f99f8b94c9355acdb14f1be28cb19aac741da68 Author: Mark Einon AuthorDate: Wed, 1 Apr 2015 22:32:04 +0100 Committer: Ingo Molnar CommitDate: Fri, 3 Apr 2015 15:21:56 +0200 x86/earlyprintk: Put

[tip:x86/cleanups] x86/earlyprintk: Put CONFIG_PCI-only functions under the #ifdef

2015-04-03 Thread tip-bot for Mark Einon
Commit-ID: 7f99f8b94c9355acdb14f1be28cb19aac741da68 Gitweb: http://git.kernel.org/tip/7f99f8b94c9355acdb14f1be28cb19aac741da68 Author: Mark Einon mark.ei...@gmail.com AuthorDate: Wed, 1 Apr 2015 22:32:04 +0100 Committer: Ingo Molnar mi...@kernel.org CommitDate: Fri, 3 Apr 2015 15:21:56

[PATCH v3] x86, earlyprintk: Fix two 'defined but not used' compile warnings

2015-04-01 Thread Mark Einon
defines. Signed-off-by: Mark Einon --- v2 - Move code to another #ifdef instead of creating a new ifdef pair after comment by Borislav Petkov . v3 - Fixed commit errors from v2, changed some uintXX_t data types to equivalent uXX. arch/x86/kernel/early_printk.c | 32

Re: [PATCH v2] x86, earlyprintk: Fix two 'defined but not used' compile warnings

2015-04-01 Thread Mark Einon
On Wed, Apr 01, 2015 at 09:54:10AM +0200, Ingo Molnar wrote: > > * Mark Einon wrote: > > > Two static functions are only used if CONFIG_PCI is defined,so only build > > them > > if this is the case. Fixes the build warnings: > > > > arch/x

Re: [PATCH v2] x86, earlyprintk: Fix two 'defined but not used' compile warnings

2015-04-01 Thread Mark Einon
On Wed, Apr 01, 2015 at 09:54:10AM +0200, Ingo Molnar wrote: * Mark Einon mark.ei...@gmail.com wrote: Two static functions are only used if CONFIG_PCI is defined,so only build them if this is the case. Fixes the build warnings: arch/x86/kernel/early_printk.c:98:13: warning

[PATCH v3] x86, earlyprintk: Fix two 'defined but not used' compile warnings

2015-04-01 Thread Mark Einon
defines. Signed-off-by: Mark Einon mark.ei...@gmail.com --- v2 - Move code to another #ifdef instead of creating a new ifdef pair after comment by Borislav Petkov b...@alien8.de. v3 - Fixed commit errors from v2, changed some uintXX_t data types to equivalent uXX. arch/x86/kernel/early_printk.c

[PATCH v2] x86, earlyprintk: Fix two 'defined but not used' compile warnings

2015-03-31 Thread Mark Einon
, int value) ^ arch/x86/kernel/early_printk.c:105:21: warning: ‘mem32_serial_in’ defined but not used [-Wunused-function] static unsigned int mem32_serial_in(unsigned long addr, int offset) ^ Signed-off-by: Mark Einon --- v2 - Move code to another #ifdef instead

[PATCH] x86, earlyprintk: Fix two 'defined but not used' compile warnings

2015-03-31 Thread Mark Einon
, int value) ^ arch/x86/kernel/early_printk.c:105:21: warning: ‘mem32_serial_in’ defined but not used [-Wunused-function] static unsigned int mem32_serial_in(unsigned long addr, int offset) ^ Signed-off-by: Mark Einon --- arch/x86/kernel/early_printk.c | 2 ++ 1

[PATCH] x86, earlyprintk: Fix two 'defined but not used' compile warnings

2015-03-31 Thread Mark Einon
, int value) ^ arch/x86/kernel/early_printk.c:105:21: warning: ‘mem32_serial_in’ defined but not used [-Wunused-function] static unsigned int mem32_serial_in(unsigned long addr, int offset) ^ Signed-off-by: Mark Einon mark.ei...@gmail.com --- arch/x86/kernel

[PATCH v2] x86, earlyprintk: Fix two 'defined but not used' compile warnings

2015-03-31 Thread Mark Einon
, int value) ^ arch/x86/kernel/early_printk.c:105:21: warning: ‘mem32_serial_in’ defined but not used [-Wunused-function] static unsigned int mem32_serial_in(unsigned long addr, int offset) ^ Signed-off-by: Mark Einon mark.ei...@gmail.com --- v2 - Move code

Re: [PATCH] et131x: use msecs_to_jiffies for conversions

2015-02-11 Thread Mark Einon
andles all > corner cases correctly. This is a minor API cleanup only. > > Signed-off-by: Nicholas Mc Guire Acked-by: Mark Einon -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo inf

Re: [PATCH] et131x: use msecs_to_jiffies for conversions

2015-02-11 Thread Mark Einon
correctly. This is a minor API cleanup only. Signed-off-by: Nicholas Mc Guire hof...@osadl.org Acked-by: Mark Einon mark.ei...@gmail.com -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http

[PATCH] MAINTAINERS: Remove duplicate entry for usbip driver

2014-10-19 Thread Mark Einon
The usbip driver was moved out of staging in 3.17-rc3 but the MAINTAINERS file still has the old staging entry as well as the new one. Remove the old entry. Signed-off-by: Mark Einon --- MAINTAINERS | 5 - 1 file changed, 5 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index 0eb6c2a

[PATCH] MAINTAINERS: Remove duplicate entry for usbip driver

2014-10-19 Thread Mark Einon
The usbip driver was moved out of staging in 3.17-rc3 but the MAINTAINERS file still has the old staging entry as well as the new one. Remove the old entry. Signed-off-by: Mark Einon mark.ei...@gmail.com --- MAINTAINERS | 5 - 1 file changed, 5 deletions(-) diff --git a/MAINTAINERS b

Re: [PATCH net-next] et131x: Add PCIe gigabit ethernet driver et131x to drivers/net

2014-10-01 Thread Mark Einon
On Wed, Oct 01, 2014 at 04:14:49PM +0200, Tobias Klauser wrote: > On 2014-10-01 at 15:43:47 +0200, Joe Perches wrote: > > On Wed, 2014-10-01 at 14:45 +0200, Tobias Klauser wrote: > > > On 2014-09-30 at 23:29:46 +0200, Mark Einon wrote: > > > > This adds the et

Re: [PATCH net-next] et131x: Add PCIe gigabit ethernet driver et131x to drivers/net

2014-10-01 Thread Mark Einon
On Wed, Oct 01, 2014 at 04:14:49PM +0200, Tobias Klauser wrote: On 2014-10-01 at 15:43:47 +0200, Joe Perches j...@perches.com wrote: On Wed, 2014-10-01 at 14:45 +0200, Tobias Klauser wrote: On 2014-09-30 at 23:29:46 +0200, Mark Einon mark.ei...@gmail.com wrote: This adds the ethernet

Re: [PATCH] et131x: Promote staging et131x driver to drivers/net

2014-09-30 Thread Mark Einon
On Tue, Sep 30, 2014 at 12:32:01PM -0700, Greg KH wrote: > > This is a fine way to make a patch, but it's no way that any of the > netdev developers could ever review it to know what the code happens to > look like in my staging-next branch :( > > How about just sending a patch that adds the

[PATCH] et131x: Promote staging et131x driver to drivers/net

2014-09-30 Thread Mark Einon
This patch moves the et131x gigabit ethernet driver from drivers/staging to drivers/net/ethernet/agere. Signed-off-by: Mark Einon --- This patch appiles to both staging-next and the linux-next tag next-20140930, and takes into account all feedback recieved from the recent RFC for this patch

Re: [PATCH] staging: et131x: remove unneccessary 'out of memory' message

2014-09-30 Thread Mark Einon
On Tue, Sep 30, 2014 at 11:25:43PM +0530, Karthik Nayak wrote: > Removes the 'out of memory' warning issued by checkpatch > > Signed-off-by: Karthik Nayak > --- Hi, I'm not sure which tree you're working agianst, but this change has already been made and is on both staging-next and linux-next.

Re: [PATCH] staging: bcm: multiple checkpatch fixes for InterfaceIdleMode.c

2014-09-30 Thread Mark Einon
On Tue, Sep 30, 2014 at 10:56:17PM +0530, Karthik Nayak wrote: > 1. WARNING: else is not generally useful after a break or return > > 2. WARNING: quoted string split across lines > > 3. WARNING: break quoted strings at a space character > > Signed-off-by: Karthik Nayak > --- Hi again, You

Re: [PATCH] staging: bcm: multiple checkpatch fixes for InterfaceIdleMode.c

2014-09-30 Thread Mark Einon
On Tue, Sep 30, 2014 at 10:56:17PM +0530, Karthik Nayak wrote: 1. WARNING: else is not generally useful after a break or return 2. WARNING: quoted string split across lines 3. WARNING: break quoted strings at a space character Signed-off-by: Karthik Nayak karthik@gmail.com --- Hi

Re: [PATCH] staging: et131x: remove unneccessary 'out of memory' message

2014-09-30 Thread Mark Einon
On Tue, Sep 30, 2014 at 11:25:43PM +0530, Karthik Nayak wrote: Removes the 'out of memory' warning issued by checkpatch Signed-off-by: Karthik Nayak karthik@gmail.com --- Hi, I'm not sure which tree you're working agianst, but this change has already been made and is on both

[PATCH] et131x: Promote staging et131x driver to drivers/net

2014-09-30 Thread Mark Einon
This patch moves the et131x gigabit ethernet driver from drivers/staging to drivers/net/ethernet/agere. Signed-off-by: Mark Einon mark.ei...@gmail.com --- This patch appiles to both staging-next and the linux-next tag next-20140930, and takes into account all feedback recieved from the recent

Re: [PATCH] et131x: Promote staging et131x driver to drivers/net

2014-09-30 Thread Mark Einon
On Tue, Sep 30, 2014 at 12:32:01PM -0700, Greg KH wrote: This is a fine way to make a patch, but it's no way that any of the netdev developers could ever review it to know what the code happens to look like in my staging-next branch :( How about just sending a patch that adds the driver to

[PATCH 2/4] staging: et131x: Cat some lines less than 80 columns

2014-09-24 Thread Mark Einon
Some split lines are less than 80 chars if rejoined, so rejoin them. Reported-by: Joe Perches Signed-off-by: Mark Einon --- drivers/staging/et131x/et131x.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/staging/et131x/et131x.c b/drivers/staging

Re: [PATCH v2] et131x: Promote staging et131x driver to drivers/net

2014-09-24 Thread Mark Einon
On Tue, Sep 23, 2014 at 02:07:17PM -0700, Joe Perches wrote: > On Tue, 2014-09-23 at 21:05 +0100, Mark Einon wrote: > > This patch moves the et131x gigabit ethernet driver from drivers/staging > > to drivers/net/ethernet/agere. > > checkpatch type modification you might

[PATCH 4/4] staging: et131x: Remove unnecessary parentheses

2014-09-24 Thread Mark Einon
Reported-by: Joe Perches Signed-off-by: Mark Einon --- drivers/staging/et131x/et131x.c |6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/et131x/et131x.c b/drivers/staging/et131x/et131x.c index cd90831..0a927d2 100644 --- a/drivers/staging/et131x

[PATCH 3/4] staging: et131x: Remove unnecessary OOM message

2014-09-24 Thread Mark Einon
Reported-by: Joe Perches Signed-off-by: Mark Einon --- drivers/staging/et131x/et131x.c |4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/staging/et131x/et131x.c b/drivers/staging/et131x/et131x.c index f5459ec..cd90831 100644 --- a/drivers/staging/et131x/et131x.c

[PATCH 1/4] staging: et131x: Use ether_addr_copy when copying ethernet addresses

2014-09-24 Thread Mark Einon
Reported-by: Joe Perches Signed-off-by: Mark Einon --- drivers/staging/et131x/et131x.c | 12 +--- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/drivers/staging/et131x/et131x.c b/drivers/staging/et131x/et131x.c index 4b2ae49..b976c75 100644 --- a/drivers/staging/et131x

[PATCH 3/4] staging: et131x: Remove unnecessary OOM message

2014-09-24 Thread Mark Einon
Reported-by: Joe Perches j...@perches.com Signed-off-by: Mark Einon mark.ei...@gmail.com --- drivers/staging/et131x/et131x.c |4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/staging/et131x/et131x.c b/drivers/staging/et131x/et131x.c index f5459ec..cd90831 100644

[PATCH 1/4] staging: et131x: Use ether_addr_copy when copying ethernet addresses

2014-09-24 Thread Mark Einon
Reported-by: Joe Perches j...@perches.com Signed-off-by: Mark Einon mark.ei...@gmail.com --- drivers/staging/et131x/et131x.c | 12 +--- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/drivers/staging/et131x/et131x.c b/drivers/staging/et131x/et131x.c index 4b2ae49..b976c75

[PATCH 4/4] staging: et131x: Remove unnecessary parentheses

2014-09-24 Thread Mark Einon
Reported-by: Joe Perches j...@perches.com Signed-off-by: Mark Einon mark.ei...@gmail.com --- drivers/staging/et131x/et131x.c |6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/et131x/et131x.c b/drivers/staging/et131x/et131x.c index cd90831..0a927d2 100644

Re: [PATCH v2] et131x: Promote staging et131x driver to drivers/net

2014-09-24 Thread Mark Einon
On Tue, Sep 23, 2014 at 02:07:17PM -0700, Joe Perches wrote: On Tue, 2014-09-23 at 21:05 +0100, Mark Einon wrote: This patch moves the et131x gigabit ethernet driver from drivers/staging to drivers/net/ethernet/agere. checkpatch type modification you might consider: Reduce logging line

[PATCH 2/4] staging: et131x: Cat some lines less than 80 columns

2014-09-24 Thread Mark Einon
Some split lines are less than 80 chars if rejoined, so rejoin them. Reported-by: Joe Perches j...@perches.com Signed-off-by: Mark Einon mark.ei...@gmail.com --- drivers/staging/et131x/et131x.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/staging

[PATCH v2] et131x: Promote staging et131x driver to drivers/net

2014-09-23 Thread Mark Einon
This patch moves the et131x gigabit ethernet driver from drivers/staging to drivers/net/ethernet/agere. Signed-off-by: Mark Einon --- Changes from v1: -modified MAINTAINERS in scope with changes -Fixed typo in Kconfig, Atheros -> Agere -Made review changes from Tobias Klauser, see patch

[PATCH 3/4] staging: et131x: Add space after { in pci ID table

2014-09-23 Thread Mark Einon
Signed-off-by: Mark Einon --- drivers/staging/et131x/et131x.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/et131x/et131x.c b/drivers/staging/et131x/et131x.c index 6539a8a..10f20b3 100644 --- a/drivers/staging/et131x/et131x.c +++ b/drivers/staging/et131x

[PATCH 1/4] staging: et131x: zero allocation of fbr to prevent random address access

2014-09-23 Thread Mark Einon
br memory on allocation. Subsequent frees of this fbr memory explicitly zeros the ring_virtaddr value. Reported-by: Tobias Klauser Signed-off-by: Mark Einon --- drivers/staging/et131x/et131x.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/et131x/et131x.c

[PATCH 4/4] staging: et131x: Remove unnecessary defines to enable driver PM

2014-09-23 Thread Mark Einon
The Power Management functions can be conditional by assigning pm ops directly to .driver.pm, instead of using #ifdef's, saving some lines of code. Reported-by: Tobias Klauser Signed-off-by: Mark Einon --- drivers/staging/et131x/et131x.c | 7 ++- 1 file changed, 2 insertions(+), 5

[PATCH 2/4] staging: et131x: don't cast a void* to a struct pointer

2014-09-23 Thread Mark Einon
Casting a void* popinter to a struct pointer is unnecessary, don't do it. Signed-off-by: Mark Einon --- drivers/staging/et131x/et131x.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/et131x/et131x.c b/drivers/staging/et131x/et131x.c index 2889f86..6539a8a

Re: [RFC PATCH linux-next] et131x: Promote staging et131x driver to drivers/net

2014-09-23 Thread Mark Einon
On Tue, Sep 23, 2014 at 12:01:07PM +0200, Tobias Klauser wrote: > On 2014-09-23 at 11:46:15 +0200, Mark Einon wrote: > > On Tue, Sep 23, 2014 at 09:22:53AM +0200, Tobias Klauser wrote: > > > > Hi Tobias, > > > > Thanks for the d

Re: [RFC PATCH linux-next] et131x: Promote staging et131x driver to drivers/net

2014-09-23 Thread Mark Einon
On Tue, Sep 23, 2014 at 09:56:53AM +1000, Angus Gibson wrote: > > +config NET_VENDOR_AGERE > > + bool "Agere devices" > > + default y > > + depends on PCI > > + ---help--- > > + If you have a network (Ethernet) card belonging to this class, say = > > Y > > + and read the

Re: [RFC PATCH linux-next] et131x: Promote staging et131x driver to drivers/net

2014-09-23 Thread Mark Einon
On Mon, Sep 22, 2014 at 06:57:25PM -0700, Joe Perches wrote: > On Mon, 2014-09-22 at 22:28 +0100, Mark Einon wrote: > > This patch moves the et131x gigabit ethernet driver from drivers/staging > > to drivers/net/ethernet/agere. > > > > There are no known issues at t

Re: [RFC PATCH linux-next] et131x: Promote staging et131x driver to drivers/net

2014-09-23 Thread Mark Einon
On Tue, Sep 23, 2014 at 09:22:53AM +0200, Tobias Klauser wrote: Hi Tobias, Thanks for the details review. I've replied below - [...] > > +/* et131x_rx_dma_memory_alloc > > + * > > + * Allocates Free buffer ring 1 for sure, free buffer ring 0 if required, > > + * and the Packet Status Ring. > >

Re: [RFC PATCH linux-next] et131x: Promote staging et131x driver to drivers/net

2014-09-23 Thread Mark Einon
On Tue, Sep 23, 2014 at 09:22:53AM +0200, Tobias Klauser wrote: Hi Tobias, Thanks for the details review. I've replied below - [...] +/* et131x_rx_dma_memory_alloc + * + * Allocates Free buffer ring 1 for sure, free buffer ring 0 if required, + * and the Packet Status Ring. + */

Re: [RFC PATCH linux-next] et131x: Promote staging et131x driver to drivers/net

2014-09-23 Thread Mark Einon
On Mon, Sep 22, 2014 at 06:57:25PM -0700, Joe Perches wrote: On Mon, 2014-09-22 at 22:28 +0100, Mark Einon wrote: This patch moves the et131x gigabit ethernet driver from drivers/staging to drivers/net/ethernet/agere. There are no known issues at this time. This doesn't apply to -next

Re: [RFC PATCH linux-next] et131x: Promote staging et131x driver to drivers/net

2014-09-23 Thread Mark Einon
On Tue, Sep 23, 2014 at 09:56:53AM +1000, Angus Gibson wrote: +config NET_VENDOR_AGERE + bool Agere devices + default y + depends on PCI + ---help--- + If you have a network (Ethernet) card belonging to this class, say = Y + and read the Ethernet-HOWTO, available

Re: [RFC PATCH linux-next] et131x: Promote staging et131x driver to drivers/net

2014-09-23 Thread Mark Einon
On Tue, Sep 23, 2014 at 12:01:07PM +0200, Tobias Klauser wrote: On 2014-09-23 at 11:46:15 +0200, Mark Einon mark.ei...@gmail.com wrote: On Tue, Sep 23, 2014 at 09:22:53AM +0200, Tobias Klauser wrote: Hi Tobias, Thanks for the details review. I've replied below

[PATCH 4/4] staging: et131x: Remove unnecessary defines to enable driver PM

2014-09-23 Thread Mark Einon
The Power Management functions can be conditional by assigning pm ops directly to .driver.pm, instead of using #ifdef's, saving some lines of code. Reported-by: Tobias Klauser tklau...@distanz.ch Signed-off-by: Mark Einon mark.ei...@gmail.com --- drivers/staging/et131x/et131x.c | 7 ++- 1

[PATCH 2/4] staging: et131x: don't cast a void* to a struct pointer

2014-09-23 Thread Mark Einon
Casting a void* popinter to a struct pointer is unnecessary, don't do it. Signed-off-by: Mark Einon mark.ei...@gmail.com --- drivers/staging/et131x/et131x.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/et131x/et131x.c b/drivers/staging/et131x/et131x.c index

[PATCH 1/4] staging: et131x: zero allocation of fbr to prevent random address access

2014-09-23 Thread Mark Einon
on allocation. Subsequent frees of this fbr memory explicitly zeros the ring_virtaddr value. Reported-by: Tobias Klauser tklau...@distanz.ch Signed-off-by: Mark Einon mark.ei...@gmail.com --- drivers/staging/et131x/et131x.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 3/4] staging: et131x: Add space after { in pci ID table

2014-09-23 Thread Mark Einon
Signed-off-by: Mark Einon mark.ei...@gmail.com --- drivers/staging/et131x/et131x.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/et131x/et131x.c b/drivers/staging/et131x/et131x.c index 6539a8a..10f20b3 100644 --- a/drivers/staging/et131x/et131x.c +++ b

[PATCH v2] et131x: Promote staging et131x driver to drivers/net

2014-09-23 Thread Mark Einon
This patch moves the et131x gigabit ethernet driver from drivers/staging to drivers/net/ethernet/agere. Signed-off-by: Mark Einon mark.ei...@gmail.com --- Changes from v1: -modified MAINTAINERS in scope with changes -Fixed typo in Kconfig, Atheros - Agere -Made review changes from Tobias

[PATCH Resend] drivers: net: b44: Fix typo in returning multicast stats

2014-09-11 Thread Mark Einon
nstat->multicast refers to received packets, not transmitted as is returned here. Change it so that received packet stats are given. Signed-off-by: Mark Einon --- changing to send to alternate address for Gary Zambrano, maintainers entry bounced. drivers/net/ethernet/broadcom/b44.c | 2 +-

[PATCH] drivers: net: b44: Fix typo in returning multicast stats

2014-09-11 Thread Mark Einon
nstat->multicast refers to received packets, not transmitted as is returned here. Change it so that received packet stats are given. Signed-off-by: Mark Einon --- drivers/net/ethernet/broadcom/b44.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/ether

[PATCH] drivers: net: b44: Fix typo in returning multicast stats

2014-09-11 Thread Mark Einon
nstat-multicast refers to received packets, not transmitted as is returned here. Change it so that received packet stats are given. Signed-off-by: Mark Einon mark.ei...@gmail.com --- drivers/net/ethernet/broadcom/b44.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers

[PATCH Resend] drivers: net: b44: Fix typo in returning multicast stats

2014-09-11 Thread Mark Einon
nstat-multicast refers to received packets, not transmitted as is returned here. Change it so that received packet stats are given. Signed-off-by: Mark Einon mark.ei...@gmail.com --- changing to send to alternate address for Gary Zambrano, maintainers entry bounced. drivers/net/ethernet

[PATCH 3/4] staging: et131x: Add auto-negotiation and 1000BT_Half as supported protocols

2014-09-03 Thread Mark Einon
The driver supports auto-negotiation and 100BaetT_Half but doesn't advertise or list it in it's phydev. Fix that. Signed-off-by: Mark Einon --- drivers/staging/et131x/et131x.c | 18 ++ 1 file changed, 10 insertions(+), 8 deletions(-) diff --git a/drivers/staging/et131x/et131x.c

[PATCH 4/4] staging: et131x: Fix whitespace - alignment matching open parenthesis

2014-09-03 Thread Mark Einon
Fix occurrences in et131x.c of: CHECK: Alignment should match open parenthesis Signed-off-by: Mark Einon --- drivers/staging/et131x/et131x.c | 130 +--- 1 file changed, 67 insertions(+), 63 deletions(-) diff --git a/drivers/staging/et131x/et131x.c b/drivers

[PATCH 1/4] staging: et131x: Remove dead code in isr

2014-09-03 Thread Mark Einon
ually need the dead code, as we're now using NAPI to handle enabling interrupts - but we do need to enable interrupts if NAPI is not scheduled - so enable interrupts if this is the case. Reported-by: Dan Carpenter Signed-off-by: Mark Einon --- drivers/staging/et131x/et131x.c | 18

[PATCH 2/4] staging: et131x: Remove spinlock fbr_lock

2014-09-03 Thread Mark Einon
The spinlock fbr_lock is only used in a single call sequence staring from et131x_poll. As this call is already locked by napi->poll_lock, we can remove it. Signed-off-by: Mark Einon --- drivers/staging/et131x/et131x.c | 6 -- 1 file changed, 6 deletions(-) diff --git a/drivers/stag

[PATCH 2/4] staging: et131x: Remove spinlock fbr_lock

2014-09-03 Thread Mark Einon
The spinlock fbr_lock is only used in a single call sequence staring from et131x_poll. As this call is already locked by napi-poll_lock, we can remove it. Signed-off-by: Mark Einon mark.ei...@gmail.com --- drivers/staging/et131x/et131x.c | 6 -- 1 file changed, 6 deletions(-) diff --git

[PATCH 1/4] staging: et131x: Remove dead code in isr

2014-09-03 Thread Mark Einon
the dead code, as we're now using NAPI to handle enabling interrupts - but we do need to enable interrupts if NAPI is not scheduled - so enable interrupts if this is the case. Reported-by: Dan Carpenter dan.carpen...@oracle.com Signed-off-by: Mark Einon mark.ei...@gmail.com --- drivers/staging/et131x

[PATCH 4/4] staging: et131x: Fix whitespace - alignment matching open parenthesis

2014-09-03 Thread Mark Einon
Fix occurrences in et131x.c of: CHECK: Alignment should match open parenthesis Signed-off-by: Mark Einon mark.ei...@gmail.com --- drivers/staging/et131x/et131x.c | 130 +--- 1 file changed, 67 insertions(+), 63 deletions(-) diff --git a/drivers/staging

[PATCH 3/4] staging: et131x: Add auto-negotiation and 1000BT_Half as supported protocols

2014-09-03 Thread Mark Einon
The driver supports auto-negotiation and 100BaetT_Half but doesn't advertise or list it in it's phydev. Fix that. Signed-off-by: Mark Einon mark.ei...@gmail.com --- drivers/staging/et131x/et131x.c | 18 ++ 1 file changed, 10 insertions(+), 8 deletions(-) diff --git a/drivers

Re: [PATCH 3/8] staging: et131x: Use for loop to initialise contiguous registers to zero

2014-08-31 Thread Mark Einon
On Sat, Aug 30, 2014 at 01:32:16PM -0700, Greg KH wrote: > On Wed, Aug 20, 2014 at 11:17:53PM +0100, Mark Einon wrote: > > Replace a long list of contiguous writel() calls with a for loop iterating > > over the same values. > > > > Signed-off-by: Mark Einon > >

Re: [PATCH 3/8] staging: et131x: Use for loop to initialise contiguous registers to zero

2014-08-31 Thread Mark Einon
On Sat, Aug 30, 2014 at 01:32:16PM -0700, Greg KH wrote: On Wed, Aug 20, 2014 at 11:17:53PM +0100, Mark Einon wrote: Replace a long list of contiguous writel() calls with a for loop iterating over the same values. Signed-off-by: Mark Einon mark.ei...@gmail.com --- drivers/staging

[PATCH 5/8 v2] staging: et131x: Remove unnecessary i2c_wack variable

2014-08-21 Thread Mark Einon
i2c_wack is only used to implement a while(1) loop, so let's remove it. Signed-off-by: Mark Einon --- Modified to remove function parameter indenting changes, which are largely unrelated. drivers/staging/et131x/et131x.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git

Re: [PATCH 5/8] staging: et131x: Remove unnecessary i2c_wack variable

2014-08-21 Thread Mark Einon
On Thu, Aug 21, 2014 at 09:06:40AM -0300, Fabio Estevam wrote: > Hi Mark, > > On Thu, Aug 21, 2014 at 6:18 AM, Mark Einon wrote: > > > > > Hi Fabio, thanks for the review. > > > > It's a space alignment of parameters to go with the previous change, to >

[PATCH 8/8 v2] staging: et131x: Implement NAPI support

2014-08-21 Thread Mark Einon
() to et131x_handle_send_pkts() -replacing bottom half workqueue with poll function which handles send & receive of skbs. -adding various other necessary standard napi calls. Also remove this item from the README TODO list. Signed-off-by: Mark Einon --- Updated after Stephen Hemminger comme

Re: [PATCH 4/8] staging: et131x: Use for loop to initialise contiguous macstat registers to zero

2014-08-21 Thread Mark Einon
On Thu, Aug 21, 2014 at 08:40:20AM +, David Laight wrote: > From: Mark Einon > > Replace a long list of contiguous writel() calls with a for loop iterating > > over the same address values. > > > > Also remove redundant comments on the macstat registers, the variabl

Re: [PATCH 8/8] staging: et131x: Implement NAPI support

2014-08-21 Thread Mark Einon
On Wed, Aug 20, 2014 at 08:25:45PM -0700, Stephen Hemminger wrote: > On Wed, 20 Aug 2014 23:17:58 +0100 > Mark Einon wrote: > > > - bool done = true; > > + int count = 0; > > + int limit = budget; > > + bool not_done = false; > > Don't use negative

Re: [PATCH 8/8] staging: et131x: Implement NAPI support

2014-08-21 Thread Mark Einon
On Wed, Aug 20, 2014 at 08:25:01PM -0700, Stephen Hemminger wrote: > On Wed, 20 Aug 2014 23:17:58 +0100 > Mark Einon wrote: > > > > > + if (budget > MAX_PACKETS_HANDLED) > > + limit = MAX_PACKETS_HANDLED; > > Why this artificial restriction

Re: [PATCH 5/8] staging: et131x: Remove unnecessary i2c_wack variable

2014-08-21 Thread Mark Einon
On Wed, Aug 20, 2014 at 07:22:54PM -0300, Fabio Estevam wrote: > On Wed, Aug 20, 2014 at 7:17 PM, Mark Einon wrote: > > > do { > > pci_read_config_dword(pdev, > > -

Re: [PATCH 5/8] staging: et131x: Remove unnecessary i2c_wack variable

2014-08-21 Thread Mark Einon
On Wed, Aug 20, 2014 at 07:22:54PM -0300, Fabio Estevam wrote: On Wed, Aug 20, 2014 at 7:17 PM, Mark Einon mark.ei...@gmail.com wrote: do { pci_read_config_dword(pdev, - LBCIF_DATA_REGISTER, val

Re: [PATCH 8/8] staging: et131x: Implement NAPI support

2014-08-21 Thread Mark Einon
On Wed, Aug 20, 2014 at 08:25:01PM -0700, Stephen Hemminger wrote: On Wed, 20 Aug 2014 23:17:58 +0100 Mark Einon mark.ei...@gmail.com wrote: + if (budget MAX_PACKETS_HANDLED) + limit = MAX_PACKETS_HANDLED; Why this artificial restriction? Hi Stephen, thanks

Re: [PATCH 8/8] staging: et131x: Implement NAPI support

2014-08-21 Thread Mark Einon
On Wed, Aug 20, 2014 at 08:25:45PM -0700, Stephen Hemminger wrote: On Wed, 20 Aug 2014 23:17:58 +0100 Mark Einon mark.ei...@gmail.com wrote: - bool done = true; + int count = 0; + int limit = budget; + bool not_done = false; Don't use negative variables. Better to keep

Re: [PATCH 4/8] staging: et131x: Use for loop to initialise contiguous macstat registers to zero

2014-08-21 Thread Mark Einon
On Thu, Aug 21, 2014 at 08:40:20AM +, David Laight wrote: From: Mark Einon Replace a long list of contiguous writel() calls with a for loop iterating over the same address values. Also remove redundant comments on the macstat registers, the variable names are good enough

[PATCH 8/8 v2] staging: et131x: Implement NAPI support

2014-08-21 Thread Mark Einon
() to et131x_handle_send_pkts() -replacing bottom half workqueue with poll function which handles send receive of skbs. -adding various other necessary standard napi calls. Also remove this item from the README TODO list. Signed-off-by: Mark Einon mark.ei...@gmail.com --- Updated after Stephen

Re: [PATCH 5/8] staging: et131x: Remove unnecessary i2c_wack variable

2014-08-21 Thread Mark Einon
On Thu, Aug 21, 2014 at 09:06:40AM -0300, Fabio Estevam wrote: Hi Mark, On Thu, Aug 21, 2014 at 6:18 AM, Mark Einon mark.ei...@gmail.com wrote: Hi Fabio, thanks for the review. It's a space alignment of parameters to go with the previous change, to keep wrapping consistent

[PATCH 5/8 v2] staging: et131x: Remove unnecessary i2c_wack variable

2014-08-21 Thread Mark Einon
i2c_wack is only used to implement a while(1) loop, so let's remove it. Signed-off-by: Mark Einon mark.ei...@gmail.com --- Modified to remove function parameter indenting changes, which are largely unrelated. drivers/staging/et131x/et131x.c | 5 + 1 file changed, 1 insertion(+), 4

[PATCH 5/8] staging: et131x: Remove unnecessary i2c_wack variable

2014-08-20 Thread Mark Einon
i2c_wack is only used to implement a while(1) loop, so let's remove it. Signed-off-by: Mark Einon --- drivers/staging/et131x/et131x.c | 10 -- 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/drivers/staging/et131x/et131x.c b/drivers/staging/et131x/et131x.c index fc18e8d

[PATCH 1/8] staging: et131x: Use eth_mac_addr() instead of duplicating the functionality

2014-08-20 Thread Mark Einon
There's already working code to set the mac address, so let's use it. Signed-off-by: Mark Einon --- drivers/staging/et131x/et131x.c | 53 + 1 file changed, 1 insertion(+), 52 deletions(-) diff --git a/drivers/staging/et131x/et131x.c b/drivers/staging

[PATCH 4/8] staging: et131x: Use for loop to initialise contiguous macstat registers to zero

2014-08-20 Thread Mark Einon
Replace a long list of contiguous writel() calls with a for loop iterating over the same address values. Also remove redundant comments on the macstat registers, the variable names are good enough. Signed-off-by: Mark Einon --- drivers/staging/et131x/et131x.c | 59

[PATCH 2/8] staging: et131x: Don't handle rx/tx packets when changing mtu

2014-08-20 Thread Mark Einon
There's no need to handle any rx/tx interrupts in the middle of an mtu change, so don't. After this change, receive and transmit interrupts are only handled in one place, which paves the way to using NAPI. Signed-off-by: Mark Einon --- drivers/staging/et131x/et131x.c | 2 -- 1 file changed, 2

[PATCH 3/8] staging: et131x: Use for loop to initialise contiguous registers to zero

2014-08-20 Thread Mark Einon
Replace a long list of contiguous writel() calls with a for loop iterating over the same values. Signed-off-by: Mark Einon --- drivers/staging/et131x/et131x.c | 27 +++ 1 file changed, 3 insertions(+), 24 deletions(-) diff --git a/drivers/staging/et131x/et131x.c b

[PATCH 7/8] staging: et131x: Fix ET_INTR_TXDMA_ISR register name typo

2014-08-20 Thread Mark Einon
We actually mean to clear the ET_INTR_TXDMA_ISR reg after handling a completed transfer, not the ET_INTR_TXDMA_ERR reg, which should get handled immediately after. Signed-off-by: Mark Einon --- drivers/staging/et131x/et131x.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 6/8] staging: et131x: Rename NUM_PACKETS_HANDLED to MAX_PACKETS_HANDLED

2014-08-20 Thread Mark Einon
To better describe it's use as a hard limit. Signed-off-by: Mark Einon --- drivers/staging/et131x/et131x.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/et131x/et131x.c b/drivers/staging/et131x/et131x.c index 551b250..df83ea3 100644 --- a/drivers

[PATCH 0/8] Implement NAPI in et131x

2014-08-20 Thread Mark Einon
so won't apply cleanly in it's current state. Please let me know if I need to do anything about this. Cheers, Mark --- Mark Einon (8): staging: et131x: Use eth_mac_addr() instead of duplicating the functionality staging: et131x: Don't handle rx/tx packets when changing mtu staging: et1

[PATCH 8/8] staging: et131x: Implement NAPI support

2014-08-20 Thread Mark Einon
() to et131x_handle_send_pkts() -replacing bottom half workqueue with poll function which handles send & receive of skbs. -adding various other necessary standard napi calls. Also remove this item from the README TODO list. Signed-off-by: Mark Einon --- drivers/staging/et131x/README |

[PATCH 8/8] staging: et131x: Implement NAPI support

2014-08-20 Thread Mark Einon
() to et131x_handle_send_pkts() -replacing bottom half workqueue with poll function which handles send receive of skbs. -adding various other necessary standard napi calls. Also remove this item from the README TODO list. Signed-off-by: Mark Einon mark.ei...@gmail.com --- drivers/staging/et131x

[PATCH 0/8] Implement NAPI in et131x

2014-08-20 Thread Mark Einon
, not staging-next - so won't apply cleanly in it's current state. Please let me know if I need to do anything about this. Cheers, Mark --- Mark Einon (8): staging: et131x: Use eth_mac_addr() instead of duplicating the functionality staging: et131x: Don't handle rx/tx packets when changing mtu

[PATCH 7/8] staging: et131x: Fix ET_INTR_TXDMA_ISR register name typo

2014-08-20 Thread Mark Einon
We actually mean to clear the ET_INTR_TXDMA_ISR reg after handling a completed transfer, not the ET_INTR_TXDMA_ERR reg, which should get handled immediately after. Signed-off-by: Mark Einon mark.ei...@gmail.com --- drivers/staging/et131x/et131x.c | 2 +- 1 file changed, 1 insertion(+), 1

[PATCH 6/8] staging: et131x: Rename NUM_PACKETS_HANDLED to MAX_PACKETS_HANDLED

2014-08-20 Thread Mark Einon
To better describe it's use as a hard limit. Signed-off-by: Mark Einon mark.ei...@gmail.com --- drivers/staging/et131x/et131x.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/et131x/et131x.c b/drivers/staging/et131x/et131x.c index 551b250..df83ea3

  1   2   >