mbox series

[net,v2,0/3] Bugfix for the netsec driver

Message ID 20181023112428.6785-1-masahisa.kojima@linaro.org
Headers show
Series Bugfix for the netsec driver | expand

Message

Masahisa Kojima Oct. 23, 2018, 11:24 a.m. UTC
From: Masahisa Kojima <masahisa.kojima@linaro.org>


This patch series include bugfix for the netsec ethernet
controller driver, fix the problem in interface down/up.

changes in v2:
 - change the place to perform the PHY power down
 - use the MACROs defiend in include/uapi/linux/mii.h
 - update commit comment

Masahisa Kojima (3):
  net: socionext: Stop PHY before resetting netsec
  net: socionext: Add dummy PHY register read in phy_write()
  net: socionext: Reset tx queue in ndo_stop

 drivers/net/ethernet/socionext/netsec.c | 40 ++++++++++++++++++++++++++++-----
 1 file changed, 34 insertions(+), 6 deletions(-)

-- 
2.14.2

Comments

Ard Biesheuvel Oct. 23, 2018, 11:32 a.m. UTC | #1
(+ Florian)

On 23 October 2018 at 08:24,  <masahisa.kojima@linaro.org> wrote:
> From: Masahisa Kojima <masahisa.kojima@linaro.org>

>

> This patch series include bugfix for the netsec ethernet

> controller driver, fix the problem in interface down/up.

>

> changes in v2:

>  - change the place to perform the PHY power down

>  - use the MACROs defiend in include/uapi/linux/mii.h

>  - update commit comment

>

> Masahisa Kojima (3):

>   net: socionext: Stop PHY before resetting netsec

>   net: socionext: Add dummy PHY register read in phy_write()

>   net: socionext: Reset tx queue in ndo_stop

>

>  drivers/net/ethernet/socionext/netsec.c | 40 ++++++++++++++++++++++++++++-----

>  1 file changed, 34 insertions(+), 6 deletions(-)

>


Hello Masahisa,

As a courtesy, please cc people that have commented on your patches on
subsequent revisions of the series.
Masahisa Kojima Oct. 23, 2018, 11:39 a.m. UTC | #2
Hi Florian, Ard,

Yes, that is my mistake. Thank you very much for pointing out, Ard.
On Tue, 23 Oct 2018 at 20:32, Ard Biesheuvel <ard.biesheuvel@linaro.org> wrote:
>

> (+ Florian)

>

> On 23 October 2018 at 08:24,  <masahisa.kojima@linaro.org> wrote:

> > From: Masahisa Kojima <masahisa.kojima@linaro.org>

> >

> > This patch series include bugfix for the netsec ethernet

> > controller driver, fix the problem in interface down/up.

> >

> > changes in v2:

> >  - change the place to perform the PHY power down

> >  - use the MACROs defiend in include/uapi/linux/mii.h

> >  - update commit comment

> >

> > Masahisa Kojima (3):

> >   net: socionext: Stop PHY before resetting netsec

> >   net: socionext: Add dummy PHY register read in phy_write()

> >   net: socionext: Reset tx queue in ndo_stop

> >

> >  drivers/net/ethernet/socionext/netsec.c | 40 ++++++++++++++++++++++++++++-----

> >  1 file changed, 34 insertions(+), 6 deletions(-)

> >

>

> Hello Masahisa,

>

> As a courtesy, please cc people that have commented on your patches on

> subsequent revisions of the series.
David Miller Oct. 23, 2018, 5:55 p.m. UTC | #3
From: masahisa.kojima@linaro.org

Date: Tue, 23 Oct 2018 20:24:25 +0900

> From: Masahisa Kojima <masahisa.kojima@linaro.org>

> 

> This patch series include bugfix for the netsec ethernet

> controller driver, fix the problem in interface down/up.

> 

> changes in v2:

>  - change the place to perform the PHY power down

>  - use the MACROs defiend in include/uapi/linux/mii.h

>  - update commit comment


Series applied.