From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from pidgin.makrotopia.org (pidgin.makrotopia.org [185.142.180.65]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id D7A2A21638A for ; Mon, 7 Apr 2025 17:26:27 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=185.142.180.65 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1744046789; cv=none; b=HEpH+dJcAK8chr6LmmLm7SC4IK+rPy8clAmNdnZKxPwWPXOeIqGdZ1R3/fiAHaAv6NN7Rw5f+eduDRXxP3MYRVR3/kUOXP7C3A1YLn+6vWzRZCU4WuFnjR/TKJBRXEzFpLYlEQC4e6SRMAi5dkW6WUf/nRe3gy9HKoztyoE1mNw= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1744046789; c=relaxed/simple; bh=qCXjZbYfOUQdvaJOulkLaljFnMlS3JxgtFCl65KJ/WA=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=golQ+xx/8+IykE0p+JnwBs0BbIcwAS3OuGddjlPmoWNC261yZfTEtvG7lFqCIH4OuYoSHsEk8MNC8VyFkV3Au34DPzx15OfU6WbTAOowpov9+jjJXVbpeKGKFLFqHsQMmQN3BXNL7jz3aDivmBd/QH2xBH7YbwNVJvYXVGsjGZI= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=makrotopia.org; spf=pass smtp.mailfrom=makrotopia.org; arc=none smtp.client-ip=185.142.180.65 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=makrotopia.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=makrotopia.org Received: from local by pidgin.makrotopia.org with esmtpsa (TLS1.3:TLS_AES_256_GCM_SHA384:256) (Exim 4.98.2) (envelope-from ) id 1u1qEH-000000008Gs-2MWH; Mon, 07 Apr 2025 17:25:49 +0000 Date: Mon, 7 Apr 2025 18:25:43 +0100 From: Daniel Golle To: "Christian Marangi (Ansuel)" Cc: Sean Anderson , Kory Maincent , netdev@vger.kernel.org, Andrew Lunn , "David S . Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Russell King , linux-kernel@vger.kernel.org, upstream@airoha.com, Heiner Kallweit , Alexandre Belloni , Alexandre Torgue , Christophe Leroy , Clark Wang , Claudiu Beznea , Claudiu Manoil , Conor Dooley , Ioana Ciornei , Jonathan Corbet , Joyce Ooi , Krzysztof Kozlowski , Krzysztof Kozlowski , Li Yang , Madalin Bucur , Madhavan Srinivasan , Maxime Coquelin , Michael Ellerman , Michal Simek , Naveen N Rao , Nicholas Piggin , Nicolas Ferre , Radhey Shyam Pandey , Rob Herring , Rob Herring , Robert Hancock , Saravana Kannan , Shawn Guo , UNGLinuxDriver@microchip.com, Vladimir Oltean , Wei Fang , devicetree@vger.kernel.org, imx@lists.linux.dev, linux-arm-kernel@lists.infradead.org, linux-doc@vger.kernel.org, linux-stm32@st-md-mailman.stormreply.com, linuxppc-dev@lists.ozlabs.org Subject: Re: [RFC net-next PATCH 00/13] Add PCS core support Message-ID: References: <20250403181907.1947517-1-sean.anderson@linux.dev> <20250407182738.498d96b0@kmaincent-XPS-13-7390> <720b6db8-49c5-47e7-98da-f044fc38fc1a@linux.dev> <1aec6dab-ed03-4ca3-8cd1-9cfbb807be10@linux.dev> Precedence: bulk X-Mailing-List: imx@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: On Mon, Apr 07, 2025 at 07:21:38PM +0200, Christian Marangi (Ansuel) wrote: > Il giorno lun 7 apr 2025 alle ore 19:00 Sean Anderson > > I agree that a "cells" approach would require this, but > > > > - There are no in-tree examples of where this is necessary > > - I think this would be easy to add when necessary > > > > There are no in-tree cause only now we are starting to support > complex configuration with multiple PCS placed outside the MAC. > > I feel it's better to define a standard API for them now before > we permit even more MAC driver to implement custom property > and have to address tons of workaround for compatibility. Qualcomm's PCS driver will require offering multiple phylink_pcs by a single device/of_node. So while it's true that there is currently no in-tree user for that, that very user is already knocking on our doors. See https://patchwork.kernel.org/project/netdevbpf/list/?series=931658&state=*