Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752030AbeACQL7 convert rfc822-to-8bit (ORCPT + 1 other); Wed, 3 Jan 2018 11:11:59 -0500 Received: from mx0b-0016f401.pphosted.com ([67.231.156.173]:43114 "EHLO mx0b-0016f401.pphosted.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750831AbeACQL5 (ORCPT ); Wed, 3 Jan 2018 11:11:57 -0500 From: Stefan Chulski To: Andrew Lunn , Antoine Tenart CC: "davem@davemloft.net" , "kishon@ti.com" , "gregory.clement@free-electrons.com" , "linux@armlinux.org.uk" , "mw@semihalf.com" , Yan Markman , "thomas.petazzoni@free-electrons.com" , "miquel.raynal@free-electrons.com" , Nadav Haklai , "netdev@vger.kernel.org" , "linux-kernel@vger.kernel.org" Subject: RE: [PATCH net-next v2 4/4] net: mvpp2: 2500baseX support Thread-Topic: [PATCH net-next v2 4/4] net: mvpp2: 2500baseX support Thread-Index: AQHThKr7eRHuEIDThkOB1mqXv5Q/DaNiTShg Date: Wed, 3 Jan 2018 16:11:27 +0000 Message-ID: References: <20180103150750.12974-1-antoine.tenart@free-electrons.com> <20180103150750.12974-5-antoine.tenart@free-electrons.com> <20180103152036.GC3401@lunn.ch> <20180103153227.GA9227@kwain> <20180103155311.GD3401@lunn.ch> In-Reply-To: <20180103155311.GD3401@lunn.ch> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-ms-exchange-transport-fromentityheader: Hosted x-originating-ip: [10.5.102.207] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:,, definitions=2018-01-03_12:,, signatures=0 X-Proofpoint-Details: rule=outbound_notspam policy=outbound score=0 priorityscore=1501 malwarescore=0 suspectscore=0 phishscore=0 bulkscore=0 spamscore=0 clxscore=1015 lowpriorityscore=0 mlxscore=0 impostorscore=0 mlxlogscore=999 adultscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.0.1-1711220000 definitions=main-1801030227 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Return-Path: > -----Original Message----- > From: Andrew Lunn [mailto:andrew@lunn.ch] > Sent: Wednesday, January 03, 2018 5:53 PM > To: Antoine Tenart > Cc: davem@davemloft.net; kishon@ti.com; gregory.clement@free- > electrons.com; linux@armlinux.org.uk; mw@semihalf.com; Stefan Chulski > ; Yan Markman ; > thomas.petazzoni@free-electrons.com; miquel.raynal@free-electrons.com; > Nadav Haklai ; netdev@vger.kernel.org; linux- > kernel@vger.kernel.org > Subject: Re: [PATCH net-next v2 4/4] net: mvpp2: 2500baseX support > > On Wed, Jan 03, 2018 at 04:32:27PM +0100, Antoine Tenart wrote: > > Hi Andrew, > > > > On Wed, Jan 03, 2018 at 04:20:36PM +0100, Andrew Lunn wrote: > > > > @@ -4612,6 +4616,9 @@ static int mvpp22_comphy_init(struct > mvpp2_port *port) > > > > case PHY_INTERFACE_MODE_1000BASEX: > > > > mode = PHY_MODE_SGMII; > > > > break; > > > > + case PHY_INTERFACE_MODE_2500BASEX: > > > > + mode = PHY_MODE_2500SGMII; > > > > + break; > > > > > > I think this is the source of confusion with linux/phy.h and > > > linux/phy/phy.h. > > > > > > What would PHY_INTERFACE_MODE_2500SGMII use? > > > > > > Where is this all getting confused? Should the caller to > > > mvpp22_comphy_init() actually be passing > PHY_INTERFACE_MODE_2500SGMII? > > > What is the MAC actually doing at this point? 2500BASEX or 2500SGMII? > > > > PHY_INTERFACE_MODE_2500BASEX is the PHY mode whereas > > PHY_MODE_2500SGMII is the mode used by the common PHY driver (i.e. the > > one configuring the serdes lanes). > > > There's no PHY_INTERFACE_MODE_2500SGMII mode. > > Hi Antoine > > At the moment there is no PHY_INTERFACE_MODE_2500SGMII. However, > there are some devices which can do 2.5G SGMII. So it will appear sometime. > This piece of code then looks even stranger. > > > Sure, I can add a comment to state this function is a translation > > between the net PHY mode and the generic PHY mode (it's a n-to-1 > > translation). > > I think from an API design point of view, passing PHY_MODE_2500BASEX to > comphy makes more sense. That is what the MAC wants to do. How the > comphy achieves that should be internal to the comphy. > > Andrew COMPHY driver configure SERDES lanes to high speed SGMII(2500SGMII) mode and doesn't care about mode used by physical layer(2500BASEX in this case). There are possibility that MAC SERDES lane is routed on a backplane to another SERDES. I think we should use SGMII naming. Stefan.