Received-SPF: pass (google.com: domain of linux-kernel+bounces-73803-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
Date: Wed, 21 Feb 2024 01:06:56 +0200
From: Vladimir Oltean <vladimir.oltean@nxp.com>
To: Sean Anderson <sean.anderson@seco.com>
Cc: zachary.goldstein@concurrent-rt.com, Shawn Guo <shawnguo@kernel.org>,
	Madalin Bucur <madalin.bucur@nxp.com>, Li Yang <leoyang.li@nxp.com>,
	Rob Herring <robh+dt@kernel.org>,
	Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
	Conor Dooley <conor+dt@kernel.org>, Andrew Lunn <andrew@lunn.ch>,
	Heiner Kallweit <hkallweit1@gmail.com>,
	Russell King <linux@armlinux.org.uk>,
	"David S. Miller" <davem@davemloft.net>,
	Eric Dumazet <edumazet@google.com>,
	Jakub Kicinski <kuba@kernel.org>, Paolo Abeni <pabeni@redhat.com>,
	linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org,
	linux-kernel@vger.kernel.org, netdev@vger.kernel.org
Subject: Re: [PATCH] arm64: ls1046ardb: Replace XGMII with 10GBASE-R phy mode
Message-ID: <20240220230656.cefvrh6avji2elrd@skbuf>
References: <20240220145037.kf3avnykjif24kkr@skbuf>
 <191b4477-7b4b-47eb-bb3e-0e4d08b3b32e@seco.com>
 <20240220223706.o7wc5r57omkmgtgh@skbuf>
 <e39c811e-ad9d-4e90-8710-629b822944e0@seco.com>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <e39c811e-ad9d-4e90-8710-629b822944e0@seco.com>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
 =?us-ascii?Q?FPBg68x0qdoroLnzZyDOg+D8eUVqYvsTOcK2++Fvz5ZvX9uw55cxWpbaKYej?=
 =?us-ascii?Q?1WOYgSKx1S9ewpwsuh3E04Onj9fhC5qMQVXcCZ+IdfQlwCiuKHByyNvcIj3A?=
 =?us-ascii?Q?uHJ1leXSB6+Efg6YXW73T7BMuVpzoo2FNNtmWhnWr+cZrdXcT/gSqrTPkOyq?=
 =?us-ascii?Q?wWeDKvkrCf8mbUVDKrYK+brJ1bJlTwQxtRTnXAJg0FKp4Qrggxb3vGTA0qvV?=
 =?us-ascii?Q?4eWlEeXnuclQ8mVK5G0CDvGNA2t0cHPsMrdT/e1MtVCSyiRn/7zVE+fxRO0Q?=
 =?us-ascii?Q?je946IIZenXxVrxcpnAfSe2IOmeMRlENfXRPjWwCjkOZIfjLpLrB10CtrnR3?=
 =?us-ascii?Q?o/32qe9sNWAJiAI2tXwubvrz+XLfKX0/wbPr5tA+t3TIQe4rzVWbwqoyQtN6?=
 =?us-ascii?Q?FzfrDHagpvnbMebRs5d39uUmxOBPmJ/bjohW4GlsZHEVoLMoO2DtR0qu2qHZ?=
 =?us-ascii?Q?cqkC8CIDP9melgXntDjpDI82FH1K+IGscwrlC6reXSqM76ixzUN2n6iLRTrp?=
 =?us-ascii?Q?+HK5BhxBZ5eEzAbQ2E5TfCF7gOT3vmFxz9huOmWXdDRyYK7MBZKj+e7Ereio?=
 =?us-ascii?Q?Mm/UT5/HN9O9TZU9RWoL875DiIEALh3RoJD3VSLEopW/RnA9Ln+C54bWzDmc?=
 =?us-ascii?Q?/N72REOYS2MbuV94FTEpi22BcDvP0r+yLJBxF8L394SvRJFFX1OOiJn24Fr3?=
 =?us-ascii?Q?WZKp1AnuQljPZgNXL2LiQQ0wMtSP0+tnui0EifnMowicqYWbbYKqqsWA8Z6q?=
 =?us-ascii?Q?0zeyM8NG30MpCQQI14fRmgv2mxh+PlSCKSV+oenterUIqG9HWtwwfutmLSrp?=
 =?us-ascii?Q?PGOd41gKXJkkUI6gOQl7Cm5/yzd/Fj4/FiLPWTzLcGavN5XIFvow0X9s2Y2M?=
 =?us-ascii?Q?y829m8eKm3IXhvxC9FW0VXohev2dI7SSBgyEuwpSC9CBbvULY9iTvaVfF48T?=
 =?us-ascii?Q?qCUhOCuGaCb+vrsa9Ffdr3IiJb8RfLIkdxD+6z7q13nK2ebYrs+pj10V5amO?=
 =?us-ascii?Q?x9yLaPFQ44T0Lgphbig070u8Ifvvj24Mi+HHFYyUfINnAb/8cD7oby9ebWyk?=
 =?us-ascii?Q?bYrh9BVgmXoj8T9IPlSv8EtaNbgclbWBjoX0xiM8GCF90OkR1SFmQUlSGetg?=
 =?us-ascii?Q?2tZmd9wnh317lbOdjk7ij5Ua1wtMuLX+WNrLw1Q4xrTddciGSUbmrR+l6TvM?=
 =?us-ascii?Q?utml6s6mSIfnCq3iZy0sOJ1zS0H/tmF2wybfpQZe2x5rUHTvARv75nMH9BuN?=
 =?us-ascii?Q?1un4rjA4rGduniTzG0ti1Xod/lmvmVpIYggCoK+Rhx52JQ9NknQRV2PijOml?=
 =?us-ascii?Q?81ccWYAbyI5XWwcBIpFe5Ewkq8uhFcrDFuBVq9/8wf0WGt522e3Yn/p8NdB0?=
 =?us-ascii?Q?WvuvgbLFCvuzNw5aw9hCaNVVNGzXNeTdycO4A8bWpGJNlQvny4++5/ljxCbB?=
 =?us-ascii?Q?4v7DAgs/5nSlBHbXnIYjYQmx+R9VBeVmCn+MdOeFd3reWeFoSgABFSj1d1xp?=
 =?us-ascii?Q?/8tTp0G/MdA9tHucwoIFWFFplm7wKmPWMAzjjhzxPS0CrND/geDZW8c1qjQg?=
 =?us-ascii?Q?jq9cVbpyKs30B8PncQrvm3MyhTvGiH9Hk1qqa1Sm0RH1krdDI/6SvNR4KO2h?=
 =?us-ascii?Q?n7b5hSukA+YDlb4glT1+KIQ=3D?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 4b3a9ec8-bee2-4c50-c007-08dc3268a4a9
X-MS-Exchange-CrossTenant-AuthSource: VE1PR04MB7374.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 20 Feb 2024 23:07:00.1508
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: 4WBpNhSUux7URasLo0d82ozPt0Rbcmjul4VASuAh/u51SGkuVYJ/H6SaoJTCJMCKScZavuPeZdn9zxjtwlDIWg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PAXPR04MB9491

On Tue, Feb 20, 2024 at 05:52:36PM -0500, Sean Anderson wrote:
> With SGMII and XFI, the PCS sits on the MAC's MDIO bus. So for SGMII and
> XFI if we don't have any labels we can just assume the PCS handle is for
> the right PCS. But for QSGMII the PCSs sit on another MAC's MDIO bus. So
> we need to tell the MAC where to find the PCS. This means we need to
> supply multiple PCSs to the MAC

So how did the other Layerscape devices with the same SerDes, PCS and
mEMAC manage to get by and support QSGMII without listing all possible
PCSes in pcs-handle-names? :-/ DPAA2 has the exact same situation with
the QSGMII PCS situated on the internal bus of another DPMAC.

It is unnecessary and buggy complexity, and it will only have to become
worse when I add support for C73 backplane autoneg in lynx-pcs and the
fman_memac driver, because I will need yet another PCS handle, this time
not even one that represents a phy-mode in particular, but a PCS handle
for C73 (with C73, the autoneg process determines the dynamic phy-mode).