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 282DFC52D6F for ; Sat, 24 Aug 2024 18:45:21 +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:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:CC:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=BxmjcEY0kFJYwqjp+O6C9NvQFjdl5/RnzwVq4OoAmFA=; b=ikYgQQMWPNiSRsk1W+jLzIm6Ce uKqVGBefIaGXxBJeL15ysr4HSDxzC2PBBgdadUJbcQLogGdl14aj3FZ3HmsjYkRoJjr2jDcgnRupr toutSAPHDXBCNrIR3LZnAjnGHYmHwVVOrcAPSWp4es9d2FlXslYaBTzLyUO6SHtdqsot1RQ119De3 3oKZQY5h7R4LNyvirRC7UGz6zzw0o/DYwY8/wTdIN+vTW5aozQrVj9ZnZw6W/SNx3IyLr2s6pddLJ HvHa70hQgfgfJjvZ6l4VeotKKsn47eahjbWoGv3hszSHnS91X/dqfee0UdgbeG3Inb6BEMGsj5ZMs jvZnnR1Q==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1shvl8-00000002iQ2-1hKc; Sat, 24 Aug 2024 18:45:10 +0000 Received: from fllv0015.ext.ti.com ([198.47.19.141]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1shvkK-00000002iK2-2tAJ for linux-arm-kernel@lists.infradead.org; Sat, 24 Aug 2024 18:44:23 +0000 Received: from lelv0266.itg.ti.com ([10.180.67.225]) by fllv0015.ext.ti.com (8.15.2/8.15.2) with ESMTP id 47OIiFF3078656; Sat, 24 Aug 2024 13:44:15 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=ti-com-17Q1; t=1724525055; bh=BxmjcEY0kFJYwqjp+O6C9NvQFjdl5/RnzwVq4OoAmFA=; h=Date:From:To:CC:Subject:References:In-Reply-To; b=vurSt9dSq4P3SlXdDLI8F6vMB6bpi8RSCpCz4EZjamfwATpynMdyFSdP+kff0C3Vg U6FSZzh/B2k/M7mFe/PstbZ27BqmgJ4CPz/B2ECQgXPVvvvetV9np8w2b16/HgpqQg rm6ArRMgCKAuPnRcCSzhA0b9EZfvACfTX3hQf6RM= Received: from DFLE110.ent.ti.com (dfle110.ent.ti.com [10.64.6.31]) by lelv0266.itg.ti.com (8.15.2/8.15.2) with ESMTPS id 47OIiFhc001601 (version=TLSv1.2 cipher=AES256-GCM-SHA384 bits=256 verify=FAIL); Sat, 24 Aug 2024 13:44:15 -0500 Received: from DFLE112.ent.ti.com (10.64.6.33) by DFLE110.ent.ti.com (10.64.6.31) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2507.23; Sat, 24 Aug 2024 13:44:15 -0500 Received: from lelvsmtp6.itg.ti.com (10.180.75.249) by DFLE112.ent.ti.com (10.64.6.33) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2507.23 via Frontend Transport; Sat, 24 Aug 2024 13:44:15 -0500 Received: from localhost (uda0133052.dhcp.ti.com [128.247.81.232]) by lelvsmtp6.itg.ti.com (8.15.2/8.15.2) with ESMTP id 47OIiFJH076201; Sat, 24 Aug 2024 13:44:15 -0500 Date: Sat, 24 Aug 2024 13:44:15 -0500 From: Nishanth Menon To: Robert Nelson CC: , , , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Vignesh Raghavendra , Andrew Davis , Jai Luthra , Roger Quadros , Siddharth Vadapalli , Jared McArthur , Jason Kridner , Deepak Khatri , Drew Fustini Subject: Re: [PATCH v3 2/2] arm64: dts: ti: Add k3-am67a-beagley-ai Message-ID: <20240824184415.o7ehjqm523igqbbi@cornhusk> References: <20240822170440.265055-1-robertcnelson@gmail.com> <20240822170440.265055-2-robertcnelson@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <20240822170440.265055-2-robertcnelson@gmail.com> X-EXCLAIMER-MD-CONFIG: e1e8a2fd-e40a-4ac6-ac9b-f7e9cc9ee180 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240824_114421_279455_2CD695BD X-CRM114-Status: GOOD ( 13.94 ) 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 On 12:04-20240822, Robert Nelson wrote: Minor nmits below: > BeagleBoard.org BeagleY-AI is an easy to use, affordable open source > hardware single board computer based on the Texas Instruments AM67A, > which features a quad-core 64-bit Arm CPU subsystem, 2 general-purpose > digital-signal-processors (DSP) and matrix-multiply-accelerators (MMA), > GPU, vision and deep learning accelerators, and multiple Arm Cortex-R5 > cores for low-power, low-latency GPIO control. > [...] > + > + vdd_3v3: regulator-2 { > + compatible = "regulator-fixed"; > + regulator-name = "vdd_3v3"; > + regulator-min-microvolt = <3300000>; > + regulator-max-microvolt = <3300000>; > + vin-supply = <&vsys_5v0>; > + regulator-always-on; > + regulator-boot-on; > + }; > + > + vdd_mmc1: regulator-mmc1 { Also responding: > Okay, i'll change these... I already see a problem, as I've got two > 3v3... can we use 'regulator-[0-9]v[0-9]-X' ? let us just call them regulator-3 and so on incrementally. You can already name with regulator-name property. We really don't need the "v' specification here. [...] > +&wkup_i2c0 { > + pinctrl-names = "default"; > + pinctrl-0 = <&wkup_i2c0_pins_default>; > + clock-frequency = <100000>; > + bootph-all; > + status = "okay"; > + > + tps65219: pmic@30 { > + compatible = "ti,tps65219"; > + reg = <0x30>; > + buck1-supply = <&vsys_5v0>; > + buck2-supply = <&vsys_5v0>; > + buck3-supply = <&vsys_5v0>; > + ldo1-supply = <&vdd_3v3>; > + ldo3-supply = <&vdd_3v3>; > + ldo4-supply = <&vdd_3v3>; > + > + pinctrl-names = "default"; > + pinctrl-0 = <&pmic_irq_pins_default>; > + interrupt-parent = <&gic500>; > + interrupts = ; > + interrupt-controller; > + #interrupt-cells = <1>; > + > + system-power-controller; > + ti,power-button; > + bootph-all; Flip the bootph-all above the system-power-controller to stay within the coding style boundaries? > + > + regulators { [...] > +}; > + > +&sdhci1 { > + /* SD/MMC */ > + vmmc-supply = <&vdd_mmc1>; > + vqmmc-supply = <&vdd_sd_dv>; > + pinctrl-names = "default"; > + pinctrl-0 = <&main_mmc1_pins_default>; > + disable-wp; > + cd-gpios = <&main_gpio1 48 GPIO_ACTIVE_LOW>; > + cd-debounce-delay-ms = <100>; > + ti,fails-without-test-cd; > + bootph-all; Nit: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/devicetree/bindings/dts-coding-style.rst#n117 keep the bootph property above the vendor prefixed ones.. > + status = "okay"; > +}; > -- > 2.39.2 > Additionally, with dtbs_check on next-20240823, I get: /workdir/arch/arm64/boot/dts/ti/k3-am67a-beagley-ai.dtb: leds: led-0: Unevaluated properties are not allowed ('linux,default-trigger' was unexpected) /workdir/arch/arm64/boot/dts/ti/k3-am67a-beagley-ai.dtb: leds: led-0:linux,default-trigger: 'oneOf' conditional failed, one must be fixed: Please fix appropriately. -- Regards, Nishanth Menon Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D