From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pj2-f8.google.com (mail-pj2-f8.google.com [74.125.227.136]) (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 2179742AFB7 for ; Tue, 6 Oct 2026 13:55:50 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=74.125.227.136 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1791294952; cv=none; b=OLyEZ5dGZbOEKjXm3GwFGXq1ahpgnXOt8Znmizt4ptyDj9peeJVyR8uvnxJjKI9vpl4OG0IHQqztYsIYKk6l6TVzmXOzcfxDjVwILfYeAP1+DzrXo59Z9he/OJQ/MMCiMVBLWz3CnXFrFyAbU0mG+mehYPmL8J2fvMz+3z4FW9s= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1791294952; c=relaxed/simple; bh=EfsknO+e8STbS4VH4mYbE6xCJxLHJAuPON4fbSQTRAc=; h=Date:From:To:CC:Subject:In-Reply-To:References:Message-ID: MIME-Version:Content-Type; b=WRaHEBBwOj2YIAXisrxFEuz8gKUOBiuoSHJZhRLNQOxQBv+bIlAwQv0dMMw6N1V/PH4HIGx0fJnjKwPQ0hXnGwkg+A+ECq4QStVzj1nyOrNEDff+lIuVjtvizbVweCp0xFqwS/05zILsBOnqUEq548bHYJSWhy77sV+dNbxFPxE= 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=SihA87bu; arc=none smtp.client-ip=74.125.227.136 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="SihA87bu" Received: by mail-pj2-f8.google.com with SMTP id 98e67ed59e1d1-398b9f722abso1461653a91.0 for ; Tue, 06 Oct 2026 06:55:50 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1791294950; x=1791899750; darn=vger.kernel.org; h=content-transfer-encoding:content-type:mime-version:message-id :references:in-reply-to:user-agent:subject:cc:to:from:date:from:to :cc:subject:date:message-id:reply-to:content-type; bh=Id6lH5GTy+yyaEPTm6FcZZ7BLsleqqnI80GvEmpDvAs=; b=SihA87buQeKBBcothTaDqfgAwjht2ZE9cMOLP8+CISRwbagc9qvCiiTdfhLLYl+CSQ Dsqd9er9K3+MhCqMEaLfTsXUgCPLMa9953obiHiabEOKFF7aw+Bmk36CyBTYsJMhv0Py i/PnM3QI2dxZhRPOuKKaMlvykoDhzmgQ1aK67H+xdj8NstPzzZisWbgq5wVr/PJuPi3t Ht5dIsfLcJB2BPQzIwS8mqDKSCsVrhPHNit7IR9qIGqUmdFEaEL/frgYdLi49G81Wgci FxPc3g9iQz91+dDTJjsCKGA94NfHocsqqA/ruQkiK0LI/2v/rKH+MR2g2sjWgVc2PjZA wBUw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20260707; t=1791294950; x=1791899750; h=content-transfer-encoding:content-type:mime-version:message-id :references:in-reply-to:user-agent:subject:cc:to:from:date:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to :content-type; bh=Id6lH5GTy+yyaEPTm6FcZZ7BLsleqqnI80GvEmpDvAs=; b=Jfk9C146NY46Bz5TQZO6nugqTgexujmEGzOvg0+EyTmoOEWQl+iZmBLicN/gmd+zmW 6RvrbdqXYl+a37S2MFtq8i/lbr8RXUh9Q2L7A934naP9YI2AqeVoD0xbABIU194UwCJM xQkGenxAqZEpJCLM/HHpV/CCWA0e/mrwTYTVPXvv0wDIUDww0DbPjQIa5AabDC2nJK/R PQgyn5wL80pB0J7g9GcojIc+fU8/7bh7WB/Ch1lW4zUfF1caThmwg2mB55KF0H7JF/S/ AO615cgZiBNobvxL6xnsqpx5a4gbm1u0pvF6gZAAI3J19HQCREHeF4fPWhxD6TE/KhzG zF8g== X-Forwarded-Encrypted: i=1; AKwUvBweKQF+ZRfXKCRttNhO4zo8bM2w84o1gj/fKrzL+DSLdlNG2YatUJhhIePMbDTeUswxRKgbvQUBG7Lq@vger.kernel.org X-Gm-Message-State: AFq9FYKi5MrmVTHg0OU3ws53v6iDRiFBYaG5rlOWsCOOg4ynUJ0gYpQJ 9Rhx8EzIhReZS3NpYTkOlejwqhiBMW5avQjFFgg5ZW6YZxWapIw4HDJIQmnClIuFj0Bsmw== X-Gm-Gg: AYBFou2vZraufn9M41rHcPg0PhC/Glu40yzfEqZzn+mKnq7EICuC+IrkkiZMMFbVjaa n6Az7IsTYEWv5OpJSw5DFbxkkfUZVK9qCQMs3X8UHzJr3OvhQ7yoamF1rYK49owQ8XRL93mSqSB HIluuuCAfu35ShNyc8W9to8ywVsoaRGG6qJl8EzsUsUL/it89UOD81bJjLVS5z/Zh7lcvu9LDJC Z1Aklwxw7/wMz5IzGzGQi/nWSIjsfopKojS5M3M/wJqchVoFo6x8HTpQNLconLSyFnxC49vedRb wMpqJ02bTjDFMBaXcBGwu+SNhHGx6Nj4+Vo1yBecS7uE2DYVunCqYQIrpqcYASVjDaR2H4xuEvd y3xVtCFRqLwj2WrFila3uy8TNcgPPJsiJwPz/dh4xDDcgKvYqU2mc0kLS9Z8XJzBj7mYj9vfMLU DrrCX07ljfQzvi1JD8LtrhHCxZyyduCna+4JvfPmWf6k/5wGBzdjlRgBcTgCPVICv7nwU= X-Received: by 2002:a17:90b:4c4e:b0:3a7:eb0:119e with SMTP id 98e67ed59e1d1-3a873554164mr844006a91.21.1791294949932; Tue, 06 Oct 2026 06:55:49 -0700 (PDT) Received: from ehlo.thunderbird.net ([2603:c021:8000:9500:233::14]) by smtp.gmail.com with ESMTPSA id 41be03b00d2f7-ccf99acacb2sm1189776a12.22.2026.10.06.06.55.48 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 06 Oct 2026 06:55:49 -0700 (PDT) Date: Tue, 06 Oct 2026 21:55:41 +0800 From: Coia Prant To: Rob Herring CC: Jakub Kicinski , Andrew Lunn , "David S . Miller" , Eric Dumazet , Paolo Abeni , Krzysztof Kozlowski , Conor Dooley , Heiko Stuebner , Maxime Chevallier , Heiner Kallweit , Russell King , David Wu , netdev@vger.kernel.org, linux-rockchip@lists.infradead.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: =?US-ASCII?Q?Re=3A_=5BPATCH_net-next_v10_4/6=5D_dt-bindings=3A?= =?US-ASCII?Q?_net=3A_rockchip-dwmac=3A_document_pcs-handle?= User-Agent: K-9 Mail for Android In-Reply-To: <20261006134802.GA1916640-robh@kernel.org> References: <20261005223011.1124347-1-coiaprant@gmail.com> <20261005223011.1124347-5-coiaprant@gmail.com> <20261006134802.GA1916640-robh@kernel.org> Message-ID: Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable On October 6, 2026 9:48:02 PM GMT+08:00, Rob Herring wr= ote: >On Tue, Oct 06, 2026 at 06:30:06AM +0800, Coia Prant wrote: >> The Rockchip GMAC binding needs to describe the PCS reference used by >> the SGMII support added later in this series=2E The property will be >> parsed by rk_pcs_init(), and a missing phandle fails the probe=2E Add i= t >> and require it when phy-mode is "sgmii" on rockchip,rk3568-gmac, the >> only SoC in this binding that has SGMII support=2E >>=20 >> Signed-off-by: Coia Prant >> --- >> =2E=2E=2E/bindings/net/rockchip-dwmac=2Eyaml | 18 ++++++++++= ++++++++ >> 1 file changed, 18 insertions(+) >>=20 >> diff --git a/Documentation/devicetree/bindings/net/rockchip-dwmac=2Eyam= l b/Documentation/devicetree/bindings/net/rockchip-dwmac=2Eyaml >> index 80c252845349c=2E=2Ebb7540e838033 100644 >> --- a/Documentation/devicetree/bindings/net/rockchip-dwmac=2Eyaml >> +++ b/Documentation/devicetree/bindings/net/rockchip-dwmac=2Eyaml >> @@ -120,6 +120,12 @@ properties: >> maximum: 0x7F >> default: 0x10 >> =20 >> + pcs-handle: >> + description: >> + Specifies a reference to a node representing the PCS device >> + connected to this GMAC=2E Required when phy-mode is "sgmii"=2E >> + maxItems: 1 >> + >> phy-supply: >> description: PHY regulator >> =20 >> @@ -159,6 +165,18 @@ allOf: >> clocks: >> minItems: 5 >> =20 >> + - if: >> + properties: >> + compatible: >> + contains: >> + const: rockchip,rk3568-gmac >> + phy-mode: >> + contains: >> + const: sgmii > >The 'if' will also be true if 'phy-mode' is not present=2E Probably not= =20 >what you want? You need 'required: [ phy-mode ]'=2E > >Rob Hi Rob, Right, thanks=2E Without 'required: [ phy-mode ]' the 'if' matches wheneve= r the compatible matches, even if phy-mode is absent=2E I'll add it and send a respin=2E Thanks, Coia