Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <44b3d722-b07d-c924-6a71-4e0f912eab61@intel.com>
Date:   Tue, 17 Jan 2023 17:07:49 -0800
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Thunderbird/102.6.1
Subject: Re: [PATCH net-next 5/6] net: phy: Decide on C45 capabilities based
 on presence of method
Content-Language: en-US
To:     Michael Walle <michael@walle.cc>,
        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>, Felix Fietkau <nbd@nbd.name>,
        John Crispin <john@phrozen.org>,
        Sean Wang <sean.wang@mediatek.com>,
        Mark Lee <Mark-MC.Lee@mediatek.com>,
        "Lorenzo Bianconi" <lorenzo@kernel.org>,
        Matthias Brugger <matthias.bgg@gmail.com>,
        Bryan Whitehead <bryan.whitehead@microchip.com>,
        <UNGLinuxDriver@microchip.com>,
        Giuseppe Cavallaro <peppe.cavallaro@st.com>,
        Alexandre Torgue <alexandre.torgue@foss.st.com>,
        Jose Abreu <joabreu@synopsys.com>,
        Maxime Coquelin <mcoquelin.stm32@gmail.com>,
        "Joel Stanley" <joel@jms.id.au>, Andrew Jeffery <andrew@aj.id.au>
CC:     <netdev@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
        <linux-arm-kernel@lists.infradead.org>,
        <linux-mediatek@lists.infradead.org>,
        <linux-stm32@st-md-mailman.stormreply.com>,
        <linux-aspeed@lists.ozlabs.org>, Andrew Lunn <andrew@lunn.ch>
References: <20230116-net-next-remove-probe-capabilities-v1-0-5aa29738a023@walle.cc>
 <20230116-net-next-remove-probe-capabilities-v1-5-5aa29738a023@walle.cc>
From:   Jesse Brandeburg <jesse.brandeburg@intel.com>
In-Reply-To: <20230116-net-next-remove-probe-capabilities-v1-5-5aa29738a023@walle.cc>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?cFdHVXk0MklQdHRiT3VOMXRldmhLWFA4N083VTdsMnJHN24zNHdUWlJoTlZQ?=
 =?utf-8?B?TEd0T2QvRGl2ZGQ3NWM4WWdnbHVIcUgrOUtidWxXZGFLOTZxV3M1ZmFlMExh?=
 =?utf-8?B?NVpTZmEyak5RbGNSUlJOZUpGSW0vczB0ZEg5dU82UTMyTElsSXNkMkpLRGY4?=
 =?utf-8?B?a0E1dHliZkxVV3JPV1g0cVowNzZzb2RocG5DK09qRXhpZFpmLzR1enNUa3R2?=
 =?utf-8?B?alo0TXhKekVXS0VRQ0Zud2RoaWhoY09jd3ZmNnlzS3I2eEJLOWZBSW9NbUM5?=
 =?utf-8?B?Vmd4elZhU3BQa2MxNy9UNGcva1VHc2JrbzFNYlpLSHVJN3J2dXBldDZxcFpL?=
 =?utf-8?B?SkVWY3MxRm1WRzlzZzVLMDYrOTZsWGlpbjhmYStVTm9raGhuemRLMGtBeHds?=
 =?utf-8?B?ZjNHcjNSNVFDNnBXQncwaW9VRXNiMmRreXk3Mm1BSkYvT1BoL2tkTkpBN24y?=
 =?utf-8?B?a1A5c0pCN0tBY0lGVC9oUk1Na2lVVnRnZnVhNHZ6SzNzQVJvb3FhdkJwVmhl?=
 =?utf-8?B?MXNpL2dpUXphTzlzWStkV1ovMWc3cHZTUEpncW1lMjRLT1EzSHZ6Q2ViMTNp?=
 =?utf-8?B?K1U4VFdRUXYybW5vazVmQ0pxeFR6cERsU1pkOVREd1J6cnVxWW92eDMyWXlX?=
 =?utf-8?B?M1k4aUtWdERnclhZbmxTQ2lSN0dKeHFrTThGR29CUHd4dUMycXJ2eEp6dE5N?=
 =?utf-8?B?Njl2N05WUGZLY2F0MWxSUjB3L01ERFdzczhYVGN3Mi9XRG1iQ0hpYzRDSCtG?=
 =?utf-8?B?SDJGejlzZEhVNUh1bEUzM21IOW5HZE1aZFYrclJydC9LdHBSUHlvNFFBTUFR?=
 =?utf-8?B?OTE3U2JqTkdDVnRkV3VLYzh0Y0Y0WFZaSEZoYkMrV3lCd0NYcE1hNnVQeklM?=
 =?utf-8?B?VC80bnF5Y3AwcVY1Znk1dE9wcDduckcvSi9GemNJajJtUFlFQms3cGtuWUto?=
 =?utf-8?B?SCtQUnlXY0c1N05hbUlpRDMxRmM4YTRvUDd3aVcrUWpzai8rbW5GRkNtRmFk?=
 =?utf-8?B?MTUxYWtwNDJ3UkM3NzlNcUYyMEx1bW84OS84Wm92bTdUY1FWRkNxRWpaTUw4?=
 =?utf-8?B?YmJxWEJZUFNzL3pqbURHakU4bTJjc25waFBBRHl4VnRtQnozSyt5a0hjTTZI?=
 =?utf-8?B?di9yUGNoRmwxRW1BSEtUeGsvenhHN1dpZWYyWjJWT24zWWd2WDFOamlyOWFl?=
 =?utf-8?B?ZnZGTGVWamEzL1Fkc20rRDhsbUFpZk5Ga0VUWlJKQ0xLb3ZzWVhkOEEwRUpS?=
 =?utf-8?B?SXlmeWg4NUt2NWk3QXRva3BQZnpwOC9sNWt4aUdHV3l6WWtZaG9CalkwUFFn?=
 =?utf-8?B?WlQxc05UMWpuQnIvc25Ua2REa09ZaUptVFdOOG1IbUc2NGVQYzI4RzV4OWRh?=
 =?utf-8?B?YXF5QzcwTHIrRmpJRVRvSm5jNXRaZGhSaXlFQWxDK05vdUJLdWdUZ0RacEpN?=
 =?utf-8?B?alVLNHVWYjFzVmJMNjhpSkZxcmNZQ2xWTmN3cmpCMmpoV1A2ZkYzRWlRWTkv?=
 =?utf-8?B?eVl5clEvYnZrTzA5YkxoeUc2Vlg0UmI5bEsxdUc3MUJCM3orMmdzOXJndzl5?=
 =?utf-8?B?UTYreUtKNzFGYWdtUE1ZaGV4T09VLy9VNFd2Z2JGNlRrR3dPUm9tMG5BTlRI?=
 =?utf-8?B?NHpXOVNsb2gycTdTUkRoQlJQc094N1lnb2Y5VVhUYUoyZytsY3I4MTZCMUJt?=
 =?utf-8?B?ak9xL3Z0cFV0YUIrdEtZbEZXTXU2RkZZM3N5MzZUa1hRM0NIQ0tzV1lPZVox?=
 =?utf-8?B?VGMxaVBRaDllVC9qellvTk43MDM3UTc1U096VWgwaWczbkRRRlpMNWQyRlk4?=
 =?utf-8?B?R0gwamt5MEVGRXpQZG5rZk9iS08yMm1RVXpRUVphZHBEcHo2eDk3a0pqbXFj?=
 =?utf-8?B?eWZaMU5keGxSeXBpcmhBSEppVUlaNThYQVllL1RMcnFidkRWckFrYUtzWTI0?=
 =?utf-8?B?dzN4RjE2UGg1VHZLb000TldLcXJQYnFVUW9HaDdoU0lTNkM0TEhObDBneDZD?=
 =?utf-8?B?bnN2OHowK0kxQkJWQWtSR3NCZGd2ZUpGVXgxOGRvZWpFbWprZUFISHh4dzdP?=
 =?utf-8?B?cUJLMk5iQXh5ejhpNVkxTFdSNm9oMU9HVUlMbm1LaDhNakpqemRVV21MVXB1?=
 =?utf-8?B?VFF2WXd1anBWN0I5azFkTFFySWo3Z3BwQllJTU1kSUNNT2N0akhtTWtXbE13?=
 =?utf-8?B?RVE9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 3678f157-294a-4ad4-7e4a-08daf8f06c05
X-MS-Exchange-CrossTenant-AuthSource: CO1PR11MB4914.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 18 Jan 2023 01:07:51.6203
 (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: H27whYQ0fVXbTMN3xqmtaIYyKBmLPREKb2XpobfTia8xIZvyLNxkHDUtIXLlVRhVgdcpm6asmA8TYDKYkuohNcHl8DQis0w2jxuH0JlUGpk=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BY1PR11MB8126
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-4.5 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,
        RCVD_IN_DNSWL_MED,SPF_HELO_NONE,SPF_NONE 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

On 1/16/2023 4:55 AM, Michael Walle wrote:
> From: Andrew Lunn <andrew@lunn.ch>
> 
> Some PHYs provide invalid IDs in C22 space. If C45 is supported on the
> bus an attempt can be made to get the IDs from the C45 space. Decide
> on this based on the presence of the C45 read method in the bus
> structure. This will allow the unreliable probe_capabilities to be
> removed.
> 
> Signed-off-by: Andrew Lunn <andrew@lunn.ch>
> Signed-off-by: Michael Walle <michael@walle.cc>

Reviewed-by: Jesse Brandeburg <jesse.brandeburg@intel.com>