Re: [PATCH v7 00/14] arm: am335x: Switch to upstream devicetree

["Markus Schneider-Pargmann" <msp@baylibre.com>]

[-- Attachment #1: Type: text/plain, Size: 7909 bytes --]

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:
>> 
>> - 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.
>> 
>> 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.
>> 
>> 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 = $(patsubst %,arch/$(ARCH)/dts/%.dtb,$(subst ",,$(CONFIG_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

>
>> 
>> Best,
>> Markus
>> 
>> Signed-off-by: Markus Schneider-Pargmann (TI) <msp@baylibre.com>
>> ---
>> 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-v2026-01-v6-0-94cde349bb01@baylibre.com
>> 
>> 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-v2026-01-v5-0-4b9c7aca0e3e@baylibre.com
>> 
>> 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-v2026-01-v4-0-0dadb0710ce7@baylibre.com
>> 
>> 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-v2026-01-v3-0-3d1566ce4dc7@baylibre.com
>> 
>> 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-v2026-01-v2-0-42a863304498@baylibre.com
>> 
>> 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-v2026-01-v1-0-91ce13c661e2@baylibre.com
>> 
>> ---
>> 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
>> 
>>  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-89e21f2d2446:v1
>> prerequisite-patch-id: 896e00f13956d28e056dd63764ad8906a0b626ef
>> prerequisite-patch-id: f612a9425d5a466c7585decbd8990d497a02cdfb
>> prerequisite-patch-id: 9b18f917297f98b76a75f3d16536283b4c1bc76b
>> prerequisite-patch-id: 2baa422fb10baa29bf4274a658eb7afba8c7b489
>> 
>> Best regards,
>> -- 
>> Markus Schneider-Pargmann (TI) <msp@baylibre.com>
>> 


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 289 bytes --]