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 lists.ozlabs.org (lists.ozlabs.org [112.213.38.117]) (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 1D8B6C531F9 for ; Mon, 27 Jul 2026 00:59:40 +0000 (UTC) Received: from boromir.ozlabs.org (localhost [127.0.0.1]) by lists.ozlabs.org (Postfix) with ESMTP id 4h7gGy2ss1z2yRl; Mon, 27 Jul 2026 10:59:38 +1000 (AEST) Authentication-Results: lists.ozlabs.org; arc=none smtp.remote-ip=203.29.241.158 ARC-Seal: i=1; a=rsa-sha256; d=lists.ozlabs.org; s=201707; t=1785113978; cv=none; b=mgAXQmruHrfqkLu3Xv0tGRWqphPJdbdo/wnaJSaPWgoBSi42+7KghrBiOlfIQ2+/qUi3GXXvd3Eq67QmpfCBOCC0N8wPDwWNHkeuCFAAX5FjI4Iv3RSTradWt7Zl0eGQ/1ktvvdlnD/Yhb+/him4VeeAirsEkfoX8bdc2MLP0jL8BoKFqEsv3GzZdZkU09Z30pUnfmzIcHELI3m8Upznf7hwCiH+DIShEFX/Uu0+BNxvLsGGsyUNfVhZbzrnQxJURJ/IPaUyz480t7AZzDKVeSMPo02QGJOvCuxvYF196RbXwynvvyptKx6tDhLx1irJcdh9mnor9b/pgAv9qvvqVg== ARC-Message-Signature: i=1; a=rsa-sha256; d=lists.ozlabs.org; s=201707; t=1785113978; c=relaxed/relaxed; bh=Wem6sNRnY30doJw+U7poipVgShNMLZwf2O/edq87Z4c=; h=Message-ID:Subject:From:To:Cc:Date:In-Reply-To:References: Content-Type:MIME-Version; b=iyBEE0jA89GtGMi/CDGnRHqFFfFxYE6mCLvdAqqmFa6AC9zOjfFEqCJnabLE0MLBK6TKFpgnaHkfriLmx19gSjyG6GLeWdcAeXOGSMq4AY0YfbGKWph4G3ZALfqFkdw/3WuMwb6CIAfA20yq9xpCY5L7UYMHoncdsttKJOXgGzU8wbI2n0RIdKcTpoBXhf6R+gcirPVUTFOKGppzXGHKonDnwwM7hsBnSbSFIksWBbVbCkLh0YncdXYojE2aBCpp6GnVT6RCsTp0V4/TroPl9AEA+6GjITmKu/8WylTydbZ7arvtRM+UV6ontPTg/NlYfcPDoiIC24CIFglJFN2bYA== ARC-Authentication-Results: i=1; lists.ozlabs.org; dmarc=pass (p=none dis=none) header.from=codeconstruct.com.au; dkim=pass (2048-bit key; unprotected) header.d=codeconstruct.com.au header.i=@codeconstruct.com.au header.a=rsa-sha256 header.s=2022a header.b=YAF55WGK; dkim-atps=neutral; spf=pass (client-ip=203.29.241.158; helo=codeconstruct.com.au; envelope-from=andrew@codeconstruct.com.au; receiver=lists.ozlabs.org) smtp.mailfrom=codeconstruct.com.au Authentication-Results: lists.ozlabs.org; dmarc=pass (p=none dis=none) header.from=codeconstruct.com.au Authentication-Results: lists.ozlabs.org; dkim=pass (2048-bit key; unprotected) header.d=codeconstruct.com.au header.i=@codeconstruct.com.au header.a=rsa-sha256 header.s=2022a header.b=YAF55WGK; dkim-atps=neutral Authentication-Results: lists.ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=codeconstruct.com.au (client-ip=203.29.241.158; helo=codeconstruct.com.au; envelope-from=andrew@codeconstruct.com.au; receiver=lists.ozlabs.org) Received: from codeconstruct.com.au (pi.codeconstruct.com.au [203.29.241.158]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange x25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 4h7gGw6vKZz2yH4 for ; Mon, 27 Jul 2026 10:59:36 +1000 (AEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=codeconstruct.com.au; s=2022a; t=1785113974; bh=Wem6sNRnY30doJw+U7poipVgShNMLZwf2O/edq87Z4c=; h=Subject:From:To:Cc:Date:In-Reply-To:References; b=YAF55WGKe7CUjz1wzdmUr4eKEYEB/bTanhieEjBYhte6/JxqrMNyHj4DHHrQJO6Mb uhRXTx0SRTtXm2QRG7uNzmWwz8rs6J/6icb7XJ13zvNDka+DxeBb6XNAmb5C0pHkkw 1OEx9jTmZY+Z2x4pOyEqUDW/v+y7aDH+Lytxj9bVqjyVKhU4aLMVy9vBsgYZYIa+jV lQOTiXwT3JYda5B9Gpflk6ncUO3n3votu1DGyCJeDVPATB31ahaZPNB13BXldO1ZqJ HHgL+A3/f3gZBMebZgTwtIK9Wz74lPrvo13awshfc6LuP1owwpVdqyeO7eNlLO3lTk ldqBNxkAOD+Xw== Received: from [192.168.68.117] (unknown [180.150.113.112]) by mail.codeconstruct.com.au (Postfix) with ESMTPSA id 2D322608E3; Mon, 27 Jul 2026 08:59:32 +0800 (AWST) Message-ID: <3a997fc11b172a5b93cb04307a6ad2bdfd5845cf.camel@codeconstruct.com.au> Subject: Re: [PATCH v8 2/2] ARM: dts: aspeed: ventura2: Add Meta ventura2 BMC From: Andrew Jeffery To: Kyle Hsieh Cc: Rob Herring , Krzysztof Kozlowski , Conor Dooley , Joel Stanley , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-aspeed@lists.ozlabs.org, linux-kernel@vger.kernel.org Date: Mon, 27 Jul 2026 10:29:31 +0930 In-Reply-To: References: <20260615-ventura2_initial_dts-v8-0-c89f92c80447@gmail.com> <20260615-ventura2_initial_dts-v8-2-c89f92c80447@gmail.com> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable User-Agent: Evolution 3.56.2-0+deb13u1 X-Mailing-List: linux-aspeed@lists.ozlabs.org List-Id: List-Help: List-Owner: List-Post: List-Archive: , List-Subscribe: , , List-Unsubscribe: Precedence: list MIME-Version: 1.0 On Thu, 2026-07-23 at 16:40 +0800, Kyle Hsieh wrote: > Hi Andrew, >=20 > Thanks for taking the time to review this. > On Wed, Jul 22, 2026 at 7:46=E2=80=AFPM Andrew Jeffery > wrote: > >=20 > > On Mon, 2026-06-15 at 10:46 +0800, Kyle Hsieh wrote: > > > Ventura2 is a Rack Management Controller. It is a modular > > > device designed to manage liquid cooling systems and monitor hardware > > > states within an IT rack. The system uses an AST2600 BMC for manageme= nt. > > >=20 > > > RMCv2 serves several critical roles: > > > - Detects liquid leakage at both tray and rack levels. > > > - Communicates with and controls liquid cooling equipment. > > > - Manages leakage events and executes system recovery protocols. > > >=20 > > > Key hardware features include: > > > - An extensive I2C and IO expander topology to support comprehensive > > > =C2=A0 sensor monitoring and backward compatibility with legacy trays= . > > > - MCTP over I2C support for asynchronous device communications. > > > - A dual-flash design for BMC firmware redundancy. > > >=20 > > > Signed-off-by: Kyle Hsieh > > > --- > > > =C2=A0arch/arm/boot/dts/aspeed/Makefile=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 |= =C2=A0=C2=A0=C2=A0 1 + > > > =C2=A0.../dts/aspeed/aspeed-bmc-facebook-ventura2.dts=C2=A0=C2=A0=C2= =A0 | 2903 ++++++++++++++++++++ > > > =C2=A02 files changed, 2904 insertions(+) > >=20 > > We now have scripts/dtc/dt-check-style. Running it over the dts with '- > > -mode relaxed' yields 49 instances of 'mixed-indent-chars' - these must > > be fixed. Further, can you please look at the output with '--mode > > strict', address all the warnings that are reasonable, and then discuss > > what's left? > I have fixed all 49 instances of the 'mixed-indent-chars' formatting issu= es. > I also ran it with '--mode strict' and addressed the formatting > warnings accordingly. Thanks. > >=20 > > Otherwise, there are still some concerns reported against the bindings: > >=20 > > =C2=A0=C2=A0 $ make CHECK_DTBS=3Dy aspeed/aspeed-bmc-facebook-ventura2.= dtb W=3D1 > > =C2=A0=C2=A0 ... > > =C2=A0=C2=A0=C2=A0=C2=A0 DTC [C] arch/arm/boot/dts/aspeed/aspeed-bmc-fa= cebook-ventura2.dtb > > =C2=A0=C2=A0 arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura2.dtb:= /ahb/apb/lpc@1e789000/lhc@a0: failed to match any schema with compatible: = ['aspeed,ast2600-lhc'] > > =C2=A0=C2=A0 /home/andrew/src/kernel.org/linux/origin/build.arm.aspeed_= g5/arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura2.dtb: fan-controlle= r@20 (maxim,max31790): 'channel@2', 'channel@5' do not match any of the reg= exes: '^fan@[0-9]+$', '^pinctrl-[0-9]+$' > > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 from schem= a $id: http://devicetree.org/schemas/hwmon/maxim,max31790.yaml > > =C2=A0=C2=A0 /home/andrew/src/kernel.org/linux/origin/build.arm.aspeed_= g5/arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura2.dtb: i2c@400 (aspe= ed,ast2600-i2c-bus): Unevaluated properties are not allowed ('aspeed,hw-tim= eout-ms' was unexpected) > > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 from schem= a $id: http://devicetree.org/schemas/i2c/aspeed,i2c.yaml > > =C2=A0=C2=A0 /home/andrew/src/kernel.org/linux/origin/build.arm.aspeed_= g5/arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura2.dtb: adc@48 (ti,ad= s1015): '#address-cells' is a required property > > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 from schem= a $id: http://devicetree.org/schemas/iio/adc/ti,ads1015.yaml > > =C2=A0=C2=A0 /home/andrew/src/kernel.org/linux/origin/build.arm.aspeed_= g5/arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura2.dtb: adc@48 (ti,ad= s1015): '#size-cells' is a required property > > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 from schem= a $id: http://devicetree.org/schemas/iio/adc/ti,ads1015.yaml > > =C2=A0=C2=A0 arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura2.dtb:= /ahb/apb/bus@1e78a000/i2c@580/power-monitor@69: failed to match any schema= with compatible: ['pmbus'] > > =C2=A0=C2=A0 arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura2.dtb:= /ahb/apb/bus@1e7a0000/syscon@0: failed to match any schema with compatible= : ['aspeed,ast2600-i3c-global', 'syscon'] > >=20 > > Can you please tidy these up? > The board-specific bindings have been fixed: > - Removed the invalid `channel@X` subnodes from the `maxim,max31790` > fan controller. > - Dropped the deprecated `aspeed,hw-timeout-ms` property from the I2C bus= . > - Removed the incomplete `ti,ads1015` ADC and generic `pmbus` power > monitor nodes to fully comply with upstream schemas. Thanks. >=20 > However, the warnings for `aspeed,ast2600-lhc` and > `aspeed,ast2600-i3c-global` still remain. > Since these nodes are inherited directly from the SoC level > (`aspeed-g6.dtsi`), it seems their YAML bindings are not yet merged > upstream. Yes, it is fine to disregard these for now. Andrew