[PATCH] phy: Add XGMII phy interface type

Used by systems based on certain Freescale SoCs (specifically the
Servergy CTS-1000 system).

Signed-off-by: Ben Collins <redacted>
Cc: netdev@vger.kernel.org
---
 include/linux/phy.h | 1 +
 1 file changed, 1 insertion(+)

diff --git a/include/linux/phy.h b/include/linux/phy.h
index 33999ad..5a94ec7 100644
--- a/include/linux/phy.h
+++ b/include/linux/phy.h

@@ -64,6 +64,7 @@ typedef enum {
 	PHY_INTERFACE_MODE_RGMII_TXID,
 	PHY_INTERFACE_MODE_RTBI,
 	PHY_INTERFACE_MODE_SMII,
+	PHY_INTERFACE_MODE_XGMII,
 } phy_interface_t;
 
 

-- 
1.8.1.2

From: Ben Collins <redacted>
Date: Mon, 18 Mar 2013 19:53:42 -0400

Used by systems based on certain Freescale SoCs (specifically the
Servergy CTS-1000 system).

Signed-off-by: Ben Collins <redacted>

You can resubmit this patch when you submit a driver upstream that
uses it, so we can see what the use case looks like.

On Mar 22, 2013, at 10:12 AM, David Miller [off-list ref] wrote:

From: Ben Collins <redacted>
Date: Mon, 18 Mar 2013 19:53:42 -0400

quoted

Used by systems based on certain Freescale SoCs (specifically the
Servergy CTS-1000 system).

Signed-off-by: Ben Collins <redacted>

You can resubmit this patch when you submit a driver upstream that
uses it, so we can see what the use case looks like.

That patch is getting sent soon. It's starting out in staging. Should I Cc you on that?

--
Servergy  : http://www.servergy.com/
SwissDisk : http://www.swissdisk.com/
Ubuntu    : http://www.ubuntu.com/
My Blog   : http://ben-collins.blogspot.com/

From: Ben Collins <redacted>
Date: Fri, 22 Mar 2013 10:17:35 -0400

On Mar 22, 2013, at 10:12 AM, David Miller [off-list ref] wrote:

quoted

From: Ben Collins <redacted>
Date: Mon, 18 Mar 2013 19:53:42 -0400

quoted

Used by systems based on certain Freescale SoCs (specifically the
Servergy CTS-1000 system).

Signed-off-by: Ben Collins <redacted>

You can resubmit this patch when you submit a driver upstream that
uses it, so we can see what the use case looks like.

That patch is getting sent soon. It's starting out in staging. Should I Cc you on that?

Why don't you wait until the driver is good enough for drivers/net proper?

On Mar 22, 2013, at 10:19 AM, David Miller [off-list ref] wrote:

From: Ben Collins <redacted>
Date: Fri, 22 Mar 2013 10:17:35 -0400

quoted

On Mar 22, 2013, at 10:12 AM, David Miller [off-list ref] wrote:

quoted

From: Ben Collins <redacted>
Date: Mon, 18 Mar 2013 19:53:42 -0400

quoted

Used by systems based on certain Freescale SoCs (specifically the
Servergy CTS-1000 system).

Signed-off-by: Ben Collins <redacted>

You can resubmit this patch when you submit a driver upstream that
uses it, so we can see what the use case looks like.

That patch is getting sent soon. It's starting out in staging. Should I Cc you on that?

Why don't you wait until the driver is good enough for drivers/net proper?

It's code that I've manually stripped down as a subset of a larger code base for Freescale's DPAA driver. I've only tested it on our (Servergy's) platform, so until it gets more broad testing (and some code review), I want to at least get it into staging so our hardware is supported.

--
Servergy  : http://www.servergy.com/
SwissDisk : http://www.swissdisk.com/
Ubuntu    : http://www.ubuntu.com/
My Blog   : http://ben-collins.blogspot.com/

From: Ben Collins <redacted>
Date: Fri, 22 Mar 2013 10:22:05 -0400

It's code that I've manually stripped down as a subset of a larger
code base for Freescale's DPAA driver. I've only tested it on our
(Servergy's) platform, so until it gets more broad testing (and some
code review), I want to at least get it into staging so our hardware
is supported.

Then why don't you submit the PHY type addition with the staging
driver, which will be the only user, and you can add my:

Acked-by: David S. Miller <davem@davemloft.net>

to that.

Thanks.

On Mar 22, 2013, at 10:27 AM, David Miller [off-list ref] wrote:

From: Ben Collins <redacted>
Date: Fri, 22 Mar 2013 10:22:05 -0400

quoted

It's code that I've manually stripped down as a subset of a larger
code base for Freescale's DPAA driver. I've only tested it on our
(Servergy's) platform, so until it gets more broad testing (and some
code review), I want to at least get it into staging so our hardware
is supported.

Then why don't you submit the PHY type addition with the staging
driver, which will be the only user, and you can add my:

Acked-by: David S. Miller <davem@davemloft.net>

That I can do. Greg had asked I get all the rest of this signed off before he could take the driver into staging.

--
Servergy  : http://www.servergy.com/
SwissDisk : http://www.swissdisk.com/
Ubuntu    : http://www.ubuntu.com/
My Blog   : http://ben-collins.blogspot.com/