Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.34 as permitted sender) client-ip=23.128.96.34;
From:   Vladimir Oltean <vladimir.oltean@nxp.com>
To:     netdev@vger.kernel.org, devicetree@vger.kernel.org,
        linux-kernel@vger.kernel.org, linux-phy@lists.infradead.org
Cc:     "Russell King (Oracle)" <rmk+kernel@armlinux.org.uk>,
        Heiner Kallweit <hkallweit1@gmail.com>,
        Andrew Lunn <andrew@lunn.ch>,
        Florian Fainelli <f.fainelli@gmail.com>,
        Madalin Bucur <madalin.bucur@nxp.com>,
        Ioana Ciornei <ioana.ciornei@nxp.com>,
        Camelia Groza <camelia.groza@nxp.com>,
        Li Yang <leoyang.li@nxp.com>, Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Conor Dooley <conor@kernel.org>,
        Sean Anderson <sean.anderson@seco.com>,
        Maxime Chevallier <maxime.chevallier@bootlin.com>,
        Vinod Koul <vkoul@kernel.org>,
        Kishon Vijay Abraham I <kishon@kernel.org>
Subject: [RFC PATCH v2 net-next 10/15] net: phylink: support all ethtool port modes with inband modes
Date:   Sat, 23 Sep 2023 16:48:59 +0300
Message-Id: <20230923134904.3627402-11-vladimir.oltean@nxp.com>
In-Reply-To: <20230923134904.3627402-1-vladimir.oltean@nxp.com>
References: <20230923134904.3627402-1-vladimir.oltean@nxp.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?MAoeq/OoZgtICxCbhF2Rmx0lgXWWUbgkYK7W8liZQprq/RlYrmV0zpUOUsmW?=
 =?us-ascii?Q?QohXKxx0WIKe3O/8H9yqRy16vldE39ZxgW+/AfQjVfv5UYM3DrMm0q25q6gt?=
 =?us-ascii?Q?zgbN4+BeM+Y7JytAPpUw574bx5nZu5HloGNyaCeqLbBY00D0GgFgvnesh9VE?=
 =?us-ascii?Q?aE4iKiWJfOwkwFSUZZXU7sXhQisDRcKXzdpwo8BO6QGAIxZOtNem9mgjb94R?=
 =?us-ascii?Q?f91j9ONEcL0W1K9I+fYLvy/yX/6+iSVlWE9CQkmRNWGbyLhGc+TkwBG65qB7?=
 =?us-ascii?Q?LDN2qPGW3t9VZdnJm6afhvcwXGToM29cF9JT7xZlw3QsMlyskQ6Z3ykF9+F3?=
 =?us-ascii?Q?5Eb+sCEeYzmzNRAXUEnracczOAo73W8eUglobbMUKU63pBNZWmTSJ7kvqvWM?=
 =?us-ascii?Q?umq1lvP+DMaYW8EXd4imzapPMWFl8MpSUkwcAb6u+LDz4R5e9ERtRkFMkWXN?=
 =?us-ascii?Q?GOUDcI9Ul42GlHn4BRcP2leR6Udneh0vXgpeV8Sd44J6BRH59i4etKyFJDKH?=
 =?us-ascii?Q?YYtGNZTs2u+7Y1pOyKYYQKO49C7fd2/0GzWRidv3QNf4bEF0vbjsRQzz9vyE?=
 =?us-ascii?Q?gyqpE6zLvyENReaFoFpJDws7ReXvo1G7Qy3psXkHd25NhuiFzEv+lHDGA4wv?=
 =?us-ascii?Q?unhA/K57CaKTpr2ZKriLQVlF0nkIYYVJHuAbTUYK2fnSa6AkaVw7V/7IZox3?=
 =?us-ascii?Q?iVd9Bc9/ImEywIZgJTfwkBx0RzZhpNPz+jcLJ24AxBwSd+iC4tXP56w5iX0n?=
 =?us-ascii?Q?JFjdqSCYQXDLpZIcZk2lCpttjPrlktVnCV5Dj8pwRFqGOFEx0MeekWg/KW/L?=
 =?us-ascii?Q?9Aupjuo4JwqmbGxq+hVdPmV6wfqDNurW3EJu9dT+vcfnmUEurdcLl4FUoAAj?=
 =?us-ascii?Q?pmiCMuIlBzdgPej7RNXHW7Y3Ul89GQX53p1Ua5K1mFz+X6VRiJxyKa3q9P5o?=
 =?us-ascii?Q?MKPyDdLcqWJuax5N48OkvGVzigFmdIAFVTXW5FLKEoTdQ3knWtgDs04X33m7?=
 =?us-ascii?Q?V0dMdivi5aN4llPlMPmfu7lBdST5oAEzA97gs/3e3LxOlGB8f0AOJ5LoDy0/?=
 =?us-ascii?Q?nBzV2tlV4Gscw3dUkHo+4li98rdUuTFx0KStrmv3NTgMTunQYNLb+BQYr80u?=
 =?us-ascii?Q?gGOMIpy7g8hH3aXZ5VJ89q2LyZHJzrAjPr/s2svEA/wz4WRWiGA9ZauoSk73?=
 =?us-ascii?Q?84IEwKslyb0apNrK4IRAE5PXp6t+NGLDddJmsC1AGKr6PhzSmKqrhVfBMriQ?=
 =?us-ascii?Q?ZMbcJVI344G7tFxpyj8WN2UmTkimPRRmakkTXKwqo0qH0V1PhgXMkeFufO2H?=
 =?us-ascii?Q?mqkQlhTBFdLFcS+TS0FF/bLbhxnm8PmHR35n6I5O+n3nQBMtuHA1npZZYaB3?=
 =?us-ascii?Q?Ke4V8jq67Yt0SYMt7By7jXZNRX1hHVBTioszzqHESHd1AFQ8Tv6BTp2eUvbH?=
 =?us-ascii?Q?VVw+qZx3yviG17jJonALZCzRRzVazVtCbux2X18MkEea36Oo1vsjcBidv/hF?=
 =?us-ascii?Q?Gk19z64wHSmD1FbEtAGXAv3nNSgbySHLWDv0IDbEI55I8cM2nM9as9Qv/2KJ?=
 =?us-ascii?Q?FwQFNlL5yJ1Ls89W68g9H8NvQ2iViWO3SSBLsSs6qlDsWuHl3+E1PhOWr3fw?=
 =?us-ascii?Q?HQ=3D=3D?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: d32c1671-df61-4eb9-3fdd-08dbbc3beb61
X-MS-Exchange-CrossTenant-AuthSource: AM0PR04MB6452.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 23 Sep 2023 13:49:34.2688
 (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: m4W2km6SRx4VPXYBATJ8UPAsGXSfUv5Ae6ghCuytEvKXLSwbuavVyf7UONE/44K5a9nat9R8/fc0KvhzakweWw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS1PR04MB9454
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_BLOCKED,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS autolearn=ham
        autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (howler.vger.email [0.0.0.0]); Sat, 23 Sep 2023 06:50:20 -0700 (PDT)

Currently, phylink_parse_mode() filters out all port modes except
ETHTOOL_LINK_MODE_MII_BIT for in-band link modes. But if we're going to
accept backplane port modes with inband autoneg, then we should also
allow ETHTOOL_LINK_MODE_Backplane_BIT.

Use phylink_set_port_modes() to just allow them all, and let the
validate() functions of the MAC and phylink_pcs restrict what they don't
support.

Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
---
v1->v2: patch is new

 drivers/net/phy/phylink.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/phy/phylink.c b/drivers/net/phy/phylink.c
index 88ace7e203c3..6415c7b91053 100644
--- a/drivers/net/phy/phylink.c
+++ b/drivers/net/phy/phylink.c
@@ -880,7 +880,7 @@ static int phylink_parse_mode(struct phylink *pl,
 		}
 
 		linkmode_zero(pl->supported);
-		phylink_set(pl->supported, MII);
+		phylink_set_port_modes(pl->supported);
 		phylink_set(pl->supported, Autoneg);
 		phylink_set(pl->supported, Asym_Pause);
 		phylink_set(pl->supported, Pause);
-- 
2.34.1