From: antoine.tenart@free-electrons.com (Antoine Tenart)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH net-next 1/6] phy: add 2.5G SGMII mode to the phy_mode enum
Date: Thu, 28 Dec 2017 11:06:57 +0100 [thread overview]
Message-ID: <20171228100656.GF2626@kwain> (raw)
In-Reply-To: <20171228072053.GA27336@lunn.ch>
Hi Andrew,
On Thu, Dec 28, 2017 at 08:20:53AM +0100, Andrew Lunn wrote:
> On Wed, Dec 27, 2017 at 11:14:41PM +0100, Antoine Tenart wrote:
> > This patch adds one more generic PHY mode to the phy_mode enum, to allow
> > configuring generic PHYs to the 2.5G SGMII mode by using the set_mode
> > callback.
> >
> > Signed-off-by: Antoine Tenart <antoine.tenart@free-electrons.com>
> > ---
> > include/linux/phy/phy.h | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/include/linux/phy/phy.h b/include/linux/phy/phy.h
> > index 4f8423a948d5..70459a28f3a1 100644
> > --- a/include/linux/phy/phy.h
> > +++ b/include/linux/phy/phy.h
> > @@ -28,6 +28,7 @@ enum phy_mode {
> > PHY_MODE_USB_DEVICE,
> > PHY_MODE_USB_OTG,
> > PHY_MODE_SGMII,
> > + PHY_MODE_SGMII_2_5G,
> > PHY_MODE_10GKR,
> > PHY_MODE_UFS_HS_A,
> > PHY_MODE_UFS_HS_B,
>
> There was a discussion maybe last month about adding 2.5G SGMII. I
> would prefer 2500SGMII. Putting the number first makes it uniform with
> the other defines, 1000BASEX, 25000BASEX, 10GKR.
Good to know. I wasn't completely sure how to name this mode properly,
but I'm fine with PHY_MODE_2500SGMII. I'll update the patches and send a
v2 (without the dt part).
Thanks!
Antoine
--
Antoine T?nart, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
WARNING: multiple messages have this Message-ID (diff)
From: Antoine Tenart <antoine.tenart@free-electrons.com>
To: Andrew Lunn <andrew@lunn.ch>
Cc: Antoine Tenart <antoine.tenart@free-electrons.com>,
davem@davemloft.net, kishon@ti.com, jason@lakedaemon.net,
sebastian.hesselbarth@gmail.com,
gregory.clement@free-electrons.com, linux@armlinux.org.uk,
mw@semihalf.com, stefanc@marvell.com, ymarkman@marvell.com,
thomas.petazzoni@free-electrons.com,
miquel.raynal@free-electrons.com, nadavh@marvell.com,
netdev@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH net-next 1/6] phy: add 2.5G SGMII mode to the phy_mode enum
Date: Thu, 28 Dec 2017 11:06:57 +0100 [thread overview]
Message-ID: <20171228100656.GF2626@kwain> (raw)
In-Reply-To: <20171228072053.GA27336@lunn.ch>
Hi Andrew,
On Thu, Dec 28, 2017 at 08:20:53AM +0100, Andrew Lunn wrote:
> On Wed, Dec 27, 2017 at 11:14:41PM +0100, Antoine Tenart wrote:
> > This patch adds one more generic PHY mode to the phy_mode enum, to allow
> > configuring generic PHYs to the 2.5G SGMII mode by using the set_mode
> > callback.
> >
> > Signed-off-by: Antoine Tenart <antoine.tenart@free-electrons.com>
> > ---
> > include/linux/phy/phy.h | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/include/linux/phy/phy.h b/include/linux/phy/phy.h
> > index 4f8423a948d5..70459a28f3a1 100644
> > --- a/include/linux/phy/phy.h
> > +++ b/include/linux/phy/phy.h
> > @@ -28,6 +28,7 @@ enum phy_mode {
> > PHY_MODE_USB_DEVICE,
> > PHY_MODE_USB_OTG,
> > PHY_MODE_SGMII,
> > + PHY_MODE_SGMII_2_5G,
> > PHY_MODE_10GKR,
> > PHY_MODE_UFS_HS_A,
> > PHY_MODE_UFS_HS_B,
>
> There was a discussion maybe last month about adding 2.5G SGMII. I
> would prefer 2500SGMII. Putting the number first makes it uniform with
> the other defines, 1000BASEX, 25000BASEX, 10GKR.
Good to know. I wasn't completely sure how to name this mode properly,
but I'm fine with PHY_MODE_2500SGMII. I'll update the patches and send a
v2 (without the dt part).
Thanks!
Antoine
--
Antoine Ténart, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
next prev parent reply other threads:[~2017-12-28 10:06 UTC|newest]
Thread overview: 77+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-12-27 22:14 [PATCH net-next 0/6] net: mvpp2: 1000BaseX and 2000BaseX support Antoine Tenart
2017-12-27 22:14 ` Antoine Tenart
2017-12-27 22:14 ` [PATCH net-next 1/6] phy: add 2.5G SGMII mode to the phy_mode enum Antoine Tenart
2017-12-27 22:14 ` Antoine Tenart
2017-12-28 7:20 ` Andrew Lunn
2017-12-28 7:20 ` Andrew Lunn
2017-12-28 10:06 ` Antoine Tenart [this message]
2017-12-28 10:06 ` Antoine Tenart
2017-12-28 14:16 ` Florian Fainelli
2017-12-28 14:16 ` Florian Fainelli
2017-12-28 18:24 ` Antoine Tenart
2017-12-28 18:24 ` Antoine Tenart
2018-01-03 14:35 ` Antoine Tenart
2018-01-03 14:35 ` Antoine Tenart
2018-01-03 15:08 ` Andrew Lunn
2018-01-03 15:08 ` Andrew Lunn
2018-01-03 17:29 ` Russell King - ARM Linux
2018-01-03 17:29 ` Russell King - ARM Linux
2017-12-27 22:14 ` [PATCH net-next 2/6] phy: cp110-comphy: 2.5G SGMII mode Antoine Tenart
2017-12-27 22:14 ` Antoine Tenart
2017-12-27 22:14 ` [PATCH net-next 3/6] net: mvpp2: 1000baseX support Antoine Tenart
2017-12-27 22:14 ` Antoine Tenart
2017-12-27 22:14 ` [PATCH net-next 4/6] net: mvpp2: 2500baseX support Antoine Tenart
2017-12-27 22:14 ` Antoine Tenart
2017-12-27 22:14 ` [PATCH net-next 5/6] arm64: dts: marvell: mcbin: enable the fourth network interface Antoine Tenart
2017-12-27 22:14 ` Antoine Tenart
2017-12-27 22:24 ` Russell King - ARM Linux
2017-12-27 22:24 ` Russell King - ARM Linux
2017-12-27 22:42 ` Antoine Tenart
2017-12-27 22:42 ` Antoine Tenart
2017-12-27 23:20 ` Russell King - ARM Linux
2017-12-27 23:20 ` Russell King - ARM Linux
2017-12-28 10:04 ` Antoine Tenart
2017-12-28 10:04 ` Antoine Tenart
2017-12-28 18:59 ` Russell King - ARM Linux
2017-12-28 18:59 ` Russell King - ARM Linux
2017-12-29 21:41 ` Antoine Tenart
2017-12-29 21:41 ` Antoine Tenart
2017-12-28 7:46 ` Andrew Lunn
2017-12-28 7:46 ` Andrew Lunn
2017-12-28 7:46 ` Andrew Lunn
2017-12-28 10:05 ` Antoine Tenart
2017-12-28 10:05 ` Antoine Tenart
2017-12-28 15:02 ` Florian Fainelli
2017-12-28 15:02 ` Florian Fainelli
2017-12-28 18:27 ` Antoine Tenart
2017-12-28 18:27 ` Antoine Tenart
2017-12-28 18:46 ` Russell King - ARM Linux
2017-12-28 18:46 ` Russell King - ARM Linux
2017-12-29 11:12 ` Marcin Wojtas
2017-12-29 11:12 ` Marcin Wojtas
2017-12-29 11:38 ` Russell King - ARM Linux
2017-12-29 11:38 ` Russell King - ARM Linux
2017-12-30 16:34 ` Marcin Wojtas
2017-12-30 16:34 ` Marcin Wojtas
2017-12-30 17:31 ` Russell King - ARM Linux
2017-12-30 17:31 ` Russell King - ARM Linux
2018-01-01 10:18 ` Marcin Wojtas
2018-01-01 10:18 ` Marcin Wojtas
2018-01-01 10:35 ` [EXT] " Stefan Chulski
2018-01-01 10:35 ` Stefan Chulski
2018-01-01 13:25 ` Russell King - ARM Linux
2018-01-01 13:25 ` Russell King - ARM Linux
2018-01-01 13:25 ` Russell King - ARM Linux
2018-01-03 17:00 ` Stefan Chulski
2018-01-03 17:00 ` Stefan Chulski
2018-01-03 17:00 ` Stefan Chulski
2018-01-03 17:54 ` Russell King - ARM Linux
2018-01-03 17:54 ` Russell King - ARM Linux
2018-01-03 17:54 ` Russell King - ARM Linux
2018-01-03 18:17 ` Marcin Wojtas
2018-01-03 18:17 ` Marcin Wojtas
2018-01-03 18:26 ` Stefan Chulski
2018-01-03 18:26 ` Stefan Chulski
2018-01-03 18:26 ` Stefan Chulski
2017-12-27 22:14 ` [PATCH net-next 6/6] arm64: dts: marvell: add Ethernet aliases Antoine Tenart
2017-12-27 22:14 ` Antoine Tenart
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20171228100656.GF2626@kwain \
--to=antoine.tenart@free-electrons.com \
--cc=linux-arm-kernel@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.