[PATCH 0/8] net: stmmac: misc patchs

2017-02-14 Thread Corentin Labbe
Hello

This is a follow up of my previous stmmac serie which address some comment
done in v2.

Corentin Labbe (8):
  net: stmmac: remove useless parenthesis
  net: stmmac: likely is useless in occasional function
  net: stmmac: use SPEED_UNKNOWN/DUPLEX_UNKNOWN
  net: stmmac: set speed at SPEED_UNKNOWN in case of broken speed
  net: stmmac: run stmmac_hw_fix_mac_speed when speed is valid
  net: stmmac: split the stmmac_adjust_link 10/100 case
  net: stmmac: reduce indentation by adding a continue
  net: stmmac: invert the logic for dumping regs

 .../net/ethernet/stmicro/stmmac/stmmac_ethtool.c   | 18 ++---
 drivers/net/ethernet/stmicro/stmmac/stmmac_main.c  | 40 ++-
 drivers/net/ethernet/stmicro/stmmac/stmmac_mdio.c  | 82 +++---
 3 files changed, 71 insertions(+), 69 deletions(-)

-- 
2.10.2



Re: [PATCH 0/8] net: stmmac: misc patchs

2017-02-14 Thread Giuseppe CAVALLARO

Hello Corentin

On 2/14/2017 8:54 PM, Corentin Labbe wrote:

Hello

This is a follow up of my previous stmmac serie which address some comment
done in v2.


I wonder if you can resend all the patches as V3
please add the Acked-by and Reviewed-by. This will help
on final review and integration.

Thx for your support and effort

Peppe



Corentin Labbe (8):
  net: stmmac: remove useless parenthesis
  net: stmmac: likely is useless in occasional function
  net: stmmac: use SPEED_UNKNOWN/DUPLEX_UNKNOWN
  net: stmmac: set speed at SPEED_UNKNOWN in case of broken speed
  net: stmmac: run stmmac_hw_fix_mac_speed when speed is valid
  net: stmmac: split the stmmac_adjust_link 10/100 case
  net: stmmac: reduce indentation by adding a continue
  net: stmmac: invert the logic for dumping regs

 .../net/ethernet/stmicro/stmmac/stmmac_ethtool.c   | 18 ++---
 drivers/net/ethernet/stmicro/stmmac/stmmac_main.c  | 40 ++-
 drivers/net/ethernet/stmicro/stmmac/stmmac_mdio.c  | 82 +++---
 3 files changed, 71 insertions(+), 69 deletions(-)





Re: [PATCH 0/8] net: stmmac: misc patchs

2017-02-14 Thread Corentin Labbe
On Wed, Feb 15, 2017 at 07:34:41AM +0100, Giuseppe CAVALLARO wrote:
> Hello Corentin
> 
> On 2/14/2017 8:54 PM, Corentin Labbe wrote:
> > Hello
> >
> > This is a follow up of my previous stmmac serie which address some comment
> > done in v2.
> 
> I wonder if you can resend all the patches as V3
> please add the Acked-by and Reviewed-by. This will help
> on final review and integration.
> 
> Thx for your support and effort
> 
> Peppe
> 

Hello

Since all patch in v2 already hit linux-next, I just want to be sure, I can add 
"Acked-by and Reviewed-by" to thoses 8 new patchs ?

Regards

> >
> > Corentin Labbe (8):
> >   net: stmmac: remove useless parenthesis
> >   net: stmmac: likely is useless in occasional function
> >   net: stmmac: use SPEED_UNKNOWN/DUPLEX_UNKNOWN
> >   net: stmmac: set speed at SPEED_UNKNOWN in case of broken speed
> >   net: stmmac: run stmmac_hw_fix_mac_speed when speed is valid
> >   net: stmmac: split the stmmac_adjust_link 10/100 case
> >   net: stmmac: reduce indentation by adding a continue
> >   net: stmmac: invert the logic for dumping regs
> >
> >  .../net/ethernet/stmicro/stmmac/stmmac_ethtool.c   | 18 ++---
> >  drivers/net/ethernet/stmicro/stmmac/stmmac_main.c  | 40 ++-
> >  drivers/net/ethernet/stmicro/stmmac/stmmac_mdio.c  | 82 
> > +++---
> >  3 files changed, 71 insertions(+), 69 deletions(-)
> >
> 


Re: [PATCH 0/8] net: stmmac: misc patchs

2017-02-15 Thread Giuseppe CAVALLARO

On 2/15/2017 8:39 AM, Corentin Labbe wrote:

Since all patch in v2 already hit linux-next, I just want to be sure, I can add 
"Acked-by and Reviewed-by" to thoses 8 new patchs ?


Right I have just seen the V2 patches are in net-next.

pls consider, for this set, my

Acked-by: Giuseppe Cavallaro 


Regards
Peppe



Regards



Corentin Labbe (8):
  net: stmmac: remove useless parenthesis
  net: stmmac: likely is useless in occasional function
  net: stmmac: use SPEED_UNKNOWN/DUPLEX_UNKNOWN
  net: stmmac: set speed at SPEED_UNKNOWN in case of broken speed
  net: stmmac: run stmmac_hw_fix_mac_speed when speed is valid
  net: stmmac: split the stmmac_adjust_link 10/100 case
  net: stmmac: reduce indentation by adding a continue
  net: stmmac: invert the logic for dumping regs

 .../net/ethernet/stmicro/stmmac/stmmac_ethtool.c   | 18 ++---
 drivers/net/ethernet/stmicro/stmmac/stmmac_main.c  | 40 ++-
 drivers/net/ethernet/stmicro/stmmac/stmmac_mdio.c  | 82 +++---
 3 files changed, 71 insertions(+), 69 deletions(-)