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 phobos.denx.de (phobos.denx.de [85.214.62.61]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 7C245CD5BB4 for ; Tue, 26 May 2026 08:35:26 +0000 (UTC) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id BF42F8482F; Tue, 26 May 2026 10:34:51 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=none (p=none dis=none) header.from=baylibre.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Authentication-Results: phobos.denx.de; dkim=pass (2048-bit key; secure) header.d=baylibre.com header.i=@baylibre.com header.b="f+vpQohY"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id 7774C83EF9; Tue, 26 May 2026 09:47:21 +0200 (CEST) Received: from mail-wm1-x32a.google.com (mail-wm1-x32a.google.com [IPv6:2a00:1450:4864:20::32a]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits)) (No client certificate requested) by phobos.denx.de (Postfix) with ESMTPS id E4247803C6 for ; Tue, 26 May 2026 09:47:18 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=none (p=none dis=none) header.from=baylibre.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=msp@baylibre.com Received: by mail-wm1-x32a.google.com with SMTP id 5b1f17b1804b1-48a563e4ef7so75426915e9.0 for ; Tue, 26 May 2026 00:47:18 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre.com; s=google; t=1779781638; x=1780386438; darn=lists.denx.de; h=in-reply-to:references:from:subject:cc:to:message-id:date :mime-version:from:to:cc:subject:date:message-id:reply-to; bh=lpmlqP1jVBzpJ7ipWbZClZXz6wbBsYHYb15myIA6Q0g=; b=f+vpQohYP1IapWBh3snUDZWvqyQxHM9vnyBD80EevH/Y6asaF+DEumxv49ESpwWTs9 AE9HUd1+92wogwx163eA9DwyTfrqVidrmzj72UjbmrVqjbF8hHlHLIJ96b2pPUIvbWBj EbvMD0okKNuB5VpVB8rBVJDKs7c93DvHAY/Ca+MmInWNj0tvvIiQzaZdu1ZrMQ6TTpue JXTTgsl2/gTzKGPwdrehykw0Xfr7lQiM/GpX/hOxvp7LSaAa1Qlqy474LpXvg92HHodO gouN0ew+5b6ZLqiqHS4BwJIC+G/geRe8cqRgKGaPpchZyo+COxauq82MtgxE5DTdsHrP lM+Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1779781638; x=1780386438; h=in-reply-to:references:from:subject:cc:to:message-id:date :mime-version:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=lpmlqP1jVBzpJ7ipWbZClZXz6wbBsYHYb15myIA6Q0g=; b=W7Eo1pA1n9L+d8eiPRtWwOp6c/6jtptJjmnR0R/l5IcO9dgipIjNx03q824CO/DBno dZOX3TqbYLXLka93oaFOUfpTXDxVrL2y9QnYXW2baoiRPtoy6Y3kl0meLjlvaBWb69pS xU4lqEzp5pFRFoOZxwRSTqRo/O7d51gOeOTvM+8kHxpAKI7t8BkevkTdj+q3H6PP5ul4 4KUKg5TZMFSvo5ivGA7uJMYk5zqOOlKblXtyfKG8fIkEPnb+z4Bz63LouTvdWRvX/hD6 5SxaH4CEfw4d/xHrYJyyiGrgOsNaHFp4NWerVLPUKEg2z9/ZSmz8XBcjl60voLkaM7wo wL2g== X-Gm-Message-State: AOJu0YyWMJF+VmewWDYR0CfrO628qPl5pFz05THjkgSFs+5L9VCwdOHv 9taz/SaEQKHNTPKjIBNRQ+8ZxwA0h74BIpGo2CNkzy5Az93NXRBZzSCN0CvYhmclaew= X-Gm-Gg: Acq92OG035LKUIH6tzfs1ptnjX2NkOh8K4zjAoSHkhtpw+UXWmSmKgQ7qQF8Ekqek+y wgKChk/8AAJst35ayoVLKz+C8zaGu9QYczVej9E/k60jiQZt2mbRtbqfNmVAQUAlDUvnxBHzs9m FZByEcWoQf3uNjWOQvinx/i/5MzqZezVtd2XPwfkOAvJdsLQnJdloRarEPk0OKuSfyR50IDxKQ4 9TFeZg4NKPWwWqZ693p4ec/rTHzQG6rDkWKYe2lCQ47UsuyL/P16V0Iug/2ayFv8FyrECBnm8fo uGc2u7S/XuHZxQJlMHF8ppunseKetRQRQBJdkmzYuDSImpBfHyUK9OpQdSGiCnZDCqfOfQQ5ajb oMI3MnimvoqrQRqpE4z/m6KMKRnYthVph3DOwzY+eWXatHEjA5Lk0rIqbL0gRfdPpgFECcVnoPp VA1J0yKuuff1gNq6ZssAL+ X-Received: by 2002:a05:600c:46cb:b0:48a:7aad:4425 with SMTP id 5b1f17b1804b1-490424a535amr283843205e9.3.1779781638246; Tue, 26 May 2026 00:47:18 -0700 (PDT) Received: from localhost ([2001:4091:a246:8595:c2f5:d375:9d8f:feeb]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-45eb6c9b980sm30533046f8f.3.2026.05.26.00.47.17 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 26 May 2026 00:47:17 -0700 (PDT) Mime-Version: 1.0 Content-Type: multipart/signed; boundary=5ab361987b71eea413031fae9d29ef44514ef13a58f1242cb994be626598; micalg=pgp-sha512; protocol="application/pgp-signature" Date: Tue, 26 May 2026 09:47:08 +0200 Message-Id: To: "Tom Rini" , "Markus Schneider-Pargmann (TI)" Cc: , "Marc Murphy" , "Rasmus Villemoes" , "Kory Maincent" , "Marek Vasut" , "Romain Gantois" , "Parvathi Pudi" , "Basharath Hussain Khaja" , "Paul Barker" , "Andrew F. Davis" , "Simon Glass" , "Anshul Dalal" , "Michael Trimarchi" , "Patrick Delaunay" , "Patrice Chotard" , "Svyatoslav Ryhel" , "Tien Fong Chee" , "Frieder Schrempf" , "Johan Jonker" , "Brian Sune" , "Hai Pham" , "Bernhard Messerklinger" , "Emanuele Ghidoli" , "Parth Pancholi" , =?utf-8?q?Jonas_Schw=C3=B6bel?= , "Paul Kocialkowski" , "Andre Przywara" , "Paresh Bhagat" , "Ion Agorria" , "Peng Fan" Subject: Re: [PATCH v7 00/14] arm: am335x: Switch to upstream devicetree From: "Markus Schneider-Pargmann" X-Mailer: aerc 0.21.0-126-g9e77103592fe References: <20260427-topic-am33-evm-oftree-v2026-01-v7-0-7e6cd1a805a1@baylibre.com> <20260518182239.GA423423@bill-the-cat> In-Reply-To: <20260518182239.GA423423@bill-the-cat> X-Mailman-Approved-At: Tue, 26 May 2026 10:34:48 +0200 X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.39 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" X-Virus-Scanned: clamav-milter 0.103.8 at phobos.denx.de X-Virus-Status: Clean --5ab361987b71eea413031fae9d29ef44514ef13a58f1242cb994be626598 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=UTF-8 On Mon May 18, 2026 at 8:22 PM CEST, Tom Rini wrote: > On Mon, Apr 27, 2026 at 11:01:16AM +0200, Markus Schneider-Pargmann (TI) = wrote: > >> The series is split into two logical groups: >>=20 >> - Patches 1-3: Fixes for am335x-evm to boot and support the LCD panel >> with the current u-boot devicetree. >> - Patches 4-14: Make upstream devicetree working with uboot. This >> adds tick-timer, adds compatibility of the board code for different >> USB probing, makes SPL smaller and updates the defconfig. >>=20 >> The series has been tested on am335x-evm. Note that I don't have all >> other boards built with the am335x_*evm_*defconfig, so tests are >> appreciated. buildman reports builds are working. >>=20 >> Dependencies: >> - net: cpsw: Add cpsw-switch DT binding support >> https://lore.kernel.org/r/20260410-topic-cpsw-switch-v2026-07-next= -v1-0-c914724b13e4@baylibre.com > > After rebasing this on top of current next, I ran in to the problem of > am335x_hs_evm* doesn't build. This is because looking at > arch/arm/mach-omap2/config_secure.mk I see: > OF_LIST_TARGETS =3D $(patsubst %,arch/$(ARCH)/dts/%.dtb,$(subst ",,$(CONF= IG_OF_LIST))) > And that won't work with OF_UPSTREAM. Can you please respin and make > sure all of am335x builds? CI can help with this too: > https://docs.u-boot.org/en/latest/develop/ci_testing.html > Thanks. Sorry for that, I will fix it. Best Markus > >>=20 >> Best, >> Markus >>=20 >> Signed-off-by: Markus Schneider-Pargmann (TI) >> --- >> Changes in v7: >> - Squash tick-timer patch for am335x-bonegreen-eco-u-boot into the patch >> that adds tick-timer for all other boards. >> - Remove dependency on (SPL_)CLK for (SPL_)SIMPLE_PM_BUS as clocks are >> optional now. Note I kept your reviewed-bys @Simon @Kory as I felt >> like this is a small change of the patch >> - Fixup USB probing for OF_UPSTREAM. Initial version of the patch probed >> USB host and USB gadget devices unconditionally. This is not necessary >> and also not what the old code did. The old code probed the >> ti-musb-wrapper driver which bound the usb host and usb gadget >> drivers, but it did not probe them. >> - Add CONFIG_PHANDLE_CHECK_SEQ to defconfig as many of the devices in >> the upstream devicetree have names like mmc@0 because they are within >> ti-sysc device nodes. The config symbol should help sorting them >> consistently anyways. >> - Link to v6: https://lore.kernel.org/r/20260415-topic-am33-evm-oftree-v= 2026-01-v6-0-94cde349bb01@baylibre.com >>=20 >> Changes in v6: >> - Add patch to add am335x-bonegreen-eco-u-boot.dtsi which sets >> tick-timer >> - Add patch to remove all devicetrees that were moved to OF_UPSTREAM >> - Link to v5: https://lore.kernel.org/r/20260410-topic-am33-evm-oftree-v= 2026-01-v5-0-4b9c7aca0e3e@baylibre.com >>=20 >> Changes in v5: >> - Add patch to add lcdc on beagle bone black to all boot phases. Without >> this patch booting uboot on bbb fails. >> - Add patch to split SIMPLE_PM_BUS into two, one for SPL and one for >> normal u-boot. This was done to reduce the size of SPL. >> - Link to v4: https://lore.kernel.org/r/20260127-topic-am33-evm-oftree-v= 2026-01-v4-0-0dadb0710ce7@baylibre.com >>=20 >> Changes in v4: >> - Add patch to enable SPL_OPTIMIZE_INLINING to reduce size and fix the >> build >> - Link to v3: https://lore.kernel.org/r/20251204-topic-am33-evm-oftree-v= 2026-01-v3-0-3d1566ce4dc7@baylibre.com >>=20 >> Changes in v3: >> - Add patch to unify all config variants for the am335x-evm and similar >> boards. This also updates the upstream defconfig enablement patch to >> effectively switch all config variants to OF_UPSTREAM. >> - Link to v2: https://lore.kernel.org/r/20251128-topic-am33-evm-oftree-v= 2026-01-v2-0-42a863304498@baylibre.com >>=20 >> Changes in v2: >> - Most of the independent patches were extracted in smaller series >> - Added patch to add vidconsole for am335x-evm >> - Add chosen tick-timer to all devicetrees that are used with the >> am335x_evm_defconfig >> - Link to v1: https://lore.kernel.org/r/20251118-topic-am33-evm-oftree-v= 2026-01-v1-0-91ce13c661e2@baylibre.com >>=20 >> --- >> Markus Schneider-Pargmann (TI) (14): >> arm: dts: am335x-evm: Add backlight to the panel >> configs: am335x_evm_defconfig: Enable panel >> include: configs: am335x_evm: Enable vidconsole >> configs: am335x_evm: Enable SPL_OPTIMIZE_INLINING >> configs: am335x_evm: Unify evm board defconfigs >> am33xx: Avoid hard failure on USB probe issue >> arm: dts: am335x-*-u-boot: Add chosen tick-timer >> arm: dts: am335x-boneblack-u-boot: Add lcdc to all boot phases >> am33xx: Fix comment about config symbols >> am33xx: Support upstream devicetree USB device >> dm: core: Split SIMPLE_PM_BUS into phases >> configs: am335x_evm_defconfig: Switch to upstream devicetree >> arm: dts: am335x: Remove unused uboot devicetrees >> dm: core: Remove dependency on CLK >>=20 >> arch/arm/dts/Makefile | 11 - >> arch/arm/dts/am335x-bone-common-u-boot.dtsi | 14 + >> arch/arm/dts/am335x-bone-u-boot.dtsi | 6 + >> arch/arm/dts/am335x-bone.dts | 23 - >> arch/arm/dts/am335x-boneblack-u-boot.dtsi | 17 + >> arch/arm/dts/am335x-boneblack.dts | 174 ----- >> arch/arm/dts/am335x-bonegreen-eco-u-boot.dtsi | 6 + >> arch/arm/dts/am335x-bonegreen-eco.dts | 53 -- >> arch/arm/dts/am335x-bonegreen-u-boot.dtsi | 6 + >> arch/arm/dts/am335x-bonegreen.dts | 14 - >> arch/arm/dts/am335x-evm-u-boot.dtsi | 6 + >> arch/arm/dts/am335x-evm.dts | 766 --------------= ------- >> arch/arm/dts/am335x-evmsk-u-boot.dtsi | 6 + >> arch/arm/dts/am335x-evmsk.dts | 730 --------------= ------ >> arch/arm/dts/am335x-icev2-u-boot.dtsi | 4 + >> arch/arm/dts/am335x-icev2.dts | 486 ------------- >> arch/arm/dts/am335x-pocketbeagle-u-boot.dtsi | 6 + >> arch/arm/dts/am335x-pocketbeagle.dts | 237 ------- >> arch/arm/dts/am335x-sancloud-bbe-common.dtsi | 67 -- >> .../am335x-sancloud-bbe-extended-wifi-u-boot.dtsi | 6 + >> arch/arm/dts/am335x-sancloud-bbe-extended-wifi.dts | 113 --- >> arch/arm/dts/am335x-sancloud-bbe-lite.dts | 50 -- >> arch/arm/dts/am335x-sancloud-bbe.dts | 53 -- >> arch/arm/mach-omap2/am33xx/board.c | 23 +- >> configs/am335x_evm.config | 87 +++ >> configs/am335x_evm_defconfig | 96 +-- >> configs/am335x_evm_spiboot_defconfig | 85 +-- >> configs/am335x_hs_evm.config | 14 + >> configs/am335x_hs_evm_defconfig | 95 +-- >> configs/am335x_hs_evm_uart_defconfig | 94 +-- >> drivers/core/Kconfig | 10 +- >> drivers/core/Makefile | 2 +- >> include/configs/am335x_evm.h | 2 + >> 33 files changed, 227 insertions(+), 3135 deletions(-) >> --- >> base-commit: 1c1bfb1cf34bbb2a5717eb6995c55093279f9341 >> change-id: 20251118-topic-am33-evm-oftree-v2026-01-ea701e03f8cf >> prerequisite-change-id: 20260410-topic-cpsw-switch-v2026-07-next-89e21f2= d2446:v1 >> prerequisite-patch-id: 896e00f13956d28e056dd63764ad8906a0b626ef >> prerequisite-patch-id: f612a9425d5a466c7585decbd8990d497a02cdfb >> prerequisite-patch-id: 9b18f917297f98b76a75f3d16536283b4c1bc76b >> prerequisite-patch-id: 2baa422fb10baa29bf4274a658eb7afba8c7b489 >>=20 >> Best regards, >> --=20 >> Markus Schneider-Pargmann (TI) >>=20 --5ab361987b71eea413031fae9d29ef44514ef13a58f1242cb994be626598 Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iKMEABYKAEsWIQSJYVVm/x+5xmOiprOFwVZpkBVKUwUCahVP/RsUgAAAAAAEAA5t YW51MiwyLjUrMS4xMiwyLDIRHG1zcEBiYXlsaWJyZS5jb20ACgkQhcFWaZAVSlN3 PwD/cUL8duAFm5PTYoH1fwJDPc7aj2+88bmzIHoNHrK7aVsA/1x2k+rcKbssrt4w WVncQ8mDBhLGOB9kxW0HYQRvB6MK =9I8E -----END PGP SIGNATURE----- --5ab361987b71eea413031fae9d29ef44514ef13a58f1242cb994be626598--