From nobody Tue Dec 16 08:54:50 2025 Received: from mail-wr1-f51.google.com (mail-wr1-f51.google.com [209.85.221.51]) (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 9A95C22616 for ; Mon, 8 Jul 2024 07:50:30 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.51 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1720425034; cv=none; b=rAIi+6hSnv7vQXVkSGe5zdBuQE51SfpStduTZiRxgY9nHFv44Y47DT9/8Yg2xX3ZcQ/BRMrxd/pxYjVIC2vdnkjZhNxuiOpO03xLQm3xYT6HYAHENnCtuKwmEqxENcjgettXTzpMb7pd3uHuOCeUTnWcUR9QZFPZ6TdSTh7ESj0= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1720425034; c=relaxed/simple; bh=9yi1anNucjAJ8p+xi+AkakLSWlU6/VCwEhS0HwZzO54=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=sUfL/dQBy0XoRtgRr+zQRC0dDUY9q5zPjGb6dkc7wVdgID18tfTACSvCxSYYxXwPUsMTWyHyER3eTx0hrplY7isu2x22nQxNnYtNiRTQdnVVwy/D80xYOWExsBRFSKigpv6J/8k39EGCnkr9FBDM8dazU5TPUTjv5pGPkz8Jg3s= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=bgdev.pl; spf=none smtp.mailfrom=bgdev.pl; dkim=pass (2048-bit key) header.d=bgdev-pl.20230601.gappssmtp.com header.i=@bgdev-pl.20230601.gappssmtp.com header.b=1CHMVnRp; arc=none smtp.client-ip=209.85.221.51 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=bgdev.pl Authentication-Results: smtp.subspace.kernel.org; spf=none smtp.mailfrom=bgdev.pl Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=bgdev-pl.20230601.gappssmtp.com header.i=@bgdev-pl.20230601.gappssmtp.com header.b="1CHMVnRp" Received: by mail-wr1-f51.google.com with SMTP id ffacd0b85a97d-367b8a60b60so978721f8f.2 for ; Mon, 08 Jul 2024 00:50:30 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bgdev-pl.20230601.gappssmtp.com; s=20230601; t=1720425029; x=1721029829; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=mvg9NGlB7Q3nnPA4DGlVOBNb6F5m/xXWm+pljGd+Dec=; b=1CHMVnRpprAnpqZEBhSRDNgQ8hVvKKnHTiDzqzSja591RtzItEyZiwh1HsLjR4LGAP fK2obxynuMxL3cTy0ikdxkMLvmXSJ3hY4q1YeZyszhDNuvQuxc5ndAD6rjvFvfmNXzUz RJFypAlTvgnlDJK7f35wBzjC5mBaTZMdNmtSUwarnvOaz+q9oBD+YedawdUFslnRhyhA zpI+Am3Qdl+qZjv3zt/8EMeGyhv2r9UjrZdd70synlpUiuoaQJO9fIaWFeAEBLnh8Qd8 6YfbspgqmFcIavkTVq2x9AMDchWjqRGUcDQMtbbEb1TvzTtgh+DB4gTHYhs2wghM7QSw DhXg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1720425029; x=1721029829; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=mvg9NGlB7Q3nnPA4DGlVOBNb6F5m/xXWm+pljGd+Dec=; b=YtXrdd4wgkRbERjdwrlJKKNDsyQiwNpXft17CoLRtHIcB0h7X4aFfXIe/S6RbY9kS+ ZbtTvRtTmG0S51OimrJ1GbzNYXh7510Xwq2SXol/jjB9XVPh6z3NgQ0jp9uGEL+U9KVK Wg7bNLVkxiWUuVo7tr8lF80DafflvlGoytl3KelQqosxZJ3YlalU8JF/nBenLkENKZAs GMIAR/JesrTlVohoUAr/j1MgjtZoLewt+o91g8gFCsLOfNqdLrBApYZ4Gr+f5LNOjqys OYs8mFIKPz0oXF8xOIA6Eac7wkph+Faw56uzweJdcs8VkddPIR1Vou3TGsUSQ5BboNHa XgCw== X-Forwarded-Encrypted: i=1; AJvYcCU2LNga7xdn8ZXVg/wNb1l+4cdAapOzPp4X+4/yFMTCXH0nMErIhI5VsyoTg0Z63bUJF3u0IWSSELmjf5TsgfiTdLp/k3gn3sXWQ9nM X-Gm-Message-State: AOJu0Yxj2XkpMRYseV4YDHslvwXStcVrGTTGbNBn++UJUweGa0fEHy5H +fkqA1LAY7ZeEyPW8+WWFBu7BGjDwL7NzI6/zhBFHQ/OoGG0uQoVr3a1j3EVgcA= X-Google-Smtp-Source: AGHT+IGEsMan5VERG5Ruuq/k28lPTx8wwmEfk9bARBnsnlCFiqZl6+JtfHYuFcpgTGHfbHSorC7eJA== X-Received: by 2002:a5d:408c:0:b0:360:7c13:761e with SMTP id ffacd0b85a97d-3679dd71a3dmr8078039f8f.65.1720425029279; Mon, 08 Jul 2024 00:50:29 -0700 (PDT) Received: from brgl-uxlite.home ([2a01:cb1d:dc:7e00:b5f9:a318:2e8a:9e50]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-3679d827789sm10160055f8f.76.2024.07.08.00.50.28 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 08 Jul 2024 00:50:29 -0700 (PDT) From: Bartosz Golaszewski To: Andrew Lunn , Heiner Kallweit , Russell King , "David S . Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org, Bartosz Golaszewski Subject: [RESEND PATCH net-next v3 1/4] net: phy: aquantia: rename and export aqr107_wait_reset_complete() Date: Mon, 8 Jul 2024 09:50:20 +0200 Message-ID: <20240708075023.14893-2-brgl@bgdev.pl> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240708075023.14893-1-brgl@bgdev.pl> References: <20240708075023.14893-1-brgl@bgdev.pl> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Bartosz Golaszewski This function is quite generic in this driver and not limited to aqr107. We will use it outside its current compilation unit soon so rename it and declare it in the header. Signed-off-by: Bartosz Golaszewski --- drivers/net/phy/aquantia/aquantia.h | 2 ++ drivers/net/phy/aquantia/aquantia_main.c | 6 +++--- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/drivers/net/phy/aquantia/aquantia.h b/drivers/net/phy/aquantia= /aquantia.h index 0bca28ec3332..2465345081f8 100644 --- a/drivers/net/phy/aquantia/aquantia.h +++ b/drivers/net/phy/aquantia/aquantia.h @@ -201,4 +201,6 @@ int aqr_phy_led_hw_control_set(struct phy_device *phyde= v, u8 index, int aqr_phy_led_active_low_set(struct phy_device *phydev, int index, bool = enable); int aqr_phy_led_polarity_set(struct phy_device *phydev, int index, unsigned long modes); +int aqr_wait_reset_complete(struct phy_device *phydev); + #endif /* AQUANTIA_H */ diff --git a/drivers/net/phy/aquantia/aquantia_main.c b/drivers/net/phy/aqu= antia/aquantia_main.c index 6c14355744b7..974795bd0860 100644 --- a/drivers/net/phy/aquantia/aquantia_main.c +++ b/drivers/net/phy/aquantia/aquantia_main.c @@ -441,7 +441,7 @@ static int aqr107_set_tunable(struct phy_device *phydev, * The chip also provides a "reset completed" bit, but it's cleared after * read. Therefore function would time out if called again. */ -static int aqr107_wait_reset_complete(struct phy_device *phydev) +int aqr_wait_reset_complete(struct phy_device *phydev) { int val; =20 @@ -494,7 +494,7 @@ static int aqr107_config_init(struct phy_device *phydev) WARN(phydev->interface =3D=3D PHY_INTERFACE_MODE_XGMII, "Your devicetree is out of date, please update it. The AQR107 family= doesn't support XGMII, maybe you mean USXGMII.\n"); =20 - ret =3D aqr107_wait_reset_complete(phydev); + ret =3D aqr_wait_reset_complete(phydev); if (!ret) aqr107_chip_info(phydev); =20 @@ -522,7 +522,7 @@ static int aqcs109_config_init(struct phy_device *phyde= v) phydev->interface !=3D PHY_INTERFACE_MODE_2500BASEX) return -ENODEV; =20 - ret =3D aqr107_wait_reset_complete(phydev); + ret =3D aqr_wait_reset_complete(phydev); if (!ret) aqr107_chip_info(phydev); =20 --=20 2.43.0 From nobody Tue Dec 16 08:54:50 2025 Received: from mail-wr1-f49.google.com (mail-wr1-f49.google.com [209.85.221.49]) (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 C6B0E38FB0 for ; Mon, 8 Jul 2024 07:50:32 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.49 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1720425035; cv=none; b=W6xtg3YpxzeJ2Vog1rbzvnHVkhUZqzwu113S4zf9rnyaC6OPiyY+EtSuyirQJZAu62aEcJ5gS407V/OPsJksUk30YhkzXMJEBpeYCemWEDqk2B0oEE4gp1wrEaFbcWv+nG6UMUMfKuxqmufUhH0Ug+kimG6/hxPAsC+zrDQiiZ0= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1720425035; c=relaxed/simple; bh=qT/I0vbZRGDpE8yP1ELoiUDNULDJ0duW33V+A0EfTRo=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=p7BqOg0SFPC1TxfnlIjGcV4mPR+qNJVvMYabXp6xrh3wDvIToY4OQ6jR2v5HCsJXfM62FOyWt0+eu+fJLrWkoZhSTuFL9L3xYiP9V4ydf1tfeQvMZEyNZlzViHM4I/VxCZGw2wlizgdCPknrfRucCLqADGb97V/50DYT6kDt+a0= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=bgdev.pl; spf=none smtp.mailfrom=bgdev.pl; dkim=pass (2048-bit key) header.d=bgdev-pl.20230601.gappssmtp.com header.i=@bgdev-pl.20230601.gappssmtp.com header.b=EYDe4A6l; arc=none smtp.client-ip=209.85.221.49 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=bgdev.pl Authentication-Results: smtp.subspace.kernel.org; spf=none smtp.mailfrom=bgdev.pl Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=bgdev-pl.20230601.gappssmtp.com header.i=@bgdev-pl.20230601.gappssmtp.com header.b="EYDe4A6l" Received: by mail-wr1-f49.google.com with SMTP id ffacd0b85a97d-36786081ac8so2223668f8f.0 for ; Mon, 08 Jul 2024 00:50:32 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bgdev-pl.20230601.gappssmtp.com; s=20230601; t=1720425031; x=1721029831; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=a9Md/Ifap9fn0ZqaVx/ir7biX84P7JSuDDxVoVQn/Dg=; b=EYDe4A6ljADu4uYo/+yKHjVNlJvACxZlSWBOvUy9AFzEaoIbWw8rxjdcodDmYPByO7 Y70F5jkWV145zREBHbMGdBohy7VoKW9o+Hh2m0VZUktQ+mlKwdHW637wJW5JL3pSlye5 m+o+obqdTKGGCspqf1WNoOUEmMZfCxXQcAQgTL/Cki+UfQ3D0t2w1WeoqM/WLiugX86h 8pcZ99IFp/ViGgKLfph0A+mJxOg2jne95BgXB8ZJo8+UzTJRmpGtx3zwprRcGN5gY2zQ w8s7Me+filDnw17jy3mUQuxNDVDgnSnJ4LWSPXa3infDj0XV+c3+gS7DzqZwb4ax8WBQ S5sA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1720425031; x=1721029831; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=a9Md/Ifap9fn0ZqaVx/ir7biX84P7JSuDDxVoVQn/Dg=; b=box35J9rCXAeOGizw3L38wg+IxdjoZYq1cvcjgQCKh5/hB3DRlSpCur8Sdwl7ROIG5 T0QW+QRdHa6YYkWQIItt0qUivL8tXFSjtPth/5jAYsWM2omRPhJ1oguwY3lbg7hYVJB1 v82pxJMQCcjI2mlzYZqkfO6Eyf5LzrCcDH3oHqP3nEG9L0o8pdJ2t3QrxbP+0U0+kNtx jKCRv5/hn8B37OymFHpWyuZmI5Qu1kDY4iUhWiXrGDuoXRgHW9kE5r1V2D9vtIxPU+q1 ZLZpeUOl6gjdAbZYDO1Fl6CL0qBroEt/CGO1J6wP4WT+pI0baIQg0b1YxTfRjQwiCrq9 Y1lA== X-Forwarded-Encrypted: i=1; AJvYcCVJNlBKmmCjO1BlBMGw6OyebHLTcUf9g3kGHsCTBUVu06/u61iWSdpdvUQGuVs2qAen0G8oDpF+TtHq4PHYc9fWKlC2SNT+FkcfNo3e X-Gm-Message-State: AOJu0YyS8vfl9n6gjC6XPVt90D6lKCQ+tB5GKv30wC0/cD8CmaGi86O3 YRwuVGyUeTbLz+UdYh1N7AVjDEazFsGCektVsz/y83v25nGbiwghpZkz8lEmXio= X-Google-Smtp-Source: AGHT+IHVB2YXB4XGZU9hQow7TYKrFDF5hlVBHYX1Zn7NqJFValokpAL01XU7F8dmUnLHKfJPoXRn5A== X-Received: by 2002:a05:6000:d04:b0:367:9107:9e11 with SMTP id ffacd0b85a97d-3679dd73c6fmr9802180f8f.62.1720425031243; Mon, 08 Jul 2024 00:50:31 -0700 (PDT) Received: from brgl-uxlite.home ([2a01:cb1d:dc:7e00:b5f9:a318:2e8a:9e50]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-3679d827789sm10160055f8f.76.2024.07.08.00.50.29 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 08 Jul 2024 00:50:29 -0700 (PDT) From: Bartosz Golaszewski To: Andrew Lunn , Heiner Kallweit , Russell King , "David S . Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org, Bartosz Golaszewski Subject: [RESEND PATCH net-next v3 2/4] net: phy: aquantia: wait for FW reset before checking the vendor ID Date: Mon, 8 Jul 2024 09:50:21 +0200 Message-ID: <20240708075023.14893-3-brgl@bgdev.pl> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240708075023.14893-1-brgl@bgdev.pl> References: <20240708075023.14893-1-brgl@bgdev.pl> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Bartosz Golaszewski Checking the firmware register before it complete the boot process makes no sense, it will report 0 even if FW is available from internal memory. Always wait for FW to boot before continuing or we'll unnecessarily try to load it from nvmem/filesystem and fail. Signed-off-by: Bartosz Golaszewski --- drivers/net/phy/aquantia/aquantia_firmware.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/drivers/net/phy/aquantia/aquantia_firmware.c b/drivers/net/phy= /aquantia/aquantia_firmware.c index 0c9640ef153b..524627a36c6f 100644 --- a/drivers/net/phy/aquantia/aquantia_firmware.c +++ b/drivers/net/phy/aquantia/aquantia_firmware.c @@ -353,6 +353,10 @@ int aqr_firmware_load(struct phy_device *phydev) { int ret; =20 + ret =3D aqr_wait_reset_complete(phydev); + if (ret) + return ret; + /* Check if the firmware is not already loaded by pooling * the current version returned by the PHY. If 0 is returned, * no firmware is loaded. --=20 2.43.0 From nobody Tue Dec 16 08:54:50 2025 Received: from mail-wm1-f48.google.com (mail-wm1-f48.google.com [209.85.128.48]) (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 5ADCD3BBEB for ; Mon, 8 Jul 2024 07:50:34 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.48 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1720425035; cv=none; b=u1gqNWqADnVqxlrSwxE9i55GtScXYjODhNidKMG8xHvFWd0PVzNhYtsEiQa3dz7ovt73sGyxqNatilYuQgqw33hLwIxAtT5lqqc031jwFLBlnjZOqjwdHMObpSYLxWau60CEVy8rUy1MXsTX1UxAwl/nyUkiw1SDLTag8UB36ls= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1720425035; c=relaxed/simple; bh=I8ZVS0D82/R3RtH++gdSA2BpSL5i3uiqUGmj0KpPRmU=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=CQcMwPNsfaZKaxqgo96MlhQFDnio1jYR2eMqaqbXIgImCiIrqSsnkXUwlOF27sLWnC7Iq9h8VW/fFzzSHeTaVR1NEvb+SEvx+M32vlXlX+iJjpLoeOfHzH35V0C2uqfpcdxdPeFDADYeWslxIdNCARgiDQwR7JiBddlKlufd4ag= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=bgdev.pl; spf=none smtp.mailfrom=bgdev.pl; dkim=pass (2048-bit key) header.d=bgdev-pl.20230601.gappssmtp.com header.i=@bgdev-pl.20230601.gappssmtp.com header.b=bcc6folD; arc=none smtp.client-ip=209.85.128.48 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=bgdev.pl Authentication-Results: smtp.subspace.kernel.org; spf=none smtp.mailfrom=bgdev.pl Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=bgdev-pl.20230601.gappssmtp.com header.i=@bgdev-pl.20230601.gappssmtp.com header.b="bcc6folD" Received: by mail-wm1-f48.google.com with SMTP id 5b1f17b1804b1-4256742f67fso25286515e9.3 for ; Mon, 08 Jul 2024 00:50:34 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bgdev-pl.20230601.gappssmtp.com; s=20230601; t=1720425033; x=1721029833; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=DH3kXkonM31DpQhN/1Hxs8pfB1Bit5q7zwIm1RL/Mck=; b=bcc6folDJj1sY3nQvsG6Hz1slvmi+IszcHp3OwgDcz2dOitWgcqcih05h5W7ivgjlo Fc4VXFXdk2MJ1aPEbW7+Zp/NrQbaLPZhUiLjg6/u+v8OeLCq8obv422hEtpe8LuWiVuq SPDxOotlvvL1GXT1yzseeNNnyn8tABxBnWsvgDeBo8Qs34cMOgsDcpLDXcJ0EV1zRJK1 mrvjGzTtN02whD4l1WgyRPTBNfyn+MRVuknSD3g8jjPLCMF7DHOVRgynAV4tL8j5FZ5S HckPw5xragSqy4YILi+DMSw5ds1OlH4WygbljVdqkNFADPcb6NgCvHPPbFlBR77W+SdE yOxQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1720425033; x=1721029833; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=DH3kXkonM31DpQhN/1Hxs8pfB1Bit5q7zwIm1RL/Mck=; b=YAwl+XeLeN2ZlQEDVhpegHoruWY8LHdT0goJ/sdYRHiGdhLSuUqVfBPdsFut/qdovQ eE76Hh/b+RfgvXe5LAHMwcvH5G26onPH6lPpMKLzueyzlNYFr3GmNjlGJ6c+3w+B+TNu tMM6C6IF3RAm013Ky9k9wl49k9KrxtLp/PcSXvbOFYdBTu+JV3IYAhg2GNGUpN+KpgKD 2YrwSaZ2UlWSBH4LH/LdKpWkomldMZx5z3ATmZ1vERLxwGfDxcu8ZOR+A9crpEyeJsSM C49N1nZa9bqpb71K4JtrIi7uHtPIRfTiP4MAhXaGcWy2z5J45zr13cO/k8ADQ7lb/TMw nPkA== X-Forwarded-Encrypted: i=1; AJvYcCWu4pyHrDTt7pDjDSdhKvd0fejDOe4H7I3Rl0i60pAJBCFrkm1emOwpTqry2DUtsVcQuVajJqo9GCns1UtoYbGPMnhp2O7Rktsks3DD X-Gm-Message-State: AOJu0Yw4B37D77ZWe7690Y/dq4lKGCW6aPOSA7pHp3rMwYhzd5YZNO2G /ic/dlyISXD7t/YUHNSNqk3RaM6vNeC1wfvV31U72lFS7OzztxlLsXlUCEdT4Cw= X-Google-Smtp-Source: AGHT+IFGHXf42qjKEYiucDNY+VpgnwFXf1LC7psqdBVMRctd1Pm+BjeZFIZfnElzKyJyj7JtJC18Lg== X-Received: by 2002:a05:600c:1d03:b0:426:6667:5c42 with SMTP id 5b1f17b1804b1-42666675d11mr20254685e9.4.1720425032668; Mon, 08 Jul 2024 00:50:32 -0700 (PDT) Received: from brgl-uxlite.home ([2a01:cb1d:dc:7e00:b5f9:a318:2e8a:9e50]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-3679d827789sm10160055f8f.76.2024.07.08.00.50.31 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 08 Jul 2024 00:50:31 -0700 (PDT) From: Bartosz Golaszewski To: Andrew Lunn , Heiner Kallweit , Russell King , "David S . Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org, Bartosz Golaszewski Subject: [RESEND PATCH net-next v3 3/4] net: phy: aquantia: wait for the GLOBAL_CFG to start returning real values Date: Mon, 8 Jul 2024 09:50:22 +0200 Message-ID: <20240708075023.14893-4-brgl@bgdev.pl> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240708075023.14893-1-brgl@bgdev.pl> References: <20240708075023.14893-1-brgl@bgdev.pl> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Bartosz Golaszewski When the PHY is first coming up (or resuming from suspend), it's possible that although the FW status shows as running, we still see zeroes in the GLOBAL_CFG set of registers and cannot determine available modes. Since all models support 10M, add a poll and wait the config to become available. Signed-off-by: Bartosz Golaszewski --- drivers/net/phy/aquantia/aquantia_main.c | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/drivers/net/phy/aquantia/aquantia_main.c b/drivers/net/phy/aqu= antia/aquantia_main.c index 974795bd0860..2c8ba2725a91 100644 --- a/drivers/net/phy/aquantia/aquantia_main.c +++ b/drivers/net/phy/aquantia/aquantia_main.c @@ -652,7 +652,13 @@ static int aqr107_fill_interface_modes(struct phy_devi= ce *phydev) unsigned long *possible =3D phydev->possible_interfaces; unsigned int serdes_mode, rate_adapt; phy_interface_t interface; - int i, val; + int i, val, ret; + + ret =3D phy_read_mmd_poll_timeout(phydev, MDIO_MMD_VEND1, + VEND1_GLOBAL_CFG_10M, val, val !=3D 0, + 1000, 100000, false); + if (ret) + return ret; =20 /* Walk the media-speed configuration registers to determine which * host-side serdes modes may be used by the PHY depending on the --=20 2.43.0 From nobody Tue Dec 16 08:54:50 2025 Received: from mail-wr1-f50.google.com (mail-wr1-f50.google.com [209.85.221.50]) (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 0F88947F5D for ; Mon, 8 Jul 2024 07:50:35 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.50 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1720425037; cv=none; b=lLiLGBJc+WpIihrZA5B/5k0K6pz03g83m+4s3OxnxRJCcMc+xSZt5AdrZBtUjYudHREw2Ca2fabWzVB1b/ZHkSlA3juLneq7TYSajCHVnmhhByqyyOG3um+R4noVlmejryg7XWoUWE428BmJrC02pW1KPZQoZTA00lF1xHq0yQk= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1720425037; c=relaxed/simple; bh=aWRGwjNl141O0N2eu4pMGXstKpSbZTAZRRlvToIsLls=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=Ql0A+tv0vCltZGx9CXS50XFDG6o7jLzCmjhaWXZPyNnjn5Q4xTE8cvWHcvX4K2O6hwbAC3DoL7Y4fn76X+Yv8Nr2OmnKFGuLNVPQjJsvdBQz2ZPYfKXR+0VgrDU8Uz0wwV96UdFhNM8i6uiy913vJbNpRTICa1rz9ZdK28lZGbg= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=bgdev.pl; spf=none smtp.mailfrom=bgdev.pl; dkim=pass (2048-bit key) header.d=bgdev-pl.20230601.gappssmtp.com header.i=@bgdev-pl.20230601.gappssmtp.com header.b=p5ZIgrUZ; arc=none smtp.client-ip=209.85.221.50 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=bgdev.pl Authentication-Results: smtp.subspace.kernel.org; spf=none smtp.mailfrom=bgdev.pl Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=bgdev-pl.20230601.gappssmtp.com header.i=@bgdev-pl.20230601.gappssmtp.com header.b="p5ZIgrUZ" Received: by mail-wr1-f50.google.com with SMTP id ffacd0b85a97d-367a23462e6so2162786f8f.3 for ; Mon, 08 Jul 2024 00:50:35 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bgdev-pl.20230601.gappssmtp.com; s=20230601; t=1720425034; x=1721029834; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=mqzfRYx/eBkp9k5OzF2pgC0wEOIaJNu82fson6xrplE=; b=p5ZIgrUZ+uadygfMxSy17GGXlCI2oeCEjVCrAw61Ydf7rSPc98lXSLPGwyO1ue++Dw 28xKFrb97PBRVAsOpk2SFw1R0rpoZo8b+0wmVebIMaLui4g1xCFNODyydsuiFj29xyIC J8w6AEII6KlxdEQdVUUpiRIwGRoGRxNi/tOh0j0XlgAA58JQlAna2BLQCHqg6KiYlaYh N5UvlAyvp/115pDhvYnHr6qE3SAzwKFc7NoTfmknO+OplHIs+0YaLhQ2nIQ4FAelKI7O t0jTkmOaJs4DHu0eFz9+/5V9Uefet6oOmgmhV1k++7ALvudSvxlRTyDhenz1xuIg+2Sz GkHw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1720425034; x=1721029834; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=mqzfRYx/eBkp9k5OzF2pgC0wEOIaJNu82fson6xrplE=; b=a94Oh4YgD94PWYbl1RQkPvmiMwVvw5RkL19mOZTlR7+ttaEbkBgWKgW1OentEGlMSb QrhGWYa+HULmM4htfoqvmSSMGDCLm9/PBtMDgxnVPrZGqkc5WX1dsfrQABnvzuF0jODt ROCaMr6/NfUDSNe7XTwmANwWANBf5TMOodHaZh0ASOWnq+rO0FIHc6XjSE/ue3u0gGbT 3f+0aOs+sVkqZq8igmXbbBToMmm9hyweUjNTQNV2ybvxMOxkcZDdE9Q5jnfHKicj8aRp XwvfILgfqF4KN9xa46OwV2gVTGdIPSxaZNqlo9jN8yC6q6yVkdp0jhmE6JuL7Kl4FRiD fsFA== X-Forwarded-Encrypted: i=1; AJvYcCWvKNJ/oDNprgw6UUkEy6dbwreIcxBM11sPSyJcavEats2Bhx3NitTa0BPzftFcO0m42ZeO4+a+tapIXDgY/o2wCqOAz3ACgKpYNTfo X-Gm-Message-State: AOJu0YzEv5dAaf1AMnFAz7rG1VOGL53/3SoA+uEdhlMYB4Bt9vUikGC9 7FXxfe6Aj2Gpyk/DMevmQSpZAYleuYKazh2JjgmNx+Tw7fxsOPsxvSj7YD3d09g= X-Google-Smtp-Source: AGHT+IHCdYtJ5gdajf8bIJppL3jAkrIqdYM6almNmdUFQezuZItVtFtLkoeiiigNoVNNGUV5o239kQ== X-Received: by 2002:adf:8bde:0:b0:35f:f58:38fc with SMTP id ffacd0b85a97d-3679dd29745mr8013567f8f.26.1720425034553; Mon, 08 Jul 2024 00:50:34 -0700 (PDT) Received: from brgl-uxlite.home ([2a01:cb1d:dc:7e00:b5f9:a318:2e8a:9e50]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-3679d827789sm10160055f8f.76.2024.07.08.00.50.32 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 08 Jul 2024 00:50:33 -0700 (PDT) From: Bartosz Golaszewski To: Andrew Lunn , Heiner Kallweit , Russell King , "David S . Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org, Bartosz Golaszewski Subject: [RESEND PATCH net-next v3 4/4] net: phy: aquantia: add support for aqr115c Date: Mon, 8 Jul 2024 09:50:23 +0200 Message-ID: <20240708075023.14893-5-brgl@bgdev.pl> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240708075023.14893-1-brgl@bgdev.pl> References: <20240708075023.14893-1-brgl@bgdev.pl> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Bartosz Golaszewski Add support for a new model to the Aquantia driver. This PHY supports 2.5 gigabit speeds. The PHY mode is referred to by the manufacturer as Overclocked SGMII (OCSGMII) but this actually is just 2500BASEX without in-band signalling so reuse the existing mode to avoid changing the uAPI. Signed-off-by: Bartosz Golaszewski --- drivers/net/phy/aquantia/aquantia_main.c | 26 ++++++++++++++++++++++++ 1 file changed, 26 insertions(+) diff --git a/drivers/net/phy/aquantia/aquantia_main.c b/drivers/net/phy/aqu= antia/aquantia_main.c index 2c8ba2725a91..d12e35374231 100644 --- a/drivers/net/phy/aquantia/aquantia_main.c +++ b/drivers/net/phy/aquantia/aquantia_main.c @@ -29,6 +29,7 @@ #define PHY_ID_AQR113 0x31c31c40 #define PHY_ID_AQR113C 0x31c31c12 #define PHY_ID_AQR114C 0x31c31c22 +#define PHY_ID_AQR115C 0x31c31c33 #define PHY_ID_AQR813 0x31c31cb2 =20 #define MDIO_PHYXS_VEND_IF_STATUS 0xe812 @@ -1005,6 +1006,30 @@ static struct phy_driver aqr_driver[] =3D { .led_hw_control_get =3D aqr_phy_led_hw_control_get, .led_polarity_set =3D aqr_phy_led_polarity_set, }, +{ + PHY_ID_MATCH_MODEL(PHY_ID_AQR115C), + .name =3D "Aquantia AQR115C", + .probe =3D aqr107_probe, + .get_rate_matching =3D aqr107_get_rate_matching, + .config_init =3D aqr113c_config_init, + .config_aneg =3D aqr_config_aneg, + .config_intr =3D aqr_config_intr, + .handle_interrupt =3D aqr_handle_interrupt, + .read_status =3D aqr107_read_status, + .get_tunable =3D aqr107_get_tunable, + .set_tunable =3D aqr107_set_tunable, + .suspend =3D aqr107_suspend, + .resume =3D aqr107_resume, + .get_sset_count =3D aqr107_get_sset_count, + .get_strings =3D aqr107_get_strings, + .get_stats =3D aqr107_get_stats, + .link_change_notify =3D aqr107_link_change_notify, + .led_brightness_set =3D aqr_phy_led_brightness_set, + .led_hw_is_supported =3D aqr_phy_led_hw_is_supported, + .led_hw_control_set =3D aqr_phy_led_hw_control_set, + .led_hw_control_get =3D aqr_phy_led_hw_control_get, + .led_polarity_set =3D aqr_phy_led_polarity_set, +}, { PHY_ID_MATCH_MODEL(PHY_ID_AQR813), .name =3D "Aquantia AQR813", @@ -1048,6 +1073,7 @@ static struct mdio_device_id __maybe_unused aqr_tbl[]= =3D { { PHY_ID_MATCH_MODEL(PHY_ID_AQR113) }, { PHY_ID_MATCH_MODEL(PHY_ID_AQR113C) }, { PHY_ID_MATCH_MODEL(PHY_ID_AQR114C) }, + { PHY_ID_MATCH_MODEL(PHY_ID_AQR115C) }, { PHY_ID_MATCH_MODEL(PHY_ID_AQR813) }, { } }; --=20 2.43.0