Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1633636 > unrolled thread

[PATCH v5 00/20] net-next: stmmac: add dwmac-sun8i ethernet driver

Started byCorentin Labbe <clabbe.montjoie@gmail.com>
First post2017-05-01 15:00 +0200
Last post2017-05-05 14:20 +0200
Articles 6 on this page of 26 — 4 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH v5 00/20] net-next: stmmac: add dwmac-sun8i ethernet driver Corentin Labbe <clabbe.montjoie@gmail.com> - 2017-05-01 15:00 +0200
    [PATCH v5 17/20] arm64: allwinner: bananapi-m64: Enable dwmac-sun8i Corentin Labbe <clabbe.montjoie@gmail.com> - 2017-05-01 15:00 +0200
    [PATCH v5 20/20] arm64: defconfig: Enable dwmac-sun8i driver on defconfig Corentin Labbe <clabbe.montjoie@gmail.com> - 2017-05-01 15:00 +0200
    [PATCH v5 09/20] arm: sun8i: orangepi-zero: Enable dwmac-sun8i Corentin Labbe <clabbe.montjoie@gmail.com> - 2017-05-01 15:00 +0200
    [PATCH v5 18/20] arm: sunxi: Enable dwmac-sun8i driver on sunxi_defconfig Corentin Labbe <clabbe.montjoie@gmail.com> - 2017-05-01 15:00 +0200
    [PATCH v5 13/20] arm64: allwinner: sun50i-a64: Add dt node for the syscon control module Corentin Labbe <clabbe.montjoie@gmail.com> - 2017-05-01 15:00 +0200
    [PATCH v5 07/20] arm: sun8i: sunxi-h3-h5: add dwmac-sun8i ethernet driver Corentin Labbe <clabbe.montjoie@gmail.com> - 2017-05-01 15:00 +0200
    [PATCH v5 14/20] arm64: allwinner: sun50i-a64: add dwmac-sun8i Ethernet driver Corentin Labbe <clabbe.montjoie@gmail.com> - 2017-05-01 15:00 +0200
    [PATCH v5 16/20] arm64: allwinner: pine64-plus: Enable dwmac-sun8i Corentin Labbe <clabbe.montjoie@gmail.com> - 2017-05-01 15:00 +0200
    [PATCH v5 15/20] arm64: allwinner: pine64: Enable dwmac-sun8i Corentin Labbe <clabbe.montjoie@gmail.com> - 2017-05-01 15:00 +0200
    [PATCH v5 06/20] arm: sun8i: sunxi-h3-h5: Add dt node for the syscon control module Corentin Labbe <clabbe.montjoie@gmail.com> - 2017-05-01 15:00 +0200
    [PATCH v5 05/20] net: stmmac: Add dwmac-sun8i Corentin Labbe <clabbe.montjoie@gmail.com> - 2017-05-01 15:00 +0200
    [PATCH v5 19/20] arm: multi_v7: Enable dwmac-sun8i driver on multi_v7_defconfig Corentin Labbe <clabbe.montjoie@gmail.com> - 2017-05-01 15:00 +0200
    [PATCH v5 08/20] arm: sun8i: orangepi-pc: Enable dwmac-sun8i Corentin Labbe <clabbe.montjoie@gmail.com> - 2017-05-01 15:00 +0200
    [PATCH v5 12/20] arm: sun8i: orangepi-pc-plus: Set EMAC activity LEDs to active high Corentin Labbe <clabbe.montjoie@gmail.com> - 2017-05-01 15:00 +0200
    [PATCH v5 10/20] arm: sun8i: orangepi-one: Enable dwmac-sun8i Corentin Labbe <clabbe.montjoie@gmail.com> - 2017-05-01 15:00 +0200
    [PATCH v5 01/20] net: stmmac: export stmmac_set_mac_addr/stmmac_get_mac_addr Corentin Labbe <clabbe.montjoie@gmail.com> - 2017-05-01 15:00 +0200
    [PATCH v5 04/20] dt-bindings: syscon: Add DT bindings documentation for Allwinner syscon Corentin Labbe <clabbe.montjoie@gmail.com> - 2017-05-01 15:00 +0200
      Re: [PATCH v5 04/20] dt-bindings: syscon: Add DT bindings  documentation for Allwinner syscon Rob Herring <robh@kernel.org> - 2017-05-05 22:30 +0200
    [PATCH v5 11/20] arm: sun8i: orangepi-2: Enable dwmac-sun8i Corentin Labbe <clabbe.montjoie@gmail.com> - 2017-05-01 15:00 +0200
    Re: [PATCH v5 00/20] net-next: stmmac: add dwmac-sun8i ethernet  driver Andrew Lunn <andrew@lunn.ch> - 2017-05-01 16:10 +0200
      Re: [PATCH v5 00/20] net-next: stmmac: add dwmac-sun8i ethernet driver Chen-Yu Tsai <wens@csie.org> - 2017-05-01 16:40 +0200
        Re: [PATCH v5 00/20] net-next: stmmac: add dwmac-sun8i ethernet  driver Andrew Lunn <andrew@lunn.ch> - 2017-05-01 16:50 +0200
          Re: [PATCH v5 00/20] net-next: stmmac: add dwmac-sun8i ethernet  driver Corentin Labbe <clabbe.montjoie@gmail.com> - 2017-05-01 18:00 +0200
            Re: [PATCH v5 00/20] net-next: stmmac: add dwmac-sun8i ethernet  driver Andrew Lunn <andrew@lunn.ch> - 2017-05-01 18:20 +0200
    Re: [PATCH v5 00/20] net-next: stmmac: add dwmac-sun8i ethernet  driver Corentin Labbe <clabbe.montjoie@gmail.com> - 2017-05-05 14:20 +0200

Page 2 of 2 — ← Prev page 1 [2]


#1633680 — Re: [PATCH v5 00/20] net-next: stmmac: add dwmac-sun8i ethernet driver

FromAndrew Lunn <andrew@lunn.ch>
Date2017-05-01 16:10 +0200
SubjectRe: [PATCH v5 00/20] net-next: stmmac: add dwmac-sun8i ethernet driver
Message-ID<tCkbw-1IO-33@gated-at.bofh.it>
In reply to#1633636
On Mon, May 01, 2017 at 02:45:00PM +0200, Corentin Labbe wrote:
> Hello
> 
> This patch series add the driver for dwmac-sun8i which handle the Ethernet MAC
> present on Allwinner H3/H5/A83T/A64 SoCs.
> 
> This driver is the continuation of the sun8i-emac driver.
> During the development, it appeared that in fact the hardware was a modified
> version of some dwmac.
> So the driver is now written as a glue driver for stmmac.
> 
> It supports 10/100/1000 Mbit/s speed with half/full duplex.
> It can use an internal PHY (MII 10/100) or an external PHY
> via RGMII/RMII.

Hi Corentin

Sorry if this has been asked before....

Does the internal PHY have a phy driver? It seems like
tx-delay-ps/rx-delay-ps are properties of this internal PHY, and so
should be in the phy driver, if it has one.

       Andrew

[toc] | [prev] | [next] | [standalone]


#1633689

FromChen-Yu Tsai <wens@csie.org>
Date2017-05-01 16:40 +0200
Message-ID<tCkEy-1S6-7@gated-at.bofh.it>
In reply to#1633680
On Mon, May 1, 2017 at 10:01 PM, Andrew Lunn <andrew@lunn.ch> wrote:
> On Mon, May 01, 2017 at 02:45:00PM +0200, Corentin Labbe wrote:
>> Hello
>>
>> This patch series add the driver for dwmac-sun8i which handle the Ethernet MAC
>> present on Allwinner H3/H5/A83T/A64 SoCs.
>>
>> This driver is the continuation of the sun8i-emac driver.
>> During the development, it appeared that in fact the hardware was a modified
>> version of some dwmac.
>> So the driver is now written as a glue driver for stmmac.
>>
>> It supports 10/100/1000 Mbit/s speed with half/full duplex.
>> It can use an internal PHY (MII 10/100) or an external PHY
>> via RGMII/RMII.
>
> Hi Corentin
>
> Sorry if this has been asked before....
>
> Does the internal PHY have a phy driver? It seems like
> tx-delay-ps/rx-delay-ps are properties of this internal PHY, and so
> should be in the phy driver, if it has one.

Nope. These affect the delay lines for the external PHY interface.
These have existed since the A20, when the GMAC hardware block and
glue layer controls were introduced.

ChenYu

[toc] | [prev] | [next] | [standalone]


#1633692 — Re: [PATCH v5 00/20] net-next: stmmac: add dwmac-sun8i ethernet driver

FromAndrew Lunn <andrew@lunn.ch>
Date2017-05-01 16:50 +0200
SubjectRe: [PATCH v5 00/20] net-next: stmmac: add dwmac-sun8i ethernet driver
Message-ID<tCkOe-1Vi-17@gated-at.bofh.it>
In reply to#1633689
On Mon, May 01, 2017 at 10:28:46PM +0800, Chen-Yu Tsai wrote:
> On Mon, May 1, 2017 at 10:01 PM, Andrew Lunn <andrew@lunn.ch> wrote:
> > On Mon, May 01, 2017 at 02:45:00PM +0200, Corentin Labbe wrote:
> >> Hello
> >>
> >> This patch series add the driver for dwmac-sun8i which handle the Ethernet MAC
> >> present on Allwinner H3/H5/A83T/A64 SoCs.
> >>
> >> This driver is the continuation of the sun8i-emac driver.
> >> During the development, it appeared that in fact the hardware was a modified
> >> version of some dwmac.
> >> So the driver is now written as a glue driver for stmmac.
> >>
> >> It supports 10/100/1000 Mbit/s speed with half/full duplex.
> >> It can use an internal PHY (MII 10/100) or an external PHY
> >> via RGMII/RMII.
> >
> > Hi Corentin
> >
> > Sorry if this has been asked before....
> >
> > Does the internal PHY have a phy driver? It seems like
> > tx-delay-ps/rx-delay-ps are properties of this internal PHY, and so
> > should be in the phy driver, if it has one.
> 
> Nope. These affect the delay lines for the external PHY interface.

Oh, yes. I understood the patch wrong.

None of the patches actually use these properties. Are they actually
needed? We should avoid adding vendor specific properties, if they are
not used.

    Andrew

[toc] | [prev] | [next] | [standalone]


#1633713 — Re: [PATCH v5 00/20] net-next: stmmac: add dwmac-sun8i ethernet driver

FromCorentin Labbe <clabbe.montjoie@gmail.com>
Date2017-05-01 18:00 +0200
SubjectRe: [PATCH v5 00/20] net-next: stmmac: add dwmac-sun8i ethernet driver
Message-ID<tClTX-2yI-5@gated-at.bofh.it>
In reply to#1633692
On Mon, May 01, 2017 at 04:45:41PM +0200, Andrew Lunn wrote:
> On Mon, May 01, 2017 at 10:28:46PM +0800, Chen-Yu Tsai wrote:
> > On Mon, May 1, 2017 at 10:01 PM, Andrew Lunn <andrew@lunn.ch> wrote:
> > > On Mon, May 01, 2017 at 02:45:00PM +0200, Corentin Labbe wrote:
> > >> Hello
> > >>
> > >> This patch series add the driver for dwmac-sun8i which handle the Ethernet MAC
> > >> present on Allwinner H3/H5/A83T/A64 SoCs.
> > >>
> > >> This driver is the continuation of the sun8i-emac driver.
> > >> During the development, it appeared that in fact the hardware was a modified
> > >> version of some dwmac.
> > >> So the driver is now written as a glue driver for stmmac.
> > >>
> > >> It supports 10/100/1000 Mbit/s speed with half/full duplex.
> > >> It can use an internal PHY (MII 10/100) or an external PHY
> > >> via RGMII/RMII.
> > >
> > > Hi Corentin
> > >
> > > Sorry if this has been asked before....
> > >
> > > Does the internal PHY have a phy driver? It seems like
> > > tx-delay-ps/rx-delay-ps are properties of this internal PHY, and so
> > > should be in the phy driver, if it has one.
> > 
> > Nope. These affect the delay lines for the external PHY interface.

I will add some sentences in the documentation to be more clear about that.

> 
> Oh, yes. I understood the patch wrong.
> 
> None of the patches actually use these properties. Are they actually
> needed? We should avoid adding vendor specific properties, if they are
> not used.
> 

The board that use it is the BananaPi M3, but the SoC A83T still lack clocks support.
So theses properties is already used out of tree, and will be used in tree soon.

Regards

[toc] | [prev] | [next] | [standalone]


#1633727 — Re: [PATCH v5 00/20] net-next: stmmac: add dwmac-sun8i ethernet driver

FromAndrew Lunn <andrew@lunn.ch>
Date2017-05-01 18:20 +0200
SubjectRe: [PATCH v5 00/20] net-next: stmmac: add dwmac-sun8i ethernet driver
Message-ID<tCmdk-2Ui-5@gated-at.bofh.it>
In reply to#1633713
> > > > Does the internal PHY have a phy driver? It seems like
> > > > tx-delay-ps/rx-delay-ps are properties of this internal PHY, and so
> > > > should be in the phy driver, if it has one.
> > > 
> > > Nope. These affect the delay lines for the external PHY interface.
> 
> I will add some sentences in the documentation to be more clear about that.

Thanks

> 
> > 
> > Oh, yes. I understood the patch wrong.
> > 
> > None of the patches actually use these properties. Are they actually
> > needed? We should avoid adding vendor specific properties, if they are
> > not used.
> > 
> 
> The board that use it is the BananaPi M3, but the SoC A83T still lack clocks support.
> So theses properties is already used out of tree, and will be used in tree soon.

Fair enough.

Thanks
	Andrew

[toc] | [prev] | [next] | [standalone]


#1636340 — Re: [PATCH v5 00/20] net-next: stmmac: add dwmac-sun8i ethernet driver

FromCorentin Labbe <clabbe.montjoie@gmail.com>
Date2017-05-05 14:20 +0200
SubjectRe: [PATCH v5 00/20] net-next: stmmac: add dwmac-sun8i ethernet driver
Message-ID<tDKnf-1Cl-3@gated-at.bofh.it>
In reply to#1633636
On Mon, May 01, 2017 at 02:45:00PM +0200, Corentin Labbe wrote:
> Hello
> 
> This patch series add the driver for dwmac-sun8i which handle the Ethernet MAC
> present on Allwinner H3/H5/A83T/A64 SoCs.
> 
> This driver is the continuation of the sun8i-emac driver.
> During the development, it appeared that in fact the hardware was a modified
> version of some dwmac.
> So the driver is now written as a glue driver for stmmac.
> 
> It supports 10/100/1000 Mbit/s speed with half/full duplex.
> It can use an internal PHY (MII 10/100) or an external PHY
> via RGMII/RMII.
> 
> This patch series enable the driver only for the H3/A64/H5 SoC since A83T
> doesn't have the necessary clocks present in mainline.
> 
> The driver have been tested on the following boards:
> - H3 Orange PI PC, BananaPI-M2+
> - A64 Pine64, BananaPi-M64
> - A83T BananaPI-M3
> 
> The first two patchs are some mandatory changes for letting dwmac-sun8i be used.
> The following three patchs add the driver and its documentation.
> The remaining are DT patch enabling it.
> 
> Regards
> Corentin Labbe
> 

Hello Giuseppe

Since this serie have only one minor comment on documentation, could you give me your view about it ?

Regards

[toc] | [prev] | [standalone]


Page 2 of 2 — ← Prev page 1 [2]

Back to top | Article view | linux.kernel


csiph-web