From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wr1-f53.google.com (mail-wr1-f53.google.com [209.85.221.53]) (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 A83DA30AAC5 for ; Fri, 21 Nov 2025 12:09:50 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.53 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763726992; cv=none; b=G8cC6Ix+zGH5CffG31nojR9UuW6X/JIy9ARnti+xbiMnPWGUBBos5fe77rbbiA8Y9ihesuiX9X+1/c7MP7va6gYTcboBbrH2Lvb44AsOh9kLRzVZNdiOWRAf7LQLJ/sZC9E65H7A+H+tnwP2HtRdEvM9ZEiE5wsV5wKM1zmR4Q8= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763726992; c=relaxed/simple; bh=gJ2JRo6SXwLf+9ulGbyK8L9kKPy5BDeC3fsCzHtzVM0=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=FocJs2OZr4dLkaxEpooU9KdSYMf8c8cTsCmL8ruG7kqflAiGyL56ntsu874evDTFJ3whDoJRPU4RFipuAPZKIoSed8DPq/+/AoB+/j+L+04S9oS2T5uiVnnnjqeVGulEJokaRRfdiCc0bxa2p4+BU07uZIVdspM0prCqy9FECYM= 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=L/BPqc2y; arc=none smtp.client-ip=209.85.221.53 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="L/BPqc2y" Received: by mail-wr1-f53.google.com with SMTP id ffacd0b85a97d-42b32a5494dso1119048f8f.2 for ; Fri, 21 Nov 2025 04:09:50 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1763726989; x=1764331789; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=ZOhuD3FYN2lFcG6XYARt5Zef8vPJNg6JJcAtWMIQOK4=; b=L/BPqc2y7KdE7VWgXGaege02CJn6LVe58B8ZUhaFcIvuNUUQbq8XURaOnl7wO3HNjW t6bhtwMLCvMueZ3S3gLiM91/Ppoa6Sp0eqA42scpM3gjkI0MB01vSP6iXf1BdImW4Wv3 as/vlvGKkFs6Sal5OZSHoOhZx7MGCBvmR28CxZ3Vxer5Za3j3LAXXNADGASHHdFLMpXp cWewvmNvP/SbYsUXXNL3R6mNRKX/EhGGH3ZbIzi9mbO2fte49nRkcsk7MZ0UHDtWSjdD Gz2XdMRmGa1/2zaFY4QW99tNtwR1ihlOki9/W/2DPwfOqynIhC5qNfwH0Cc+RhW/DcnM zBPA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1763726989; x=1764331789; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-gg:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=ZOhuD3FYN2lFcG6XYARt5Zef8vPJNg6JJcAtWMIQOK4=; b=TUIfK7AZrqIwPrPWw+xsFbF8iSnvl1oRn/mm8jL1S7MoZeuLBsEE/nuAn5tp/I/g7W pD1ezPZ+TSjvWdKs1pisTV8vTs19raUqzPaiELNb7kkb4CKpzK2FgdmLw7jjMVTcP/YH kP+4muda06NXn2aNQVnaq9AigoMKpIXwG3o4gXCaxJSSaV4Rzc58qg28g6QHNAdySOi0 /Qzo74MSy/vTbAj1/Ykrb8Zh8ihKXtsEB69KCP7cop7/IPmFXpwvyHB+7ZgJf4lSLkwY m7WnwLssgF9TUSxQ1rUdcZQ9juYdDcGl/2uoK91FhPzdWAwB3sOZ7eKnmxuNd69nQW3G C/fw== X-Forwarded-Encrypted: i=1; AJvYcCUCxREn+QuAM7E1Yt68CamQYH95wNn5Ngpx7TMDZdhRBlu49JsqYuvVfvBv4Zl/SMRYkc6G4gjR/zTFCL8=@vger.kernel.org X-Gm-Message-State: AOJu0Yzv3LU6/e3Cm1oEDJ1ne464VC6+BsrPAENaA450dJ6kOBWPbxmB UvyKqabdHBeXXFhoWo8kgJtVL7je9g6bN62J8dzZeKB/SMcCtwIvIrVR X-Gm-Gg: ASbGncuhQr5ulD0u/cP1hlIt1GIFojzWsuQjrByGEkHkbbmCINnNUUaVTelbPjCxQzD tu+WXTG249PK+MvgPpM85a48blNRM5f5uPdb0y/mGrO9CBifm3zoo9BNGYdSiCOYTLdDCTdURSX WwY7e7Z0u+aWCa56t1EGIP2VlASwakaUgLeX3tjve0Q1QwvmHKE3ZZfzhm6483x5lILWBq1CwgH nMLJEpvlybNvtTeCqGVcbKdmZZfNIPffd09C2vY1biKptHGmfrMtfP5A9GiLt/4aVTGtlA1Lr4Q zkGLynjK3owrKzyTfJz7wa7WTxHEVwprzr5eyhXM5nbiapP0Ws8l5fVl8Z6Cy25aqCbFyec/2tl wWp+cGBL8VKxRD+Lv/MzWYkTI+OOXJhpKaordDqOVj+1ODShoCG+6An/6uKibiby3sS8JuM88M+ BG6rSPlSzD8me2sRbIARJ91jUQ7Q== X-Google-Smtp-Source: AGHT+IH8n/QkOXtXGlOicb4wsLtjqPV6gsrQePDU0F8CvJviDa7jL2c2AmKQvd/w+gkVUIyGcvOD9A== X-Received: by 2002:a5d:5f96:0:b0:42b:30d4:e3f0 with SMTP id ffacd0b85a97d-42cc1cf0fa3mr2085273f8f.22.1763726988710; Fri, 21 Nov 2025 04:09:48 -0800 (PST) Received: from google.com ([37.228.206.31]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-42cb7fd8d97sm10293196f8f.42.2025.11.21.04.09.47 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 21 Nov 2025 04:09:48 -0800 (PST) Date: Fri, 21 Nov 2025 12:09:46 +0000 From: Fabio Baltieri To: Heiner Kallweit Cc: Michael Zimmermann , Andrew Lunn , nic_swsd@realtek.com, Andrew Lunn , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , netdev@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] r8169: add support for RTL8127ATF Message-ID: References: <89298d49-d85f-4dfd-954c-f8ca9b47f386@lunn.ch> <22b15123-b134-467c-835c-c9e0f1e19e29@gmail.com> 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=us-ascii Content-Disposition: inline In-Reply-To: <22b15123-b134-467c-835c-c9e0f1e19e29@gmail.com> On Fri, Nov 21, 2025 at 12:17:33AM +0100, Heiner Kallweit wrote: > Could you please test whether the following fixes the chip hang on suspend / shutdown? > > diff --git a/drivers/net/ethernet/realtek/r8169_main.c b/drivers/net/ethernet/realtek/r8169_main.c > index de304d1eb..97dbe8f89 100644 > --- a/drivers/net/ethernet/realtek/r8169_main.c > +++ b/drivers/net/ethernet/realtek/r8169_main.c > @@ -1517,11 +1517,20 @@ static enum rtl_dash_type rtl_get_dash_type(struct rtl8169_private *tp) > > static void rtl_set_d3_pll_down(struct rtl8169_private *tp, bool enable) > { > - if (tp->mac_version >= RTL_GIGA_MAC_VER_25 && > - tp->mac_version != RTL_GIGA_MAC_VER_28 && > - tp->mac_version != RTL_GIGA_MAC_VER_31 && > - tp->mac_version != RTL_GIGA_MAC_VER_38) > - r8169_mod_reg8_cond(tp, PMCH, D3_NO_PLL_DOWN, !enable); > + switch (tp->mac_version) { > + case RTL_GIGA_MAC_VER_02 ... RTL_GIGA_MAC_VER_24: > + case RTL_GIGA_MAC_VER_28: > + case RTL_GIGA_MAC_VER_31: > + case RTL_GIGA_MAC_VER_38: > + break; > + case RTL_GIGA_MAC_VER_80: > + r8169_mod_reg8_cond(tp, PMCH, D3_NO_PLL_DOWN, true); > + break; > + default: > + r8169_mod_reg8_cond(tp, PMCH, D3HOT_NO_PLL_DOWN, true); > + r8169_mod_reg8_cond(tp, PMCH, D3COLD_NO_PLL_DOWN, !enable); > + break; > + } > } > > static void rtl_reset_packet_filter(struct rtl8169_private *tp) Yes, patched it in and tested on both suspend and reboot without touching the wol flags, seems to be working correctly. Thanks!