From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from bali.collaboradmins.com (bali.collaboradmins.com [148.251.105.195]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 9A70119939D; Thu, 14 Nov 2024 19:22:46 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=148.251.105.195 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1731612168; cv=none; b=KOMtTg9v9zuQ9TlFBPrvmlnD9rbSld5QHs+BfqqvviXzQOt2m1cETDST5odC+NFNLq1unl6fke+KQ7X6v/CpYq3d7aVb2wwbAjfhrX7s6YtahPqUlHcRTM2pseEqWtkCD0JHAgHkU1mYPFyf/Gt7AqFw2oqMFkjVdoR9seTdEs8= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1731612168; c=relaxed/simple; bh=JRi8xKxGuuE+b0jHJ+Vql53V2Sw0uR0okGWsErcdffQ=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=IdrOtqKtRE80koWJIAUu8ZposQJKt3CbAZmxV42+AXXeLl1yVJsI+gwHh0Yv9Zuw6zlZ7XmeDeSvzTTq/fjy8ReeRtgHkYygLfurc+Lmgi+C1rVah6W+E7a5wPPe/fYTORgnMCVbTesqCSL3swMEWtnMi1MCOfyuCa17rcDBMC4= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=collabora.com; spf=pass smtp.mailfrom=collabora.com; dkim=pass (2048-bit key) header.d=collabora.com header.i=@collabora.com header.b=Yqkehs3T; arc=none smtp.client-ip=148.251.105.195 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=collabora.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=collabora.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=collabora.com header.i=@collabora.com header.b="Yqkehs3T" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1731612164; bh=JRi8xKxGuuE+b0jHJ+Vql53V2Sw0uR0okGWsErcdffQ=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=Yqkehs3TrUIuXJhUJQV0HpIYhnyj2ep62TqiK6605+0AuNb+Ie3PObfpQ3qgk5Bdu PLhwrXLEWqQEHPdM3en8D8z79kKxPxDDh4M+C4yC6aNxeJio+hyvyZ0aN/ixk6rNJb UduvP3x+5asy+/QrE7Qu36caEXlA+Y5GJ5lFJnX5it8BERejomH5Blxmm5kCsL9Bjh RVp28VQ1E3WeOVM+o3m8a4OiFpHPx8EWEfx9BH//KOLwhpoBlIqsM+uwZcb+v4tdsj MNlj22eoru+8XGoTvwzC5bk00i+mITI1XejHZc8ZlBAb0Nl7FYt0t3FqRY7ws0+rxo WqQvyaW17HrXQ== Received: from notapiano (pool-100-2-116-133.nycmny.fios.verizon.net [100.2.116.133]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (prime256v1) server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) (Authenticated sender: nfraprado) by bali.collaboradmins.com (Postfix) with ESMTPSA id 6856D17E377C; Thu, 14 Nov 2024 20:22:40 +0100 (CET) Date: Thu, 14 Nov 2024 14:22:38 -0500 From: =?utf-8?B?TsOtY29sYXMgRi4gUi4gQS4=?= Prado To: AngeloGioacchino Del Regno Cc: Michael Walle , kernel@collabora.com, netdev@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, Andrew Lunn , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Matthias Brugger , Biao Huang , Alexandre Torgue , Jose Abreu , Maxime Coquelin , Bartosz Golaszewski , Andrew Halaney , Simon Horman Subject: Re: [PATCH v2 2/2] arm64: dts: mediatek: Set mediatek,mac-wol on DWMAC node for all boards Message-ID: References: <20241109-mediatek-mac-wol-noninverted-v2-0-0e264e213878@collabora.com> <20241109-mediatek-mac-wol-noninverted-v2-2-0e264e213878@collabora.com> Precedence: bulk X-Mailing-List: netdev@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: On Thu, Nov 14, 2024 at 10:26:34AM +0100, AngeloGioacchino Del Regno wrote: > Il 09/11/24 16:16, Nícolas F. R. A. Prado ha scritto: > > Due to the mediatek,mac-wol property previously being handled backwards > > by the dwmac-mediatek driver, its use in the DTs seems to have been > > inconsistent. > > > > Now that the driver has been fixed, correct this description. All the > > currently upstream boards support MAC WOL, so add the mediatek,mac-wol > > property to the missing ones. > > > > Signed-off-by: Nícolas F. R. A. Prado > > --- > > arch/arm64/boot/dts/mediatek/mt2712-evb.dts | 1 + > > arch/arm64/boot/dts/mediatek/mt8195-demo.dts | 1 + > > arch/arm64/boot/dts/mediatek/mt8395-kontron-3-5-sbc-i1200.dts | 1 + > > 3 files changed, 3 insertions(+) > > > > ..snip.. > > > diff --git a/arch/arm64/boot/dts/mediatek/mt8195-demo.dts b/arch/arm64/boot/dts/mediatek/mt8195-demo.dts > > index 31d424b8fc7cedef65489392eb279b7fd2194a4a..c12684e8c449b2d7b3b3a79086925bfe5ae0d8f8 100644 > > --- a/arch/arm64/boot/dts/mediatek/mt8195-demo.dts > > +++ b/arch/arm64/boot/dts/mediatek/mt8195-demo.dts > > @@ -109,6 +109,7 @@ ð { > > pinctrl-names = "default", "sleep"; > > pinctrl-0 = <ð_default_pins>; > > pinctrl-1 = <ð_sleep_pins>; > > + mediatek,mac-wol; > > The demo board has the same WoL capability as the EVK, so you can avoid adding the > mac-wol property here. Not sure I follow... If we omit the property here it will use PHY WOL instead, while the genio 1200 EVK has the property, so it will be using MAC WOL, so they're already the same and omitting will make them behave differently... Let me recap to make sure we're all on the same page: This was the WOL configuration for each board before this series: MAC mt2712-evb.dts MAC mt8195-demo.dts PHY mt8395-genio-1200-evk.dts MAC mt8395-kontron-3-5-sbc-i1200.dts PHY mt8395-radxa-nio-12l.dts PHY mt8390-genio-700-evk.dts After patch 1, they all get inverted: PHY mt2712-evb.dts PHY mt8195-demo.dts MAC mt8395-genio-1200-evk.dts PHY mt8395-kontron-3-5-sbc-i1200.dts MAC mt8395-radxa-nio-12l.dts MAC mt8390-genio-700-evk.dts And after patch 2, the remaining PHY ones are set to MAC: MAC mt2712-evb.dts MAC mt8195-demo.dts MAC mt8395-genio-1200-evk.dts MAC mt8395-kontron-3-5-sbc-i1200.dts MAC mt8395-radxa-nio-12l.dts MAC mt8390-genio-700-evk.dts The only board I have in hands and am able to test is mt8390-genio-700-evk.dts, which requires MAC WOL to work. For the others, your feedback on v1 was that they should all be set to MAC WOL. Except for mt2712, which you were not sure about, but it was already set to MAC WOL so we're keeping the same behavior. That's how we got to adding mediatek,mac-wol to mt8195-demo.dts, mt8395-kontron-3-5-sbc-i1200.dts and mt2712-evb.dts. Let me know if there has been some misunderstanding. Thanks, Nícolas > > > status = "okay"; > > mdio { > > diff --git a/arch/arm64/boot/dts/mediatek/mt8395-kontron-3-5-sbc-i1200.dts b/arch/arm64/boot/dts/mediatek/mt8395-kontron-3-5-sbc-i1200.dts > > index e2e75b8ff91880711c82f783c7ccbef4128b7ab4..4985b65925a9ed10ad44a6e58b9657a9dd48751f 100644 > > --- a/arch/arm64/boot/dts/mediatek/mt8395-kontron-3-5-sbc-i1200.dts > > +++ b/arch/arm64/boot/dts/mediatek/mt8395-kontron-3-5-sbc-i1200.dts > > @@ -271,6 +271,7 @@ ð { > > pinctrl-names = "default", "sleep"; > > pinctrl-0 = <ð_default_pins>; > > pinctrl-1 = <ð_sleep_pins>; > > + mediatek,mac-wol; > > I'm mostly sure that Kontron's i1200 works the same as the EVK in regards to WoL. > > Michael, I recall you worked on this board - can you please confirm? > > Thanks, > Angelo >