From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 508D4CFD641 for ; Wed, 7 Jan 2026 14:55:27 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Type: Content-Transfer-Encoding:MIME-Version:References:In-Reply-To:Message-ID:Date :Subject:Cc:To:From:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=hcBGP+YEL31N0blv4+v2NuEIpQB28RQ0vx0uWq2aRTo=; b=nYCxXIQCYmkpB7wEasajorV17m Zwaecgov2+nvC4ewgAUz/23FLFUfXC+3PiA+jNLA/06NrwtYE8/wfe4wYpoTlpNIlxFEWdv5OcNPl ytR2Xy61o+BUfUwKoIIRCI8TY3PDVVkeFJo1W4uDcQS0VphMAsksKd+l1nI9hQysm7QJ4cJw+o8ue KB8Gjbz1nDNkDG0mjL9oUqj7F1JY/mVyl9EuB83UANuC+oe9uaMwHMnhGv+QJOyaoYwBZxVCD/0t6 g+pAi3feP6ZzyfmoFFs98MJw9aoed46KuWCQ8JupDoH2RjTfIGHDlW6g5sT2RLpd56xn9G6HRg2gm kNGSm5fg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1vdUwS-0000000F8Lj-45It; Wed, 07 Jan 2026 14:55:20 +0000 Received: from gloria.sntech.de ([185.11.138.130]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1vdUwP-0000000F8LG-0BD9; Wed, 07 Jan 2026 14:55:19 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=sntech.de; s=gloria202408; h=Content-Type:Content-Transfer-Encoding:MIME-Version: References:In-Reply-To:Message-ID:Date:Subject:Cc:To:From:Reply-To; bh=hcBGP+YEL31N0blv4+v2NuEIpQB28RQ0vx0uWq2aRTo=; b=Lom2AXd9dsigseGn1s7xGCiVIv /EKODcuzWY/SjvFU0VEexo4xSmkY3YpL0HexQ+CmXrKIj2FBvXUERsSewPX8Dkc9eOfezvxMjo4eP E0iaq9ntdXmolmGytsP1JO0eUXm+c1N3m+ukKzzpN2HkDMZn3+85miVMd1YWGx0qeD53/SF1TiRwd UE+PXSFjn03XEOJV46iM7T7gXDmzTpyYs+vCtw6/6RI0Tt7Ld8fse94tNPlCrEU5UpYIhPS+8j5XZ yg7EyF15l1tC9Ys9ftCUaWnWCeWqt1tb8J6QPaS4YRhFr/nAD8WouPjrqTFxLr1Pa/caNp6mD7hFT zWRAP3zQ==; Received: from i53875b57.versanet.de ([83.135.91.87] helo=diego.localnet) by gloria.sntech.de with esmtpsa (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1vdUvw-001OJ9-Fm; Wed, 07 Jan 2026 15:54:49 +0100 From: Heiko =?UTF-8?B?U3TDvGJuZXI=?= To: Chaoyi Chen , Alexey Charkov , Chaoyi Chen Cc: Rob Herring , Krzysztof Kozlowski , Conor Dooley , Quentin Schulz , Kever Yang , Jonas Karlman , John Clark , FUKAUMI Naoki , Jimmy Hon , Dragan Simic , Michael Riesch , Peter Robinson , Shawn Lin , Sebastian Reichel , Andy Yan , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-rockchip@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2 2/2] arm64: dts: rockchip: Add rk3576 evb2 board Date: Wed, 07 Jan 2026 15:54:47 +0100 Message-ID: <1956308.1xdlsreqCQ@diego> In-Reply-To: References: <20260107070322.323-1-kernel@airkyi.com> <881548f1-7dd9-40bb-9267-e5a03fb97820@rock-chips.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20260107_065517_127571_CF5A4E37 X-CRM114-Status: GOOD ( 22.44 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Am Mittwoch, 7. Januar 2026, 11:04:42 Mitteleurop=C3=A4ische Normalzeit sch= rieb Chaoyi Chen: > On 1/7/2026 5:57 PM, Chaoyi Chen wrote: > > On 1/7/2026 4:21 PM, Heiko St=C3=BCbner wrote: > >> Am Mittwoch, 7. Januar 2026, 08:56:04 Mitteleurop=C3=A4ische Normalzei= t schrieb Alexey Charkov: > >>> On Wed, Jan 7, 2026 at 11:04=E2=80=AFAM Chaoyi Chen wrote: [...] > >>>> + vcc3v3_hubreset: vcc3v3-hubreset { > >>>> + compatible =3D "regulator-fixed"; > >>>> + regulator-name =3D "vcc3v3_hubreset"; > >>>> + regulator-boot-on; > >>>> + regulator-always-on; > >>> > >>> If this regulator supplies a soldered-on discrete hub and is required > >>> to power it up, won't it be better to describe the hub in the device > >>> tree (see binding at [1]), make the regulator its supply, and perhaps > >>> drop the "regulator-boot-on/regulator-always-on" annotation here, > >>> letting the regulator core deal with its enabling instead? > >>> > >>> [1] https://github.com/torvalds/linux/blob/master/Documentation/devic= etree/bindings/usb/usb-device.yaml > >> > >> Yep, it would be nicer to it this way. > >> A live example can be found in the Rock 5 ITX [2] > >> > >> [2] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git= /tree/arch/arm64/boot/dts/rockchip/rk3588-rock-5-itx.dts#n1266 > >=20 > > Thank you for the great example. BTW the hub used here is CH344. It > > looks like we need to add a new binding :) > > >=20 > Typo... It is WCH CH334. I don't think you need a new compatible at all :-)=20 When you look at the usb-device.yaml linked above you'll the compatible already defined as a pattern: compatible: contains: pattern: "^usb[0-9a-f]{1,4},[0-9a-f]{1,4}$" description: Device nodes or combined nodes. "usbVID,PID", where VID is the vendor id and PID the product id. The textual representation of VID and PID shall be in lower case hexadecimal with leading zeroes suppressed. The other compatible strings from the above standard binding could also be used, but a device adhering to this binding may leave out all except for "usbVID,PID". Which will match everything VID + PID combination, so you just need to use the VID+PID from your hub. Heiko