RE: [PATCH net] lan78xx: changed to use updated phy-ignore-interrupts

2016-01-25 Thread Woojung.Huh
Florian, > Looks fine, just one nit below: > > > @@ -954,6 +956,7 @@ static int lan78xx_link_reset(struct lan78xx_net > *dev) > > > > ret = lan78xx_update_flowcontrol(dev, ecmd.duplex, ladv, > radv); > > netif_carrier_on(dev->net); > > Do you need this netif_carrier_on()

Re: [PATCH net] lan78xx: changed to use updated phy-ignore-interrupts

2016-01-22 Thread Florian Fainelli
On 21/01/2016 12:15, woojung@microchip.com wrote: > Update lan78xx to use patch of commit 4f2aaf7dd95b > ("Merge branch 'fix-phy-ignore-interrupts'") Looks fine, just one nit below: > > Signed-off-by: Woojung Huh > --- > drivers/net/usb/lan78xx.c | 28 ++-- > 1 file

[PATCH net] lan78xx: changed to use updated phy-ignore-interrupts

2016-01-21 Thread Woojung.Huh
Update lan78xx to use patch of commit 4f2aaf7dd95b ("Merge branch 'fix-phy-ignore-interrupts'") Signed-off-by: Woojung Huh --- drivers/net/usb/lan78xx.c | 28 ++-- 1 file changed, 14 insertions(+), 14 deletions(-) diff --git a/drivers/net/usb/lan78xx.c b/drivers/net/usb/