All of lore.kernel.org
 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.