Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756435Ab2J2AVs (ORCPT ); Sun, 28 Oct 2012 20:21:48 -0400 Received: from glenfiddich.mraw.org ([88.191.124.82]:52663 "EHLO glenfiddich.mraw.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751612Ab2J2AVr (ORCPT ); Sun, 28 Oct 2012 20:21:47 -0400 Date: Mon, 29 Oct 2012 01:21:40 +0100 From: Cyril Brulebois To: Francois Romieu Cc: nic_swsd@realtek.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] r8169: Fix WoL on RTL8168d/8111d. Message-ID: <20121029002140.GK6195@mraw.org> References: <1351392414-32219-1-git-send-email-kibi@debian.org> <20121028230523.GA26778@electric-eye.fr.zoreil.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="aqWxf8ydqYKP8htK" Content-Disposition: inline In-Reply-To: <20121028230523.GA26778@electric-eye.fr.zoreil.com> Organization: Debian User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2136 Lines: 62 --aqWxf8ydqYKP8htK Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Francois Romieu (29/10/2012): > Cyril Brulebois : > > This regression was spotted between Debian squeeze and Debian wheezy > > kernels (respectively based on 2.6.32 and 3.2). The fix was inspired > > by , using > > RTL_GIGA_MAC_VER_{25,26} for the RTL8168d/8111d chipset. >=20 > If someone experiences problems or has questions about WoL with > Realtek's 816x chipsets, http://marc.info/?t=3D132079219400004 would > imho be a better reference. Thanks. Should I re-send with that as reference, or will you amend the patch once you're done with your testing? > [...] > > Tested on top of a 3.2.23 kernel. >=20 > It should still be relevant for current kernel. I'll give it a try. FWIW, further testing was suggested on some Debian lists, and 3.2.32 (which includes some commits related to r8169) is also affected. > [...] > > It looks like RTL_GIGA_MAC_VER_{25,26} are kinda supposed to go > > together so I kept both. Florent's testing gave the following results: > > - RTL_GIGA_MAC_VER_25 only: FAIL. > > - RTL_GIGA_MAC_VER_26 only: SUCCESS. > > - RTL_GIGA_MAC_VER_25 + RTL_GIGA_MAC_VER_26: SUCCESS. >=20 > I don't understand the VER_25 vs VER_26 difference. Neither do I. We just figured that it would nice to test all three cases since those two macros popped up when looking at the 8168D family. Mraw, KiBi. --aqWxf8ydqYKP8htK Content-Type: application/pgp-signature; name="signature.asc" Content-Description: Digital signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.12 (GNU/Linux) iEYEARECAAYFAlCNzAgACgkQeGfVPHR5Nd3bNgCdGk0pTyrS9cCOQVpr3xqMj/aX N+sAoKRaBIZZFH9OHiNeFxH7CJFreMci =JSky -----END PGP SIGNATURE----- --aqWxf8ydqYKP8htK-- -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/