public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Mattijs Korpershoek <mkorpershoek@baylibre.com>
To: Nishanth Menon <nm@ti.com>, Neha Francis <n-francis@ti.com>,
	Tom Rini <trini@konsulko.com>
Cc: Bryan <bb@ti.com>, Praneeth <praneeth@ti.com>,
	Andrew <afd@ti.com>, Robert Nelson <robertcnelson@gmail.com>,
	Vignesh <vigneshr@ti.com>,
	u-boot@lists.denx.de, Jan Kiszka <jan.kiszka@siemens.com>,
	Nishanth Menon <nm@ti.com>
Subject: Re: [PATCH V4 0/8] board: ti: Add support for BeaglePlay
Date: Wed, 23 Aug 2023 09:44:09 +0200	[thread overview]
Message-ID: <874jkqgq46.fsf@baylibre.com> (raw)
In-Reply-To: <20230822184135.2328409-1-nm@ti.com>

Hi Nishanth,

On mar., août 22, 2023 at 13:41, Nishanth Menon <nm@ti.com> wrote:

> Hi,
>
> Add support for BeaglePlay - rev 4
>
> Full series: https://github.com/nmenon/fix-k3-dt-u-boot/commits/beagleplay-v4.2
>
> Caveats:
> * networking: pending https://lore.kernel.org/all/20230822121350.51324-1-rogerq@kernel.org/
> * 32kclk and usb: pending:
>   https://lore.kernel.org/u-boot/20230725185253.2123433-4-nm@ti.com/
>   OR https://github.com/nmenon/fix-k3-dt-u-boot/commit/853b29d63c1ca642be316f1afb0fb778610dec46
>   being properly resolved (NOTE: without this patch, wlan is broken in
>   Linux as the 32kclk from SoC is incorrectly supplied to wlan as 25MHz)
> * There seems to be a bug in Linux kernel with sdhci that seems to
>   depend on u-boot initialization of sdhci for functionality.
>
> Changes since V3:
> * GPIO, LEDs, PMIC enabled at u-boot level.
> * eMMC boot fixed in a manner not to break existing SK and other EVM
>   solutions. UDA FS solution similar to commit 5019170970ad
>   ("arch: arm: mach-k3: j721e: add support for UDA FS") introduced.
> * env file for beagleplay introduced to keep things consistent with the
>   behavior expected on BeaglePlay
> * Documentation updates to fixup alt, references, added OpenOCD debug
>   info etc.
> * New patch to ensure env files have priority over header included env
>   variables.
>
> Bootlog:
> emmc with no SD: https://gist.github.com/nmenon/5755228ff424c55245f4534548827ab9
> emmc with SD: https://gist.github.com/nmenon/8571dba9e942e77e48e52fe59cf71aac
> SD boot: https://gist.github.com/nmenon/1cc850c37469fe7c86b26842fe0910f8
>
> Baseline: 7e6e40c57233 (origin/next) Merge tag 'v2023.10-rc3' into next
>
> V3: https://lore.kernel.org/all/20230815164440.2713726-1-nm@ti.com/
> V2: https://lore.kernel.org/u-boot/20230727234446.3651836-1-nm@ti.com/
> V1: https://lore.kernel.org/all/20230725185253.2123433-1-nm@ti.com/
>
> Nishanth Menon (6):
>   env_default: Allow CONFIG_EXTRA_ENV_TEXT to override
>     CFG_EXTRA_ENV_SETTINGS
>   configs: am62x_evm*: Enable EMMC_BOOT configuration
>   arm: mach-k3: am625: Add support for UDA FS
>   arm: dts: k3-am625-sk-binman: Add labels for unsigned binary
>   board: ti: am62x: Add am62x_beagleplay_* defconfigs and env file
>   doc: board: ti: Add BeaglePlay documentation
>
> Nitin Yadav (1):
>   drivers: mmc: am654_sdhci: Update OTAP/ITAP delay
>
> Robert Nelson (1):
>   arm: dts: Add k3-am625-beagleplay

Using baseline 7e6e40c57233 ("Merge tag 'v2023.10-rc3' into next"), I
could boot all the way to U-boot proper shell.

And the LEDs work for me as well with:

  => led led-0 toggle

So for the whole series:

Tested-by: Mattijs Korpershoek <mkorpershoek@baylibre.com>

>
>  arch/arm/dts/Makefile                         |    2 +
>  .../dts/k3-am625-beagleplay-ddr4-1600MTs.dtsi | 2195 +++++++++++++++++
>  arch/arm/dts/k3-am625-beagleplay-u-boot.dtsi  |  196 ++
>  arch/arm/dts/k3-am625-beagleplay.dts          |  758 ++++++
>  arch/arm/dts/k3-am625-r5-beagleplay.dts       |   87 +
>  arch/arm/dts/k3-am625-sk-binman.dtsi          |    4 +-
>  arch/arm/mach-k3/am625_init.c                 |    6 +
>  board/ti/am62x/MAINTAINERS                    |    7 +
>  board/ti/am62x/beagleplay.env                 |   23 +
>  board/ti/am62x/beagleplay_a53.config          |   55 +
>  board/ti/am62x/beagleplay_r5.config           |   15 +
>  configs/am62x_evm_a53_defconfig               |    1 +
>  configs/am62x_evm_r5_defconfig                |    1 +
>  doc/board/ti/am62x_beagleplay.rst             |  256 ++
>  doc/board/ti/img/beagleplay_emmc.svg          |  697 ++++++
>  doc/board/ti/k3.rst                           |    1 +
>  drivers/mmc/am654_sdhci.c                     |   14 +-
>  include/env_default.h                         |    6 +-
>  18 files changed, 4315 insertions(+), 9 deletions(-)
>  create mode 100644 arch/arm/dts/k3-am625-beagleplay-ddr4-1600MTs.dtsi
>  create mode 100644 arch/arm/dts/k3-am625-beagleplay-u-boot.dtsi
>  create mode 100644 arch/arm/dts/k3-am625-beagleplay.dts
>  create mode 100644 arch/arm/dts/k3-am625-r5-beagleplay.dts
>  create mode 100644 board/ti/am62x/beagleplay.env
>  create mode 100644 board/ti/am62x/beagleplay_a53.config
>  create mode 100644 board/ti/am62x/beagleplay_r5.config
>  create mode 100644 doc/board/ti/am62x_beagleplay.rst
>  create mode 100644 doc/board/ti/img/beagleplay_emmc.svg
>
> -- 
> 2.40.0

      parent reply	other threads:[~2023-08-23  7:44 UTC|newest]

Thread overview: 53+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-08-22 18:41 [PATCH V4 0/8] board: ti: Add support for BeaglePlay Nishanth Menon
2023-08-22 18:41 ` [PATCH V4 1/8] env_default: Allow CONFIG_EXTRA_ENV_TEXT to override CFG_EXTRA_ENV_SETTINGS Nishanth Menon
2023-08-22 23:16   ` Simon Glass
2023-08-22 23:33     ` Nishanth Menon
2023-08-23  2:34       ` Simon Glass
2023-08-23 11:42         ` Nishanth Menon
2023-08-23  7:47   ` Mattijs Korpershoek
2023-08-23 14:42   ` Tom Rini
2023-08-23 15:06     ` Nishanth Menon
2023-08-23 15:17       ` Tom Rini
2023-08-22 18:41 ` [PATCH V4 2/8] configs: am62x_evm*: Enable EMMC_BOOT configuration Nishanth Menon
2023-08-23  7:50   ` Mattijs Korpershoek
2023-08-22 18:41 ` [PATCH V4 3/8] arm: mach-k3: am625: Add support for UDA FS Nishanth Menon
2023-08-23  7:54   ` Mattijs Korpershoek
2023-08-22 18:41 ` [PATCH V4 4/8] drivers: mmc: am654_sdhci: Update OTAP/ITAP delay Nishanth Menon
2023-08-23  8:06   ` Mattijs Korpershoek
2023-08-23 10:15     ` Nitin Yadav
2023-08-23 10:18       ` Nitin Yadav
2023-08-23 11:38         ` Nishanth Menon
2023-08-23 14:21           ` Nishanth Menon
2023-08-22 18:41 ` [PATCH V4 5/8] arm: dts: k3-am625-sk-binman: Add labels for unsigned binary Nishanth Menon
2023-08-23  8:09   ` Mattijs Korpershoek
2023-08-22 18:41 ` [PATCH V4 6/8] arm: dts: Add k3-am625-beagleplay Nishanth Menon
2023-08-23  8:26   ` Mattijs Korpershoek
2023-08-22 18:41 ` [PATCH V4 7/8] board: ti: am62x: Add am62x_beagleplay_* defconfigs and env file Nishanth Menon
2023-08-23  8:29   ` Mattijs Korpershoek
2023-08-23 14:42   ` Tom Rini
2023-08-22 18:41 ` [PATCH V4 8/8] doc: board: ti: Add BeaglePlay documentation Nishanth Menon
2023-08-23  8:36   ` Mattijs Korpershoek
2023-08-23 15:30   ` Simon Glass
2023-08-23 17:15     ` Tom Rini
2023-08-23 23:57       ` Simon Glass
2023-08-24  0:18         ` Nishanth Menon
2023-08-24  3:01           ` Simon Glass
2023-08-24  3:27             ` Nishanth Menon
2023-08-24  8:21               ` Mattijs Korpershoek
2023-08-24 12:07                 ` Nishanth Menon
2023-08-24 13:16               ` Neha Malcom Francis
2023-08-24 14:20                 ` Tom Rini
2023-08-24 14:41                   ` Simon Glass
2023-08-24 14:43                     ` Tom Rini
2023-08-24 14:46                       ` Simon Glass
2023-08-25 14:17                         ` Neha Malcom Francis
2023-08-24 14:16               ` Tom Rini
2023-08-25  1:04   ` Simon Glass
2023-08-25  1:22     ` Nishanth Menon
2023-08-25  1:31       ` Simon Glass
2023-08-25  1:59         ` Nishanth Menon
2023-08-25 23:45           ` Simon Glass
2023-08-26  1:06             ` Robert Nelson
2023-08-26  2:04               ` Simon Glass
2023-08-28 13:01                 ` Nishanth Menon
2023-08-23  7:44 ` Mattijs Korpershoek [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=874jkqgq46.fsf@baylibre.com \
    --to=mkorpershoek@baylibre.com \
    --cc=afd@ti.com \
    --cc=bb@ti.com \
    --cc=jan.kiszka@siemens.com \
    --cc=n-francis@ti.com \
    --cc=nm@ti.com \
    --cc=praneeth@ti.com \
    --cc=robertcnelson@gmail.com \
    --cc=trini@konsulko.com \
    --cc=u-boot@lists.denx.de \
    --cc=vigneshr@ti.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox