From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wr1-f43.google.com (mail-wr1-f43.google.com [209.85.221.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 213FC38D6BD for ; Tue, 10 Mar 2026 14:02:04 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.43 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773151326; cv=none; b=VyV9d7YFgj6fqT2BnqikNuRk+50nE+aXNW0swFM6mCp692E/th8zmCnx/ob0V09NHEd1n+vIfwS6IkRh7lAXFN8VkKBF6PT3RuBrwOATSizdYtFawVzQDOiJqHhLlNFvyobIjz8UlSzPGZ/CT6QxsfRZZRCA02vf5u07Li7lFnM= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773151326; c=relaxed/simple; bh=0hNNZYnPE67nF1mHX8U8sOx0nA6XVig9tZv/Hddtnw4=; h=From:To:Cc:Subject:Date:Message-Id:MIME-Version:Content-Type; b=t8/jj23HECPpUeN7pM1LqTHT7uIYL3rMWMrwvKKOi8WmNle+Osne4mKLLY+112NnTXrym90QjLQ3uSMZVBptzu/zGSvYdVQArzneOgVHKU5GRsh6wFQTr3EJttg4s/5HaFu9sOjynv/3ve48ZI8RSV3OzroG8GAjyiLV3fsl7ac= 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=M+7j/zvj; arc=none smtp.client-ip=209.85.221.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="M+7j/zvj" Received: by mail-wr1-f43.google.com with SMTP id ffacd0b85a97d-439b78b638eso10199249f8f.2 for ; Tue, 10 Mar 2026 07:02:04 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1773151323; x=1773756123; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=Ohidppn0DDN3kquQ+UkqZeq+6ADdLoEMttZZlZ8FD5Y=; b=M+7j/zvjumKtUZpS4BLAjINjDMiUwYIwXyCKQbfIp14hPu8vuZF/zJkzxJzAerjkO8 p47igbUa4QBAx2R15MmOwyzgtZg0r4CM7C75aVcdHcsoVWQPaDWby3yaxJCe2NY60HsR RcXAF49WPFdKGT9nHVR0HWlZiFeJaVgRiKxWmcMcPC4y4jCivbRvA3+z1pgStPMtYZqu agCvHmtodV2ui+9J5vUDoAQwK9lp/t4Vg/wJyQsSs4tfLtYgbZt8ceJpv023lVGVSD/E 1tGOk5N+PxcmHKn3kCI4syOMGWby8kInpuIEFdSBid+GdsO9mhnu0Oy2FeYmYuAAajKG 4aHw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1773151323; x=1773756123; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=Ohidppn0DDN3kquQ+UkqZeq+6ADdLoEMttZZlZ8FD5Y=; b=Ee5imefQlq31ZScDyJ0LmysQ22aWPB+uL8jJrvRIuw+yrYqfISEXeLsEf0HlP5TDqF yk4GcMZkMkWM++ytD+ez/9M1YRdCGT8epeXtkd/A/Q2yBZpHk8Wwr3hRCQiDe0KnJ1iE 4R/RY/qKmmdPCJpKnGWLmDyNTnwOTXFrNl2eyxdeNzs+aVQ6bvak7MizV/EobiWxeut/ YevRLfgnSiQWbVuX8msLmCl1wPA1RuLwHLkrCsJmo9W7bny4pRBluJFgxFn55lfzQ7dX iMFjUKJOiNlbwbv/NsVA3Ba26Ob18Gv3w8C3NXrmF6MySJgLqwgKFFF2cldQgNyhB1oE vr/g== X-Forwarded-Encrypted: i=1; AJvYcCU0CaRtZCL3ugWvHbpwBg8MEmFngBWCF/nhJG6MB0jEQFS4PSTMwu1cDEiWGvzzgkAvMjAFfyXCGc0N6n4=@vger.kernel.org X-Gm-Message-State: AOJu0YwhY8xdHvCh1lF8r/G8rRag/KtACF6FNvKDK1KJQhkA8YP1YEeT bFuKJvWgJ+YsRpuybqKIFDhPKbuYicPWiNTYaoWnWlc7FzpziZn9fsyC X-Gm-Gg: ATEYQzxebTDnzbqju6E6Y7zpvQbCCuruvSBafDKX7NVSqWIbGS3wzqvhnnxMDHbFnuc PPCxdCUx/f6ShIXK98dn/Osqe4Fo6ikoqN2L3AaxZv9sS5Xo7102lewOEs9VrfPFRXJK1Hrvpy9 b5HHKavoR/vskske2KhN3KjIpF5y5HVSy9jH2XQH41YHXb1swGb7wuR6BEEm5U68UV74QC8kkbx WKnAGTvLCL23JLu7nSJ5ig2F/VzYxwiMUQKa707Rmm02wl+cd0A91aRM/gwkgN1pep/oWYFMZGC 91awAFZe1WfllQVf/BUA1gezuQBiw2Hc3A0/a9jtYNIxiE7RRFQdNRxQH4XxBDKhkE19o5eZy3z jUtYzfFkohXQ88s/5FkHXDZNFbYNoz97FSQUw6EotcCxYsz7OealvOaPoHBvKrvWdYY98kToicO I961umPyayd2iFobSluQBpBPywNU0yFHaAe5T+P7Z0lpz62QrsPH3xJ/T8OkqmDG6LSkdzGW16P XjvCwI5Frbm X-Received: by 2002:a05:6000:144c:b0:439:b7c9:2ef0 with SMTP id ffacd0b85a97d-439da88cebcmr24963018f8f.33.1773151323051; Tue, 10 Mar 2026 07:02:03 -0700 (PDT) Received: from snowdrop.snailnet.com (82-69-66-36.dsl.in-addr.zen.co.uk. [82.69.66.36]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-439dae3f267sm34409645f8f.31.2026.03.10.07.02.01 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 10 Mar 2026 07:02:02 -0700 (PDT) From: david.laight.linux@gmail.com To: Johannes Zink , Alexandre Torgue , =?UTF-8?q?Alexis=20Lothor=C3=A9?= , Andrew Lunn , Bartosz Golaszewski , Chen-Yu Tsai , "David S. Miller" , Eric Dumazet , Gatien Chevallier , Jakub Kicinski , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-stm32@st-md-mailman.stormreply.com, Maxime Chevallier , Maxime Coquelin , netdev@vger.kernel.org, Paolo Abeni , "Russell King (Oracle)" , Richard Cochran Cc: David Laight Subject: [PATCH resend net-next 1/1] net: stmmac: actually error requests to change the auxiliary snapshot capture channel Date: Tue, 10 Mar 2026 14:01:55 +0000 Message-Id: <20260310140155.3359-1-david.laight.linux@gmail.com> X-Mailer: git-send-email 2.39.5 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit From: David Laight Commit 2ddd05d1d5ed ("net: stmmac: do not silently change auxiliary snapshot capture channel") added code that attempted to return -EBUSY to a PTP_CLK_REQ_EXTTS request whan a snapshot was already enabled. However it tested bits in 'acr_value' after they had been masked off so the check would never return an error. Change the code so that the test actually works. Note that when the commit message says: Previously in case of a PTP_CLK_REQ_EXTTS request, previously active auxiliary snapshot capture channels were silently dropped and the new channel was activated. this only refers to two commits earlier (a few minutes earlier). Prior to that only a single fixed snapshot channel could be enabled. Note that the check will reject requests to re-enable the currently enabled channel. Plausibly the best fix is just to delete the check completely. Signed-off-by: David Laight --- Resend with corrected subject and To: Richard Cochran. Found by a test for FIELD_GET() returning 'constant zero' from a non-constant 'reg' value. drivers/net/ethernet/stmicro/stmmac/stmmac_ptp.c | 13 +++++-------- 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/drivers/net/ethernet/stmicro/stmmac/stmmac_ptp.c b/drivers/net/ethernet/stmicro/stmmac/stmmac_ptp.c index 3e30172fa129..aa17335a2031 100644 --- a/drivers/net/ethernet/stmicro/stmmac/stmmac_ptp.c +++ b/drivers/net/ethernet/stmicro/stmmac/stmmac_ptp.c @@ -222,19 +222,16 @@ static int stmmac_enable(struct ptp_clock_info *ptp, write_unlock_irqrestore(&priv->ptp_lock, flags); break; } - case PTP_CLK_REQ_EXTTS: { - u8 channel; - + case PTP_CLK_REQ_EXTTS: mutex_lock(&priv->aux_ts_lock); acr_value = readl(ptpaddr + PTP_ACR); - channel = ilog2(FIELD_GET(PTP_ACR_MASK, acr_value)); - acr_value &= ~PTP_ACR_MASK; if (on) { - if (FIELD_GET(PTP_ACR_MASK, acr_value)) { + u32 enabled_snapshots = FIELD_GET(PTP_ACR_MASK, acr_value); + if (enabled_snapshots) { netdev_err(priv->dev, "Cannot enable auxiliary snapshot %d as auxiliary snapshot %d is already enabled", - rq->extts.index, channel); + rq->extts.index, ilog2(enabled_snapshots)); mutex_unlock(&priv->aux_ts_lock); return -EBUSY; } @@ -245,6 +242,7 @@ static int stmmac_enable(struct ptp_clock_info *ptp, acr_value |= PTP_ACR_ATSEN(rq->extts.index); acr_value |= PTP_ACR_ATSFC; } else { + acr_value &= ~PTP_ACR_MASK; priv->plat->flags &= ~STMMAC_FLAG_EXT_SNAPSHOT_EN; } netdev_dbg(priv->dev, "Auxiliary Snapshot %d %s.\n", @@ -256,7 +254,6 @@ static int stmmac_enable(struct ptp_clock_info *ptp, !(acr_value & PTP_ACR_ATSFC), 10, 10000); break; - } default: break; -- 2.39.5