From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ed1-f43.google.com (mail-ed1-f43.google.com [209.85.208.43]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 6A43E1E1322; Tue, 7 Jan 2025 13:14:07 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.208.43 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1736255649; cv=none; b=d/rq+iNNss88gCGez7FehMOWl3O8tvpqFCSDwoBx4l4Es4EqqCP1mFpXl1EEVVJXrlpfT/9jJGBOHUFbnL6qagFbrjaSMJtFoVzrgldiEojo16DgDmMJd188wcnAdFEuf4IUVVtBH4S7eUEG47qI2GWxghIJjxNpfhXhGrZOSgg= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1736255649; c=relaxed/simple; bh=dEXv7Cj+BC57a/C4F+5/2Rm0FhmxI6RRXkVRjhyCXkY=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=d+InHA7ia1noUitcidRoVrwHNQ8yWdfduTxkyjcSqSI8XVnlrMjXbjPKP3hkYsfPhVhNvKkh2HRIWK1062SJf0siimvunSXmY0o+FDD5fHuCRgazfkepb9Q6nf/7rntiV7ZL2mr53yCMQ3vfQN6hlvDYllV1LKcEZ5Y/64MxWEc= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=nB6SqPxp; arc=none smtp.client-ip=209.85.208.43 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="nB6SqPxp" Received: by mail-ed1-f43.google.com with SMTP id 4fb4d7f45d1cf-5d0d32cd31aso23126420a12.0; Tue, 07 Jan 2025 05:14:07 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1736255646; x=1736860446; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:content-language:from :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=TaKbNQkeyDwSjQGIQ+m5SGuLh75ZMMIa7nlAuxNwI4s=; b=nB6SqPxpCWt77xF3nYeG3JLsw7BStLJUFLe8ZM2p5PvTcIDo2pPSwnCvM61J/1PwQ1 CMNhTaRar74kN3gH7sjOrrfZVs2QcC/z43oLvGqjAQ/PrQBPIcI/hiNcZBoTIDuYuqNL ar+Bj4hEGpRP4+ATti7CJ1CfG8lCDUjjxxaZLhDs3UIa2CgVIhUWFguqmOjt1FYe/aOA Swqk7nH7QnGWv69+BkTIjlUbT3Kc7Vp5gE5Ie9mtdZ4LbDqhNTmPK2+fWBkKHwWdZkUe bQagC6bjZrTw5IQhwg6se26Xi/YPVqXwaa6mOt1o9g2Yk9hKRtxOpJd1mv0t+YS9ShHM aHTg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1736255646; x=1736860446; h=content-transfer-encoding:in-reply-to:content-language:from :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=TaKbNQkeyDwSjQGIQ+m5SGuLh75ZMMIa7nlAuxNwI4s=; b=NEubKqb/4qS0kpELCQ+mYjKiB0T+pElt8TZf1aG8/+0MDDtLeLZ9VYpiukawWiRg1i LExy49dSehA77K1Qm4rQ778Shc6xlKE83NWQ5tKX9z1kzq1Fyl7c2TdYFSO5h5I1Z3Sh m7s+kLDAbBQiHVfi4HFgMbKvqCgeEOge6xDXYkkqg1xQdnEeOmb8/ZgsznPMB56WLVDs 2WkZZpiBBaHwZNHwWTwkf9f3kWslmY5LBa+Hor045RtkJgNsx6c0+BfAEq/XHq+hX5qG 5Vo1KscmYRz4AyF9J+ZVzPA0EEJ6JfRC7ZZVazjeK462W6NCyk63ZTuGAxiHDaanBbMU ll/g== X-Forwarded-Encrypted: i=1; AJvYcCVVgg7mB+/78FX51lnp7PhuCLNHDbpXmWnlZnpxmp64RyxnPe6LESbwJrp3esyhDofG9b7AjSO6KBJcjjs=@vger.kernel.org, AJvYcCWSgfFSf7CXBRcPKgOC2mGrS99rOnZNGPLYfRpunnaKlXPtDzU2lplPjxEhXWtei25vLRN7GsHuLTBxnuw+iBiB@vger.kernel.org, AJvYcCXqow2VRiwYaSDJ1Vme9iM9wNLMSBM7BdP0vgNpjif4MAF7y97roALb2KXKoOyuJyM2694kDB2x@vger.kernel.org X-Gm-Message-State: AOJu0Yzxlj+BYIFI0UIojbUgGbXIxVvFMzlJd+9gYikLBtpBdpWsvhXk nO2KU7slmlxLUjSubRePlr4q9xu1Y5C2cx/4tubawaa4Bu+1RvMZ X-Gm-Gg: ASbGncus0AKxXxrdy8zA3yxK62djyuOKAIR1XDPlsskHT860IqC3D1+TEjC1Dr3YoN4 lrBdvcgyyfTWpxwQdpGH8mqfnx4MS8qVBN42t7Yvtrukk2b5cnS7DKoj4YGBuH9QFfzHHN9X/TH mnwRFXhNtdF6dZUs/iPWmnUNUZXaskM/F8UgRdYLgh3/zhfDQekarA/E2v4zLiLxQwYpjIIb2mH MqToJ9zEcFG/IDPw82+xixTDgjAEgvB9Gv1BCwv3WeSi6XcGcAdO/QLW7DUuY2bUlFXKMDsv6yV 5WYH/CouoTJEJssIfBXNVIwr1xxPLnHedT8C3tc21kxsDAZaUOn5L4vFN+O47fGwd6Q8V5y82rc EdsZBBF1b4V3y/PcljRaz01lB0KHJUGo= X-Google-Smtp-Source: AGHT+IG685u/TuxRo4gOyPFhExD9aC7tMDskOonBEuq2S9e6D8OwVVwO7wNmsSRZ+5auRMu6yPKriA== X-Received: by 2002:a05:6402:35c2:b0:5d0:ea4f:972f with SMTP id 4fb4d7f45d1cf-5d81dd9af30mr134224547a12.8.1736255645405; Tue, 07 Jan 2025 05:14:05 -0800 (PST) Received: from ?IPV6:2001:1c00:20d:1300:1b1c:4449:176a:89ea? (2001-1c00-020d-1300-1b1c-4449-176a-89ea.cable.dynamic.v6.ziggo.nl. [2001:1c00:20d:1300:1b1c:4449:176a:89ea]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-aac0f015b1asm2383448366b.148.2025.01.07.05.14.04 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 07 Jan 2025 05:14:05 -0800 (PST) Message-ID: <4b9b2a9a-061b-43ad-b402-a49aee317f41@gmail.com> Date: Tue, 7 Jan 2025 14:14:03 +0100 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH RFC net-next] net: phylink: always config mac for (delayed) phy To: "Russell King (Oracle)" Cc: Andrew Lunn , Heiner Kallweit , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Matthias Brugger , AngeloGioacchino Del Regno , Frank Wunderlich , Daniel Golle , netdev@vger.kernel.org, linux-kernel@vger.kernel.org, netfilter-devel@vger.kernel.org, coreteam@netfilter.org, bridge@lists.linux.dev, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org References: <20250107123615.161095-1-ericwouds@gmail.com> From: Eric Woudstra Content-Language: en-US In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit On 1/7/25 1:47 PM, Russell King (Oracle) wrote: > Going through the log... > > On Tue, Jan 07, 2025 at 01:36:15PM +0100, Eric Woudstra wrote: >> Log before this patch is applied: >> [root@bpir3 ~]# dmesg | grep eth1 >> [ 2.515179] mtk_soc_eth 15100000.ethernet eth1: mediatek frame engine at 0xffff800082380000, irq 123 >> [ 38.271431] mtk_soc_eth 15100000.ethernet eth1: configuring for inband/2500base-x link mode >> [ 38.279828] mtk_soc_eth 15100000.ethernet eth1: major config, requested inband/2500base-x >> [ 38.288009] mtk_soc_eth 15100000.ethernet eth1: interface 2500base-x inband modes: pcs=01 phy=00 >> [ 38.296800] mtk_soc_eth 15100000.ethernet eth1: major config, active inband/inband,an-disabled/2500base-x > > This is indeed without the PHY. We're using inband, although the PCS > mode is PHYLINK_PCS_NEG_INBAND_DISABLED, meaning inband won't be > used. As there is no PHY, we can't switch to MLO_AN_PHY. > >> [ 38.306362] mtk_soc_eth 15100000.ethernet eth1: phylink_mac_config: mode=inband/2500base-x/none adv=00,00000000,00008000,0000e240 pause=04 >> [ 39.220149] mtk_soc_eth 15100000.ethernet eth1: interface 2 (mii) rate match none supports 0-3,6-7,13-14 >> [ 39.229758] mtk_soc_eth 15100000.ethernet eth1: interface 3 (gmii) rate match none supports 0-3,5-7,13-14 >> [ 39.239420] mtk_soc_eth 15100000.ethernet eth1: interface 4 (sgmii) rate match none supports 0-3,5-7,13-14 >> [ 39.249173] mtk_soc_eth 15100000.ethernet eth1: interface 22 (1000base-x) rate match none supports 5-7,13-14 >> [ 39.259080] mtk_soc_eth 15100000.ethernet eth1: interface 23 (2500base-x) rate match none supports 6-7,13-14,47 >> [ 39.594676] mtk_soc_eth 15100000.ethernet eth1: PHY i2c:sfp-1:11 uses interfaces 4,23, validating 4,23 > > The PHY comes along... > >> [ 39.603992] mtk_soc_eth 15100000.ethernet eth1: interface 4 (sgmii) rate match none supports 0-3,5-7,13-14 >> [ 39.650080] mtk_soc_eth 15100000.ethernet eth1: interface 23 (2500base-x) rate match none supports 6-7,13-14,47 >> [ 39.660266] mtk_soc_eth 15100000.ethernet eth1: PHY [i2c:sfp-1:11] driver [RTL8221B-VB-CG 2.5Gbps PHY (C45)] (irq=POLL) >> [ 39.671037] mtk_soc_eth 15100000.ethernet eth1: phy: 2500base-x setting supported 00,00000000,00008000,000060ef advertising 00,00000000,00008000,000060ef >> [ 39.684761] mtk_soc_eth 15100000.ethernet eth1: requesting link mode inband/2500base-x with support 00,00000000,00008000,000060ef > > We decide to use MLO_AN_INBAND and 2500base-X, which we're already using. > >> [ 40.380076] mtk_soc_eth 15100000.ethernet eth1: phy link down 2500base-x/Unknown/Unknown/none/off >> [ 40.397090] brlan: port 5(eth1) entered blocking state >> [ 40.402223] brlan: port 5(eth1) entered disabled state >> [ 40.407437] mtk_soc_eth 15100000.ethernet eth1: entered allmulticast mode >> [ 40.414400] mtk_soc_eth 15100000.ethernet eth1: entered promiscuous mode >> [ 44.500077] mtk_soc_eth 15100000.ethernet eth1: phy link up 2500base-x/2.5Gbps/Full/none/off >> [ 44.508528] mtk_soc_eth 15100000.ethernet eth1: No phy led trigger registered for speed(2500) > > ... but we don't see link-up reported by the PCS after the PHY comes > up. Why is that - I think that needs investigation before we proceed > to patch the issue, because that suggests the PCS isn't seeing > valid 2500base-X from the PHY. > I think it is because pl->act_link_an_mode stays at MLO_AN_INBAND, but it needs to be set to MLO_AN_PHY, so that only the phy determines the link state: phylink_resolve() { ... } else if (pl->act_link_an_mode == MLO_AN_PHY) { link_state = pl->phy_state; ... }