From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (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 3B0BA1C5489; Mon, 4 Aug 2025 03:09:37 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1754276979; cv=none; b=i+a7xMya+ZKA8Mauu+yaUuj6Nj4oBRlDY6ERIzFmjxX5fVf0S8JvG7x21tR4wk6FDgicP6Z3WEdBX0vd/U8KyrreVgyX63a3JYDuoAfy7aKQqDYQNlP2DDm9unZ06RChiKiqqqoIquG/mvVLvWFmWejbOhi8hMGSgBrOI8cdzps= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1754276979; c=relaxed/simple; bh=voCsL460D8V6dc8thN7QJQy/hbr0f8CssRhZcgxCPn4=; h=Date:Content-Type:MIME-Version:From:Cc:To:In-Reply-To:References: Message-Id:Subject; b=TEmCPpb9nDUUqu+XGqE+alxnUzwLDynHtLs0vbYtl28Nvm3yS4WIoPoyT2j1/Dk7Q5KtuhkO55u7reNDkV9BM6TYGcnJq2HcN3gGLqKXhYnOTQbi/6klqOd4B3TDtMha+OCUXzn/DnxvI1vxoTK8wiYc6Ptrbyj37soImnwgXK0= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=bPJjFPrg; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="bPJjFPrg" Received: by smtp.kernel.org (Postfix) with ESMTPSA id A306CC4CEEB; Mon, 4 Aug 2025 03:09:37 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1754276977; bh=voCsL460D8V6dc8thN7QJQy/hbr0f8CssRhZcgxCPn4=; h=Date:From:Cc:To:In-Reply-To:References:Subject:From; b=bPJjFPrgobqxRaL7nNB4qwHkkdlWle3vvvMb4jHmIngMqsLNBptON5X9EEA+wizye haR2jeB82up0BVka2d9lkyPW38vdE8wRr0uF4/fPtD9jdiwpKE/XY2PEL6UIxAlSez 6HcVVy8Y8Ax1KwoAcvfKOsgmXrcekVwg7Q4cw4yMh2mHqpVDq2cHTAa4MiuA0YpCFs P8Xl+5EIcJJnoV1DNhP3WJZ/m7VrwHkuVWDY+e+YxS75tooCRnursXYYDwDETO2TvT PiTGk2RFmnDCjFdDMzJgMASjDTHeE31u6BDkG2+R2I5kNtMTT37VAvcTL/BTHYp1dD ZpCtsrgW/chFg== Date: Sun, 03 Aug 2025 22:09:36 -0500 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 From: "Rob Herring (Arm)" Cc: joel@jms.id.au, p.k.lee@quantatw.com, linux-kernel@vger.kernel.org, Jason-Hsu@quantatw.com, linux-aspeed@lists.ozlabs.org, devicetree@vger.kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org, yang.chen@quantatw.com, Jerry.Lin@quantatw.com, patrick@stwcx.xyz, linux-arm-kernel@lists.infradead.org, andrew@codeconstruct.com.au To: "P.K. Lee" In-Reply-To: <20250801141131.2238599-1-pkleequanta@gmail.com> References: <20250801141131.2238599-1-pkleequanta@gmail.com> Message-Id: <175427693317.1227278.3466107434143462265.robh@kernel.org> Subject: Re: [PATCH v10 0/2] Add Meta (Facebook) Ventura BMC (AST2600) On Fri, 01 Aug 2025 22:11:29 +0800, P.K. Lee wrote: > Add Linux device tree entry related to Meta (Facebook) Ventura specific > devices connected to the BMC (AST2600) SoC. The purpose of Ventura is to > detect liquid leakage from all compute trays, switch trays and rack > sensors within the rack, log the events, and take necessary actions > accordingly. > > --- > v1: > 1. Create ventura dts file. > 2. Add commit msg. > 3. Use format-patch to generate patch. > 4. Add subject prefixes matching the subsystem. > --- > v2: > 1. Modify email content. > --- > v3: > 1. Add mail list. > --- > v4: > 1. Apply git send-email --thread option. > 2. Sort nodes in the dts alphanumerically. > --- > v5: > 1. Run scripts/checkpatch.pl and fix reported warnings. > 2. Remove unnecessary 88E6393X CONFIG FRU. > --- > v6: > 1. Add a new stage for the DTS change. > 2. Run scripts/checkpatch.pl and fix reported error. > 3. Fix the issue in a separate patch. > --- > v7: > 1. Fix broken indentation in the device tree file. > 2. Sort nodes alphabetically, then by address if equal. > 3. Rename fan sensor nodes from 'hwmon' to 'fan-controller'. > --- > v8: > 1. This patch series has significant changes compared to > previous versions, and quite some time has passed since the last > submission.Therefore, previously received Acked-by/Reviewed-by/Tested-by > tags are not included in this version. > If needed, tags can be added again after review of thisnew version. > --- > v9: > 1. Reordered the node sequence under i2c5. > 2. Added a description of the platform's intended use to the commit > messages. > 3. Added 3 GPIO expanders to i2c10 and defined the necessary GPIO > line names. > --- > v10: > 1. Added IRQ support in GPIO expanders under i2c10 to handle edge-triggered > events. > 2. Reordered the nodes. > --- > P.K. Lee (2): > dt-bindings: arm:aspeed add Meta Ventura board > ARM: dts: aspeed: ventura: add Meta Ventura BMC > > .../bindings/arm/aspeed/aspeed.yaml | 1 + > arch/arm/boot/dts/aspeed/Makefile | 1 + > .../aspeed/aspeed-bmc-facebook-ventura.dts | 1578 +++++++++++++++++ > 3 files changed, 1580 insertions(+) > create mode 100644 arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura.dts > > -- > 2.43.0 > > > My bot found new DTB warnings on the .dts files added or changed in this series. Some warnings may be from an existing SoC .dtsi. Or perhaps the warnings are fixed by another series. Ultimately, it is up to the platform maintainer whether these warnings are acceptable or not. No need to reply unless the platform maintainer has comments. If you already ran DT checks and didn't see these error(s), then make sure dt-schema is up to date: pip3 install dtschema --upgrade This patch series was applied (using b4) to base: Base: attempting to guess base-commit... Base: tags/next-20250801 (exact match) If this is not the correct base, please add 'base-commit' tag (or use b4 which does this automatically) New warnings running 'make CHECK_DTBS=y for arch/arm/boot/dts/aspeed/' for 20250801141131.2238599-1-pkleequanta@gmail.com: arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura.dtb: timer (arm,armv7-timer): 'clocks' does not match any of the regexes: '^pinctrl-[0-9]+$' from schema $id: http://devicetree.org/schemas/timer/arm,arch_timer.yaml# arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura.dtb: /sdram@1e6e0000: failed to match any schema with compatible: ['aspeed,ast2600-sdram-edac', 'syscon'] arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura.dtb: bus@1e600000 (aspeed,ast2600-ahbc): compatible: ['aspeed,ast2600-ahbc', 'syscon'] is too long from schema $id: http://devicetree.org/schemas/bus/aspeed,ast2600-ahbc.yaml# arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura.dtb: syscon@1e6e2000 (aspeed,ast2600-scu): 'smp-memram@180' does not match any of the regexes: '^interrupt-controller@[0-9a-f]+$', '^p2a-control@[0-9a-f]+$', '^pinctrl(@[0-9a-f]+)?$', '^pinctrl-[0-9]+$', '^silicon-id@[0-9a-f]+$' from schema $id: http://devicetree.org/schemas/mfd/aspeed,ast2x00-scu.yaml# arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura.dtb: /ahb/apb/syscon@1e6e2000/smp-memram@180: failed to match any schema with compatible: ['aspeed,ast2600-smpmem'] arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura.dtb: /ahb/apb/display@1e6e6000: failed to match any schema with compatible: ['aspeed,ast2600-gfx', 'syscon'] arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura.dtb: adc@1e6e9000 (aspeed,ast2600-adc0): 'interrupts' does not match any of the regexes: '^pinctrl-[0-9]+$' from schema $id: http://devicetree.org/schemas/iio/adc/aspeed,ast2600-adc.yaml# arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura.dtb: adc@1e6e9100 (aspeed,ast2600-adc1): 'interrupts' does not match any of the regexes: '^pinctrl-[0-9]+$' from schema $id: http://devicetree.org/schemas/iio/adc/aspeed,ast2600-adc.yaml# arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura.dtb: crypto@1e6fa000 (aspeed,ast2600-acry): 'aspeed,ahbc' does not match any of the regexes: '^pinctrl-[0-9]+$' from schema $id: http://devicetree.org/schemas/crypto/aspeed,ast2600-acry.yaml# arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura.dtb: lpc@1e789000 (aspeed,ast2600-lpc-v2): reg-io-width: 4 is not of type 'object' from schema $id: http://devicetree.org/schemas/mfd/aspeed-lpc.yaml# arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura.dtb: lpc@1e789000 (aspeed,ast2600-lpc-v2): lpc-snoop@80: 'clocks' does not match any of the regexes: '^pinctrl-[0-9]+$' from schema $id: http://devicetree.org/schemas/mfd/aspeed-lpc.yaml# arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura.dtb: kcs@24 (aspeed,ast2500-kcs-bmc-v2): 'clocks' does not match any of the regexes: '^pinctrl-[0-9]+$' from schema $id: http://devicetree.org/schemas/ipmi/aspeed,ast2400-kcs-bmc.yaml# arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura.dtb: kcs@28 (aspeed,ast2500-kcs-bmc-v2): 'clocks' does not match any of the regexes: '^pinctrl-[0-9]+$' from schema $id: http://devicetree.org/schemas/ipmi/aspeed,ast2400-kcs-bmc.yaml# arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura.dtb: kcs@2c (aspeed,ast2500-kcs-bmc-v2): 'clocks' does not match any of the regexes: '^pinctrl-[0-9]+$' from schema $id: http://devicetree.org/schemas/ipmi/aspeed,ast2400-kcs-bmc.yaml# arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura.dtb: kcs@114 (aspeed,ast2500-kcs-bmc-v2): 'clocks' does not match any of the regexes: '^pinctrl-[0-9]+$' from schema $id: http://devicetree.org/schemas/ipmi/aspeed,ast2400-kcs-bmc.yaml# arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura.dtb: /ahb/apb/lpc@1e789000/lhc@a0: failed to match any schema with compatible: ['aspeed,ast2600-lhc'] arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura.dtb: /ahb/apb/lpc@1e789000/ibt@140: failed to match any schema with compatible: ['aspeed,ast2600-ibt-bmc'] arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura.dtb: sdc@1e740000 (aspeed,ast2600-sd-controller): sdhci@1e740100:compatible: ['aspeed,ast2600-sdhci', 'sdhci'] is too long from schema $id: http://devicetree.org/schemas/mmc/aspeed,sdhci.yaml# arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura.dtb: sdc@1e740000 (aspeed,ast2600-sd-controller): sdhci@1e740200:compatible: ['aspeed,ast2600-sdhci', 'sdhci'] is too long from schema $id: http://devicetree.org/schemas/mmc/aspeed,sdhci.yaml# arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura.dtb: /ahb/apb/sdc@1e740000/sdhci@1e740100: failed to match any schema with compatible: ['aspeed,ast2600-sdhci', 'sdhci'] arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura.dtb: /ahb/apb/sdc@1e740000/sdhci@1e740200: failed to match any schema with compatible: ['aspeed,ast2600-sdhci', 'sdhci'] arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura.dtb: /ahb/apb/bus@1e78a000/i2c@200/i2c-mux@77/i2c@5/pwm@20: failed to match any schema with compatible: ['max31790'] arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura.dtb: i2c@400 (aspeed,ast2600-i2c-bus): Unevaluated properties are not allowed ('aspeed,hw-timeout-ms' was unexpected) from schema $id: http://devicetree.org/schemas/i2c/aspeed,i2c.yaml# arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura.dtb: /ahb/apb/bus@1e78a000/i2c@580/power-monitor@69: failed to match any schema with compatible: ['pmbus'] arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura.dtb: fsi@1e79b000 (aspeed,ast2600-fsi-master): compatible: ['aspeed,ast2600-fsi-master', 'fsi-master'] is too long from schema $id: http://devicetree.org/schemas/fsi/aspeed,ast2600-fsi-master.yaml# arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura.dtb: /ahb/apb/fsi@1e79b000: failed to match any schema with compatible: ['aspeed,ast2600-fsi-master', 'fsi-master'] arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura.dtb: fsi@1e79b100 (aspeed,ast2600-fsi-master): compatible: ['aspeed,ast2600-fsi-master', 'fsi-master'] is too long from schema $id: http://devicetree.org/schemas/fsi/aspeed,ast2600-fsi-master.yaml# arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura.dtb: /ahb/apb/fsi@1e79b100: failed to match any schema with compatible: ['aspeed,ast2600-fsi-master', 'fsi-master'] arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura.dtb: /ahb/apb/dma-controller@1e79e000: failed to match any schema with compatible: ['aspeed,ast2600-udma'] arch/arm/boot/dts/aspeed/aspeed-bmc-inspur-on5263m5.dtb: fan@0: aspeed,fan-tach-ch: b'\x00\x01' is not of type 'object', 'integer', 'array', 'boolean', 'null' from schema $id: http://devicetree.org/schemas/dt-core.yaml#