Re: [PATCH 0/9] board: beagle: beagley-ai: Cleanups and

2025-04-17 Thread Jon Humphreys
Nishanth Menon writes: > Just happened to get a BeagleY-AI at desk and happened to test master > branch (7dd49a9264a6 drivers: scsi: Add 'erase' support), noticed a few > issues which were rather easy to solve.. so, here we go: > > Bootlog: https://gist.github.com/nmenon/a3a18825a783c0c3798cc7250

Re: [PATCH 8/9] configs: am67a_beagley_ai_a53: Enable additional features

2025-04-17 Thread Jon Humphreys
Nishanth Menon writes: > Enable basic efi capsule features, switch over to stdboot, enable > GPIO, LED and PMIC used on this platform. And enable UHS mode support > for SD cards in U-Boot. > Nishanth, I would update the description to be "Enable basic efi debug features, ..." since you are not e

Re: [PATCH v4 0/3] EFI Capsule update explicitly sets dfu_alt_info

2025-04-09 Thread Jon Humphreys
Michal Simek writes: > Hi, > > st 26. 2. 2025 v 23:36 odesílatel Jonathan Humphreys > napsal: >> >> For capsule update, explicitly set the dfu_alt_info environment variable >> before the DFU operation, and then restore it to the original value. >> Previously, the dfu_alt_info environment variabl

Re: [PATCH 1/2] efi_loader: Move public cert for capsules to .rodata

2025-04-04 Thread Jon Humphreys
Ilias Apalodimas writes: > commit ddf67daac39d ("efi_capsule: Move signature from DTB to .rodata") > was reverted in > commit 47a25e81d35c ("Revert "efi_capsule: Move signature from DTB to > .rodata"") > because that's what U-Boot was usually doing -- using the DT to store > configuration and da

Re: [PATCH 1/2] configs: am62x_r5: introduce fragment for USB MSC boot

2025-03-01 Thread Jon Humphreys
Siddharth Vadapalli writes: > Introduce the config fragment for enabling USB MSC boot. USB MSC boot > involves fetching the next stage of the bootloader from a USB Mass Storage > device such as a USB Flash Drive with the USB controller on the SoC acting > as the USB Host. > > Signed-off-by: Siddh

Re: [PATCH v4 1/3] xilinx: dfu: Fill directly update_info.dfu_string

2025-02-28 Thread Jon Humphreys
Heinrich Schuchardt writes: > On 28.02.25 16:41, Michal Simek wrote: >> >> >> On 2/28/25 15:33, Heinrich Schuchardt wrote: >>> On 26.02.25 23:35, Jonathan Humphreys wrote: From: Michal Simek Directly fill update_info.dfu_string to prepare platforms to switch from using dfu_al

Re: [PATCH v3 2/3] efi_firmware: set EFI capsule dfu_alt_info env explicitly

2025-02-23 Thread Jon Humphreys
Ilias Apalodimas writes: > Hi Jonathan, > > The patch looks correct but there's a few more things to address. > > On Thu, Feb 13, 2025 at 01:53:50PM -0600, Jonathan Humphreys wrote: >> The current implementation of EFI capsule update uses set_dfu_alt_info() to >> set the dfu_alt_info environment

Re: [PATCH] dfu: Prevent set_dfu_alt_info() from overwriting a previous value

2025-02-03 Thread Jon Humphreys
Ilias Apalodimas writes: > Hi Jon, > > On Fri, 17 Jan 2025 at 00:02, Jon Humphreys wrote: >> >> Sughosh Ganu writes: >> >> > On Thu, 16 Jan 2025 at 14:07, Mattijs Korpershoek >> > wrote: >> >> >> >> Hi Jon, >> >&g

Re: [PATCH] dfu: Prevent set_dfu_alt_info() from overwriting a previous value

2025-01-16 Thread Jon Humphreys
f my backlog. >> >> On mer., déc. 18, 2024 at 17:09, Jon Humphreys wrote: >> >> > Mattijs Korpershoek writes: >> > >> >> Hi Jonathan, >> >> >> >> Thank you for the patch. >> >> >> >> On mar.,

Re: [PATCH] dfu: Prevent set_dfu_alt_info() from overwriting a previous value

2024-12-18 Thread Jon Humphreys
Mattijs Korpershoek writes: > Hi Jonathan, > > Thank you for the patch. > > On mar., déc. 17, 2024 at 14:48, Jonathan Humphreys > wrote: > >> If CONFIG_SET_DFU_ALT_INFO is enabled, the dfu_alt_info environment >> variable is dynamically set when initializing the DFU entities, which is >> done a

Re: [PATCH 1/4] dfu: do not overwrite "dfu_alt_info"

2024-12-17 Thread Jon Humphreys
Siddharth Vadapalli writes: > From: Jonathan Humphreys > > For use-cases such as USB DFU Boot, "spl_dfu_cmd()" will setup the > appropriate value for "dfu_alt_info". To facilitate such use-cases and > in order to avoid overwriting the value of "dfu_alt_info", invoke the > "set_dfu_alt_info()" fu

Re: [PATCH] dma: ti: k3-udma: Fix BCDMA probe by adding check for MMR_RFLOW

2024-12-17 Thread Jon Humphreys
Prasanth Babu Mantena writes: > RFLOW config related MMR does not exist incase of BCDMA. > Add check to bypass the RFLOW MMR extraction. > Without this, the probe sequence fails checking for > the MMR_RFLOW region, which is valid only for packet based > DMA and obselete for BCDMA. > > Signed-off-

Re: [PATCH v2] mtd: spi-nor: Fix the spi_nor_read() when config SPI_STACKED_PARALLEL is enabled

2024-12-12 Thread Jon Humphreys
Venkatesh Yadav Abbarapu writes: > Update the spi_nor_read() function only for parallel configuration > when config SPI_STACKED_PARALLEL is enabled and keep the default > code as is. For parallel configurations fix the read issue for > 4byte address width by passing the entire length to the read

RE: [PATCH 1/6] Revert "spi: zynq_qspi: Add parallel memories support in QSPI driver"

2024-12-05 Thread Jon Humphreys
"Abbarapu, Venkatesh" writes: > Hi, > >> -Original Message- >> From: Jon Humphreys >> Sent: Thursday, December 5, 2024 9:47 AM >> To: Marek Vasut ; Simek, Michal >> ; u-boot@lists.denx.de; Tom Rini >> Cc: Andre Przywara ; Ashok Red

Re: [PATCH 1/6] Revert "spi: zynq_qspi: Add parallel memories support in QSPI driver"

2024-12-04 Thread Jon Humphreys
Jon Humphreys writes: > Marek Vasut writes: > >> On 11/7/24 4:49 PM, Jon Humphreys wrote: >>> Marek Vasut writes: >>> >>>> On 11/6/24 10:58 PM, Jon Humphreys wrote: >>>>> Marek Vasut writes: >>>>> >

Re: [PATCH v2 0/5] EFI Capsule "dfu_alt_info" fixes for TI Boards

2024-11-27 Thread Jon Humphreys
Siddharth Vadapalli writes: > On Mon, Nov 25, 2024 at 07:51:54PM -0600, Jon Humphreys wrote: > > Hello Jon, > > [...] > >> >> Hi Siddharth, this is an issue that I also discovered while working on a >> separate issue with Collabora, but haven't posted a

Re: [PATCH v2 0/5] EFI Capsule "dfu_alt_info" fixes for TI Boards

2024-11-25 Thread Jon Humphreys
Siddharth Vadapalli writes: > Hello, > > This series fixes the commits which introduced EFI Capsule support. The > blamed commits unconditionally updated "dfu_alt_info" with the > "dfu_string" corresponding to Serial Flash and therefore its associated > partitions. While this addresses use-cases

Re: [PATCH 1/6] Revert "spi: zynq_qspi: Add parallel memories support in QSPI driver"

2024-11-11 Thread Jon Humphreys
Marek Vasut writes: > On 11/7/24 4:49 PM, Jon Humphreys wrote: >> Marek Vasut writes: >> >>> On 11/6/24 10:58 PM, Jon Humphreys wrote: >>>> Marek Vasut writes: >>>> >>>>> On 11/6/24 8:18 PM, Jon Humphreys wrote: >>>&g

Re: [PATCH 1/6] Revert "spi: zynq_qspi: Add parallel memories support in QSPI driver"

2024-11-07 Thread Jon Humphreys
Marek Vasut writes: > On 11/6/24 10:58 PM, Jon Humphreys wrote: >> Marek Vasut writes: >> >>> On 11/6/24 8:18 PM, Jon Humphreys wrote: >>>> Marek Vasut writes: >>>> >>>>> On 10/23/24 10:17 AM, Michal Simek wrote: >>>>

Re: [PATCH 1/6] Revert "spi: zynq_qspi: Add parallel memories support in QSPI driver"

2024-11-06 Thread Jon Humphreys
Marek Vasut writes: > On 11/6/24 8:18 PM, Jon Humphreys wrote: >> Marek Vasut writes: >> >>> On 10/23/24 10:17 AM, Michal Simek wrote: >>>> >>>> >>>> On 10/22/24 23:06, Marek Vasut wrote: >>>>> This reverts commit

Re: [PATCH 1/6] Revert "spi: zynq_qspi: Add parallel memories support in QSPI driver"

2024-11-06 Thread Jon Humphreys
Marek Vasut writes: > On 10/23/24 10:17 AM, Michal Simek wrote: >> >> >> On 10/22/24 23:06, Marek Vasut wrote: >>> This reverts commit 1e36d34b52e7a1ebe5a2a5339d6905540f4253aa. >>> >>> This parallel/stacked support breaks basic SPI NOR support, >>> e.g. this no longer works: >>> >>> => sf probe

Re: [PATCH v11 20/29] net-lwip: lwIP wget supports user defined port in the uri, so allow it.

2024-10-04 Thread Jon Humphreys
Jerome Forissier writes: > On 10/4/24 09:11, Ilias Apalodimas wrote: >> On Thu, 3 Oct 2024 at 18:47, Jerome Forissier >> wrote: >>> >>> From: Jonathan Humphreys >> >> A short description is missing here > > I will fold this patch into the previous one since it was a review > comment from Jonat

Re: [PATCH v5 4/6] configs: am62x_evm_*: Enable USB and DFU support

2024-07-11 Thread Jon Humphreys
Jon Humphreys writes: > Martyn Welch writes: > >> On Thu, 2024-05-23 at 15:08 -0500, Jon Humphreys wrote: >>> Martyn Welch writes: >>> >>> > From: Sjoerd Simons >>> > >>> > Provide config fragments to enable USB host as w

Re: [PATCH v5 4/6] configs: am62x_evm_*: Enable USB and DFU support

2024-07-11 Thread Jon Humphreys
Martyn Welch writes: > On Thu, 2024-05-23 at 15:08 -0500, Jon Humphreys wrote: >> Martyn Welch writes: >> >> > From: Sjoerd Simons >> > >> > Provide config fragments to enable USB host as well as USB gadget >> > and DFU >> > support

Re: [PATCH v5 4/6] configs: am62x_evm_*: Enable USB and DFU support

2024-07-03 Thread Jon Humphreys
Martyn Welch writes: > On Thu, 2024-05-23 at 15:08 -0500, Jon Humphreys wrote: >> Martyn Welch writes: >> >> > From: Sjoerd Simons >> > >> > Provide config fragments to enable USB host as well as USB gadget >> > and DFU >> > support

Re: [PATCH v4 10/14] net-lwip: add wget command

2024-06-24 Thread Jon Humphreys
Jerome Forissier writes: > On 6/24/24 08:21, Jon Humphreys wrote: >> Jerome Forissier writes: >> >>> Add support for the wget command with NET_LWIP. >>> >>> Based on code initially developed by Maxim U. >>> >>> Signed-off-by: Jerome

Re: [PATCH v4 10/14] net-lwip: add wget command

2024-06-24 Thread Jon Humphreys
Subject: [PATCH] net-lwip: Add message if not using http:// for wget U-Boot's wget only supports http://, so give the user a clue if they don't use it. Signed-off-by: Jonathan Humphreys --- net-lwip/wget.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/net-lwip/wget.c b/

Re: [PATCH v4 10/14] net-lwip: add wget command

2024-06-24 Thread Jon Humphreys
Subject: [PATCH] net-lwip: fixes off-by-one array access error with wget When wget parses the url and extracts the host, it is off by one on the index to terminate the character array. Signed-off-by: Jonathan Humphreys --- net-lwip/wget.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) d

Re: [PATCH v4 10/14] net-lwip: add wget command

2024-06-24 Thread Jon Humphreys
Jerome Forissier writes: > Add support for the wget command with NET_LWIP. > > Based on code initially developed by Maxim U. > > Signed-off-by: Jerome Forissier > Co-developed-by: Maxim Uvarov > Cc: Maxim Uvarov > Signed-off-by: Jerome Forissier > --- > cmd/Kconfig| 7 ++ > cmd/net

Re: [PATCH] efi_loader: adjust config options for capsule updates

2024-06-18 Thread Jon Humphreys
Ilias Apalodimas writes: > Hi Jon, > > On Tue, 18 Jun 2024 at 19:49, Jon Humphreys wrote: >> >> Ilias Apalodimas writes: >> >> > EFI_IGNORE_OSINDICATIONS is used to ignore OsIndications if setvariable >> > at runtime is not supported and allow th

Re: [PATCH] efi_loader: adjust config options for capsule updates

2024-06-18 Thread Jon Humphreys
Ilias Apalodimas writes: > EFI_IGNORE_OSINDICATIONS is used to ignore OsIndications if setvariable > at runtime is not supported and allow the platform to perform capsule > updates on disk. With the recent changes boards can conditionally enable > setvariable at runtime using EFI_RT_VOLATILE_STOR

Re: [PATCH v2 0/2] scripts/Makefile.lib: EFI: Use capsule CRT instead of ESL

2024-06-14 Thread Jon Humphreys
Ilias Apalodimas writes: > Hi Jonathan > > On Thu, 13 Jun 2024 at 23:28, Jonathan Humphreys wrote: >> >> Use the capsule's public key certificate rather than a prebuilt ESL >> generated from the certificate. The ESL is now generated as part of the >> build. > > Is there a reason to do this? I un

Re: [PATCH] efi: capsule: Improve capsule update porting to new boards

2024-06-13 Thread Jon Humphreys
Ilias Apalodimas writes: > Hi Jon, > > On Sat, 8 Jun 2024 at 02:44, Jonathan Humphreys wrote: >> >> Add to the capsule update porting documentation that when capsule >> authentication is enabled (via EFI_CAPSULE_AUTHENTICATE), a public key >> certificate corresponding to the capsule signing key

Re: [PATCH v4 15/19] board: am62px: Define capsule update firmware info

2024-05-30 Thread Jon Humphreys
Ilias Apalodimas writes: > Hi Jon, > > On Fri, 24 May 2024 at 18:38, Jon Humphreys wrote: >> >> Ilias Apalodimas writes: >> >> > Hi Jonathan >> > >> > Thanks for working on this >> > >> > On Thu, May 09, 2024 at 11:41:19

Re: [PATCH v2 4/4] configs: j721s2_evm_*_defconfig: Enable OSPI configs

2024-05-30 Thread Jon Humphreys
Manorit Chawdhry writes: > Enable OSPI related configs to boot using OSPI > > Reviewed-by: Apurva Nandan > Signed-off-by: Manorit Chawdhry > --- > configs/j721s2_evm_a72_defconfig | 3 +++ > configs/j721s2_evm_r5_defconfig | 3 +++ > 2 files changed, 6 insertions(+) > > diff --git a/configs/j

Re: [PATCH v4 15/19] board: am62px: Define capsule update firmware info

2024-05-24 Thread Jon Humphreys
Ilias Apalodimas writes: > Hi Jonathan > > Thanks for working on this > > On Thu, May 09, 2024 at 11:41:19AM -0500, Jonathan Humphreys wrote: >> Define the firmware components updatable via EFI capsule update, including >> defining capsule GUIDs for the various firmware components for the AM62px

Re: [PATCH v2 4/7] dts: beagleplay: binman: Include firmware capsules binman nodes

2024-05-23 Thread Jon Humphreys
Tom Rini writes: > On Wed, May 22, 2024 at 11:12:35PM -0500, Jon Humphreys wrote: >> Tom Rini writes: >> >> > On Tue, May 21, 2024 at 09:20:26PM -0500, Jon Humphreys wrote: >> >> Tom Rini writes: >> >> >> >> > On

Re: [PATCH v5 4/6] configs: am62x_evm_*: Enable USB and DFU support

2024-05-23 Thread Jon Humphreys
Martyn Welch writes: > From: Sjoerd Simons > > Provide config fragments to enable USB host as well as USB gadget and DFU > support for a53 and r5. This relevant fragment is included into the > am62x EVM a53 defconfig. For the r5, due to the smaller available size, > the config fragment also disa

Re: [PATCH v2 4/7] dts: beagleplay: binman: Include firmware capsules binman nodes

2024-05-22 Thread Jon Humphreys
Tom Rini writes: > On Tue, May 21, 2024 at 09:20:26PM -0500, Jon Humphreys wrote: >> Tom Rini writes: >> >> > On Fri, Apr 19, 2024 at 04:28:16PM -0500, Jonathan Humphreys wrote: >> > >> >> Fill in the BeaglePlay's capsule GUID proper

Re: [PATCH v2 4/7] dts: beagleplay: binman: Include firmware capsules binman nodes

2024-05-21 Thread Jon Humphreys
Tom Rini writes: > On Fri, Apr 19, 2024 at 04:28:16PM -0500, Jonathan Humphreys wrote: > >> Fill in the BeaglePlay's capsule GUID properties of the base binman capsule >> nodes. >> >> Signed-off-by: Jonathan Humphreys >> --- >> arch/arm/dts/k3-am625-beagleplay-u-boot.dtsi | 27

Re: [PATCH] mmc: sdhci: Correct ADMA_DESC_LEN to 12

2024-05-01 Thread Jon Humphreys
"A. Sverdlin" writes: > From: Alexander Sverdlin > > Commit 37cb626da25d0d ("mmc: sdhci: Add Support for ADMA2") introduced > ADMA_DESC_LEN == 16 (64 bit case), but it was never used before commit > 74755c1fed1b0 ("mmc: sdhci: introduce adma_write_desc() hook to struct > sdhci_ops"). > > "sizeo

Re: [PATCH v2 01/16] board: Define GUIDs for firmware images

2024-04-12 Thread Jon Humphreys
Ilias Apalodimas writes: > Hi both > > On Wed, 10 Apr 2024 at 10:36, Heinrich Schuchardt wrote: >> >> On 09.04.24 23:05, Jon Humphreys wrote: >> > Heinrich Schuchardt writes: >> > >> >> On 4/9/24 00:31, Jonathan Humphreys wrote: >> &g

Re: [PATCH 01/13] ti:keys Add EFI signature list

2024-04-12 Thread Jon Humphreys
Ilias Apalodimas writes: > Hi Jon, > > > On Wed, 10 Apr 2024 at 20:35, Jon Humphreys wrote: >> >> Ilias Apalodimas writes: >> >> > On Tue, 9 Apr 2024 at 23:14, Andrew Davis wrote: >> >> >> >> On 4/9/24 2:26 PM, Heinric

Re: [PATCH v2 12/16] arm: dts: k3-am625-sk-u-boot: Add sysreset-controller node

2024-04-11 Thread Jon Humphreys
Mattijs Korpershoek writes: > Hi Jonathan, > > Thank you for the patch. > > On lun., avril 08, 2024 at 17:31, Jonathan Humphreys > wrote: > >> Signed-off-by: Jonathan Humphreys > > Please consider adding a commit message body. Got it. thanks. BTW, the next version of this series will drop t

Re: [PATCH 2/2] arm: dts: k3: Remove unneeded ti, sci-sysreset binding and nodes

2024-04-10 Thread Jon Humphreys
Andrew Davis writes: > This extra binding is non-standard and now unneeded as we bind the > sysreset driver automatically. This matches what is done in Linux > and allows us to more closely match the DTBs. Remove the binding > and all users. > > Signed-off-by: Andrew Davis > --- > arch/arm/dts/

Re: [PATCH 1/2] firmware: ti_sci: Bind sysreset driver when enabled

2024-04-10 Thread Jon Humphreys
Andrew Davis writes: > The sysreset TI-SCI API is available with TI-SCI always, there is no need > for a DT node to describe the availability of this. If the sysreset driver > is available then bind it during ti-sci probe. > > Remove the unneeded device tree matching. > > Signed-off-by: Andrew Da

Re: [PATCH] arm: mach-k3: security: Lower verbosity of cert message for GP

2024-04-10 Thread Jon Humphreys
Andrew Davis writes: > When we find a certificate on an image to be booted on a GP device we > print out a message explaining that the certificate is being skipped. > This message is rather long and is printed for every image. Shorten > the message and make the long version into a debug message.

Re: [PATCH 01/13] ti:keys Add EFI signature list

2024-04-10 Thread Jon Humphreys
Ilias Apalodimas writes: > On Tue, 9 Apr 2024 at 23:14, Andrew Davis wrote: >> >> On 4/9/24 2:26 PM, Heinrich Schuchardt wrote: >> > On 4/9/24 14:14, Andrew Davis wrote: >> >> On 4/8/24 10:34 PM, Heinrich Schuchardt wrote: >> >>> On 4/8/24 23:33, Jonathan Humphreys wrote: >> EFI signature l

Re: [PATCH 3/7] dts: j721e: binman: Include firmware capsules binman nodes

2024-04-10 Thread Jon Humphreys
Andrew Davis writes: > On 4/8/24 5:17 PM, Jonathan Humphreys wrote: >> Signed-off-by: Jonathan Humphreys >> --- >> arch/arm/dts/k3-j721e-binman.dtsi | 32 +++ >> 1 file changed, 32 insertions(+) >> >> diff --git a/arch/arm/dts/k3-j721e-binman.dtsi >> b/arch/arm/d

Re: [PATCH v2 02/16] board: am64x: Define capsule update firmware info

2024-04-09 Thread Jon Humphreys
Heinrich Schuchardt writes: > On 4/9/24 00:31, Jonathan Humphreys wrote: >> Define the firmwares updatable via EFI capsule update. > > Nits: > > %s/firmwares/firmware/ (firmware is uncountable). > >> >> Signed-off-by: Jonathan Humphreys >> --- >> board/ti/am64x/evm.c | 33 +

Re: [PATCH v2 01/16] board: Define GUIDs for firmware images

2024-04-09 Thread Jon Humphreys
Heinrich Schuchardt writes: > On 4/9/24 00:31, Jonathan Humphreys wrote: >> Define GUIDs for the different firmware images (tiboot3.bin, tispl.bin, >> u-boot.img, sysfw). > >> Signed-off-by: Jonathan Humphreys >> --- >> include/configs/ti_armv7_common.h | 17 + >> 1 file chang

Re: [PATCH 1/4] mtd: spi-nor-core: Do not start or end writes at odd address in DTR mode

2024-04-01 Thread Jon Humphreys
Manorit Chawdhry writes: > From: Pratyush Yadav > > On DTR capable flashes like Micron Xcella the writes cannot start or end > at an odd address in DTR mode. Extra 0xff bytes need to be prepended or > appended respectively to make sure both the start and end addresses are > even. > > Signed-off-

Re: [RFC] Makefile.lib: find capsule ESL dtsi file when CONFIG_OF_UPSTREAM=y

2024-03-29 Thread Jon Humphreys
Sughosh Ganu writes: > On Thu, 28 Mar 2024 at 09:34, Jon Humphreys wrote: >> >> Sughosh Ganu writes: >> >> > hi Jonathan, >> > >> > On Wed, 27 Mar 2024 at 08:05, Jonathan Humphreys >> > wrote: >> >> >> >> When

Re: [RFC] Makefile.lib: find capsule ESL dtsi file when CONFIG_OF_UPSTREAM=y

2024-03-28 Thread Jon Humphreys
Sughosh Ganu writes: > hi Jonathan, > > On Wed, 27 Mar 2024 at 08:05, Jonathan Humphreys wrote: >> >> When CONFIG_OF_UPSTREAM is enabled, DTS files are in SOC subdirectories (vs >> the >> top level dts directory), but when CONFIG_EFI_CAPSULE_AUTHENTICATE is >> enabled, >> the dynamically creat

Re: [PATCH v2 2/2] arm: dts: k3-am642-evm/sk: Enable OSPI support in SPL

2024-02-27 Thread Jon Humphreys
Tom Rini writes: > On Sat, Feb 24, 2024 at 11:34:36AM -0600, Jon Humphreys wrote: >> Tom Rini writes: >> >> > On Fri, Feb 23, 2024 at 06:17:02PM -0600, Jonathan Humphreys wrote: >> >> Add bootph DT tags to enable OSPI in SPL. >> >> Set OS

Re: [PATCH v2 3/3] arm: dts: k3-j721e-sk: Remove OSPI phypattern partition

2024-02-25 Thread Jon Humphreys
"Kumar, Udit" writes: > Thanks Jon > > On 2/24/2024 5:53 AM, Jonathan Humphreys wrote: >> The phy calibration pattern partition isn't needed as the Cadence driver >> isn't >> calibrating the phys. > > > Please do mention Fixes tag here > > 58d61fb5a77e ("arm: dts: k3-j721e-sk: Add initial A72 sp

Re: [PATCH v2 2/2] arm: dts: k3-am642-evm/sk: Enable OSPI support in SPL

2024-02-24 Thread Jon Humphreys
Tom Rini writes: > On Fri, Feb 23, 2024 at 06:17:02PM -0600, Jonathan Humphreys wrote: >> Add bootph DT tags to enable OSPI in SPL. >> Set OSPI regs for R5 SPL to address OSPI's boot region. >> >> Signed-off-by: Jonathan Humphreys >> --- >> arch/arm/dts/k3-am642-evm-u-boot.dtsi | 16 ++

Re: [PATCH 2/2] arm: dts: k3-am642-evm/sk: Enable OSPI support in SPL

2024-02-23 Thread Jon Humphreys
Tom Rini writes: > On Thu, Feb 22, 2024 at 11:24:38PM -0600, Jonathan Humphreys wrote: >> Add bootph DT tags to enable OSPI in SPL. >> Set OSPI regs for R5 SPL to address OSPI's boot region. >> >> Signed-off-by: Jonathan Humphreys >> --- >> arch/arm/dts/k3-am642-evm-u-boot.dtsi | 24 ++

Re: [PATCH V2 10/10] include: env: ti: Drop default_findfdt

2024-01-12 Thread Jon Humphreys
Nishanth Menon writes: > We shouldn't need finfdt anymore. Drop the env script. > > Signed-off-by: Nishanth Menon > --- > Changes from V1: None. > > V1: https://lore.kernel.org/r/20240108173301.2692332-11...@ti.com > include/env/ti/default_findfdt.env | 12 > 1 file changed, 12 del

Re: [PATCH V2 09/10] board: beagle: beagleplay: Set fdtfile from C code instead of findfdt script

2024-01-12 Thread Jon Humphreys
Nishanth Menon writes: > Stop using the findfdt script and switch to setting the fdtfile from C > code. > > Signed-off-by: Nishanth Menon > --- > Changes from V1: > * Just macro name change s/TI_EVM_FDT_FOLDER_PATH/TI_FDT_FOLDER_PATH > * Commit message update to drop the "warning added to findfd

Re: [PATCH V2 08/10] board: beagle: beagleboneai64: Set fdtfile from C code instead of findfdt script

2024-01-12 Thread Jon Humphreys
Nishanth Menon writes: > Stop using the findfdt script and switch to setting the fdtfile from C > code. > > Signed-off-by: Nishanth Menon > --- > Changes from V1: > * Just macro name change s/TI_EVM_FDT_FOLDER_PATH/TI_FDT_FOLDER_PATH > * Commit message update to drop the "warning added to findfd

Re: [PATCH V2 07/10] board: ti: j721s2: Set fdtfile from C code instead of findfdt script

2024-01-12 Thread Jon Humphreys
Nishanth Menon writes: > We now can provide a map and have the standard fdtfile variable set from > code itself. This allows for bootstd to "just work". > > While at this, replace findfdt in environment with a warning as it is no > longer needed. > > Signed-off-by: Nishanth Menon > --- > Changes

Re: [PATCH V2 06/10] board: ti: j721e: Set fdtfile from C code instead of findfdt script

2024-01-12 Thread Jon Humphreys
Nishanth Menon writes: > We now can provide a map and have the standard fdtfile variable set from > code itself. This allows for bootstd to "just work". > > While at this, replace findfdt in environment with a warning as it is no > longer needed. > > Signed-off-by: Nishanth Menon > --- > Changes

Re: [PATCH V2 05/10] board: ti: am65x: Set fdtfile from C code instead of findfdt script

2024-01-12 Thread Jon Humphreys
Nishanth Menon writes: > We now can provide a map and have the standard fdtfile variable set from > code itself. This allows for bootstd to "just work". > > While at this, replace findfdt in environment with a warning as it is no > longer needed. > > Signed-off-by: Nishanth Menon > --- > Changes

Re: [PATCH V2 04/10] board: ti: am64x: Set fdtfile from C code instead of findfdt script

2024-01-12 Thread Jon Humphreys
Nishanth Menon writes: > We now can provide a map and have the standard fdtfile variable set from > code itself. This allows for bootstd to "just work". > > While at this, replace findfdt in environment with a warning as it is no > longer needed. > > Signed-off-by: Nishanth Menon > --- > Changes

Re: [PATCH V2 03/10] board: ti: am62x: Set fdtfile from C code instead of findfdt script

2024-01-12 Thread Jon Humphreys
Nishanth Menon writes: > Stop using the findfdt script and switch to setting the fdtfile from > C code. > > While at this, replace findfdt in environment with a warning as it is > no longer needed > > Signed-off-by: Nishanth Menon > --- > Changes from V1: None. > I have retained the central call

Re: [PATCH V2 02/10] board: ti: am62ax: Set fdtfile from C code instead of findfdt script

2024-01-12 Thread Jon Humphreys
Nishanth Menon writes: > Stop using the findfdt script and switch to setting the fdtfile from > C code. > > While at this, replace findfdt in environment with a warning as it is > no longer needed > > Signed-off-by: Nishanth Menon > --- > Changes from V1: None. > I have retained the central call

Re: [PATCH V2 01/10] board: ti: common: Introduce a common fdt ops library

2024-01-12 Thread Jon Humphreys
Nishanth Menon writes: > Introduce a common fdt operations library for basic device tree > operations that are common between various boards. > > The first library to introduce here is the capability to set up > fdtfile as a standard variable as part of board identification rather > than depend o

Re: [PATCH 02/10] board: ti: am62ax: Set fdtfile from C code instead of findfdt script

2024-01-08 Thread Jon Humphreys
Nishanth Menon writes: > Stop using the findfdt script and switch to setting the fdtfile from > C code. > > While at this, replace findfdt in environment with a warning as it is > no longer needed > > Signed-off-by: Nishanth Menon > --- > board/ti/am62ax/am62ax.env | 1 - > board/ti/am62

Re: [PATCH 08/10] board: beagle: beagleboneai64: Set fdtfile from C code instead of findfdt script

2024-01-08 Thread Jon Humphreys
Nishanth Menon writes: > Stop using the findfdt script and switch to setting the fdtfile from C > code. > > While at this, replace findfdt in environment with a warning as it is > no longer needed > > Signed-off-by: Nishanth Menon > --- > board/beagle/beagleboneai64/beagleboneai64.c | 14

Re: [PATCH 01/10] board: ti: common: Introduce a common fdt ops library

2024-01-08 Thread Jon Humphreys
Nishanth Menon writes: > Introduce a common fdt operations library for basic device tree > operations that are common between various boards. > > The first library to introduce here is the capability to set up > fdtfile as a standard variable as part of board identification rather > than depend o

Re: [PATCH 1/2] board: ti: am62x: am62x.env: Fix boot_targets

2024-01-04 Thread Jon Humphreys
Andrew Davis writes: * uEnv.txt processing >>> >>> What command can do this? >>> >> >> run envboot; >> >> https://source.denx.de/u-boot/u-boot/-/blob/master/include/env/ti/mmc.env?ref_type=heads#L20 > > This could be a new bootmeth which looks for

Re: [PATCH] include: env: ti: default_findfdt: Follow the bootstd/distro conventions

2023-12-28 Thread Jon Humphreys
Nishanth Menon writes: > Distroboot and bootstd both mandate a findfdt variable pointing to the > correct device tree blob. Current mechanism calls a find_fdt function > to set this variable. We do not need a find_fdt command to set the > environment variable to a single dtb. Simplify the default