From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 7BC5ACAC5A5 for ; Wed, 24 Sep 2025 19:13:39 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:MIME-Version:Content-Type: In-Reply-To:From:References:CC:To:Subject:Date:Message-ID:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=u7A/HEB90qtE7/0sB+Et3dCMtkr7Qn2Bz4RNTcF9B0g=; b=SiwoRcsK18beExugQvhe1Zhwsb YEpyvqrXy+kAP+o2qfdkghRH9u4P2DnH7HTV1UzVtL9vrMmmVajjErFKHXx/0mMKOe5K3LM4qYs3q E75r+MiuxTSeB+AlyPYeb/pD1giAvoc0/L9fdX2gu1WSybvxiizWLNLNgxEJ7csxoOaZ2Cd8jV7nX GWUYy0vlycMfD8ESO2Pa855ubLViCrR3jpOlvLWTlcuhB6+JWW4ZwfHpzYiiNlZy/e/hMvm1u6y5I 9Az/NsKso8A0TVt/p/jvvWsqiWQD4I2WWiez1jFXy3IqZ/BDE18HO+l39Ygpm/tZ10V55SiNWM7bb msGamYqQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1v1Uvk-00000002xGk-2EzJ; Wed, 24 Sep 2025 19:13:32 +0000 Received: from mgamail.intel.com ([198.175.65.12]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1v1Uvi-00000002xF2-0ANY for linux-arm-kernel@lists.infradead.org; Wed, 24 Sep 2025 19:13:31 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1758741210; x=1790277210; h=message-id:date:subject:to:cc:references:from: in-reply-to:mime-version; bh=PGuUJ+n+OJdfLV5yTipwnMpEFIlzVd86ld1i4rnpkkM=; b=eFfwBroQoLGaVBzYK/F9w/5tCwbq+mRQ/qcHOClqrnrRl4+YsPK0vyWV A8D1aAiXe2XOrUprgxC1rjGfygfWMjMhqcnkuxaykBf1n5FGCEAiLVHW3 5Jizuw05ITMoBKB8USlTv7OA+0rjUjiYViTt6hq/pTPz1CBwe3e2wfzPO PNJfs+FohorGZkl1F66K28/FPPHN+uyaIBFC9O8wa4WpVARJ92eGmul2S Qs9oTJaaAstEOoarAOC5hPUYgPNto/uQKVfpLc90ClnZ+/fhA4hsZWGPR TaPYH1sMdL0oF8EBS4m26iSR5bQeLjBac/iBGv3k06BtmZX9M6QH2AwRu Q==; X-CSE-ConnectionGUID: IDNtvAvlTRS5KptZO1ZiKg== X-CSE-MsgGUID: w7ZvJILQRfWw5SGzQw4QJg== X-IronPort-AV: E=McAfee;i="6800,10657,11563"; a="72475422" X-IronPort-AV: E=Sophos;i="6.18,291,1751266800"; d="asc'?scan'208";a="72475422" Received: from orviesa008.jf.intel.com ([10.64.159.148]) by orvoesa104.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 24 Sep 2025 12:13:27 -0700 X-CSE-ConnectionGUID: PKX7ffxmTHerqfjfiv8U/g== X-CSE-MsgGUID: 4KppkykVTAmg84lGWrPDyg== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.18,291,1751266800"; d="asc'?scan'208";a="177190973" Received: from fmsmsx901.amr.corp.intel.com ([10.18.126.90]) by orviesa008.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 24 Sep 2025 12:13:27 -0700 Received: from FMSMSX902.amr.corp.intel.com (10.18.126.91) by fmsmsx901.amr.corp.intel.com (10.18.126.90) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.2562.27; Wed, 24 Sep 2025 12:13:25 -0700 Received: from fmsedg902.ED.cps.intel.com (10.1.192.144) by FMSMSX902.amr.corp.intel.com (10.18.126.91) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.2562.27 via Frontend Transport; Wed, 24 Sep 2025 12:13:25 -0700 Received: from CH5PR02CU005.outbound.protection.outlook.com (40.107.200.49) by edgegateway.intel.com (192.55.55.82) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.2562.17; Wed, 24 Sep 2025 12:13:25 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=dp11YnQHfSOisLJdXe+4qtHzV1QIIZVPFfOVr4rYm6CdtF8GNybCJFVejJ3me+6qUETTkKVBhNsnXOFkC18qqnuO69g9i7zfoabhfnajB4mKU8RMkg5uAiQKxI4vNBmKErR5LSluU2gGTxgdqgB8kxcCaBAi4EuJ6W7mtBPR5U6qVWupqW6k+qU3DcszUo/c5b/E5tLWv8ArWp9wol0QvDmMeHueviorIyPj6YCyBg8T32yEYOrE0DFFB1wO/8U7xMDYUtWC3BUXWP7irqdcrcsLP7CDN9eTG4fNCScm0MlHMvwhQ86t1CHxMuAmCGAwz9kfb1UkoqBraXVgp855/g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=u7A/HEB90qtE7/0sB+Et3dCMtkr7Qn2Bz4RNTcF9B0g=; b=Lzf7eq2sh7We+nAAx2S0P0qCZRvRJevmUzC1jgascIyFfHQT1Yhz6AEAdGsYUGGdnL/T/s4ulDNaOdJNxa1KqSam4QvgzyCK39EvkDyOcYA3wlr1bxgT2D+tSdokBFQdpC2n11EaQiQZkUxpezH2CtciYPkNCTHuVnzWOrR2cB3uYzagy765Bq0zuvDg6MHAauNtUrFyxjMOLezX8m4PZR260usegn5EQGaujqjc4CT7rdHvmC3nhMQ1gyKeGDpyHGmXeVasXekao39KO8BHk/JgTlTIzWHQCFsCwBsWHAJyYAx/RVLE5h2Fwy420KLVt9OeorMaoi6GSiVyShgrWg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=intel.com; dmarc=pass action=none header.from=intel.com; dkim=pass header.d=intel.com; arc=none Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=intel.com; Received: from CO1PR11MB5089.namprd11.prod.outlook.com (2603:10b6:303:9b::16) by IA3PR11MB9015.namprd11.prod.outlook.com (2603:10b6:208:57e::22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9137.19; Wed, 24 Sep 2025 19:13:22 +0000 Received: from CO1PR11MB5089.namprd11.prod.outlook.com ([fe80::81f7:c6c0:ca43:11c3]) by CO1PR11MB5089.namprd11.prod.outlook.com ([fe80::81f7:c6c0:ca43:11c3%4]) with mapi id 15.20.9137.018; Wed, 24 Sep 2025 19:13:20 +0000 Message-ID: Date: Wed, 24 Sep 2025 12:13:18 -0700 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH RFC net-next 0/9] net: stmmac: experimental PCS conversion To: "Russell King (Oracle)" , Andrew Lunn , Heiner Kallweit CC: Abhishek Chauhan , Alexandre Torgue , Alexis Lothore , Andrew Lunn , Boon Khai Ng , Choong Yong Liang , Daniel Machon , "David S. Miller" , "Drew Fustini" , Emil Renner Berthing , Eric Dumazet , Faizal Rahim , Furong Xu <0x1207@gmail.com>, Inochi Amaoto , Jakub Kicinski , "Jan Petrous (OSS)" , "Jisheng Zhang" , Kees Cook , Kunihiko Hayashi , Lad Prabhakar , Ley Foon Tan , , , , Matthew Gerlach , Maxime Chevallier , Maxime Coquelin , Michal Swiatkowski , , Oleksij Rempel , "Paolo Abeni" , Rohan G Thomas , Shenwei Wang , Simon Horman , "Song Yoong Siang" , Swathi K S , Tiezhu Yang , Vinod Koul , "Vladimir Oltean" , Vladimir Oltean , Yu-Chun Lin References: Content-Language: en-US From: Jacob Keller Autocrypt: addr=jacob.e.keller@intel.com; keydata= xjMEaFx9ShYJKwYBBAHaRw8BAQdAE+TQsi9s60VNWijGeBIKU6hsXLwMt/JY9ni1wnsVd7nN J0phY29iIEtlbGxlciA8amFjb2IuZS5rZWxsZXJAaW50ZWwuY29tPsKTBBMWCgA7FiEEIEBU qdczkFYq7EMeapZdPm8PKOgFAmhcfUoCGwMFCwkIBwICIgIGFQoJCAsCBBYCAwECHgcCF4AA CgkQapZdPm8PKOiZAAEA4UV0uM2PhFAw+tlK81gP+fgRqBVYlhmMyroXadv0lH4BAIf4jLxI UPEL4+zzp4ekaw8IyFz+mRMUBaS2l+cpoBUBzjgEaFx9ShIKKwYBBAGXVQEFAQEHQF386lYe MPZBiQHGXwjbBWS5OMBems5rgajcBMKc4W4aAwEIB8J4BBgWCgAgFiEEIEBUqdczkFYq7EMe apZdPm8PKOgFAmhcfUoCGwwACgkQapZdPm8PKOjbUQD+MsPBANqBUiNt+7w0dC73R6UcQzbg cFx4Yvms6cJjeD4BAKf193xbq7W3T7r9BdfTw6HRFYDiHXgkyoc/2Q4/T+8H In-Reply-To: Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="------------3HQBP092X4ZovoddKl4yq7jW" X-ClientProxiedBy: MW4PR04CA0323.namprd04.prod.outlook.com (2603:10b6:303:82::28) To CO1PR11MB5089.namprd11.prod.outlook.com (2603:10b6:303:9b::16) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: CO1PR11MB5089:EE_|IA3PR11MB9015:EE_ X-MS-Office365-Filtering-Correlation-Id: 3737b43e-8b8b-4cd0-1500-08ddfb9e6c9f X-LD-Processed: 46c98d88-e344-4ed4-8496-4ed7712e255d,ExtAddr X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|1800799024|376014|7416014|366016; X-Microsoft-Antispam-Message-Info: =?utf-8?B?WjlqejZHK0IxdHExQWE5YkdlQlEySFhvM0lwK3phcSswcTZkeUM1RWd0SHp1?= =?utf-8?B?cmtIa051aklzK2gxYWhkdVUwbHpSNDE1MmZCSHFvaER2aVZVWGY0YTBvZ0kx?= =?utf-8?B?am9YaGw2b0ZLUFdYaTlCbFM5dDU0ZlVsUG85Mk9hU1hSWWVyYVExRDN5R0pW?= =?utf-8?B?SWZKL2VVUmxxQVBDOU9kcVFyMWFJaXJ6MFlMdU9LOFo2Y0JkQm1zVk91QThB?= =?utf-8?B?RHczRzJUb1RrNjg2QlgrNis0VVAxWDcwZXFKUHpFWm1LRmliQkVrWGtaTmpw?= =?utf-8?B?TWZnSGxVSmdIOHZUc2pQWFhuS01WZ05FOUp5YXRDcGxkQ0ttOExFb0JiVEtr?= =?utf-8?B?TUN2cHF5eGoybWxzT1E1d0JnWjE1LzJUTjVEOEh6ek12ZHl2WVlydmRiN1BL?= =?utf-8?B?YnpUZEs2OVJjV2V5eUlQK2dQZGVWSjFNZ0pKR05xV01SVGJVTnpjR0FURzlG?= =?utf-8?B?TmdXSnpaLzlYZVNCd2l2SmpxdVRzT0o5OGVOa0VsQ2VIWWVLc2hKV3QzTEFt?= =?utf-8?B?ck45R2J3WWliNzNEOE5hNXB2UmYwM20xUjRNbS92c1pmUVBrZ2JsVmlsWGZU?= =?utf-8?B?TFRGRkVvS012aGhtNDNMczJCMkdVVzBBTzVSUG5EMUJXZTlsSGVtRzArWFRy?= =?utf-8?B?Y0VVdmF5SDBTV2R0TE50bjNpc1ZHeFVwUW10anNlSkl2YlJ5bEpSYzRFMG1F?= =?utf-8?B?SjJ5MlNPNE1rRVI3WWx2N2FFalc1Y3JaN2pydW5XRnlvcHNRcFpZTU5obmkx?= =?utf-8?B?dUw1YkN2VldmeWZoY3JydjFMM3g0bzFTZkNlaWM0cGNtU3NVd1BGY21WZXht?= =?utf-8?B?SkxpeXFSTHg1TXlwVSt4aTlTNGNFeUFhTmQyMjFRRmEveHZXbFN5WXVNMU5t?= =?utf-8?B?aVVLa2lPYnJ0c0FQcUMvWFo0UFRHRkFjQjNxSlB6TXFzSGNPOG5yQWxGQXQw?= =?utf-8?B?djU3NzNsQWFtSkZRdlhPb25vdGJmdzZMVklFTlUxWjRQRWhSSUE4bGN5dmgz?= =?utf-8?B?bzFYMWM5cytZQmR0N2tSaGR6SmxiTzAvcFZSWE41dUJDRUxyZVVMajRTQTdV?= =?utf-8?B?aHpDUDF0cVpkb21oQ3ZRclZqdnFwclFTd2tFNFNrRVd3bnIvemtsc3hRNit5?= =?utf-8?B?aWpFaDk5cU0vbnBLQ2czd2VOSU9MSU9lSUVON2lNZmNVQ3g1RVluSHRmR216?= =?utf-8?B?L1gzTTZ0SlY1NXNCSHBLeXBEZ1g5RVdzeDhSTFk5b3pWdkhIZjBzYVNZS2Rm?= =?utf-8?B?VmpFay84R1VHQ3lZUUFkTlJsRVRvZHBheFZKRWUvTDRRU0pkTE00WTRXUWl4?= =?utf-8?B?V1hBWHZ6dlJ0UjQ3cXRjN3FHMFhVNnNSWVduMS9xak55dVlIaXdlYnp1eUMr?= =?utf-8?B?LzFuUEV2VkNpS1JCc3hzOVA1STM3OEY2ZjJLYVpOdW4wNTQ1THQyeVR4SHZw?= =?utf-8?B?QVhwbFhrRnJSK3MzRVA0cEZVbzZuVDR2L2c5cnJUdTNOMGpkc1Q2c0V1R1FF?= =?utf-8?B?WlNrVXY4TWNrV0w2ZytRL1NjMStaQnFZRHRjbEthOHNiL0U5ZThFNUlZdU9v?= =?utf-8?B?Qk5YMGgzTWdJSElUd3pkd2JuOWFaWCthZ1U1cHRTdmN3N1pqYWxSbUhNU2U0?= =?utf-8?B?cFZjTmJsUTFNRVJiUW5pQlZ4cGRiQlVabkZlTlRESEVXamlmNHJOd0xMeHFT?= =?utf-8?B?M1prV0ZoMmRpSXBNZnZDKzk0T1dxdEZSVFo1Q1dONGJ4dzdKS1dYV1c5L085?= =?utf-8?B?c3hIYVRESzRHMUJodDVKNjJFbC9ESGxzbGNBQTV5R0NMbUE3U0Z3OEpoNjM1?= =?utf-8?B?TlB3d29hbEorSGNPYlI2SzZFNlBmdU1pb09jdlVFN3JKVHEzSzhuOGVHK2xx?= =?utf-8?B?NU81R1JVM1I2TlZORDRGWXVGdEFsb2ZLVno0TXdPYkt1eGpRZllXNUx1cS9P?= =?utf-8?Q?lJiMrK8OMR4=3D?= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:CO1PR11MB5089.namprd11.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(1800799024)(376014)(7416014)(366016);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?bmZFSTVTV0IyMXNQR2s4T3NUWmIxV01qZDZFK1NpVkUrYlh5S3RSMnYrL3ds?= =?utf-8?B?TnlkU204ek1GRWt1cFVaT1lUN3JnRnU1YTB5VzEwS0xURkF4UzZrQ2lKN1Iy?= =?utf-8?B?YjhnSjZ3bVVhdzVldG94ZW9uUXhOUUZYS2k0SndTeG51VE9hOTdFSXFqZTlF?= =?utf-8?B?eTVGdEsyM3hOaHBNQTNsOSt5NzM0cG54Ni9nWUdMNmtuRk5GRGNUdjF2TFkw?= =?utf-8?B?TkNSY29wSnVwUEgvWmN0MWlsV0U5bEhCMjdEVTlUMDAwZlVtWVNMWHorUGdW?= =?utf-8?B?dzR1VVExZXlQRWhKeHFqZ2RnMkQrOUhqUnlBSzBDL2RQcnlLOVR6SjJSZjdY?= =?utf-8?B?YThoemljSnRRMnVkTDF0MnIrM1VTZExMdHAwYXV0UzkzR05MWGtoaERMSEpa?= =?utf-8?B?aXdiUEJlUlVSWGpObExLOFVoeUlTZjVLR3pRK0xXc0tpSUl4UTdrRFBEUlFj?= =?utf-8?B?Y2VLYjIwSUU2cmNNNGNrTXYyaWIxVERBandUZU0xbDhMUVMzc2lYSkovSGZB?= =?utf-8?B?bWFmSHNvSWw0VDd6V003Uzg1cGhEQWpXYVAzSmc1dFV3bWRXT3NyS2VVcVN0?= =?utf-8?B?RmNWV1BDWnVVcFJveE16RXExTWY0OXd5SWtabS9JYXZxbU0zKzBmVFZpcm55?= =?utf-8?B?cThiQWlLdnk4NDRkcGdQUlJYVkxrSW1PK0FRa01qVDM2RUxqY2ZjZHJXeXBJ?= =?utf-8?B?UmxpZFd4Q0J4bjF3OGxyVVhsZjl5SkxQMHJWNWlLUzVFNDRWVUJpUXE4Q0JI?= =?utf-8?B?ZDdwUUE1REcvUURhTFBwcVI4WnEyeG1udE1TVnNBMkxkenNzK2JIUkVvTU1L?= =?utf-8?B?VmQvbGtQUDhxb3NvWGpBd3lWc0lKRWRHNGZFT084bW1YSjAzUDlkbkVqRFox?= =?utf-8?B?SmVZU3IxZE5hWGNMbFpmWHpQSWZiT3JoTC9uSzhtSFQzbjZXaGxta29Qd2ZD?= =?utf-8?B?MFRaS2FzRVQ1dXpBWDE1WUordjVHd0ZmZC90QUxYQlgyOGtvSTNVcWtScU14?= =?utf-8?B?SlkwZUlPcGRkb2JLNzlKRmpZMWFUQ0FRMVo1bVNsTExFM0pzTFUwUGVMcDRw?= =?utf-8?B?amg1SHpHUHREY1ZqKzBQZ0RMcWMrZXFEVXV3V3hrVk15Uk9JR2daWStYckhV?= =?utf-8?B?WkJwWkUrK29oSzdWZDRlelgzYUo5VDJZRkI5Unhoc2IvNHVvMkdhWmVScnBm?= =?utf-8?B?MGZrY0s4ZFlRcVZYdW1uTkpEWG1oUHMvSFlIK2RtMWJkYUI1WVBvVDhZZTlo?= =?utf-8?B?U2Zaa1cxSzdiVThocDBmcFN6Mk1TU0V2b2gzbm5ObmtqSGMrMTVQOThPckRB?= =?utf-8?B?SWFrY0NTbnU1aUdNbnZ3SXloT1p0a1czMG9TdENmMjF5emVVdENaVXNxT1pS?= =?utf-8?B?N2Juc1JTUlorWnNLalduTVpCdm9QUlVpV054UGJDaDM0dXhHNllSa2U1SU11?= =?utf-8?B?OUxMdWFKZHpOb2xEckdpc1dXbk9NR3ZTQnNOelBBQTRIOVAxNjhvZmZONVBv?= =?utf-8?B?OFpTTGpzdVIzNzd2WXVVbzVRZ3p5czVCYUlXOFJhbyt3c2ZQbXRWSkFIUnov?= =?utf-8?B?YzBLVGNnM284RUc5YlVQT1Q4VFFOMDFxcmROV0pvQmxKMm9Fdzc3R1c5YkV6?= =?utf-8?B?a2Roc0liem9nQ2hucWZKYXMwV0VoTzM3dloyYUdNZURMWk56UmM3bExJUTVs?= =?utf-8?B?aS9sYTluM2U0Lzh4Nm1KL2UxUG90SkhFV1NvUTNmUStxSEljbnR2N3NXYUFw?= =?utf-8?B?WmkraE43YVlVNENNTi85TGhpZkZjVmFSaGpUcWJ6VUxscDZoTk5MUnBNdytR?= =?utf-8?B?SDBwZDZYM0dsRVlCK0wxcHFyeUFVOWh6ek45ZUExRmNkUjh3MGtPS3B5QVJC?= =?utf-8?B?NTJwL3dPQzFWZE9pOC9TTTkrU1NycUY4M1JaSzBrNG03cisxTEZ1eU9ocVh1?= =?utf-8?B?V2I4SUYrLzA0VGIzMTM5anFHdjliWWs4NGlnTlVaekYwVnBPVEhiOXZseXFI?= =?utf-8?B?eHNEZ0ZYR0orNXdVTHJGSjhoSkg4ZzVzSWRxQlp0YzZGRDJ2SFlZZFRXV3ln?= =?utf-8?B?QUNpYy80L2ZxVGlSWkVmNXgrUVVXOCtUcHd2cm5sdVJmN2JqbUNqU2xZRzNG?= =?utf-8?B?V3RVY3JPdUxjcHVOa3lEelkvT3YvTTZqUFFkYmVGcGh4aFNRMzNxM1NQUHpl?= =?utf-8?B?ZUE9PQ==?= X-MS-Exchange-CrossTenant-Network-Message-Id: 3737b43e-8b8b-4cd0-1500-08ddfb9e6c9f X-MS-Exchange-CrossTenant-AuthSource: CO1PR11MB5089.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 24 Sep 2025 19:13:20.4050 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: WW1BG7h12UV8PUtZwlyAl1BJTL+FXDBo4D396rsls/6QINTjnSJdus5c70OF66fsQFIKRwGQPdN2+JGsOEL1dpcp3IdpxfdwQzLgz8r3JTI= X-MS-Exchange-Transport-CrossTenantHeadersStamped: IA3PR11MB9015 X-OriginatorOrg: intel.com X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20250924_121330_143735_85B479C2 X-CRM114-Status: UNSURE ( 6.40 ) X-CRM114-Notice: Please train this message. X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --------------3HQBP092X4ZovoddKl4yq7jW Content-Type: multipart/mixed; boundary="------------cezbLSVVCgFm4dw49RQteAst"; protected-headers="v1" From: Jacob Keller To: "Russell King (Oracle)" , Andrew Lunn , Heiner Kallweit Cc: Abhishek Chauhan , Alexandre Torgue , Alexis Lothore , Andrew Lunn , Boon Khai Ng , Choong Yong Liang , Daniel Machon , "David S. Miller" , Drew Fustini , Emil Renner Berthing , Eric Dumazet , Faizal Rahim , Furong Xu <0x1207@gmail.com>, Inochi Amaoto , Jakub Kicinski , "Jan Petrous (OSS)" , Jisheng Zhang , Kees Cook , Kunihiko Hayashi , Lad Prabhakar , Ley Foon Tan , linux-arm-kernel@lists.infradead.org, linux-arm-msm@vger.kernel.org, linux-stm32@st-md-mailman.stormreply.com, Matthew Gerlach , Maxime Chevallier , Maxime Coquelin , Michal Swiatkowski , netdev@vger.kernel.org, Oleksij Rempel , Paolo Abeni , Rohan G Thomas , Shenwei Wang , Simon Horman , Song Yoong Siang , Swathi K S , Tiezhu Yang , Vinod Koul , Vladimir Oltean , Vladimir Oltean , Yu-Chun Lin Message-ID: Subject: Re: [PATCH RFC net-next 0/9] net: stmmac: experimental PCS conversion References: In-Reply-To: --------------cezbLSVVCgFm4dw49RQteAst Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable On 9/24/2025 11:17 AM, Russell King (Oracle) wrote: > This series is radical - it takes the brave step of ripping out much of= > the existing PCS support code and throwing it all away. >=20 > I have discussed the introduction of the STMMAC_FLAG_HAS_INTEGRATED_PCS= > flag with Bartosz Golaszewski, and the conclusion I came to is that > this is to workaround the breakage that I've been going on about > concerning the phylink conversion for the last five or six years. >=20 > The problem is that the stmmac PCS code manipulates the netif carrier > state, which confuses phylink. >=20 > There is a way of testing this out on the Jetson Xavier NX platform as > the "PCS" code paths can be exercised while in RGMII mode - because > RGMII also has in-band status and the status register is shared with > SGMII. Testing this out confirms my long held theory: the interrupt > handler manipulates the netif carrier state before phylink gets a > look-in, which means that the mac_link_up() and mac_link_down() methods= > are never called, resulting in the device being non-functional. >=20 > Moreover, on dwmac4 cores, ethtool reports incorrect information - > despite having a full-duplex link, ethtool reports that it is > half-dupex. >=20 > Thus, this code is completely broken - anyone using it will not have > a functional platform, and thus it doesn't deserve to live any longer, > especially as it's a thorn in phylink. >=20 > Rip all this out, leaving just the bare bones initialisation in place. >=20 > However, this is not the last of what's broken. We have this hw->ps > integer which is really not descriptive, and the DT property from > which it comes from does little to help understand what's going on. > Putting all the clues together: >=20 > - early configuration of the GMAC configuration register for the > speed. > - setting the SGMII rate adapter layer to take its speed from the > GMAC configuration register. >=20 > Lastly, setting the transmit enable (TE) bit, which is a typo that puts= > the nail in the coffin of this code. It should be the transmit > configuration (TC) bit. Given that when the link comes up, phylink > will call mac_link_up() which will overwrite the speed in the GMAC > configuration register, the only part of this that is functional is > changing where the SGMII rate adapter layer gets its speed from, > which is a boolean. >=20 > From what I've found so far, everyone who sets the snps,ps-speed > property which configures this mode also configures a fixed link, > so the pre-configuration is unnecessary - the link will come up > anyway. >=20 > So, this series rips that out the preconfiguration as well, and > replaces hw->ps with a boolean hw->reverse_sgmii_enable flag. >=20 > We then move the sole PCS configuration into a phylink_pcs instance, > which configures the PCS control register in the same way as is done > during the probe function. >=20 > Thus, we end up with much easier and simpler conversion to phylink PCS > than previous attempts. >=20 > Even so, this still results in inband mode always being enabled at the > moment in the new .pcs_config() method to reflect what the probe > function was doing. The next stage will be to change that to allow > phylink to correctly configure the PCS. This needs fixing to allow > platform glue maintainers who are currently blocked to progress. >=20 > Please note, however, that this has not been tested with any SGMII > platform. >=20 > I've tried to get as many people into the Cc list with get_maintainers,= > I hope that's sufficient to get enough eyeballs on this. >=20 I'm no expert with this hardware or driver, but all of your explanations seem reasonable to me. I'd guess the real step is to try and get this tested against the variety of hardware supported by stmmac? --------------cezbLSVVCgFm4dw49RQteAst-- --------------3HQBP092X4ZovoddKl4yq7jW Content-Type: application/pgp-signature; name="OpenPGP_signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="OpenPGP_signature.asc" -----BEGIN PGP SIGNATURE----- wnsEABYIACMWIQQgQFSp1zOQVirsQx5qll0+bw8o6AUCaNRCzgUDAAAAAAAKCRBqll0+bw8o6LxF AP9YN3Ecux9wX5jJjt/lyf1j3o6ZfCoAkE5o3M1fMh63JQEA5LOANQtz9afAnbwcWnuisaWLkgPE emFvkRCcOjTx1gE= =6nXV -----END PGP SIGNATURE----- --------------3HQBP092X4ZovoddKl4yq7jW--